javascriptで値を返す時ネストのスタンダードな書き方は？

Question

###前提・実現したいこと
Javascriptで値を返す時ネストのスタンダードな書き方を知りたいです。

###該当のソースコード

// 1.
function hoge (va) {
  if (va) {
    some();
    return true;
  } else {
    some();
    return false;
  }
}

// 2
function hoge (va) {
  if (!va) {
    some();
    return false;
  } else {
    some();
    return true;
  }
}

// 3
function hoge (va) {
  if (va) {
    some();
    return true;
  }
  some();
  return false;
}

// 4
function hoge (va) {
  if (!va) {
    some();
    return false;
  }
  some();
  return true;
}

最後に

上記の 1 - 4 のうち スタンダードなのはどれなのでしょうか？

Answer 1

コードが抽象的すぎてなんとも言えませんが、読みやすいコードは4でしょうか。

最初のほうで問題があればすぐreturnしてしまうという書き方をすれば、そこから後の主要な処理はではその問題を考慮せずに読むことができます。インデントも浅くなりますしね。

「ガード節」や「早期リターン」という名前が付いている書き方です。

Answer 2

vaの型がわかりませんが、ifの分岐をみるにその内容なら
vaを最後に返すだけでは？

function hoge (va) {
some();
return va;
}

vaの真偽ごとのsome()が実はsome1(),some2()と違いものなら分岐が必要ですが
それにしても、戻り値は所詮vaですよね？
if(va)もホントはif(vaをなんらかのチェック)という制御がはいっているのでしょうか？

Answer 3

if 文内の条件がそのまま返り値になるのですから、次のようなコードであれば、

function hoge (va) {
  if (va) {
    some();
    return true;
  } else {
    some();
    return false;
  }
}

次のように書き直せます。

function hoge (va) {
  some();
  return Boolean(va);
}

Re: SorrowBlue さん

Comments

[think49]

質問にあるコードが変わった為、回答したコードも修正しました。
return Boolean(va); で良いと思います。