else を禁止してはいけない - Qiita

複雑なプログラムを書いてしまうのを避けるために、if 文の else を避ける制約を設けてプログラミングしてみましょう、というアプローチがあります。これはあくまで、ThoughtWorks アンソロジーに書かれていた、ある種のプログラミング思考のトレーニング方法のひとつにすぎないものです。 しかし、権威だからか、あるいは、あまりコードを書かない人がわかりやすいと感じるからか、プロダクションコードの規約に使おうとする誤った考えがあります。そんなことをすると、コード品質は下がります。 else 禁止制約というのは、「これまで if-else で記述していた多くのコードは、実は、生成するオブジェクトの多態性によって、分岐の前倒しが可能になるものだったのか」という認識を得られれば、すぐに忘れるべきルールです。 ある実在の OSS コードを例に説明します。 https://github.com/xr

[tattyu]

最初の例だとそもそも1番上の条件を反転して早期returnにして、ネストを浅くする様にするかな。ifのネストがそもそもバグを仕込み易いので避ける様にしてるわ。

[strawberryhunter]

元はelseの中で必ずreturnするならelseのブロック化は不要だという静的解析ツールの指摘だったんじゃないかと思う。プロジェクトの決め事で現代のプログラミング言語の文法を制限しても良い結果につながらない。

[n314]

これはelse無しの方がシンプルなので例が良くない、と思ったらコメントに既に書かれていた。

[xlc]

大手SIer様の規約にwhere句でのOR禁止というのがあって仰天したことがあった。

[kako-jun]

例がポリモーフィズムと関係ない？elseなしのほうが美しいじゃん？phpワケワカラナイ……と思ったけど、ほかの人もそう思ったようで安心しました

[arx0balest]

まずPHPを禁止した方がいいと思うよ。というかPHPってまだ使ってる人いたんだ。

[uxoru]

？？elseがどうかというより、ネスト浅くすればええという話やないの？🤔

[wwolf]

elseを禁止〜という主張の為にあえて早期returnを封印するのは不誠実な気がする

[su_zu_ki_1010]

スト２（古い）の波動拳みたいなif文のネストさえ書かなければいいですよ。読みやすさは常に模索していきましょ。

[natto21]

例題はちょっと微妙なので元記事コメントで反論されているが、主張そのものには賛成。別のコメントにもあったけど、SIIerは馬鹿げた規約を作ったりしがち。それがどれだけ末端を苦しめているか、PMは考えるべき。

[daimaruonline]

あまりメリットを感じないのは、例がよくないからな気がする

[Wafer]

きっしょ！きっしょ！else使って書くとreturnで返さないケースが実際はどんなパスもたどり着かないけどあるようにみえるやんけ！気持ちわる！一番最初の例が一番ええコードやん。最初以降は目つぶって書いたんけ？

[yamadar]

オライリーの ThoughtWorks アンソロジーに出てくる「elseを使わないで書いてみよう」はあくまでトレーニングであって、実務では目的に応じて適切にelseも使いましょうという記事。

[mak_in]

プログラミング「言語」というくらいなので、同じ内容を英語や日本語にしても短く簡潔に表現できるようにロジックを作ることを心がけてる。それが命名だったり、ネストだったり、if条件の表現だったりするわけで

[suika3417]

Factoryなら生成するためにどっかに列挙型みたいなの用意してそれに準拠してnewしてほしいな…（意図が違う）

[sgo2]

例えば後からtry〜catchが追加されて、returnで抜けるはずのフローの途中で例外発生→elseのフローに突入というバグが仕込まれる可能性もあるので、開発メンバーの信頼性次第ではきちんとelseを付けた方が良いです。

[fukken]

適切に反転して順番を入れ替えれば単独ifでも問題ないところ、自説の説明のためにその辺を誤魔化しているのが邪悪。嘘つきは悪いコード以上に有害。

[Knoa]

switch

[syuu256]

“ガード節”とかポリフォーリズムの例は無いのか？

[zentarou]

大爆笑 “return のインデントが揃っていません。”

[dozo]

elseなしの意味が解ってない記事

[Guro]

（こういう（説明文章の）書き方はよいな）