1: 名無し

命名規則に関連するクソコード

クラス名、メソッド名、変数名などのネーミングを誤るとクソコード認定されてしまいます。会社やプロジェクトごとに多少のルールの違いはあるにせよ、どこに行っても漏れなくクソコード認定されてしまうネーミングパターンのご紹介です。

ネーミングが「記号+番号」
クラス名や変数名はわかりやすい名称にしましょう。ネーミングを見て内容を推測できるようになっていることが重要です。「記号+番号」ではそれを見るだけでは何のプログラムであるかを推測することは不可能です。

ネーミングに日本語、英語、ローマ字が混在
プロジェクトによってクラス名や変数名のネーミングルールは異なりますので、何がダメだというわけではありませんが、自由すぎるネーミングを行うのはやめましょう。きちんとプロジェクトでルールを統一することは重要です。

またにクラス名や変数名に日本語を使用することは言語仕様上可能とはなっておりますが、アルファベットを使うことが慣習となっていることと、日本語だとIDEの補完機能がうまく機能しないことがあって非効率化の原因となりますので、避けた方が無難です。

ネーミングにスペルミスがある
ネーミングでスペルミスがあると、後でソースコードから文字列で該当箇所を検索する時に検索にヒットせず、改修漏れの原因にもなります。正しいスペルと間違ったスペルが混在していたりするともう最悪です。スペルミスのないように気をつけましょう。

ネーミングに個人名が使われている
ネーミングはプログラムの中身がわかるような名前にするという観点からも、プログラムの中に自分の名前にすることは適切ではないのでやめましょう。

またソースコードレビューの時に思いがけず恥ずかしい思いをすることになるかもしれません。私は新人の時に「yonemura.sh」という名前で自分用に作ったシェルが他社に買い取られることになってしまい、他の会社のエンジニア20名くらいの前で「よねむらシェルとは・・・」と説明会で大きな声で読み上げるはめになって大変恥ずかしい思いをしたことがあります。

個人で使うプログラムでもプログラムの中身を表した無難なネーミングにしておくことを強くお勧めします。

ネーミングに番号やアルファベットの連番が使われている
クラスや変数のネーミングに、1からの連番やaからの連番を使うと、クラスや変数の中身を推測することが不可能になってしまうのでやめましょう。こういうことをすると後でそのプログラムをメンテナンスする人に、一々プログラムの処理を細かく解析することを強いることとなり、「このクソコード書いたやつまじで氏ね」と言われてしまいますのでやめましょう。

可読性に関連するクソコード
プログラムは後でメンテナンスするためにも、読みやすく書くことが非常に重要です。処理の内容だけ見ると読みやすくても読みにくくても実行される内容は同じかもしれませんが、読みやすいソースコードは改修の工数を下げますし、バグが混入するリスクも下げてくれます。

ネストが異様に深い
ソースコードの中にネストが何重にもなっている箇所があると可読性を下げてしまいます。ネストを何重まで許可するかはプロジェクトによって異なりますが、個人的には3重か4重くらいまでにおさまるようにコーディングするよう心がけていました。

これとセットで「1行の文字数は80文字まで」みたいなコーディング規約があるとさらにカオスな感じになってきます。ネストが10階層+1行80文字までとか、考えただけでも嫌になりますね。

インデントがずれている
今どきエディタが良い感じにインデントしてくれるのに、まさかインデントがずれているソースコードなんて存在しないと信じたいところですが、昔作られたソースコードだとそういう化石みたいなクソコードにお目にかかることはあるようですね。

カッコの閉じ位置のインデントがズレていたりすると、著しく可読性を下げますし、コードの解析を誤るリスクも増えてしまいます。こういうことをすると漏れなくクソコード認定されてしまうでしょう。

1つのメソッドが異様に長い
たまに1つのメソッドが異様に長いソースコードにお目にかかることがあります。私の個人的な感想だと某国にオフショア開発に出されてウミガメのように日本に帰ってきたソースコードにそういうメソッド分割の概念が消失してしまったかのようなソースコードが多いように思います。

1つのメソッドの長さが数千行にも及ぶような男前なソースコードにバグが混入してしまい、解析及び改修をしなければならなくなった時には絶望するしかありませんね。
以下ソース
https://axia.co.jp/2018-04-27

9: 名無し

>>1
マイクラのjavaソースなんて一発目から該当するけどなw

2: 名無し

プロパティを自分で作れるのを知ったのが
IT土方5年目

3: 名無し

今まで見た一番難解だったプログラムは、
GOTO文でデータ部にジャンプして、そのデータ部が実はマシン語だったケース。

8: 名無し

>>3
おう、すごいな。
プロテクト外し対策だな。

4: 名無し

1行目がGOTO文なBASICプログラム

5: 名無し

10 50ニイケ

6: 名無し

メンテナンスなんてしなければいいじゃない320

7: 名無し

ITドカタの世界なんて知ってもなぁ

10: 名無し

YAMAUCHI

11: 名無し

漢なら数万行のスパゲッティ!

12: 名無し

本文よりもコメントの方がやばいなw

13: 名無し

昔は日常会話の様にマシン語で会話してるヤツが居たんだが、彼らはどこ行ってしまったんだ?

14: 名無し

おかしいな~、変だな〜、こわいな〜

15: 名無し

16: 名無し

ユニットテストのメソッド名とかに日本語を使うと意外とイイ

17: 名無し

スミマセン、クソこーど作ってしまった自覚はあります

18: 名無し

氏通とかNTTとかひたちとかああいうところが一発目から該当するな
そういう話を聞いただけだがそうだろうな

19: 名無し

>私は新人の時に「yonemura.sh」という名前で自分用に作ったシェルが

>他社に買い取られることになってしまい

おいしいシチュエーションじゃん
仕事回ってくるかもよw

20: 名無し

スペルミスは問題ないよ
全てがミスで統一されていれば

21: 名無し

if(a){while(a){}}

22: 名無し

これより下にコードを書くな

23: 名無し

どうでもいいわ。

24: 名無し

creat関数

25: 名無し

1行目のコメントに「触るな!!!」と書かれている

26: 名無し

rm -rf .*で全部消えるってほんと?

27: 名無し

>>26
ああ、有名な都市伝説ね

28: 名無し

秀丸って本名?

29: 名無し

//よく分からないけどこれで動く

30: 名無し

虫取り用

if 1=1 then go to 10