javascript: проверьте наличие нескольких специфических свойств объекта (также в coffeescript)

Question

Мне нужно проверить, существует ли в объекте несколько конкретных свойств. И для этого у меня есть два вопроса:

1. Как я обычно должен проверять наличие нескольких существующих свойств в JavaScript?
2. Есть ли ярлык в CoffeeScript, чтобы сделать это элегантным способом?

Мой Javascript решение будет:

if(obj && obj.pOne && obj.pTwo && obj.pThree) doStuff(); 

В кофе сценарий, который не будет короче:

if obj and obj.pOne and obj.pTwo and obj.pThree then doStuff() 

Не стоит ввести в заблуждение здесь, это не будет работать:

И затем я столкнулся с оператором in, если предложения (которые я никогда не используется для таких проверок). Это правильный способ сделать то, что я хочу достичь? Или это что-то вроде не-go, или есть лучшее/более чистое решение?

Мой код будет выглядеть следующим образом:

if(obj && ('pOne' && 'pTwo' && 'pThree' in obj)) doStuff(); 

и CoffeeScript так:

if obj and ('pOne' and 'pTwo' and 'pThree' of obj) then doStuff() 

Answer 1

Ваша первая строка - это то, что вы хотите. Нет ничего плохого в этом, как есть. Возможно, подумайте о том, чтобы написать свою функцию по-другому, если вы считаете, что вам нужно проверить 4 вещи перед вызовом функции.

if(obj && obj.pOne && obj.pTwo && obj.pThree) doStuff(); 

Смотреть это: Why Defensive Programming is Rubbish

PS.Твои последние примеры не хорошо и не оценивать так, как вы думаете, что они делают

---

Рассмотрят переписывание функции, чтобы сделать его лучше использовать

var doStuff = function(obj) { 
    obj  = obj  || {}; 
    obj.pOne = obj.pOne || "pOneDefault"; 
    obj.pTwo = obj.pTwo || "pTwoDefault"; 
    obj.pThree = obj.pThree || "pThreeDefault"; 

    console.log(obj); 
}; 

doStuff(); 
// {pOne: "pOneDefault", pTwo: "pTwoDefault", pThree: "pThreeDefault"} 

doStuff({pOne: "foo"}); 
// {pOne: "foo", pTwo: "pTwoDefault", pThree: "pThreeDefault"} 

doStuff({pTwo: "bar"}); 
// {pTwo: "bar", pOne: "pOneDefault", pThree: "pThreeDefault"} 

doStuff({pTwo: "hello", pThree: "world"}); 
// {pTwo: "hello", pThree: "world", pOne: "pOneDefault"} 

Answer 2

pOne in obj не то же самое, как obj.pOne. Например, для obj = { a: 0 }, a in obj - true, но obj.a - это фальшь.

('pOne' && 'pTwo' && 'pThree' in obj) не означает, что вы думаете; он оценивается как ('pOne' && 'pTwo' && ('pThree' in obj)), что аналогично true && true && ('pThree' in obj).

Хотя многословный, если вы хотите, чтобы поймать разницу между falsy значениями и несуществующими ключами, это еще лучшим способом:

if (obj && 'pOne' in obj && 'pTwo' in obj && 'pThree' in obj) 

Если вы считаете, что свойства могут быть унаследованы, а затем с помощью obj.hasOwnProperty('pOne') вместо 'pOne' in obj еще лучше.