私的アンリーダブルコード―他人を発狂させるための 9 のテクニック

コードはたいてい一度しか書かれませんが、何度も何人も読むことになります。 普段何気なく書いているコードが他人の時間と精神を削っているかもしれません。 そんなわけで、個人的に辛いなと思うことを 9 つ挙げてみました。共感してもらえるものもいくつかあるんじゃないかと思います。 実体にそぐわない変数名 見分けの付かない配列とハッシュの変数名 呼び出し元で true/false を指定するだけの引数 暗黙の実行順序 [] メソッドの定義・Array の継承 ハッシュの乱用 密結合した mixin 過剰な nil guard 条件によって異なる返り値の型 推薦図書 静的型付き言語を使うことで解消される問題もありますが、その選択肢はひとまずなしということで。 Ruby 前提になっていますが、他の言語にも言えることも多いと思います。 実体にそぐわない変数名 例えば Vehicle というクラスが定義され

[kabukawa]

言われれば「せやな」だけど、書いてる時は気付かないんだよな。自分の頭の中に有るから。

[gfx]

せやなという感じはする。どれもよくあるパターンなので見かけても発狂はしないけど。

[kent-where-the-light-is]

ありありすぎる。後メソッドのreturnの型が条件分岐で違うとか。

[rokujyouhitoma]

なにこれつらい。そしてRubyだからなのか読みにくいなぁ

[takaesu]

変数名の付け方など参考になる、ハッシュ(hash)やArrayの命名

[rosylilly]

いいアンチパターンだ

[iww]

コメントに書いとくのじゃダメなんかな

[rryu]

すごいわかる。グルーピングによって生成されたハッシュは何によってグルーピングされたかが名前で分らないと後で必ず悩むのでby_hogeと付けるようになった。

[Horiuchi_H]

型の無い言語は大変ですね、としか。

[Youchan]

“[] メソッドの定義”のやつは`element`を返してしまうと`element`の破壊的なメソッドも呼べてしまうからいやというのはあると思う。`[]`定義できるのRubyの良いところだと思うんだけどねー

[koyancya]

極端じゃない、ジワジワくるやつだ

[YassLab]

“「デザインパターン何それ？」な人は「Java言語で学ぶデザインパターン入門」がオススメです。全てのパターンを覚える必要はないでしょうが、Adapter パターン、Template Method パターン、Strategy パターン辺りはよく使う”

[shiro_goma]

だいたいわかるわかる、という感じだけど `見分けの付かない配列とハッシュの変数名` のとこだけそれは良いんじゃないかなぁ(複数形だったら配列orハッシュくらいまではスッと考える)とは思った。

[lumely]

バッドノウハウというか，リーダブルコードだけではなくこのような悪い例も明確化することは有用だと思う

[rlho]

気をつけないといけない

[t-wada]

Ruby で書かれたコードを例に、読みにくいコードをどう直していくかを説明したエントリ。あるある、わかる、というコードが並んでいる。

[tbpg]

こういうエントリを書くきっかけって、こういうコードによるツラミが閾値を超えた瞬間なのかな。関係ないかもだけど

[vndn]

vehicleと呼ばれていたものが別の箇所でrideとかmachineとか呼ばれてたり(問題外

[teracy_junk]

あるある

[tyru]

（わかる）

[tofu-kun]

あるな

[satoshie]

アンチパターン

[maguro1111]

[]や演算子のオーバーライドって誰でも簡単に書ける割にカッコいいから初心者が不必要に使いがち

[ttakezawa]

せやな

[netcraft3]

あるあるすぎる。