「悪〜いコード」を読んだので、ついでにコードメトリクスを計測してみた - Qiita

はじめに 先日、「悪〜いコード」を読む機会がありました。 どんな風に悪いのか、軽くですが分析してみたので、ポエムとして投稿したいと思います。 古のコード 私は普段Ruby on Railsをメインに開発を行っているのですが、ユーザーからの質問に答えるために、普段の開発や保守しているのとは全く別のシステムのコードを読む機会がありました。 そのシステムはPHPで書かれた古いコードでした。ユーザーの質問はシンプルだったので、コードを見れば一瞬で答えは見つかるだろうと思ったのですが、とても読み難いコードだったので30分ほど頭を悩ませながら読むことになりました。 何が読み難いのか 結果、ユーザーからの質問には答えることができたのですが 「僕の30分を返してくれーーー！」と叫びたい気分です。 と愚痴ってしまいましたが、それだけでは何の進歩もないので、何が読み難かったのかを明らかにしてみたいと思います。

[peperon_brain]

すごい行数・すごいネスト・巨大関数なコード見ると「はぇ〜、こんな複雑なコード書いて挙動把握できるなんてめちゃくちゃ頭いいんですねぇ〜」って思う。

[yarumato]

“PHPで書かれた古いコードが読み難い。コードメトリクス（コードの質を評価する測定基準）をツールを使って計測。「ネスト11段」で破壊力がわかる。コードメトリクスで指摘された値を閾値以下に抑えるよう修正”

[euclidean]

悪いまま読まなくてもChatGPTに投げたらよしなにしてくれるんじゃなの？

[pwatermark]

昔はそれで良かったんだよ それでマズイと気づいた人たちが一生懸命整理して、今に繋がってるんだよ....

[tinsep19]

この複雑さを30分で読み解いたの？

[hogeaegxa]

「実現したいものを上から順番に行っている」コードってのは、関数とかで分割して切るより、ただ上から下に読むだけでいい分理解してる本人的にはむしろ可読性高く見えるんだろうな。まあ、忘れた後は言わずもがな

[ducktoon]

PHPの時点で察しよう

[pmint]

同じパターンのネストや繰り返しならいい。コピペで書かれたコードは読みやすい。テストのカバレッジは上げられなくなるけど。分割すりゃいいってのは、初心者が考えがちなリファクタリング方法。分割はモデリング。

[kagehiens]

ネスト11段はヤバいなと思いつつ、この半分ぐらいの規模(長さが半分で複雑性が1/2乗程度)なら全然普通に見るよなぁ？とも思った。しかし、うん・・・バグ出さずに改修するのは至難の業やろうな。。。

[fukken]

「リファクタリング」を良もう。本来の意味での「リファクタリング」は、どんなコードでも安全に変更を加える方法で、そのための具体的な機械的操作が書かれている。

[rryu]

NPath複雑性がやばすぎる。どれだけ分岐を入れればこの数値を叩き出せるのか…

[yamadar]

これは酷い。ポエりたくなる気持ちにもなるな

[buhoho]

PHPの日常

[hatest]

架け子、受け子、出し子、入れ子（ネスト）は、特殊詐欺だと思え

[syu-m-5151]

お前も悪いやつやでぇ…