if-then-else文の順番 : 小野和俊のブログ

ペアプロで if-then-else 文が出てきた際、「これ、else if の順序、こっちの方が良くない？」というような会話をすることが時折ある。 どれも当たり前のものかもしれないが、「ああ、確かに」という反応があることもあるので、今日はそんな会話の際に出てくる視点についてまとめてみた。 if (よくあるケース/正常なケース) { // 処理 } else if (比較的特殊なケース) { // 処理 } else if (さらに特殊なケース) { // 処理 } else { // 処理 } 条件式の結果がtrueになる確率が高く、「ノーマル」に近いものを上に書く。可読性が上がる他、特に2.で触れる条件式の判定に時間のかかる場合や、ループの最奥にある処理などのif-then-else文の実行される回数が極めて多い場合には体感レベルで実行速度にも大きな差が出ることもある。 Code Co

[bowbow99]

ノーマルケースの判定って if (!レアケース) になることが多いので大抵はレアケースを先に切って return しちゃうな

[Hamachiya2]

昔はSQLのwhere句とかも順番考えて書いてたわーなどということを思いだしました

[udzura]

一方ルビーストはunlessをうおっなんだおまえらはやめろ

[asakura-t]

↓確かに「!」は見落としやすいので「not」 が使える言語のほうが好き。「! invisible」みたいにスペース空けるようにしてる／perl は冒頭で「return if 例外」とか書けるから分かりやすいよね。気持ち悪いって言われるけどｗ

[FTTH]

ifの評価順が仕様規定に無かったコードは書いたことない（仕事の話）

[ktakeda47]

昔ね、Javaで if (!!visible) って！が２つになっているのに気づかないで、なんでここ入んないんだろーって唸ってる人見たことある。

[mohno]

↓冒頭で if (例外) return; はたしかにある。/「!invisible」←昔は気にしなかったのに、年を取ってくると「invisible == false」とか書きたくなって困る（主に!を見落としそうという理由で）。

[p260-2001fp]

「== TRUE」っと…うっ、い、いや「!= FALSE」…しかし可読性が…いやしかしTRUEがbooleanとは限らないし…とか悩まなくていい所でコード書きたい

[chlono]

1はまあ当たり前だよねと思ったけど、それ以外は人によるという感じだなあ。個性がでるところというか。｜そもそも閉じ括弧と次の行を一緒にする書き方が好きじゃないので…（そこか

[ad2217]

最初は再帰終了条件だろう。

[tmatsuu]

perlでif(condition){}とxxx if condition;を混在させてしまう癖をなんとかしたい。

[kappaseijin]

テストを考えると意味的順序が楽だけど大規模開発は比較式負荷昇順かなあ。

[Argentear]

参考になります

[koogawa]

!invisible とかホントにやめてほしいｗ

[pochi-mk]

だいたいそうしてると思う。他人のソースでコレジャナイ感たっぷりのものを見てげんなりすることもしばしば。

[typista]

ぽけったー 小野和俊のブログ

[t-murachi]

むしろ else if を極力書かずに済むようなコーディングを心がけるべきなんだと思うんだが…

[agw]

5番は何か共感できる。思えば最近は肯定を処理しない書き方をすることが増えた

[holypp]

ごく個人的に、３は好みじゃない。

[airj12]

こういう事を語らいながら仕事したいっす

[e24ns]

if-then-else文の順番 - 小野和俊のブログ

[ngsw]

プログラミングってやっぱり文章なんだなあ、という感想をもった。

[Nan_Homewood]

最近はあまり else if しないようにしてる。

[adsty]

実行速度や意味的順序や可読性等を考慮。

[anakahala]

elseif自体書かない

[kuma606]

確かに当たり前だけど言われてみればそーだな。

[f_oggy]

はてな民はツッコマビリティーに溢れている

[s-tomo]

else節使うってことはif内の末尾でreturnしておらず、else後という離れた位置に続きの処理が書いてあるわけで、その内容は別関数に分けるべき処理じゃないか？という疑問は上げておく。

[nisemono_san]

「ある連想配列のキーを使って値を比べる（無いときはエラー）」のあとに「その連想配列にキーが存在するかどうか」の判定が来たときは震えたなー。

[te2u]

特殊なケースを先行して判定し、できるなら即座にreturnするようにしてる。できるだけelseは使わないように。

[hundret]

ツッコマビリティーwwwwwwwww

[gologo13]

当たり前そうだけど、大事な気配りだよね