チェック平等「の場合 - 他の」ステートメント

Question

は、我々はそのようなコード（JS）を持って言う：

/** 
* @param {String} type Could be only 'high' or 'low' 
* @return {String} 
*/ 
function getSome(type) { 
    if (type == 'high') { 
     return 'This is high'; 
    } else if (type == 'low') { 
     return 'This is low'; 
    } 
} 

これよりも、このバリアントが望ましい場合には（私は、コメントが含まれていませんが、彼らは同じです） ？：

function getSome(type) { 
    if (type == 'high') { 
     return 'This is high'; 
    } else { 
     return 'This is low'; 
    } 
} 

私は同じような状況が何度もありました。通常、私は最高の異形については考えず、最初に来たものを書きました。しかし今、私は将来どのような変種を使うかを決めたいと思っています。

最後の質問です。変数が2つの値しか持たず、最初の値がifステートメントで与えられた場合、elseの変数の等価性をチェックする必要があるのですか？私はまた、関数のコメントを使用して答えに影響を与える可能性があることを知りたい。

Answer 1

を。知られていない型の場合、関数は未定義または他のデフォルト値を返します。デフォルト

function getSome(type) { 
    return { high: 'This is high', low: 'This is low'}[type] || 'type not known.'; 
} 

Answer 2

値のみhighとlowが有効である場合、第三の選択肢を検討してください。

function getSome(type) { 
    if (type === 'high') { 
     return 'This is high'; 
    } else if (type === 'low') { 
     return 'This is low'; 
    } else { 
     throw new Error(type + ' is invalid'); 
    } 
} 

が速い失敗。これはバグをもっと早く見つけるのに役立ちます。

Answer 3

それはあなたがより多くの可能な値持っている場合は特に、switch演算子を使用することがはるかに便利になります：二つの値のグループ化などのオブジェクトを使用しないのはなぜ

function getSome(type) { 

    switch(type){ 

     case 'hight': 
      return 'This is a high'; 
      break; 
     case 'low': 
      return 'This is a low'; 
      break; 
     case 'third': 
      return 'This is a third'; 
      break; 
     default: 
      throw new Error(type + ' is invalid.'); 
      break; 
    } 
}