ニュー速vip

やばいプログラムにありがちなこと

元スレ

1 ：風吹けば名無し：2022/06/23(木) 12:52:25.05 ID:lVdor4Ef0.net
ルーチン名メソッド名と明らかに違う処理が組み込まれている

64 ：風吹けば名無し：2022/06/23(木) 13:25:58.70 ID:jHKU6X/F0.net

コメントに愚痴書いてあるやつ

56 ：風吹けば名無し：2022/06/23(木) 13:21:58.30 ID:4Ynq6eQG0.net

あっちこっちで書き換えられるグローバル変数

61 ：風吹けば名無し：2022/06/23(木) 13:24:35.08 ID:keiElekk0.net

>>40
テーブル名もカラム名も日本語でつけたろ！

36 ：風吹けば名無し：2022/06/23(木) 13:05:52.89 ID:hZ3B4thVM.net

vscodeなら変数名一文字だったりローマ字だったら大体ワーニング出るやろ…

65 ：風吹けば名無し：2022/06/23(木) 13:26:14.81 ID:lVdor4Ef0.net

TORIKESI_PROC

35 ：風吹けば名無し：2022/06/23(木) 13:05:04.95 ID:lVdor4Ef0.net

>>34
作ったやつのこめかみぐりぐりしたくなる☺

76 ：風吹けば名無し：2022/06/23(木) 13:32:18.68 ID:dlyycOvu0.net

>>55
再帰処理を固定回数でこなす
樹形図みたいな内部分岐
とかか？

20 ：風吹けば名無し：2022/06/23(木) 12:59:07.03 ID:Cm13xirza.net

final String nullStr = “null”;

32 ：風吹けば名無し：2022/06/23(木) 13:03:50.86 ID:lVdor4Ef0.net

_tmp
_work
↑
これにさらに連番つけるやつ

43 ：風吹けば名無し：2022/06/23(木) 13:11:27.57 ID:lVdor4Ef0.net

>>42
あああああああああああああ

71 ：風吹けば名無し：2022/06/23(木) 13:28:19.33 ID:pSUw4iIR0.net

>>63
WordPress使うやつはそもそもコード書けないからあんまり関係ない

59 ：風吹けば名無し：2022/06/23(木) 13:23:35.34 ID:4Ynq6eQG0.net

>>32
5行ぐらいのメソッドならまだ良いけど実際は100行以上とかざら

25 ：風吹けば名無し：2022/06/23(木) 13:01:11.06 ID:r9DiuZ2kd.net

クラスがオブジェクトと乖離している
もうただの便利な変数入れ

40 ：風吹けば名無し：2022/06/23(木) 13:08:21.50 ID:GlPoB4Gld.net

>>34
実際のDB定義とDB定義書が違うのとかも下手したら死人が出るな

57 ：風吹けば名無し：2022/06/23(木) 13:22:02.99 ID:j7EJJpY/0.net

本当にパスタ

63 ：風吹けば名無し：2022/06/23(木) 13:25:57.45 ID:Jxmk7Swi0.net

>>53
WordPress が存在しない世界線では使われなくなってるんやろか

28 ：風吹けば名無し：2022/06/23(木) 13:02:06.76 ID:pSUw4iIR0.net

>>27
異常事態が日常になっとるんやで

3 ：風吹けば名無し：2022/06/23(木) 12:53:19.44 ID:GWlwfveMd.net

変数名が1文字

52 ：風吹けば名無し：2022/06/23(木) 13:17:38.63 ID:lVdor4Ef0.net

>>50
アホみたいに乱立するコメントアウト

62 ：風吹けば名無し：2022/06/23(木) 13:25:42.18 ID:1GPQbtW2M.net

プログラミング言語によってグローバル変数の挙動違うのどうにかならんの？
テストするのめんどくて全部グローバル変数とかにしたらデバッグで偉い目にあった

69 ：風吹けば名無し：2022/06/23(木) 13:27:27.95 ID:0q2GQWVua.net

テスト簡単にしようって発想がない時点でプログラマーの素質ないから辞めろって言いたくなる

11 ：風吹けば名無し：2022/06/23(木) 12:55:38.81 ID:lVdor4Ef0.net

変更履歴だけで1万行オーバー

85 ：風吹けば名無し：2022/06/23(木) 13:38:39.05 ID:bj1ypYlWd.net

if(xxx = true){
}else{
　～～～
　～～～
}
↑処理無いほうも何故か書くやつあほしね

68 ：風吹けば名無し：2022/06/23(木) 13:27:11.00 ID:LONcbVcr0.net

private変数…？参照できないし不便だから全部public staticにしたろ！ｗ
こういうノリが実在するのを見て眩暈がした

22 ：風吹けば名無し：2022/06/23(木) 13:00:40.52 ID:lVdor4Ef0.net

マスタ設定で柔軟な作りにしようとしてるのにプログラム側でガッチガチにロジック作り込んでるのとか相当たち悪い

2 ：風吹けば名無し：2022/06/23(木) 12:53:03.94 ID:inTNzTLt0.net

よくわからないけど動く

10 ：風吹けば名無し：2022/06/23(木) 12:54:59.95 ID:7akmhrA8M.net

5000行超えの関数

50 ：風吹けば名無し：2022/06/23(木) 13:16:37.26 ID:JgNDhGHr0.net

1クラス2万行
まじで意味わからん

45 ：風吹けば名無し：2022/06/23(木) 13:12:23.38 ID:/OqEQQdJ0.net

アカン…このシステム2100年にエラー起きるわ…まあそれまで使わんしええやろ！

83 ：風吹けば名無し：2022/06/23(木) 13:36:25.40 ID:0q2GQWVua.net

public staticは無駄にメモリ食うやろ

39 ：風吹けば名無し：2022/06/23(木) 13:07:34.51 ID:lVdor4Ef0.net

>>37
組んだやつに逆にどうしてこれでうまく行くと思ったと聞いてみたい

39 ：風吹けば名無し：2022/06/23(木) 13:07:34.51 ID:lVdor4Ef0.net

>>37
組んだやつに逆にどうしてこれでうまく行くと思ったと聞いてみたい

41 ：風吹けば名無し：2022/06/23(木) 13:09:10.65 ID:r9DiuZ2kd.net

>>37
これを参照渡しで誤魔化してやってるやつマジで糞

89 ：風吹けば名無し：2022/06/23(木) 13:41:09.62 ID:SpHCOlWv0.net

使ってねえ変数放置

70 ：風吹けば名無し：2022/06/23(木) 13:28:09.41 ID:c+MN0sKB0.net

コードが複雑になるときは設計からやり直せほんとマジで

66 ：風吹けば名無し：2022/06/23(木) 13:26:59.95 ID:pSUw4iIR0.net

ラムダ式使うメリットがいまだにわからん
別の行列にforループで移植させたら何があかんのや？

33 ：風吹けば名無し：2022/06/23(木) 13:04:03.05 ID:Hb4XNQuda.net

解凍した段階で何故かエラーある

38 ：風吹けば名無し：2022/06/23(木) 13:07:28.78 ID:kx7EdqiS0.net

どこ検索しても代入されていない変数の値が変わっている
メモリ壊してる？

37 ：風吹けば名無し：2022/06/23(木) 13:06:37.14 ID:B5R5l9E9M.net

4ネスト以上のループ

73 ：風吹けば名無し：2022/06/23(木) 13:29:06.44 ID:nG/HQ30gM.net

>>66
そうやって何万行ものコードが生み出されていくんやろ

12 ：風吹けば名無し：2022/06/23(木) 12:55:47.16 ID:8VNDOfyNd.net

//↓消すとなぜか動かない

67 ：風吹けば名無し：2022/06/23(木) 13:27:01.84 ID:7i6vYvyc0.net

>>61
とはいえ訳せない業務用語を直訳するのやつもつらい

49 ：風吹けば名無し：2022/06/23(木) 13:15:35.64 ID:rxZUcgvF0.net

PHPで設計されてるだけでもう吐きそう

8 ：風吹けば名無し：2022/06/23(木) 12:54:31.14 ID:EsKIpmtBp.net

同じ変数を使い回す

6 ：風吹けば名無し：2022/06/23(木) 12:53:58.50 ID:qh+WjYDo0.net

えっ？なんで動くの？バグもないの？なんで？ってしばらく眺めてみても分からない

23 ：風吹けば名無し：2022/06/23(木) 13:01:00.35 ID:NSfRHea30.net

エラー投げてるけど親が受け取ってない

44 ：風吹けば名無し：2022/06/23(木) 13:12:13.63 ID:KgkF5e2/0.net

ワイが書いたやつ

31 ：風吹けば名無し：2022/06/23(木) 13:03:14.82 ID:8dj2oK6K0.net

とりあえずで作られたサービス層がある

84 ：風吹けば名無し：2022/06/23(木) 13:37:40.47 ID:pSUw4iIR0.net

public変数が多いほどコンパイル時間が増大する

24 ：風吹けば名無し：2022/06/23(木) 13:01:07.15 ID:pSUw4iIR0.net

変　数　が　ロ　ー　マ　字

RECOMMENDこちらの記事も人気です。