おまえらが目撃した「絶望的なソースコード」　どんなのある？

1 ：名無しさん＠涙目です。：2018/10/31(水) 11:45:57.78 ID:nUxs9tl80.net ?PLT(12015)

http://img.5ch.net/ico/folder1_03.gif
#!/bin/bash
function f() {
　　sleep "$1"
　　echo "$1"
}
while [ -n "$1" ]
do
　　f "$1" &
　　shift
done
wait

https://monobook.org/wiki/%E3%82%B9%E3%83%AA%E3%83%BC%E3%83%97%E3%82%BD%E3%83%BC%E3%83%88

2 ：名無しさん＠涙目です。：2018/10/31(水) 11:46:42.79 ID:4qAgK6zQ0.net

画面描画クラスが3万行あった

3 ：名無しさん＠涙目です。：2018/10/31(水) 11:47:07.24 ID:8J18AdgH0.net

10 MOTOR : GOTO 10

4 ：名無しさん＠涙目です。：2018/10/31(水) 11:47:16.48 ID:OVnNcVZB0.net

kinou1
kinou2
kinou999

関数名だぜ？

5 ：名無しさん＠涙目です。：2018/10/31(水) 11:48:00.46 ID:oFCqkQrC0.net

function ログイン(){
}

6 ：名無しさん＠涙目です。：2018/10/31(水) 11:48:32.84 ID:rCz48hBo0.net

//この処理を消すな

7 ：名無しさん＠涙目です。：2018/10/31(水) 11:48:47.60 ID:siKfB4kt0.net

とあるシステムの一部分のソースだけでC#のファイルが1万超えてて震えた

8 ：名無しさん＠涙目です。：2018/10/31(水) 11:49:43.06 ID:ZJmCHVXd0.net

if文のインデントだけで80文字超えて改行

9 ：名無しさん＠涙目です。：2018/10/31(水) 11:50:38.85 ID:gQOJRX2Z0.net

マ板でやれ、つーかあるだろ。

10 ：名無しさん＠涙目です。：2018/10/31(水) 11:50:52.67 ID:rCz48hBo0.net

>>7
1万行じゃなく1万ファイル？

11 ：名無しさん＠涙目です。：2018/10/31(水) 11:51:35.33 ID:Ec9YB1ju0.net

>>7
そこまで行くと、作った奴が凄いなと褒めたくなるなw

12 ：名無しさん＠涙目です。：2018/10/31(水) 11:52:50.97 ID:KrSf/gyW0.net

>>4
何かあったら全部自分で面倒を見る、という覚悟だな

13 ：名無しさん＠涙目です。：2018/10/31(水) 11:54:05.90 ID:ytJ6BinA0.net

ttps://techwave.jp/images_inbox/2017/04/DSC_8998-1-640x464.jpg

14 ：名無しさん＠涙目です。：2018/10/31(水) 11:54:39.60 ID:mSVJ8Kpi0.net

さっぱりわからんけどすごそう

15 ：名無しさん＠涙目です。：2018/10/31(水) 11:54:54.93 ID:3Cch75DY0.net

コメントで、汚いソースでゴメンね！ってあった

16 ：名無しさん＠涙目です。：2018/10/31(水) 11:55:13.85 ID:BPS7rBpx0.net

ループの外でクラス構築して
ループの中でそのクラスに値をListに追加し続けてるソース

そのコードを書いたバカの一言
「リストをダンプすると全部同じ値なんですよ、何でなんですかね？」

17 ：名無しさん＠涙目です。：2018/10/31(水) 11:55:19.26 ID:r4zvGZCF0.net

コントロール名がデフォルト

18 ：名無しさん＠涙目です。：2018/10/31(水) 11:55:26.12 ID:pXVOobwe0.net

https://pbs.twimg.com/media/Dkt3fnqUcAAVaUN.jpg

19 ：名無しさん＠涙目です。：2018/10/31(水) 11:56:08.99 ID:eJWG3tWI0.net

3万行超え

20 ：名無しさん＠涙目です。：2018/10/31(水) 11:58:56.09 ID:yUeqTaYe0.net

>>16
ループの外でクラス構築して
ループの中でそのクラスの値をListに追加し続けてるソース

だろ？

21 ：名無しさん＠涙目です。：2018/10/31(水) 11:59:53.13 ID:yUeqTaYe0.net

Axsis2でコード生成したら3万行の
コードが吐き出されたでござる。

22 ：名無しさん＠涙目です。：2018/10/31(水) 12:00:03.25 ID:fpSFF5xe0.net

exec “mkdir C:¥データ”

23 ：名無しさん＠涙目です。：2018/10/31(水) 12:00:03.55 ID:Qx6z7zup0.net

if(penis == true) {
dead = true;
}

24 ：名無しさん＠涙目です。：2018/10/31(水) 12:02:19.26 ID:1+Hdtqjo0.net

元祖スーパーモンキーのあれ

25 ：名無しさん＠涙目です。：2018/10/31(水) 12:04:15.98 ID:FfrJM+Hn0.net

>>19
いい加減、行数でコードを評価しようとするのやめなよ、おじいちゃん

26 ：名無しさん＠涙目です。：2018/10/31(水) 12:05:01.55 ID:9RtNy98o0.net

黒田清子 最新画像
https://i.imgur.com/J83yWcQ.jpg

27 ：名無しさん＠涙目です。：2018/10/31(水) 12:05:33.62 ID:DxHGMlh80.net

10 A=0
20 A=A+1
30 PRINT A
40 GOTO 20

28 ：名無しさん＠涙目です。：2018/10/31(水) 12:05:38.11 ID:BPS7rBpx0.net

>>20
いやこういう糞ソース

A a = new A()
List list = new ArrayList()

for (結果のレコード : dbの検索結果) {
a.set(結果のレコード)
list.add(a)
}

29 ：名無しさん＠涙目です。：2018/10/31(水) 12:05:44.34 ID:1XrHBUn60.net

グローバル変数の名前がa

VSみたいな統合開発環境なんてないから追跡不可能

30 ：名無しさん＠涙目です。：2018/10/31(水) 12:06:08.17 ID:fpSFF5xe0.net

if(条件2){
if(条件3){
if(条件4){
if(条件5){
if(条件6){
if(条件7){
if(条件8){
......;
}}}}}}}}

31 ：名無しさん＠涙目です。：2018/10/31(水) 12:06:45.97 ID:SLc3qNAS0.net

タブと2、4文字インデントが混在してる

32 ：名無しさん＠涙目です。：2018/10/31(水) 12:07:36.68 ID:1Ar8Z5/O0.net

https://i.imgur.com/rTXlWRN.jpg

33 ：名無しさん＠涙目です。：2018/10/31(水) 12:08:02.00 ID:hHaT4VVy0.net

本当は効率やメンテナンスを考えて処理を関数化すべきなんだろうが
あまりにも納期にまで時間なくてif文大量に書いた
引き継ぎの人、ごめんな

でも1機能あたりシェルスクリプト1万行書いてた隣のチームよりは優しいと思うから許して

34 ：名無しさん＠涙目です。：2018/10/31(水) 12:08:45.13 ID:fpSFF5xe0.net

https://t.co/C2bG93ZCkj 👀
Rock54: Caution(BBR-MD5:b73a9cd27f0065c395082e3925dacf01)

35 ：名無しさん＠涙目です。：2018/10/31(水) 12:09:30.67 ID:OcIw2Jm10.net

IT速報に転載すんの？

36 ：名無しさん＠涙目です。：2018/10/31(水) 12:10:03.03 ID:5juZ2CAX0.net

>>13
楽天の社長？
プログラマだったの？

37 ：名無しさん＠涙目です。：2018/10/31(水) 12:10:37.82 ID:G9qsSK7z0.net

上の見てると俺まだ楽なんだなぁ

設計書なしで、3000行のファイル、うち2/3がデッドコード
関数化してなくて重複処理多すぎ
保守すんのめんどくさい

38 ：名無しさん＠涙目です。：2018/10/31(水) 12:11:06.72 ID:N0ZGX2tr0.net

1から10まで足した答えを印字するプログラム

PRINT　”55”

39 ：名無しさん＠涙目です。：2018/10/31(水) 12:11:16.71 ID:NXp4qEmO0.net

//なぜかこれをやらないと動かない？

40 ：名無しさん＠涙目です。：2018/10/31(水) 12:11:18.56 ID:KDUyGUww0.net

>>28
ちょっと考えないと何がダメなのかわからなかった

41 ：名無しさん＠涙目です。：2018/10/31(水) 12:13:21.85 ID:/8Gwspff0.net

>>13
なんでこんな演算してるんだ

42 ：名無しさん＠涙目です。：2018/10/31(水) 12:14:23.40 ID:IX1FbuRc0.net

とにかくありとあらゆるリテラルをシステムで１つの定数クラスに入れてある
別のクラスで使う、同じ意味、内容のリテラルの共通化もされてない
おかげで定数クラスが数万行になっていた

43 ：名無しさん＠涙目です。：2018/10/31(水) 12:14:35.68 ID:p7zAKkO+0.net

関数名がchinpoとかsex

44 ：名無しさん＠涙目です。：2018/10/31(水) 12:14:55.33 ID:NXp4qEmO0.net

>>4
普通 func だよな

45 ：名無しさん＠涙目です。：2018/10/31(水) 12:15:10.48 ID:RVMAscDj0.net

昔ステップ数で金変わるからforループ使うな、て言う天才がいたわ。
固定で100回す処理とか100個引数変えて書いてた

46 ：名無しさん＠涙目です。：2018/10/31(水) 12:15:25.76 ID:K/dnnotQ0.net

コメントが見当たらない

47 ：名無しさん＠涙目です。：2018/10/31(水) 12:15:29.22 ID:MSZtuOOi0.net

>>18
if(true==true)return false;
これすこ

48 ：名無しさん＠涙目です。：2018/10/31(水) 12:16:57.63 ID:Wv0SxeBs0.net

1クラスに何千行も書かないでくれ

49 ：名無しさん＠涙目です。：2018/10/31(水) 12:17:49.79 ID:NXp4qEmO0.net

なぜかgitから取ってきたばかりなのにコンパイルが通らない

50 ：名無しさん＠涙目です。：2018/10/31(水) 12:21:49.37 ID:wMZ1yO/x0.net

retuan0;が6個くらいある

51 ：名無しさん＠涙目です。：2018/10/31(水) 12:21:59.11 ID:bb5tESQn0.net

マシン語のプラグラムをマシン語で書き換える

52 ：名無しさん＠涙目です。：2018/10/31(水) 12:22:26.85 ID:pXVOobwe0.net

>>47
そうかそうか！是非現物を見ていきたまえ！
この素晴らしいログイン処理を！

https://pbs.twimg.com/media/DkPM68iVAAAZLQ_.jpg

53 ：名無しさん＠涙目です。：2018/10/31(水) 12:24:38.18 ID:bgmMrjG50.net

1文で5000行のSQL
文にもテーブルにもコメントなどない

54 ：名無しさん＠涙目です。：2018/10/31(水) 12:24:59.36 ID:OwD7bGc10.net

VB6の標準モジュールが限界を迎えた

55 ：名無しさん＠涙目です。：2018/10/31(水) 12:25:13.40 ID:8J18AdgH0.net

10 FOR I=0 TO 100000 :NEXT
20 TALK "TIXKO" :GOTO 20

56 ：名無しさん＠涙目です。：2018/10/31(水) 12:25:29.43 ID:8617H4Y90.net

継承したクラスコピペして作るのはいいけど
使っていない変数は消せ

57 ：名無しさん＠涙目です。：2018/10/31(水) 12:25:55.74 ID:0kNeAlZJ0.net

コメントに会社や上司の悪口が書いてある

内容も事実なんだけどね

58 ：名無しさん＠涙目です。：2018/10/31(水) 12:26:07.69 ID:/8Gwspff0.net

>>51
太古のパソコン雑誌は16進のダンプリストを自力で逆アセンブルして移植なんてよくあったなあ
俺もアドレスずれても再アセンブル通る程度にならやったことがある
学生の暇に任せた行動力はほんとに感心する

59 ：名無しさん＠涙目です。：2018/10/31(水) 12:28:12.29 ID:YimmOUEx0.net

ファイルを開いたらエディタがフリーズ(1クラスがでかすぎ)

60 ：名無しさん＠涙目です。：2018/10/31(水) 12:28:44.73 ID:J0K7w8UZ0.net

jmp $

61 ：名無しさん＠涙目です。：2018/10/31(水) 12:30:12.61 ID:OwD7bGc10.net

If 判定をする then
判定で正常な結果が帰ってきた時の処理
Else
判定で正常な結果が帰ってこなかった時の処理
End If

62 ：名無しさん＠涙目です。：2018/10/31(水) 12:30:52.73 ID:YimmOUEx0.net

>>43
すまん、自動テスト用の文字列にunkoとfuckはよく使う

63 ：名無しさん＠涙目です。：2018/10/31(水) 12:31:53.98 ID:NqUY/Sox0.net

関数名が東海道の宿場名

64 ：名無しさん＠涙目です。：2018/10/31(水) 12:32:59.46 ID:n3Ji/Y/l0.net

>>25
いまだにstep数で見積もり出せって言ってくる某大企業様

65 ：名無しさん＠涙目です。：2018/10/31(水) 12:34:52.19 ID:c48G6Wyd0.net

format cなんちゃらかんちゃら
実行したらPC壊れた
お前ら許さんからな

66 ：名無しさん＠涙目です。：2018/10/31(水) 12:35:06.87 ID:Ujiz3F9V0.net

COBOLのワーキンクストレージセクションの定義名が全て女の名前

67 ：名無しさん＠涙目です。：2018/10/31(水) 12:35:14.32 ID:L78dO8Eh0.net

関数がmain()のみ

68 ：名無しさん＠涙目です。：2018/10/31(水) 12:36:51.56 ID:8j7qVduS0.net

VBでtempA,tempB,tempC...がグルグル回ってるソースでよくよく調べてみると
tempAが途中でtempaに生まれ変わって動いてた

69 ：名無しさん＠涙目です。：2018/10/31(水) 12:38:34.93 ID:LWlCnwhH0.net

>>52
このコードは凄いなwwwww

70 ：名無しさん＠涙目です。：2018/10/31(水) 12:38:44.65 ID:O3iF2eK+0.net

変数名関数名をdo、go、toの組み合わせに変換してみよう。もちろん予約語は除いて。

71 ：名無しさん＠涙目です。：2018/10/31(水) 12:39:09.61 ID:6u30kAjF0.net

>>51

昔はたまにあった

もちろんトリッキーと認識してたけど

データ領域のフリをしてコード入れて
走らせたらまた自分で書き換えて、、

もちろん普通の人はしない方が良いって言われてた

72 ：名無しさん＠涙目です。：2018/10/31(水) 12:40:43.12 ID:p7zAKkO+0.net

>>62
いや、俺がそう書いてるんだ。誰も読まねえから商用にもそのままつっこんだぜ！！！！！

73 ：名無しさん＠涙目です。：2018/10/31(水) 12:43:04.37 ID:u7TOA6Us0.net

業務上の仕様通りに書けばいいのに、
意地でも三項演算子使ったり、
変な変換かまして行数減らしたがるアホは死んで欲しい

74 ：名無しさん＠涙目です。：2018/10/31(水) 12:43:31.57 ID:yeuX0xFh0.net

数学屋さんがn88basicあたりで書いたやつをまんまcへ
変数全部グローバルで
int i,j,i2,i3,i7,i31,i55,j2,j3,j4,j44; /* ループ用 */

75 ：名無しさん＠涙目です。：2018/10/31(水) 12:43:56.91 ID:hBPXw6990.net

!fusianasan

76 ：名無しさん＠涙目です。：2018/10/31(水) 12:45:45.78 ID:lXilVaxF0.net

SendCmd(コマンドA);
ThreadSleep(3000);
SendCmd(コマンドB);
ThreadSleep(2000);
SendCmd(コマンドC);
ThreadSleep(2000);

引き継いだとき絶望した
そら動作が遅いわけだ

77 ：名無しさん＠涙目です。：2018/10/31(水) 12:51:00.39 ID:fMLdtm4M0.net

1画面、1 exe

78 ：名無しさん＠涙目です。：2018/10/31(水) 12:53:13.19 ID:v6UsUN2r0.net

三項演算子はネストしないでほすい
ネストするならせめて改行入れてくれ

79 ：名無しさん＠涙目です。：2018/10/31(水) 12:54:12.71 ID:yeuX0xFh0.net

if (a = b) { /* ==のはずだがココは通らないのかも */

80 ：名無しさん＠涙目です。：2018/10/31(水) 12:56:16.36 ID:/8Gwspff0.net

>>78
条件が複雑になるとifで分けてるけどその方が良いよね
自分でも読めなくなっちゃう

81 ：名無しさん＠涙目です。：2018/10/31(水) 12:58:33.56 ID:1MNXujN10.net

ハードウェア設計の言語だけど
クロック信号やリセット信号のツリーを自前で数百行書いてるやつ

82 ：名無しさん＠涙目です。：2018/10/31(水) 12:58:58.98 ID:sMfwCTwm0.net

4年前に書いた、自分のコメントがないスクレイピング用ソースコード。

83 ：名無しさん＠涙目です。：2018/10/31(水) 13:05:58.30 ID:65E1dani0.net

基本はソースコード＝ドキュメント
おかしなトリックとか使わない基本命令しか使わない
３項演算子でてたがもちろん１行に１回

84 ：名無しさん＠涙目です。：2018/10/31(水) 13:06:00.40 ID:NSjUxjhk0.net

不要なコメント行がソースコードの全体の８割を占めているとか。

85 ：名無しさん＠涙目です。：2018/10/31(水) 13:07:04.41 ID:65E1dani0.net

>>86
アセンブラ始めたときは１行１コメント書いてたw

86 ：名無しさん＠涙目です。：2018/10/31(水) 13:08:43.18 ID:UODjql3t0.net

一万行のクラスの変数がグローバル変数で三次元配列になってた。

87 ：名無しさん＠涙目です。：2018/10/31(水) 13:15:08.35 ID:PpyTYzHb0.net

ＩＴ速報アフィ的には、派遣社員が他社から持ち込んだソースだと知った時

88 ：名無しさん＠涙目です。：2018/10/31(水) 13:16:38.72 ID:miM13rev0.net

次元数が多すぎる連想配列。
7くらいで絶望的になるには十分だが、10次元の面倒を見させられて、メンテしたら12次元になった。

89 ：名無しさん＠涙目です。：2018/10/31(水) 13:22:28.63 ID:KM0YH5pd0.net

ザラキ ザラキ ザラキ

90 ：名無しさん＠涙目です。：2018/10/31(水) 13:23:30.71 ID:3ruYDhBs0.net

お前らって頭えーねんな 何言うてるんかサッパリや

91 ：名無しさん＠涙目です。：2018/10/31(水) 13:26:05.20 ID:ZXSHhXNf0.net

10　print“ﾊｹﾞ”
20　goto10

92 ：名無しさん＠涙目です。：2018/10/31(水) 13:30:33.88 ID:c1ZQHcJE0.net

自分で書いたソースが読めなかった

93 ：名無しさん＠涙目です。：2018/10/31(水) 13:30:52.52 ID:miM13rev0.net

>>90
コメントで喧嘩していたり、誰が見ても明らかに痛いソースコードもあるぞ。
スペルが間違ってるとかそんなことで30行も言い争うことなのか？

94 ：名無しさん＠涙目です。：2018/10/31(水) 13:34:11.01 ID:mIYWSgn60.net

>>52
どこにも問題ないように見える
ちょっと遊び心に溢れてはいるが

95 ：名無しさん＠涙目です。：2018/10/31(水) 13:42:48.62 ID:7eNX02YB0.net

>>4
こういうのって他の人が理解できないようにわざとやってんのかね？

96 ：名無しさん＠涙目です。：2018/10/31(水) 13:44:30.51 ID:8kLg/W8q0.net

ぬるぽ

97 ：名無しさん＠涙目です。：2018/10/31(水) 13:46:45.27 ID:NYwklrJS0.net

がっ!!

98 ：名無しさん＠涙目です。：2018/10/31(水) 13:52:25.01 ID:sRbb579p0.net

>>83
それがベストだと思うわ

99 ：名無しさん＠涙目です。：2018/10/31(水) 13:52:42.72 ID:2/w7hC4C0.net

関数名が hoge( )

100 ：名無しさん＠涙目です。：2018/10/31(水) 13:53:39.35 ID:tDI5N9DN0.net

>>4
おそらく仕様書に書いてある「機能1、機能2・・・」というのをそのまま実装しようとしたんだろう。
後から保守することを何も考えていなかったんだろうな。仕様書が残っていれば一応追いかけられるが、
どこかにいってしまうと悲惨なことになる。