右寄りのプログラマを粛正する。 - こせきの技術日記

まずこれを10回読もう。 うねりコード - 【アンチパターン】遠くから見ると、画面上でコードがうねっている。 コードが2段階以上右に寄ったら、右足はウンコ踏んでると思いましょう。無闇とエディタの横幅を広げるのも良くない。広いディスプレイは甘え。 つぎこれ。 ガード節による入れ子条件記述の置き換え - ぐるぐる～ function foo() { if (cond) { : : : : } }こういうのは最低です。下のように書きます。 function foo() { if (! cond) return; // ガード節で、さっさと帰らせる。 : (正常処理) : }無駄にインデントが深くなりません。意図が明確になります。 下のようなコードも良くない。 function foo() { if (condA) { : : } if (condB) { : : } }右に寄りすぎ。メソッドを分

[lizy]

保守的って意味か、あるいはネトウヨの事かと思ったら全然違った

[tyru]

「出口は一つに」の信条のもとガード文を極端に嫌う人たちは思考停止に陥ってると思う / 右寄りが保守的って意味に見えたからってわけでは決して(ry

[tofu-kun]

ネトウヨプログラマかと思ったら違った。でも、見て良かった。

[trashtoy]

自分も同じ考えだ。「returnは1つだけにしないと処理が追いにくくなる」なんて意見をたまに見かけるけど、特殊な条件をサクっとreturnさせたほうがずっと読みやすい。if文が多くなったらStrategyパターンを検討する。

[Jxck]

if で早期/即 return するスタイル=ガード節 この名前をいつも忘れる。(フィルタ？だっけ？となる)

[invent]

右寄りのプログラマを粛正する。 - こせきの技術日記

[sr10]

ガード節って、初心者にはわかりにくい気がする。でも慣れるとすごく見やすい "右寄りのプログラマを粛正する。 - こせきの技術日記" -

[ledsun]

returnじゃなくて例外投げた方がいいよ（cとかjsとかかな？）。二つ目の例はelse if じゃないなら間に一行開けてほしい。詳細によるけど、この手のは「メソッド分けて呼出し元で分岐」に直すことが多い。

[regicat]

ifは整理すればネスト防げるけど、多段ループは悩む。データ自体が階層構造になってるのを逐次処理する必要がある場合とか。素直に考えるとどーしてもｆｏｒｅａｃｈのネストに……

[r-west]

ごもっともで自分もそうしてる、けど最近前者は手続きチックでちょっとアレな感じという気もしている。

[habuakihiro]

俺はインデントの深いコードを「女の腐ったような腰をナヨナヨクネクネしならせてるようなコード」と呼んでる。男は竹を割った如くまっすぐ！

[masutaka26]

『コードが2段階以上右に寄ったら、右足はウンコ踏んでると思いましょう。』