このコードをリファクタリングする良い方法はありますか？

Question

以下は私のコードのすべての

if(props.plan[plan_id]) { 
    if(props.plan[plan_id].number== 0) { 
     return null; 
    } 
    else { 
     return 'some value'; 
    } 
    } 

まずあり、私はそのデータを使用して存在しているprop.plan [plan_id]かどうかを確認する必要があります。 if文を二度書いたので、このコードをもっと美しくする方法はありますか？

Answer 1

はところで、私は===が、この場合には優れている、代わりに==を使用するかもしれないと思う：&& operatorを使用して

if(props.plan[plan_id]) { 
    return props.plan[plan_id].number === 0 ? null : 'some value'; 
    } 

Answer 2

は三項演算子を使用します。

if (props.plan[plan_id]) 
    return (props.plan[plan.id].number == 0) ? null : 'some value'; 

Answer 3

は2 if sからあなたを救うことができます最初のステートメントで終了するため、falseに変換することができます（存在しない場合はprops.plan[plan_id]の値をチェックしようとせず、左側がと評価されます）。）：

return props.plan[plan_id] && props.plan[plan_id].number != 0 ? 'some value' : null; 

Answer 4

なしランタイムエラーがないことを確認するために、私はこれを行うだろう：

return props && props.plan && props.plan[plan_id] ? 'some value' : null 

props.plan[plan_id]が存在していたか、その値は以下のいずれかであるされていない場合はsome valueを取得します。0 、undefined、null、または空の文字列。