あえてelif, elseを使わない条件判定はありですか？

Question

[追記]
以下の入力のバリデーションを追記しました。

if not isinstance(x, int):
    raise Exception("Input must be int")

---

以下は単純な例ですが、

例えばこのような関数を

def test(x):
    if not isinstance(x, int):
        raise Exception("Input must be int")

    if x < 0:
        print("x is negative")
    elif x <= 100:
        print("x is positive and not too large")
    else:
        print("x is too large")

以下のように書くのはありですか？(または、一般的に受け入れられていますか？）

def test(x):
    if not isinstance(x, int):
        raise Exception("Input must be int")

    if x < 0:
        print("x is negative")
    if 0 <= x <= 100:
        print("x is positive and not too large")
    if 100 < x:
        print("x is too large")

このように書くことのメリット（と私が考えること）は、

• 条件が対等に見えることによる可読性の向上

考えられるデメリットとしては、

• 条件の抜け漏れ・オーバーラップなどが起こりやすくなる
• 実行時間が気持ち長くなる(可能性がある？）

これらの問題が大きく影響しない場合には取り入れられる書き方でしょうか？

Answer 1

ありかなしかであれば、言語的にはありですが個人的にはナシだと思います。

if 〜 elif 〜 else 〜 は、処理がその3つのうちのいずれか一箇所のみを通ることがわかりやすいですが、if 〜 if 〜 if 〜　と書くと3つのifを複数通るパターンがあるのではないか？という疑念が湧くからです。

今回のパターンであれば、複数ヶ所を通過することはありませんが、条件まで確認しないとそのことがわかりません。数カ月後の自分は他人ですので、ヒネった書き方をすると後でなんだこれ？となりますのでおすすめできません。（経験上）

Answer 2

私は受け入れません。理由は以下です。

• 同じ判定を何度もしているのは気持ち悪い
• 網羅性・重複チェックを読む人がしなければならず可読性が落ちる

Comments

[maisumakun]

言語によっては、返り値が必要な関数で全部をifで書いてしまうと「値を返さない可能性があります」とコンパイルを通らないこともありますね。

[Zuishin]

そうですね。今回は返り値が必要ないんでしょうが、逆に返り値が必要ないことがバグを気づきにくくしていましたね。

Answer 3

以下のように書くのはありですか？

この場合、動作が異なります。x = nanのような変な値を投げ込まれた場合、NaNは大小比較できない（結果がすべてfalseとなる）ので、elseを使わずに書いたパターンではどれにも当てはまらず、何も返さないことになります。

Comments

[maisumakun]

なお、今回の例のように各項目が1行しかないような場合、「条件判定をして、満たしていれば必要な処理をしてreturn」とすることでelseを書く必要をなくす、という流儀も存在します。

[maisumakun]

（回答後に質問のコードが修正されています）

[nouken]

すみません、自分の中では本題ではなかったので修正しましたが、結局このようなことが起こる可能性が高いということ自体が問題かもしれません。

[sazi]

> 「条件判定をして、満たしていれば必要な処理をしてreturn」とすることでelseを書く必要をなくす、という流儀も存在します。

処理時間短縮の為でしたよね。流儀に名前ってありましたっけ？

[Zuishin]

> 流儀に名前ってありましたっけ？

この質問の回答にもいくつかありますが、「早期リターン」じゃないですか？
私もこの質問を見て真っ先にそれを思い浮かべたんですが、

> 条件が対等に見えることによる

とあったので、それをメリットと思うなら、その「メリット」が無くなるため、外しました。

[sazi]

@Zuishinさん
「早期リターン」聞き覚えがありました。
ありがとうございます。

Answer 4

質問のコードとは違いますが、"return early pattern"や"早期リターン"という名前が付いて議論されている概念が参考になると思うので、検索してみて読むとよいと思います

def test(x):
    if not isinstance(x, int):
        raise Exception("Input must be int")

    if x < 0:
        print("x is negative")
        return
    if x <= 100:
        print("x is positive and not too large")
        return
    print("x is too large")

Answer 5

すみませんが何とも言えません。
詳しくは会社のルールやチームのルールに従って下さい。

ちなみに私の場合は下記のような早期リターンを使った書き方を好んで使います。

def test(x):
    if x < 0:
        print("x is negative")
        return

    if x > 100:
        print("x is too large")
        return
    
    print("x is positive and not too large")

Answer 6

「あえて」ならよろしいのではないでしょうか。

上記のコードは動作が異なりますので、「可読性向上のために置き換えた」とコードレビュー等で説明されたらNGをだします。
こういう問題が起こる可能性があるならelif/elseを排除する意味は無いと思います。
一般的にはどうかはレビュワーなどによると思いますが、個人的には受け入れられません。

ifしか無いコードを見たことがありますが、else書くより面倒なことをしている印象があるので、やめた方がいいんじゃないですかね。

Answer 7

条件が対等に見えることによる可読性の向上

を重視したのか、追加削除の考慮なのか、

if False:
    pass
elif ～～～:
    ～～～
elif ～～～:
    ～～～
elif ～～～:
    ～～～

のようなコードを見た気がします。Pythonだったかどうか覚えてませんが。

Rubyだと、case+when一択ですね。