プログラムの可読性を上げるための条件分岐を減らす方法7個 - Qiita

Help us understand the problem. What is going on with this article?

[posmaru1]

気持ちはわかるけどelseを書かなくてもいけると思うのはデータパターンを全て理解してると思っているからで、レガシーシステムとか本番でしか存在しないパターンに遭遇した時else書いておけばっていうのは度々遭遇した

[odakaho]

ガードのifにもブレース {} 付けて欲しい派

[dbfireball]

初心者を指導すると分かるけど、一番最初のif2連続よりif〜elseのほうがバグになる率が少ないです。うっかり2つのifに当たる値が入ってくるケースの考慮が初心者は抜けてるのでバグになりやすい。

[nyarou]

ガード節を使う目的は、事前条件を満たしてない場合とか特殊な条件の場合に早期リターンして意図を示しつつメインの処理の見通しを上げることで、そういう視点だと人気コメみたいなケースは出てこない気がするが。

[buhoho]

値を切り替えたいなら三項演算子。処理を切り替えたいならif

[osnq]

ガード節がデバッグしにくいってどういうケースなのか普通にわからんけどなんでそんなにスター貰ってんの？？？ / このブコメにスターつけてんのなんなの

[kkobayashi]

ガード節の例、複数箇所でreturnするとデバッグがやりにくいというデメリットもあるんだよなぁ。何でもかんでもネストを減らせば良いのかと言うと・・・

[manma_shiro]

ガード節を使ってもelseは必要だと思う。/ガードで前提・事前チェックとして利用して、その後本質のif分岐。elseは想定外の条件で入ってきたことを想定してException投げるようにしてる

[ShoCoh]

↓リーダブルコードにも書かれてるんだけど、ガード節を使う(ネストを浅くする)のは「精神的スタック」への負担を軽くする意味があるんですよ、だいたいの人間はブロックの終端を見て条件を簡単に思い出せないので

[Futaro99]

お前らサンプルコードがPHPだから文句言ってんだろ！そうなんだろ！

[hylom]

PHPではブレースなしでのif文は普通なの……？と思ったらやっぱりはてブコメントで突っ込まれていた

[ockeghem]

すごく異論があるわけではないですが、条件分岐はあまり減ってないので、単に「可読性を上げる方法」でよくない?

[nc44v-134-nvtyqfgq]

「条件分岐を減らす方法」とかいう我々の興味を引きそうなワードがタイトルに入っています そしてめっちゃコメントついてました

[t1mvverr]

null合体演算子最近出てきたんだな。実務で使うとIEとかEdge(Chromium移行前)では未対応だから、babelで対応しないといけなさそう。

[girled]

ふーむと流し読んでたけど最後のJavaScriptのNull合体演算子の認識が甘かったので勉強になった

[kaputte]

else が初心者バグをからめ取ってくれるというブコメの指摘を念頭に読むと有用

[chirotec]

あまりにトリッキーな技法に手を出したりひねくれた事をすると大抵後で悲惨なことになるので、シンプルな素直さを忘れないのも大事かと思う（抽象的ですみません）

[sgo2]

「関数化でコードを分割してコンパクトにする」ではなく、「命名できる処理を関数に切り出す」ようにしないと「コードが散らばって読みにくい」だけだし、思わぬ所から参照されて変更に困ったりする。

[maecchi]

elseを書くか書かないかは常に議論されている気がします。

[tumo300-500]

ブクマ付きすぎていて自転車置場の議論感ある (といいつつブクマ

[bocuno]

こういうの見るたびに、馬鹿でも分かる様に書いてもらわないと馬鹿がバグをこしらえると思ってしまうのだが。

[zgmf-x20a]

schemeだったらcall/cc？

[igni3]

よくあるパターンなので見やすくはなるけど分岐数は減らない。

[sasanori-0126]

PHPに限らず、Kotlin,Swiftとか他の言語の扱う人も参考になるだろうね。

[ooblog]

算数で国語クイズすんな≒「演算子を活用したり言語特性を生かすことによって条件分岐を減らす」という話のはずなんだけど、switch文配列化＆default節null合体演算子置き換えにアンチミニマリスト湧きそうなイメージ。

[Lagenaria]

この記事の例くらいの複雑さだと、どちらのコードが可読性が高くなるかは読む人次第になりそう。

[higgsino]

最適解は状況によって変わるので、ちょっと参考にするかくらいの気持ちで使えばよい。重箱の隅をつついてこうするべきあーするべきと言うのはあまり意味がない

[honma200]

Option使ったり、パターンマッチングとかねー。改行と&&使って見易くとかするとか

[assaulter]

なんでこんなバズってるんだろう

[ata00000]

簡潔さと可読性の高さはイコールなのだろうか。自然に読みやすい文とシンプルな文は違うのではないだろうか。申し訳ないけど、言語を使いこなせるようになった中級者、という感じが。全部おかしいとは言わないが⋯

[agano]

短く書くより、可読性をあげる方を大事にしています。ここで書かれてない手法としては関数ポインタかな。

[xjack]

想定している言語が何か，まず書けよ．．．（C++にnull合体演算子は無い）

[ssids]

少なくともGCがない言語ではガード節はちょっと怖いかな

[ene0kcal]

良いエントリやと思う。これは可読性が悪いコードのリファクタリング法と読めばいい。あるいは、初期コーティング時に目的のコードを整理されたものにする為のイディオムと読めばいい。てかM.ファウラーの本のまま。

[quabbin]

うへ。このレベルに反対する人いるのか。驚きだ。最低、Cyclomatic complexityはプロジェクト健全性の指標にしたいものなんだがなぁ(つか、してる)

[rideonshooting]

異常系とは別に、ロジックを通らなくても例外処理はちゃんと必要

[takhasegawa]

多くはマーチンファウラー氏の名著「リファクタリング」に書かれている内容 / 去年の12月に2nd Editionの日本語版が出たよ

[kenzy_n]

anything else ?

[Harnoncourt]

可読性よりも大事なことを忘れていませんかね。