今までみた絶望的なソースコード [転載禁止]©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/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以上…

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

解けない因数分解ω

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

>>920
まずユニットテスト書いてリファクタ

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

古いコードを延々と残しておくのをやめてほしいな。
ソースコードの半分以上が使われてないコードだったりして見通しが悪いったらありゃしない。
その割に修正履歴はクソの役にも立たないコメントが付いてる。

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

>>923
使われとらんのやったら消せばいいやろ
消さないおまえが悪い

**デフォルトの名無しさん** · 2018/10/14(日) 21:25:39.07

めっちゃ長いネストをプリントアウトして一生懸命読んでたら
コメントだったでござる

**デフォルトの名無しさん** · 2018/10/14(日) 21:29:08.49

未だに紙出ししてコード読むおじいちゃん居るんかよw

**デフォルトの名無しさん** · 2018/10/14(日) 21:35:16.62

画面に収まりきらないとかメモ書き込んだりするならやるけどな

それだけ元がひどいんだろ

**デフォルトの名無しさん** · 2018/10/14(日) 21:41:26.50

画面に収まりきるコードてHello, Worldかなw

**デフォルトの名無しさん** · 2018/10/14(日) 21:50:49.34

ずいぶん狭い画面だね

**デフォルトの名無しさん** · 2018/10/14(日) 21:55:26.77

>>929
画面に収まるコードて25行以下のことやぞw
今どきのディスプレイにあわせて制約ゆるく解釈てんじゃねーよタコwww

**デフォルトの名無しさん** · 2018/10/14(日) 21:57:38.53

それ何か面白いの？

**デフォルトの名無しさん** · 2018/10/14(日) 21:58:28.19

>>931
おまえの無知がおもろいんやけどw

**デフォルトの名無しさん** · 2018/10/14(日) 21:58:53.76

よかったね