■ダメなコードを見ていて疑問に思うこと

数百行、果ては数千行もある関数やメソッドが何故生まれるのか、どうしても理解できない。それ仕様の通りに動かそうと思ったら、テストもデバッグもライフワークになるよね？それとも未完成でも納品するってこと？もしかしたら「勝手に関数/メソッド作るな」司令が出てるのかもしれないけど、だったら「できません」と断ったほうが絶対楽だと思うぞ。あ、楽といえばこういうコードのレビューは楽だよ、「長すぎて読めない、書き直し」で終わるから。

条件分岐やループを何重にもネストしたコード。スクロールさせるとうねって見える。それさ、正常系の処理だとして一番最後の行の直前はどこ通るの？即答できないなら書き直せ。即答できても「こういう複雑なコードを書けるのがプロ」とか誇らしげな表情しちゃう人はただの勘違いバカだから、その姿勢改めるまで可愛がられるかサクッと見捨てられるかでしょう。

ANDやORが入り組みまくった条件式だけどさ、お前一体何がしたいの？それ多分お前しか理解できないから、システムがEOLになるまで面倒見てあげてね。

「1からnまでの総和」とか、公式がありそうなものをロクに調べもせず、n回ループする方法しか思いつきませんでした的に書いたコード。そんなバカ丸出しの実装で恥ずかしくないの？バカはプログラマに向いてません。それは前述の勘違いバカもそうだけど、頭を捻るセンスが無いのも致命的。

閏年を始めとする日付の計算アルゴリズムとか、標準提供されていそうなものまで手で実装したコード。お前それ辛くないか？てかマニュアルやヘルプはちゃんと読もう、な？

なんで同じコードをコピペするの？共通ルーチン作らなくてもgrepで直して回れば修正漏れも起きないって魂胆？同じ処理が出てくるたびに読んでてうんざりしてくる人も少なからずいることを理解して仕事してください。頼むよ。

DBのテーブルから全レコード取得して、必要なデータか判定して処理するコード。WHERE句って知ってる？データが数億件とかあってもそれでいいと思ってる？

その処理は本当にコードで書かないとダメなの？書かないで済みそうなことまで何故書くの？書かなきゃいけないとして、どうやって短く書くか工夫しないの？コードが増えただけバグも増えることを、もっと深刻に受け止めて欲しい。

最後にコードじゃないけど、巨大なディスプレイに高解像度で極小フォントという環境で開発ってどうなのよ。そこまでしてコード全部表示しないと書けないんだ？今まさに書いている部分なんて、変数もアルゴリズムも自分の頭の中にあるんだから画面に表示する必要なくね？確かに開発用のディスプレイは大きいものが複数あったほうがいいけど、それらはそういう使い方をするためにあるものじゃないと思うぞ。

Permalink | 記事への反応(4) | 13:22

ツイートシェア