関数内で複数の return文 は避けた方が良いか？

Question

関数内で複数の return文 は避けた方が良いか？

以下の様な関数は可読性から考えて不適切でしょうか？

var someFunc = function(){
  if('someCond1'){
    return false;
  }

  if('someCond2'){
    return false
  }

  return true;
}

Answer 1

「条件を満たしていないことがわかったら早めにリターンする」というのは、ガード節と名前もついているぐらいに定着している技法です。

Comments

[foobar810]

回答ありがとうございます。
ガード節、知りませんでした。勉強になります。

Answer 2

闇雲に return を書くのはダメですが、処理を簡潔するために使用するガード節はむしろ推奨されていると思います。

コーディングのための定番本「リーダブルコード」でも紹介されていますね。

Comments

[foobar810]

回答ありがとうございます。
リーダブルコード買ってみます。

Answer 3

私もよく書きますし、例外をthrowする場合も同様のロジックなので
悪くはないと思いますが、処理をぶった切る感もあるので
戻り値を変数にうけておいて、最後に戻すというやり方も嫌いではありません。
とくにコールバックを多用していくと関数から意図した戻り値を
吐いていないバグに繋がりかねないので、関数の戻り箇所は絞ったほうが
管理がしやすくなるかもしれません。

Comments

[foobar810]

ご回答ありがとうございます。

> 悪くはないと思いますが、処理をぶった切る感もあるので
質問時点でこの点が気になっていたのですが、確かに例外のロジックでも使われているので、アリなのかなと思う様になりました。

> 戻り値を変数にうけておいて、最後に戻すというやり方も嫌いではありません。
どちらにするか、いつも悩んでいるのですが、やはり状況に応じて選ぶのが良いのでしょうね。

Answer 4

好みやプロジェクトによるので何とも言えませんが、私はよくこういった書き方します。