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

受託開発エンジニアでプロジェクトリーダーという立場になるにあたって品質保証について考えるようになりました。 品質と一口に言っても「内部品質」と「外部品質」の2点がありますが、エンジニアである以上、内部品質の向上が外部品質を上げる最適解だと思います。 ※内部品質、外部品質についてはこちらを参照。 そして内部品質を上げるためにはアーキテクチャを考えることが大切ですが、既存プロジェクトの場合はアーキテクチャが 存在しないレベルでぐっちゃぐちゃ 十分に検討されていないケースも多いです。 そこで、一番手っ取り早くプログラムの可読性を上げる方法は、新規に書くコードをできるだけシンプルに実装することです。今回はその中でも個人的に使っている条件分岐を少なくする or 見やすくするテクニックについてまとめてみました。

[posmaru1]

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

[odakaho]

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

[dbfireball]

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

[sai0ias]

経験が浅い人とベテランの言う可読性って結構差があるよなぁと言う印象ある。

[nyarou]

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

[osnq]

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

[buhoho]

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

[onesplat]

プロジェクトリーダーがif文でグダグダ言ってるようなプロジェクトは駄目だと思うのね

[kkobayashi]

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

[manma_shiro]

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

[fellfield]

自転車置場の議論だとは思うけど、OSSのコード読んだりしてると、この水準のコードでも書けていない人は多い（もちろん俺も怪しい）。定期的に話題になるのは良いことだ。

[maecchi]

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

[souiufunidekiteiru]

割と上司を説得するのが1番得策な気が

[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]

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

[kougekiyou]

デバッグしづらい、のかな？確かにprint debugなら一箇所で済むかもしれないが。一個の関数に大量にreturnがある場合は辛いと思うが。

[takhasegawa]

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

[ShoCoh]

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

[kenzy_n]

anything else ?

[Harnoncourt]

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

[cozyconemotel]

可読性なんて人それぞれ、社内でcoding styleが確立していて認知されていれば問題ない

[colonoe]

読みやすさを重視しすぎて、何をしてるのかわかりにくくなってる気がする

[celaeno_w]

シンプルなことと、「可読性」が上がるかどうかは別で、true/false や配列の例は、悪い例の方が「可読性」は高い。

[yaboot]

オブジェクト指向教、三項演算子クソ教、switch大好き教、PHPクソ教

[NOV1975]

分岐を減らすというのが必ずしも可読性を上げるわけではなく、そういった例も入っちゃってる気がする。