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

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

248 ：名無しさん＠涙目です。：2018/11/01(木) 14:00:21.62 ID:T42QKBbp0.net

>>234
ダメな部分を理解できない頭バグってるバカが
精神勝利法でドヤってもバカを晒すだけだよ
どーせお前、思い込みで糞コード書いてバグを量産してるバグ量産機だろwww

249 ：名無しさん＠涙目です。：2018/11/01(木) 14:04:39.68 ID:PrRdv8PP0.net

>>246
コメントがあるだけマシだよ…

250 ：名無しさん＠涙目です。：2018/11/01(木) 15:19:33.19 ID:rw39L8n20.net

>>28
addが中でコピーしてれば何の問題もないから
これだけでは何とも判定しがたい

251 ：名無しさん＠涙目です。：2018/11/01(木) 16:08:12.86 ID:O9cGBZpU0.net

>>192
変数名が日本語をローマ字にして母音を省略

解析が大変大変

252 ：名無しさん＠涙目です。：2018/11/01(木) 18:09:54.16 ID:CCbj8+hB0.net

//　見ちゃダメ！

253 ：名無しさん＠涙目です。：2018/11/01(木) 18:17:53.54 ID:cwK7EwXW0.net

>>13
愚かだなあ

254 ：名無しさん＠涙目です。：2018/11/01(木) 18:21:59.97 ID:cwK7EwXW0.net

>>52
全舐め大サービス

255 ：名無しさん＠涙目です。：2018/11/01(木) 18:25:23.49 ID:GwAnQYsD0.net

>>200
コードをというかアーキだよね

256 ：名無しさん＠涙目です。：2018/11/01(木) 19:52:03.79 ID:f5sXap7W0.net

rem文なしで数万行。
作成者は他社へ転職済み。社内システム管理者が対応してくれない。

257 ：名無しさん＠涙目です。：2018/11/01(木) 20:09:01.67 ID:+mnQzEhg0.net

>>242
自前センサーをごにょごにょするコードは謎の計算式でいっぱいだ

258 ：名無しさん＠涙目です。：2018/11/01(木) 21:57:23.70 ID:ndyc0KM50.net

ログインの判定SQLが

SELECT * FROM M_USER WHERE
LOGIN_ID=#loginId# AND
LOGIN_PASS=#passWord#

でデータが取れたかどうかで判断しちゃう

259 ：名無しさん＠涙目です。：2018/11/01(木) 22:08:08.24 ID:ivaUMjbK0.net

>>230 鬼のようにコメントアウトされてて何万行あるんだよ。って感じだったけどな。
何のためにgit使ってたんだよ。って感じ。

260 ：名無しさん＠涙目です。：2018/11/01(木) 22:11:08.75 ID:i/vdxjEh0.net

演習かなんかでキャラクタで四角を作るコードを書けと言ったら
ループとか全く思いつかずprintfをひたすら並べてたやつがいたな

261 ：名無しさん＠涙目です。：2018/11/01(木) 22:30:14.51 ID:c5vbbOPs0.net

>>244
アセンブラで考えるんだ

262 ：名無しさん＠涙目です。：2018/11/01(木) 22:30:52.09 ID:cxnygPbj0.net

>>259
横浜プリズンの軍曹さんの話にも出てくる
https://lolipop-teru.ssl-lolipop.jp/gunsou/

263 ：名無しさん＠涙目です。：2018/11/01(木) 22:40:51.73 ID:/tPvvwhJ0.net

>>261 読んだけど壮絶だな。。。
うちは一応仕様書は共有されてるしコミュニケーションはslackでし放題だ。

264 ：名無しさん＠涙目です。：2018/11/01(木) 22:47:29.69 ID:9VJzWfSe0.net

>>150
ガッ

265 ：名無しさん＠涙目です。：2018/11/01(木) 22:51:34.85 ID:d2HCLt0p0.net

>>261
アセンブラは単純だし組みやすいけどいかんせん最適化されないんだよな。プログラムサイズは小さいけどさ

266 ：名無しさん＠涙目です。：2018/11/01(木) 22:52:36.53 ID:eBBtznEI0.net

caseが500個くらい並んでた

267 ：名無しさん＠涙目です。：2018/11/01(木) 22:58:11.15 ID:gJeU7VG60.net

全部左寄せで3000ステップほどかきなぐり、
俺に

見てください！ものすごくきれいにかけましたよ！
といってきたおっさんがいたなぁ

あいついきてるかなぁ。
その後も色々やらかし、昼休憩で嫁に聞いてきます！とかわけわからんこと言ってそのまま戻らなかったおっさん…

嫁が謝りに来てたぞ………

268 ：名無しさん＠涙目です。：2018/11/01(木) 23:00:13.71 ID:K0k1hdCz0.net

>>39
そうだな。
// おまじない。
とか

269 ：名無しさん＠涙目です。：2018/11/01(木) 23:21:25.89 ID:ObAG9sRT0.net

>>268
// to make sure
とか見るとうんざりするけどな

270 ：名無しさん＠涙目です。：2018/11/01(木) 23:23:44.45 ID:UswWVyLG0.net

>>258
あかんのか？

271 ：名無しさん＠涙目です。：2018/11/01(木) 23:29:00.77 ID:QdST6m1x0.net

>>30
状況によりありかなｗ

条件8まで式が「変わる可能性」と
「優先順位の変更」（要は順番）が決まってない
状態だで、しかも関連性から同じソース内で処理しないといけない状況

いつでも式を大胆に変更でき、順番を変えれる様その形にしておく

合理化して式を単純化したら
分解し辛い・・・・

272 ：名無しさん＠涙目です。：2018/11/01(木) 23:43:53.35 ID:2zyZC/pr0.net

便利メソッドを集めたクラス。
シングルトンじゃないのでインスタンス化必須だが色々な所から使われるのでnewされまくり。
GCがすぐにメモリ解放してくれると勘違いしとる。

273 ：名無しさん＠涙目です。：2018/11/01(木) 23:59:18.53 ID:UswWVyLG0.net

>>272
あかんのか？

274 ：名無しさん＠涙目です。：2018/11/02(金) 00:01:36.23 ID:dRD9dNja0.net

//ここのウェイトを取るとなんか動かない

275 ：名無しさん＠涙目です。：2018/11/02(金) 01:11:36.14 ID:qVQL70vU0.net

>>270
平文でパスワード抜けるよ

276 ：名無しさん＠涙目です。：2018/11/02(金) 01:13:39.47 ID:qVQL70vU0.net

>>260
バカのコード：人が考えた結果を羅列する
天才のコード：人の考える過程を表現する

277 ：名無しさん＠涙目です。：2018/11/02(金) 03:19:39.29 ID:pseYBSVl0.net

doevent

278 ：名無しさん＠涙目です。：2018/11/02(金) 05:59:56.37 ID:GY+5gKIN0.net

for(int i= 〜
ってまあ一カ所なら許せるがどれもこれもiとnだけとか勘弁してほしい

279 ：名無しさん＠涙目です。：2018/11/02(金) 07:38:16.09 ID:pseYBSVl0.net

>>258
IDは0or1にしよっかな

280 ：名無しさん＠涙目です。：2018/11/02(金) 10:55:43.76 ID:TsDlLKMP0.net

一万ステップ以上でコピー多用してるヤツ

281 ：名無しさん＠涙目です。：2018/11/02(金) 11:53:16.38 ID:VOvfm3jw0.net

20行くらいで文字列が違うだけの塊が500個ぐらいで1万行。

20行を関数化して500+20行くらいにしてやった。

282 ：名無しさん＠涙目です。：2018/11/02(金) 18:22:39.73 ID:+e6R+Xx70.net

オーバーヘッドが…

283 ：名無しさん＠涙目です。：2018/11/02(金) 19:04:12.79 ID:BAyOF+1k0.net

世の中には同じ処理は関数でまとめるのが当たり前だと思ってる人もいると思うが
RAMのついてない環境の制御プログラムのソースでそれやるとぶっ飛ぶからな

284 ：名無しさん＠涙目です。：2018/11/02(金) 19:32:32.23 ID:VOvfm3jw0.net

>>282
気になるならinline化

285 ：名無しさん＠涙目です。：2018/11/02(金) 21:03:03.95 ID:RNWgGPda0.net

1万行は萎えた
ザラにあるのかもしれないが
書いたやつ病んでたし

286 ：名無しさん＠涙目です。：2018/11/02(金) 22:24:12.35 ID:ZcyA8oPH0.net

>>283
意味不明。
RAMのついてない環境って、書き換え可能記憶領域はCPUの汎用性レジスタしか使えないことだと思うけど、関数まとめるとか直接関係ない。

287 ：名無しさん＠涙目です。：2018/11/02(金) 23:01:03.66 ID:nsp6y73n0.net

おいおいオンエラーゴートゥって…ゴートゥ文使っとるぞ外せ外せwww
↓
なんだよトライキャッチ全然入ってねえじゃん手抜きしやがって入れろ入れろ！

同じでは…？

288 ：名無しさん＠涙目です。：2018/11/02(金) 23:09:09.63 ID:BAyOF+1k0.net

>>286
関数はスタック使うからRAMが無かったら戻り番地保持できなくてぶっ飛ぶよ

289 ：名無しさん＠涙目です。：2018/11/03(土) 00:11:28.52 ID:OGox+ur40.net

>>288
RAM無いのに関数呼び出しでスタック使おうとする糞みたいなコンパイラは窓から投げ捨てろ

てか強制inline位あるだろ

290 ：名無しさん＠涙目です。：2018/11/03(土) 00:16:33.25 ID:0dL1/mcL0.net

>>289
コンパイラの性能に頼ってるなんて相当経験浅いな

291 ：名無しさん＠涙目です。：2018/11/03(土) 00:20:12.10 ID:OGox+ur40.net

スタック使えないから関数使わないと言う方がダメだろ
コンパイラは面倒でもソースコードジェネレータ位ヘロヘロっと自作しろよ

292 ：名無しさん＠涙目です。：2018/11/03(土) 00:23:39.76 ID:0dL1/mcL0.net

コンパイラ、逆コンパイラ、アランブラ、逆アセンブラは全て自作経験あるよ

293 ：名無しさん＠涙目です。：2018/11/03(土) 00:26:19.30 ID:qvdxB8jk0.net

>>227
もう一個付け足すなら、エンジニアが糞

294 ：名無しさん＠涙目です。：2018/11/03(土) 00:27:11.36 ID:OGox+ur40.net

流石にディスコンパイラは作ったこと無いわw

ハンドでならやることもあるけど

295 ：名無しさん＠涙目です。：2018/11/03(土) 00:30:22.41 ID:0dL1/mcL0.net

自分で設計したDSP用の並列計算コードなんて既存ツールが存在しないから自作するしかないんだよ

296 ：名無しさん＠涙目です。：2018/11/03(土) 00:55:34.53 ID:f6zkXCue0.net

前バージョンソースをラッピングの繰り返し
Direct3Dが凄かった

297 ：名無しさん＠涙目です。：2018/11/03(土) 03:15:56.75 ID:WF88qLDY0.net

>>287
MISRAもそうだが規約が腐ってる
確かにバグになりやすい箇所の指摘もあって重宝もするけど
多くはコードを読みにくくするだけの効果も何もない独善的ルール