おまえらが目撃した「絶望的なソースコード」 どんなのある?
■ このスレッドは過去ログ倉庫に格納されています
#!/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 kinou1
kinou2
kinou999
関数名だぜ? とあるシステムの一部分のソースだけでC#のファイルが1万超えてて震えた >>7
そこまで行くと、作った奴が凄いなと褒めたくなるなw >>4
何かあったら全部自分で面倒を見る、という覚悟だな ttps://techwave.jp/images_inbox/2017/04/DSC_8998-1-640x464.jpg ループの外でクラス構築して
ループの中でそのクラスに値をListに追加し続けてるソース
そのコードを書いたバカの一言
「リストをダンプすると全部同じ値なんですよ、何でなんですかね?」 >>16
ループの外でクラス構築して
ループの中でそのクラスの値をListに追加し続けてるソース
だろ? Axsis2でコード生成したら3万行の
コードが吐き出されたでござる。 if(penis == true) {
dead = true;
} >>19
いい加減、行数でコードを評価しようとするのやめなよ、おじいちゃん 10 A=0
20 A=A+1
30 PRINT A
40 GOTO 20 >>20
いやこういう糞ソース
A a = new A()
List list = new ArrayList()
for (結果のレコード : dbの検索結果) {
a.set(結果のレコード)
list.add(a)
} グローバル変数の名前がa
VSみたいな統合開発環境なんてないから追跡不可能 if(条件2){
if(条件3){
if(条件4){
if(条件5){
if(条件6){
if(条件7){
if(条件8){
......;
}}}}}}}} 本当は効率やメンテナンスを考えて処理を関数化すべきなんだろうが
あまりにも納期にまで時間なくてif文大量に書いた
引き継ぎの人、ごめんな
でも1機能あたりシェルスクリプト1万行書いてた隣のチームよりは優しいと思うから許して https://t.co/C2bG93ZCkj 👀
Rock54: Caution(BBR-MD5:b73a9cd27f0065c395082e3925dacf01) 上の見てると俺まだ楽なんだなぁ
設計書なしで、3000行のファイル、うち2/3がデッドコード
関数化してなくて重複処理多すぎ
保守すんのめんどくさい 1から10まで足した答えを印字するプログラム
PRINT ”55” >>28
ちょっと考えないと何がダメなのかわからなかった とにかくありとあらゆるリテラルをシステムで1つの定数クラスに入れてある
別のクラスで使う、同じ意味、内容のリテラルの共通化もされてない
おかげで定数クラスが数万行になっていた 昔ステップ数で金変わるからforループ使うな、て言う天才がいたわ。
固定で100回す処理とか100個引数変えて書いてた >>18
if(true==true)return false;
これすこ なぜかgitから取ってきたばかりなのにコンパイルが通らない 1文で5000行のSQL
文にもテーブルにもコメントなどない 10 FOR I=0 TO 100000 :NEXT
20 TALK "TIXKO" :GOTO 20 継承したクラスコピペして作るのはいいけど
使っていない変数は消せ コメントに会社や上司の悪口が書いてある
内容も事実なんだけどね >>51
太古のパソコン雑誌は16進のダンプリストを自力で逆アセンブルして移植なんてよくあったなあ
俺もアドレスずれても再アセンブル通る程度にならやったことがある
学生の暇に任せた行動力はほんとに感心する ファイルを開いたらエディタがフリーズ(1クラスがでかすぎ) If 判定をする then
判定で正常な結果が帰ってきた時の処理
Else
判定で正常な結果が帰ってこなかった時の処理
End If >>43
すまん、自動テスト用の文字列にunkoとfuckはよく使う >>25
いまだにstep数で見積もり出せって言ってくる某大企業様 format cなんちゃらかんちゃら
実行したらPC壊れた
お前ら許さんからな COBOLのワーキンクストレージセクションの定義名が全て女の名前 VBでtempA,tempB,tempC...がグルグル回ってるソースでよくよく調べてみると
tempAが途中でtempaに生まれ変わって動いてた 変数名関数名をdo、go、toの組み合わせに変換してみよう。もちろん予約語は除いて。 >>51
昔はたまにあった
もちろんトリッキーと認識してたけど
データ領域のフリをしてコード入れて
走らせたらまた自分で書き換えて、、
もちろん普通の人はしない方が良いって言われてた >>62
いや、俺がそう書いてるんだ。誰も読まねえから商用にもそのままつっこんだぜ!!!!! 業務上の仕様通りに書けばいいのに、
意地でも三項演算子使ったり、
変な変換かまして行数減らしたがるアホは死んで欲しい 数学屋さんがn88basicあたりで書いたやつをまんまcへ
変数全部グローバルで
int i,j,i2,i3,i7,i31,i55,j2,j3,j4,j44; /* ループ用 */ SendCmd(コマンドA);
ThreadSleep(3000);
SendCmd(コマンドB);
ThreadSleep(2000);
SendCmd(コマンドC);
ThreadSleep(2000);
引き継いだとき絶望した
そら動作が遅いわけだ 三項演算子はネストしないでほすい
ネストするならせめて改行入れてくれ if (a = b) { /* ==のはずだがココは通らないのかも */ >>78
条件が複雑になるとifで分けてるけどその方が良いよね
自分でも読めなくなっちゃう ハードウェア設計の言語だけど
クロック信号やリセット信号のツリーを自前で数百行書いてるやつ 4年前に書いた、自分のコメントがないスクレイピング用ソースコード。 基本はソースコード=ドキュメント
おかしなトリックとか使わない基本命令しか使わない
3項演算子でてたがもちろん1行に1回 不要なコメント行がソースコードの全体の8割を占めているとか。 >>86
アセンブラ始めたときは1行1コメント書いてたw 一万行のクラスの変数がグローバル変数で三次元配列になってた。 IT速報アフィ的には、派遣社員が他社から持ち込んだソースだと知った時 次元数が多すぎる連想配列。
7くらいで絶望的になるには十分だが、10次元の面倒を見させられて、メンテしたら12次元になった。 >>90
コメントで喧嘩していたり、誰が見ても明らかに痛いソースコードもあるぞ。
スペルが間違ってるとかそんなことで30行も言い争うことなのか? >>52
どこにも問題ないように見える
ちょっと遊び心に溢れてはいるが >>4
こういうのって他の人が理解できないようにわざとやってんのかね? >>4
おそらく仕様書に書いてある「機能1、機能2・・・」というのをそのまま実装しようとしたんだろう。
後から保守することを何も考えていなかったんだろうな。仕様書が残っていれば一応追いかけられるが、
どこかにいってしまうと悲惨なことになる。 ■ このスレッドは過去ログ倉庫に格納されています