今までみた絶望的なソースコード [転載禁止]©2ch.net

**デフォルトの名無しさん** · 2015/04/17(金) 23:00:30.63

今井氏：ソースコード公開は、社長のティム（*2）の意向です。彼はバリバリのプログラマーで、初期の「Unreal Engine 1」を
1人で書いた人ですが、若い時に雑誌に載っていたコードを書き写して勉強したそうです。それで今の若い人にも、プロのソー
コードとはこういうものだというのを見せたいという願いがあって、ソースコードを公開しています。本当に今のゲーム業界の
事情を憂いてる1人だと思います。（*2）Epic Gamesの創業者兼CEOであるTim Sweeney氏

出村氏：読みやすいコードですよ。「C++」というのは、黒魔術（高度な計算）が多くなりがちな言語ですが、
そういうこともなく、すっきりしていて目的の機能も探しやすい。解読しやすいコードなので、確かにお手本になると思います。

僕は初代のゲームボーイからプレイステーション 2の頃くらいまでゲームプログラマーだったのですが、ゲームプログラミングでは
必ず数学が出てきます。行列とか三角関数とか。もちろん今でもまったく不要になったわけではありませんが、そういう知識の
重要性は薄れてきていると思います。「Unreal Engine」では特にそうです。
http://game.watch.impress.co.jp/docs/interview/20150417_698349.html

初級者から中級者へ昇格する時期は、ほぼどのようなソースコードでも読める程度にプログラミング言語に精通し、
また偉いプログラマーの提唱したデザインパターンも一通り理解したくらいの時期である。

すると、プログラミング言語の持つあらゆる機能と、偉いプログラマーの提唱するあらゆる技術を使わねばならない
という思い込みが発生する。そしてHello Worldにまで崇高なオブジェクト指向や壮大なデザインパターンを
適用しようとしだすのである。

その結果、
* 大量のクラス
* 迷路のような変数渡し
* 底なしに深いネスト
などといった凄いものが生まれる。また、条件分岐に三項演算子を乱用するなどの症状も多く見受けられる。
最終的には第三者にとって読みにくい保守性の悪いスパゲッティコードが生成されることになる。
http://monobook.org/wiki/%E4%B8%AD%E7%B4%9A%E8%80%85%E7%97%85

**デフォルトの名無しさん** · 2017/02/07(火) 11:04:23.35

情報エキスパートしか知らないのか、入門書を真に受けたのか
受注クラスに受注のすべてが書いてあった

**デフォルトの名無しさん** · 2017/02/24(金) 06:22:47.09

// よくここで止まっていた（いつの間にか何故か解消しました）

**デフォルトの名無しさん** · 2017/02/27(月) 12:20:37.06

一年前の自分のコード

◆QZaw55cn4c · 2017/04/08(土) 18:56:00.33

>>822
それそれ
自分が書いたかどうかすら忘れて今っているので、今必死に読んでいるんだが、まるで人のコードを読んでいる気分だ

**デフォルトの名無しさん** · 2017/05/06(土) 09:46:21.28

>>804　
シングルタスク(CPU1個)処理しか古い計算ソフトでは使えない。
並列化の描き方とは違う技術が必要。

逆に、当時のコンピューターに並列化したソースを読ませるとかなり遅くなる。

**デフォルトの名無しさん** · 2017/05/06(土) 10:40:45.49

いや保守運用を全く考えない使い捨てレベルの俺俺コードを保守させられたって話だろ
素人が描いたVBAに対する愚痴と大して変わらん

**デフォルトの名無しさん** · 2017/05/24(水) 05:02:46.93

一年前の自分のソースを見たときの俺

　　＿　□□　　　＿　　　　　＿＿＿､､､
　／／＿　　[][]／／　 ,,-―''':::::::::::::::ヽヾヽ':::::/､　　誰　　書　　こ
／／　＼＼　／／　／::::::::::::::::::::::::::::::i l | l i:::::::ミ　　だ　　い　　の
￣　　　　￣　￣／ /:::::::::,,,-‐,／i/`'''￣￣￣｀i::;|　　あ　　た　　汚
―`―--^--､＿_ 　 /:::::::::=ｿ　　 /　ヽ､／　　　,,|/　　っ　　の　　い
/f ）,fヽ,-､　　ﾉ　|　三　i　<ニ`-, ノ　/､-ﾆニ' 」'）　！！　は　　ソ
　 i'/　/^~i f-iノ　　|三　彡　t￣｡` ｿﾊ_ﾞ'､￣｡,ﾌ | ）　　　　　　　　 |
,,,　　　　 l'ノ　j　　ﾉ::i⌒ヽ;;|　￣￣　/ _ヽ､￣　ﾞi　）　　　　　　　　ス
　 ` '' -　　／　　　ﾉ::|　ヽミ　　`＿,(＿　　i＼_　 `i ヽ､　∧ ∧ ∧ ∧
　　　　　／／／　　|:::|　( ミ　 /　＿＿ニ'＿＿｀i　|　　Ｙ　Ｙ　Ｙ　Ｙ　Ｙ
　　　,-"　　　　　,|:::ヽ　ミ　　/-───―-`l 　|　　／／　　　 |
　　 |　　／／　　　l::::::::l＼　　　 ||||||||||||||||||||||/　 |　　　　　／／ |
　　/　　　　＿＿__.|:::::::| 　　､　`ー-―――┴　/　　　＿＿,,..-'|
　/ﾞｰ､,-―'''ＸＸＸＸ `''l::,/|　　　　ｰ- ､＿＿￣＿,,-"､＿,-''XXXXX |
/XX/ XXXXXXXXXX|　|　　　　　　　　　＿,　　/ﾉXXXXXXXXXX|

**デフォルトの名無しさん** · 2017/05/25(木) 01:23:57.90

>>822
それは自分のスキルが上がったらそう思うんだよ（震え声）

**デフォルトの名無しさん** · 2017/05/26(金) 15:58:13.06

まだ衰えを知らんと見える

**デフォルトの名無しさん** · 2017/06/11(日) 18:40:50.70

cでもなくc++ でもないみたいなソース渡された時はcの仕事受けるのやめようと思った
cメインで書いてる癖に難しい処理で半端にc++ に逃げるくらいなら最初から全部c++ で書けよと

**デフォルトの名無しさん** · 2017/06/11(日) 19:15:52.72

GDIplusですね判ります

**デフォルトの名無しさん** · 2017/07/04(火) 10:13:42.35

vector<int> v(2);
v[0]=1;
v[1]=2;

**デフォルトの名無しさん** · 2017/07/16(日) 15:02:14.04

>829
betterCとしては想定内の使い方なんじゃないの？

**デフォルトの名無しさん** · 2017/07/19(水) 09:35:02.32

簡単なコマンドラインツールなんかを c で書いてて map 使いたい時なんかはそんな風になるな。
だからって printf を cout で書き直したりはしない

**デフォルトの名無しさん** · 2017/07/19(水) 11:32:18.81

cout使いにくすぎる

**デフォルトの名無しさん** · 2017/07/22(土) 02:05:47.41

C#ってどんなだったか
結局printfの書式文字列がオーパーツすぎた

**デフォルトの名無しさん** · 2017/12/16(土) 10:53:17.72

江東区立深川図書館特殊

銅和加配

奇声あげて人をボコボコにぶんなぐってもＯＫ　お咎めなし
被害者が警察を呼んでくれと何度も言っているのに公務員は無視し続けてた
幼児が歓声上げて走り回る　　　　　見ぬふり
小学生が歓声上げて走り回る　　　　見ぬふり
中学生が大声で談笑して走り回る　見ぬふり
高校生が閲覧机で談笑雑談　　　　　見ぬふり
公務員による恣意行為
etc

なんのための施設か？　→特殊な関係用

翌日、被害者を公務員が脅していた

**デフォルトの名無しさん** · 2018/01/20(土) 14:52:43.07

俺のなかでは>>806が最狂

**デフォルトの名無しさん** · 2018/02/06(火) 13:41:29.39

ファイルローカルか公開へッダかでだいぶ違うが、後者だろうな

**デフォルトの名無しさん** · 2018/02/07(水) 11:16:22.68

https://github.com/oreilly-japan/deep-learning-from-scratch/blob/master/ch04/gradient_simplenet.py
https://github.com/oreilly-japan/deep-learning-from-scratch/blob/master/common/functions.py
https://github.com/oreilly-japan/deep-learning-from-scratch/blob/master/common/gradient.py

**デフォルトの名無しさん** · 2018/02/14(水) 23:21:09.68

ソースコード自体じゃないんだが、俺が抜けた後にそのプロジェクトの保守になった人の運用方法が……

/source_new
/source_new2
/source_new3
/source_new_rXXXX
/source_new_rXXXY
...

ってディレクトリがいつの間にか、各リビジョン毎に出来てて、中身丸々コピーして書き換えてる

かつ、ソースのコメントには
// リビジョンXXXXでの変更です！
とだけ書いてあり、更に元々は
// 下記の条件を満たす
// 詳細は○○の仕様書を参照
みたいに書いてあったところがことごとく
// リビジョンXXUU以前での変更です！
というコメントに書き換えられて行ってるんだが……

※なお、このプロジェクトはSVNでバージョン管理されている

**デフォルトの名無しさん** · 2018/02/14(水) 23:43:27.72

コメントに勢いがあってよろしい

**デフォルトの名無しさん** · 2018/02/16(金) 14:51:35.99

!とか?とかか

**デフォルトの名無しさん** · 2018/02/19(月) 23:12:20.19

俺が見た中で最凶。C言語。

len = strlen(hoge);
strncpy(piyo, hoge, len);

**デフォルトの名無しさん** · 2018/02/25(日) 15:48:10.58

ソースファイルがコメント含めて 12669行
このファイルで
最大のメソドがコメント含めて 1109行
メソド引数が 36個
非スタティックメソドにフィールド値を ref渡し

-->MS純正

**デフォルトの名無しさん** · 2018/02/25(日) 16:45:59.94

創業者のコードが汚いって従業員から言われていたからな

**デフォルトの名無しさん** · 2018/03/16(金) 13:22:49.42

hello Cで書いといて

**デフォルトの名無しさん** · 2018/04/16(月) 07:14:25.33

【悲報】Fラン大学生「説明会どこも満席やんけ…せや！ソースコード見たろ！」→学歴フィルターの存在がバレる [973639274]
https://leia.5ch.net/test/read.cgi/poverty/1523795256/

**デフォルトの名無しさん** · 2018/04/24(火) 18:20:16.99

",".join(tokens)

**デフォルトの名無しさん** · 2018/05/23(水) 19:46:52.59

僕の知り合いの知り合いができたパソコン一台でお金持ちになれるやり方
役に立つかもしれません
グーグルで検索するといいかも『ネットで稼ぐ方法　モニアレフヌノ』

95649

**デフォルトの名無しさん** · 2018/05/24(木) 10:56:26.89

95649

**デフォルトの名無しさん** · 2018/06/17(日) 12:09:14.85

Web系だとJavaScriptの広告スクリプトはインデントもなく難読化していてgoto文に似た
身近なクソコードの代表例といえるだろう。

おいらが見た最悪のコードはN88BASICをQuickBASICに書き直したの古い計算コードで、
行番号のN88BASIC構文と構造化(プロシージャ使用)が混載して併記され、プロシージャ
内部からいきなりサブルーチンコールし、すべてリターン値はグローバル変数、コメント無し、
goto文多用、マルチラインetc、行番号を模したラベル等々、などで書かれたソース。
これで難しい多項式計算していた。誰も保守できない年代物。

**デフォルトの名無しさん** · 2018/06/17(日) 13:25:48.64

test

**デフォルトの名無しさん** · 2018/06/25(月) 01:09:11.76

>>637
その気持ちは非常に理解出来るが、最終的に宗教論争になるから面倒臭い話になる。

**デフォルトの名無しさん** · 2018/06/25(月) 14:40:31.63

>>637
同意

**デフォルトの名無しさん** · 2018/06/25(月) 15:37:18.94

え、
if(kinoko == stick){

}
else{

}
普通こうじゃねえの？2つとも気持ち悪いんだけどｗ

**デフォルトの名無しさん** · 2018/06/26(火) 08:50:23.13

宗教戦争なら俺も参加するぞ
if(kinoko == stick) {

} else {

}

**デフォルトの名無しさん** · 2018/06/26(火) 20:56:53.52

宗教戦争なら俺も参加するぞ
if(kinoko == stick)
{

}
else
{

}

**デフォルトの名無しさん** · 2018/06/26(火) 21:23:05.10

{}だけの行やめろ

**デフォルトの名無しさん** · 2018/06/26(火) 22:50:54.21

詰めて書くの嫌い

**デフォルトの名無しさん** · 2018/06/29(金) 17:48:57.75

function
{
　　if () {

　　} else {

　　}
}

つかどんな言語であれ
その言語を代表するフレームワークとかで
コーディング規約として明示されてるから
それを守ればいいだけ

頭が悪いやつほど俺々ルールを作ろうとする

**デフォルトの名無しさん** · 2018/06/29(金) 19:08:43.50

プロジェクトによっては整形ツールの設定やlintのルールが決まってたりするからな

**デフォルトの名無しさん** · 2018/07/04(水) 22:08:32.84

X69

**デフォルトの名無しさん** · 2018/07/05(木) 16:33:12.12

X69

**デフォルトの名無しさん** · 2018/07/09(月) 22:05:35.39

>>857
その書き方は、ぶら下がりif文が書ける言語でぶら下がりif文を検出するためだけに意味があっただけ。
最近の言語は1行が長い場合が多いから、1行1文字でスカスカなコードはかえって読みづらい。

と、俺は思うのだが。

**デフォルトの名無しさん** · 2018/07/10(火) 08:55:02.87

なんで同じソースましてや関数内にブレスの開始が違うものが混在してるのかが不思議、開始と終了は揃えたい

**デフォルトの名無しさん** · 2018/07/10(火) 09:31:12.22

担当が代わったんじゃね

**デフォルトの名無しさん** · 2018/07/10(火) 16:08:06.16

99%の凡人は可能な限り独自の書き方は避けて
優秀な人間の考えたフレームワークやライブラリのコーディング規約に倣うべし

過去の財産に倣うというのはプログラマに必要な資質の1つ

**デフォルトの名無しさん** · 2018/07/11(水) 12:24:36.51

過去の財産に倣うというフレーズに資質とゆう単語を繋げてしまうと
この人は抽象化して考えるくせがないんやなと思われてしまうで
プログラマーに重要な資質の一つが欠けとるな、と

**デフォルトの名無しさん** · 2018/07/11(水) 18:22:34.72

ノンプログラミングのBIツールを700万で導入
GUIで作れる優れもの

入れた奴はプログラマの人件費が浮くから長期的に安いと息巻いてた

でも現在管理してるのは派遣プログラマの俺
VBAと違ってネットに情報無いから詰んでる

**デフォルトの名無しさん** · 2018/07/11(水) 20:03:46.10

>>869
よかったな
めっちゃ費用浮いてるやん！

会社が

**デフォルトの名無しさん** · 2018/07/11(水) 20:08:11.14

ノンプログラミングツールだからってプログラムの技量が低くていいってことにはならない
やっぱり知識も注意力もいる

ノンプログラミングツールのいいところは
それに従事した人間が、汎用的な知識を得ることなく
そのツール以外のことができない役立たずになることだ
その会社とその仕事に依存せざるを得ないからずっと安くこきつかえる

やったね！

**デフォルトの名無しさん** · 2018/07/13(金) 07:24:12.48

>>871
汎用的な知識を持って初めてノンプログラミングツールだよな

知らない人間がやったらソフトウェア開発失敗の歴史を一通り体験することになる

**デフォルトの名無しさん** · 2018/07/13(金) 07:25:12.25

でも知ってる人間はプログラム組めるという虚しさ

**デフォルトの名無しさん** · 2018/07/13(金) 12:23:22.47

フレームワークとかでも一緒
フレームワークの使い方は少し知ってても
言語の仕様をろくにしてない馬鹿とかザラにいる

潰しがきかないというか、お前は一生そのフレームワークで食っていくつもりなのかとｗ
そりゃIT土方にもなりますわ

**デフォルトの名無しさん** · 2018/07/13(金) 22:02:32.65

プログラマになった時点で
なんぼ汎用的でも所詮は人のつくったオモチャの知識で
頭埋め尽くされてるんやで…

**デフォルトの名無しさん** · 2018/07/14(土) 09:00:00.17

>>875
コンピュータサイエンスの研究して提案してる側でないとそうなるよね

**デフォルトの名無しさん** · 2018/07/14(土) 09:26:23.70

>>874
それでもフレームワークはコミュニティが存在して詰まるポイントは情報交換してるから良い

弱小ベンダが作った独自ノンプログラミングツールはサポートも含め使い方解らない

**デフォルトの名無しさん** · 2018/07/14(土) 10:28:38.58

>>875

**デフォルトの名無しさん** · 2018/07/14(土) 15:34:32.10

某一部上場企業に出向してた時がcobolをラップした日本語でプログラミングする企業だったな。
もう史上最低レベルでわからんかった。それで皆やってたから俺の応用力がないんだろうけど。
プログラミングじゃなくてコピペで作る積み木をやってる感覚だった。ベースのソースファイルなんか1つで100万行超えてたからな・・・

**デフォルトの名無しさん** · 2018/07/14(土) 15:37:32.14

>>879
それ地獄の所業だな

**デフォルトの名無しさん** · 2018/07/14(土) 15:54:39.16

>>880
しかも単体テストが出来ないから結合テストの時まで目視。作った段階じゃ動くかどうかもわからない。
本番環境じゃなくてテスト環境なのに、テストする為だけにプロセスコンパイル申請して（自分で勝手にコンパイル出来ない）
リンカが動いてる時はなんと100人近いプログラマが一切動けない。だからバグ頻発させる案件担当は恨まれる。

1箇所でも変数やバグ修正する度に、修正した変数が使われてる関数全ソースを「紙に出力」して案件担当に報告する。
無論その関数も二段組で10枚を超えるようなものもザラ。

ソースも絶望だったが会社そのものも絶望だった・・・これで一部上場企業でISO取得してるってんだから
世の中狂ってると思ったわ。今もあんなんやってんだろうか・・・

**デフォルトの名無しさん** · 2018/07/14(土) 16:13:46.94

ISOなんてソフトに関しちゃダメな証でしかない。

**デフォルトの名無しさん** · 2018/07/14(土) 23:35:54.44

ある有名大学を卒業した若手女子プログラマーが書いていたもの

if
　for
if
else
if
else
　　if >>4
else
//上書き
　　　　　　result

**デフォルトの名無しさん** · 2018/07/14(土) 23:48:20.01

それがどうした？

**デフォルトの名無しさん** · 2018/07/15(日) 00:05:51.22

再帰ｎ個まで対応として同じ処理をネストで内部にたくさん継ぎ足すのはなかなか

**デフォルトの名無しさん** · 2018/07/15(日) 14:23:36.32

しょうもねえ

**デフォルトの名無しさん** · 2018/07/17(火) 15:33:56.20

ネストが4階層以上あるコードが頻出してるかどうかで
プログラマとしてのセンスの9割以上が分かると思う

**デフォルトの名無しさん** · 2018/07/18(水) 12:19:37.23

センスのない経験則やなw

**デフォルトの名無しさん** · 2018/07/18(水) 13:59:31.91

ネストの4階層はよく言われてる事だろ
ネストの深さは闇の深さってな

**デフォルトの名無しさん** · 2018/07/19(木) 21:56:52.07

ネストの深さで分かるのは無能である事だけやぞ？
センス関係ないやんw
てか無能はコード見るまでもなく一言話せば分かるしw
迸っとるからな無能がwおまえらみたいにw

**デフォルトの名無しさん** · 2018/07/19(木) 22:48:50.17

だいぶ仕上がってますね

**デフォルトの名無しさん** · 2018/07/19(木) 22:56:52.61

ネスト取っ払うのなんてセンスいるか？

**デフォルトの名無しさん** · 2018/07/19(木) 23:03:03.88

メソッドに抽出したりcontinue,break,returnで浅くする工夫なら普通はやるわな

**デフォルトの名無しさん** · 2018/07/20(金) 13:13:13.22

>>890
おっと自己紹介はそこまでだ

**デフォルトの名無しさん** · 2018/07/20(金) 16:09:29.36

もうさぁ・・・一般的に言われてるソースじゃなくて、実際の仕事で見たソースの話してくんない？
本でよく見るような話いらんわ。しょうもない。

**デフォルトの名無しさん** · 2018/07/20(金) 20:33:44.27

実際あるから本のネタになるんだよ

**デフォルトの名無しさん** · 2018/07/20(金) 23:12:22.02

底辺の話を一般論のようにする馬鹿

**デフォルトの名無しさん** · 2018/07/20(金) 23:52:33.01

どのレスに対してなのか安価ぐらい付けなさいよ馬鹿

**デフォルトの名無しさん** · 2018/07/21(土) 07:46:57.20

自覚があるけどあえて名指ししてほしい馬鹿

**デフォルトの名無しさん** · 2018/07/23(月) 13:56:06.67

こんな3年以上前にあるスレでお前ら元気だなｗ

**デフォルトの名無しさん** · 2018/07/23(月) 16:44:38.45

褒めてもクソコードしか出せんぞ

**デフォルトの名無しさん** · 2018/07/23(月) 20:32:42.08

最近糞コードより糞メールに心を痛める毎日

**デフォルトの名無しさん** · 2018/07/24(火) 14:06:33.45

中国やアメリカで仕事して日本に帰ってくると
未だにメールでやり取りしてて心を痛める毎日

**デフォルトの名無しさん** · 2018/07/24(火) 15:37:09.77

ほんそれ

**デフォルトの名無しさん** · 2018/07/24(火) 20:50:50.75

むこうは何つかってるん

**デフォルトの名無しさん** · 2018/07/24(火) 22:01:57.77

すらっくやろ

**デフォルトの名無しさん** · 2018/08/06(月) 19:05:58.90

>>905
アメリカはSlack
中国はQQとWeChat（日本のLINEみたいなやつ）

中国は一般人でもメールはほとんど使わない
あとは何でもQRコードで電子決済
道路の両脇に並んでる崩れ落ちそうな肉まん屋ですら電子決済に対応してる
自転車乗るのはスマホ3タッチするだけ

深センあたりに行けば分かるがもう日本は完全手遅れ

**デフォルトの名無しさん** · 2018/08/08(水) 20:23:17.99

そんなにちがうものだろうか
国は嬉しいかもしれんが個人は

**デフォルトの名無しさん** · 2018/08/23(木) 09:04:38.91

履歴書手書きもうやだ

**デフォルトの名無しさん** · 2018/08/23(木) 09:43:28.39

1000人以上水増し

**デフォルトの名無しさん** · 2018/08/28(火) 22:37:03.51

methodという名前のクラスを見つけた

**デフォルトの名無しさん** · 2018/08/28(火) 22:46:56.46

>>911
ｗｗｗ

**デフォルトの名無しさん** · 2018/08/29(水) 01:03:25.64

Methodオブジェクトパターンかメタプログラミング文脈ならありえる

**デフォルトの名無しさん** · 2018/08/29(水) 19:19:49.93

>>911-913←なかなかの絶望的なコーダーwwww

**デフォルトの名無しさん** · 2018/10/10(水) 01:27:15.56

深いネストを隠蔽するために関数化。

**デフォルトの名無しさん** · 2018/10/13(土) 13:54:10.63

バクレポ「機能しない」

中身
{
;//ここに実装する
}

**デフォルトの名無しさん** · 2018/10/13(土) 16:31:33.86

>>916
それみて思い出したのがあったw

BOOL xxx(xxx)
{
// なんかコンパイル通らないのでコメントアウト
/*
:
*/
return TRUE;
}

**デフォルトの名無しさん** · 2018/10/14(日) 12:49:48.40

予定として工事中のまま残しとくのはあるな

**デフォルトの名無しさん** · 2018/10/14(日) 13:11:12.33

int func(prm1,prm2, …中略…,prmN)
{

//
// 色々処理する(見通し悪い1400ライン)
//
return 0;
}

**デフォルトの名無しさん** · 2018/10/14(日) 20:13:14.41

修正に修正を重ねて稼働し続けてきたリソース。
ソース分析しろ言われて開いたらネストが13以上…