条件文の可読性を考慮した適切な書き方

Question

条件文の適切な書き方

以下の条件文は可読性の上でどちらが適切でしょうか。

• [1] 条件式の結果を変数に格納し、if文の条件とする
• [2] 条件式をそのまま、if文の条件とする
• [3] その他

var func1 = function(apple) {
  var appleExists = typeof apple !== 'undefined';
  if(appleExists) { // [1] 条件式の結果を変数に格納し、if文の条件とする
    // appleの処理
  }
}

var func2 = function(apple) {
  if(typeof apple !== 'undefined') { // [2] 条件式をそのまま、if文の条件とする
    // appleの処理
  }
}

Comments

[m.ts10806]

目的はundefinedの判定ということでしょうか。
挙動が同じとういことでしたら何を持って「適切」と質問者さんは判断されるおつもりですか？

[foobar810]

ご指摘ありがとうございます。
本文には「可読性の上で」と書いていたのですが、念のためタイトルにも追加しました。

Answer 1

arguments.length

ケースバイケースてすが、質問文のケースなら、arguments.length を私は使います。

function sample1 (apple) {
   if (!arguments.length) {
     return;
   }

   // apple の処理
}

なお、「早期リターン」にするか、「関数の出口を一つにする」かは、ポリシーに依るところが大きいので、自分に合ったものを選ぶ事が重要と思います。

undefined 判定

歴史的に undefined はキーワードではない為、ローカル変数 undefined を定義する事が可能てす。
その為、typeof演算子やvoid演算子で確実に undefined を判定する考え方が生まれています。
typeof演算子は ReferenceErrorを回避出来ますが、void演算子は回避出来ません。
どちらでも良いケースなら、好きに選べば良いと思います。

Re: foobar8 さん

Comments

[foobar810]

ご回答ありがとうございます。
typeoof演算子、void演算子の仕様について、わかってない点が多いので、勉強してみようと思います。

Answer 2

以下の条件文は可読性の上でどちらが適切でしょうか。

場合によります。

「同じ条件を複数箇所で使い回す」なら変数に入れる一択ですし、条件式が極端に長い場合も一度変数に入れたほうが可読性が増します。

一方で、typeof apple !== 'undefined'のようなシンプルな式の場合は、変数に入れるまでもないでしょう。

---

本題ではありませんが、appleという変数が存在する場面でtypeof apple !== 'undefined'と書くのは冗長です。apple !== undefinedもしくはapple !== void 0で十分です。

Comments

[foobar810]

ご回答ありがとうございます。
> 場合による
そうなのですね。
決まった書き方があればと思ったのですが、状況次第なんですね。

> apple !== void 0
この表記は初めてみました。参考になります。

Answer 3

書き方ということであれば「ヨーダ記法」ってのもありますねｗ
適切かどうかは別として。

Answer 4

const func1 = (...args)=>{
  if(args.length>0){
    console.log(args[0]);
  }else{
    console.log("empty");
  }
};

func1();
func1("hoge");