■メーカー系SIerの作成したソースコードが酷すぎる

メーカー系SIer（仮にF社とする）が過去に作成したJavaのソースコードを読んでいるんだけど、非常に読みにくい。

適切な単位でメソッドとしてくくりだすことができていない

• 1つのメソッドの行が多すぎる（いわゆる神メソッド）。数百行/1メソッドとかがざら。ほんとにざら。何なら10000行/1メソッドとかがある。読めねーよ。
• 制御文のネストが深すぎる。for{if{if{if{for{if{}}}}}}とかがざら。ほんとにざら。何なら50多重ネストとかがある。読めねーよ。
• 意味単位でメソッドでくくりだしてくれない。if( /* ここに10行くらいにわたってなんかの式が書かれている */ ){}とかがざら。ほんとにざら。

命名が適当すぎる

• 変数名の命名が適当すぎる。boolean型の変数名がisTrueとかがざら。ほんとにざら。
• メソッド名の命名が適当すぎる。メソッド名の末尾に番号を付与して区別している。誰かこのdoSomething1メソッドとdoSomething2メソッドの違いを教えてくれ...。
• クラス名の命名が適当すぎる。謎の文字列が頭に付与されている。A1234Controller.javaとかB3456Dao.javaって何だよ...。

英語ができない

• そもそも言葉の意味を理解していない。hasErrorメソッドの戻り値Trueが「エラーなし」を意味していたりする。真逆じゃねーか。
• 受動態が使いこなせない。selectItemが「選択された項目」を意味していたりする。
• むしろ英語が読み書きできればできるほど読めなくなる。insertUpdateDeleteUser(int i)メソッド（300行/1メソッド）とか。どうしろと。

まとめ

これ読んでメンテナンスする身にもなってくれよ...。

もう疲れた。

Permalink | 記事への反応(3) | 17:29

ツイートシェア