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

**デフォルトの名無しさん** · 2015/04/20(月) 12:48:14.68

などと下手糞が一丁前に語っちゃってます

**デフォルトの名無しさん** · 2015/04/20(月) 13:47:35.36

変数名が日本語ならまだわかるが、演算子まで日本語にしたらかえってわけわからん。

**デフォルトの名無しさん** · 2015/04/20(月) 13:53:08.80

ガラパってるな

**デフォルトの名無しさん** · 2015/04/20(月) 16:31:22.14

大企業に限ってソースはダメなテンプレだよ。1ファイル5万行に及ぶクソコードと
分岐ネスト&goto乱発の超絶スパゲッティで某携帯会社のシステムは稼働している。
この会社で働いてた時は一時期プログラミング出来なくなって鬱で休職したわ。俺の知ってるプログラミングじゃなかった。

**デフォルトの名無しさん** · 2015/04/20(月) 18:40:19.36

RoofRaised って何だと思って、長い時間をかけてヘッダーファイルを調べてみたら、ただの double だった、なんてことがよくあったでしょ。
大規模なプロジェクトのすべてのクラスで暗黙の typedef を見つけ出すのにどれくらい時間がかかると思う？

**デフォルトの名無しさん** · 2015/04/21(火) 08:08:57.40

>>34
ダブルだけに二段屋根？

**デフォルトの名無しさん** · 2015/04/21(火) 08:34:59.52

仕事選べない立場ってやっぱクズですね
同情します

**デフォルトの名無しさん** · 2015/04/21(火) 10:01:54.00

>>34
findとgrep組み合わせるだけで、どんなに大きな大規模プロジェクトでも
10分はかからないと思うが。馬鹿なの？

**デフォルトの名無しさん** · 2015/04/21(火) 12:33:47.15

> 10分はかからないと思うが。馬鹿なの？
そんなにかかったらだめだろ。
せいぜい10秒程度だろ。

処理は読まないといけないからもう少しかかるが
定義は辞書を調べるようなものなんだから
grepとかfindなんか使わずに、単語を選択して
「定義場所を表示」ぐらいの調べられないとだめ。

この程度で分レベルの時間がかかってしまったら
調べている間に、何してたんだっけ？ってなるよ。
思考を中断させる原因となってしまう。

**デフォルトの名無しさん** · 2015/04/22(水) 20:53:51.72

☆ 日本の核ブ装は絶対に必須ですわ。☆
http://www.soumu.go.jp/senkyo/kokumin_touhyou/index.html

☆ 日本国民の皆様方、2016年7月の『第２４回参議院選挙』で、改憲の参議院議員が
3分の2以上を超えると改憲が実現します。皆様方、必ず投票に自ら足を運んでください。
私たちの『日本国憲法』を絶対に改正しましょう。☆

**デフォルトの名無しさん** · 2015/04/22(水) 22:45:32.37

増改築を繰り返せばどんなシステムでもスパゲティになる

**デフォルトの名無しさん** · 2015/04/23(木) 14:29:59.79

>>38
それくらいで止まるとかw

**デフォルトの名無しさん** · 2015/04/23(木) 16:20:02.46

お前たちは真の絶望を知らない

**デフォルトの名無しさん** · 2015/04/24(金) 13:08:16.02

昔だがUNIX C使いがMSDOS Cで作ったソフト
auto変数でサイズ10000バイトとか普通にやってる上に子プロセスも同じだから全見直し

もっと酷かったのがC言語で書かれた移動体の基地局側ファームウェア
処理毎に移動体の数だけフラグ分けとif文の羅列で設計から全見直し

**デフォルトの名無しさん** · 2015/04/24(金) 14:39:36.66

>>43
その「auto変数でサイズ10000バイトとか普通にやってる」ところが
全部staticに直されていてスレッドで排他されていないコードも
みたことある。絶望的だ。

**デフォルトの名無しさん** · 2015/04/26(日) 08:28:01.28

シングルスレッドの組み込みなら常識的なコードだが

**デフォルトの名無しさん** · 2015/04/26(日) 13:23:54.32

昔のMSDOS環境はメモリ利用領域500KBほどだよ
stackでガバガバ取ったら、あっという間にエラーで落ちるからさw

**デフォルトの名無しさん** · 2015/04/26(日) 16:19:44.76

ロートルの何の価値もないXMS講座が始まるからDOSの話は止めろ

**デフォルトの名無しさん** · 2015/04/26(日) 16:40:53.05

>>46
ほう。じゃあ今はガバガバの2000倍は
使わないといけないから、
あっという間の2000倍かかるなｗ

**デフォルトの名無しさん** · 2015/04/26(日) 19:38:09.53

>>48
環境を考慮できないお前みたいな馬鹿ならできんじゃねw

**デフォルトの名無しさん** · 2015/04/29(水) 20:48:46.78

んと、UNIXではmallocの領域とスタックの領域は重なった四角形の上辺と下辺から衝突する方向で成長する。
だから、どちらが大きな領域を取れるかというと対等であるのだけれど
ヒープの側はfreeした場合、管理領域に虫食い穴ができてしまう。
そうした事情があるので、仮想記憶領域をムダに成長させたくなければ
可能な限り大きなメモリーはスタックに確保したほうが合理的なの。

**デフォルトの名無しさん** · 2015/04/29(水) 20:54:30.77

Windowsはスタックが1MBだったかな

**デフォルトの名無しさん** · 2015/04/29(水) 21:02:54.96

>>50
UNIXはそうかもしれないけど、LinuxもBSDもWindowsも違うよ。

**デフォルトの名無しさん** · 2015/04/29(水) 21:24:24.10

>>52

気になってlinuxのメモリモデル確認したら

テキスト(コード)、初期化、非初期化、（モート）、スタック

になってるじゃん。
ヒープが非初期化領域の末端に来て、反対側からスタックが衝突方向
だからLinuxもやっぱりUNIXと同じモデル。
ユーザーランドから見た論理モデルは変わってないじゃん。

**デフォルトの名無しさん** · 2015/04/29(水) 21:34:48.30

>>53
どこを見たのか知らないけど、違うよ。

**デフォルトの名無しさん** · 2015/04/29(水) 21:45:11.23

そもそも実装依存過ぎて話にならねえ

**デフォルトの名無しさん** · 2015/04/29(水) 22:01:22.35

わかった今確認した。
カーネル2.6では
text, data, (moat?), heap, bss
になってる。
sbrk()にゼロを渡すと dataと heapの間の数値になるから
初期化データに対してヒープが衝突方向に成長し、
ヒープの後ろに非初期化データが続く。

ただ、ドキュメントによれば2.4ではUNIXと同じはず。

**デフォルトの名無しさん** · 2015/04/29(水) 22:33:43.39

ほーん、で？いちいち書き込むなカス

**デフォルトの名無しさん** · 2015/05/02(土) 14:37:22.70

>>28
正直な経験で言うと、社内システムを開発外注して業務改善てのがゆとりな神経だと思う。
成功した会社を一つたりとも見たことがない

**デフォルトの名無しさん** · 2015/05/02(土) 14:43:12.43

switch分岐が5段連なった奴は絶望感が酷かったな
そのなかにたらたら転写、フラグ操作した上で、サブルーチンを呼び、サブルーチンでもまた分岐
調べると同じコードでフラグの設定値だけ違う

某地デジ

**聖マリアンナ医大病院20人資格取り消し** · 2015/05/05(火) 20:23:24.18

【神回】美人ニコ生主えりぃがFME切り忘れで公開オナニー！？【電マ】

https://www.youtube.com/watch?v=0U15dDLipOw

**デフォルトの名無しさん** · 2015/05/05(火) 22:49:52.71

CRubyでは、compile.c 内の、iseq_compile_each 関数で、
AST(抽象構文木)を、Ruby仮想マシン用の中間コードに、
コンパイルするのだが、

ASTノードの種類によって分岐する、
switch文は数千行もある

switch (ASTノードの種類){ }

「Rubyのしくみ」という本を参照

**デフォルトの名無しさん** · 2015/05/06(水) 04:06:56.31

rubyのソースが糞ってのは同意
中身見て驚いたわ

**デフォルトの名無しさん** · 2015/05/06(水) 16:44:48.96

変数の頭に＄をつけなきゃならん言語は全てゴミ扱いでいいと思うの
というかC++以外滅びろ

**デフォルトの名無しさん** · 2015/05/06(水) 17:23:42.32

htmlとかも滅びてしまうが

**聖マリアンナ医大病院20人資格取り消し** · 2015/05/10(日) 03:27:12.77

【海外】「さすが日本だな」　自衛隊の国産ヘリ「ＯＨ－１」の性能に外国人衝撃！！　やはり日本は世界一だった【大和】

https://www.youtube.com/watch?v=kDfbXZZ15s4

**デフォルトの名無しさん** · 2015/05/10(日) 11:59:03.70

#本当はうごかないはず

**デフォルトの名無しさん** · 2015/05/14(木) 19:48:56.69

行数が多すぎてコンパイルできない。

**デフォルトの名無しさん** · 2015/05/14(木) 20:03:23.27

なんだそれ

**デフォルトの名無しさん** · 2015/05/15(金) 19:31:56.50

どうみても間違っているソースだな
明らかに昨日要件とは違う実装
数週間前に別のやつが触っているのに誰も申告しない

後から知ったが、彼奴にあてがえば、そこも直した上で機能変更も設計変更もやってくれる
と考えた仕打ちだそうです。

安くて真面目に仕事して、能力あると思われると、死ぬまで仕事が降ってくる。

**デフォルトの名無しさん** · 2015/05/16(土) 00:20:16.24

今悩まされてるのが同じロジック別モジュール
作成者が統一化に猛反対
馬鹿は本当に死んでくれ

**デフォルトの名無しさん** · 2015/05/16(土) 01:05:34.17

>>70
既存に似たものあるから統合するって感じ？
作成者は少々いじられるのも嫌と？
可能な限り共通化して分離必要な箇所はインターフェース化して個別実装だな
ソースコピペとかは技術者を名乗ってはいけない

**デフォルトの名無しさん** · 2015/05/16(土) 01:51:37.25

>>70
ロジックが同じかどうかでまとめるのではなくて、
役割で考えないとダメだよ。

同じロジックであってもまとめるべきではないこともたまにある。

例えばTwitter APIとAWS APIの内部で同じ処理をやっていたとしても、
役割はそれぞれ別なので分けるべきだろう。

もちろん汎用モジュールに分けられるようなものであれば
分けた方がいい。

**デフォルトの名無しさん** · 2015/05/16(土) 06:32:17.99

自動車の大規模リコール騒ぎを見てれば
なんでも共通化すればいいってもんじゃない
ということは言える

**デフォルトの名無しさん** · 2015/05/16(土) 09:42:11.75

同じロジックでも制御対象が別ならコピペして分けておく
現在たまたま同じロジックで済むだけだから

**デフォルトの名無しさん** · 2015/05/16(土) 10:51:43.92

>>74
分ける必要が出た時に差分を個別実装すればいいだけ
コピペされたコードなんて誰もメンテしないぞ
特に他社のコード引っ張って来たら絶対触れない
不具合出ても一方は修正されたのに一方は修正されない
客からしたら不具合治ってないとクレーム
それが続けば信用問題にもなりかねない
月日が経てばもはやどっちがオリジナルか判別不明

**デフォルトの名無しさん** · 2015/05/16(土) 11:56:34.39

>>71が正解
日付け事にコピペしてんだよ！！
モジュール統一化してパラメーター渡しにしたらえらく御立腹w

**デフォルトの名無しさん** · 2015/05/16(土) 12:13:54.92

要するに自分の管理外にコードが出ていくのが嫌なのね
気持ちは分からんでもないけど、そいつテスト書いてないだろ

**デフォルトの名無しさん** · 2015/05/16(土) 13:13:51.56

コードは自分以外が読むものとして書くべき。

**デフォルトの名無しさん** · 2015/05/16(土) 15:16:33.13

>>72－>>74と仕事すると大変だろうな

**デフォルトの名無しさん** · 2015/05/16(土) 15:20:46.44

>>79と仕事するほうが大変だろｗ
理由はわかっている人には明らかだらあえて言うまでもないが。

**デフォルトの名無しさん** · 2015/05/16(土) 15:32:07.60

同じコードだからって領域が違うものをまとめるのはアホ。
コピペの方がまだマシ

**デフォルトの名無しさん** · 2015/05/16(土) 15:57:15.01

たぶんここから濃厚な罵倒合戦の火蓋が切って落とされると思うんですけど

**デフォルトの名無しさん** · 2015/05/16(土) 16:13:02.95

結果的に同じものができたから、一つにするという考えは間違っている。

共通モジュールは設計段階で考えることで、実装段階で考えることではない。

ましてや他の機能やシステムとの間で共通化することなどありえない。

汎用モジュールとして使うとしてもパッケージソフトくらいにしか適用できない。

**デフォルトの名無しさん** · 2015/05/16(土) 17:41:36.12

>>82
禿同w

>>76見ると日付毎言うてるから相当悲惨と思われる
俺はオブジェクトのシリアライズでバージョン毎に完全に分けられてて泣いた
どうせフィールド追加しただけなのに全部コピーするから一万行超えてたwww
差分だけ追加するようにしたら200行…

**デフォルトの名無しさん** · 2015/05/16(土) 18:21:40.06

このスレの誰と仕事すると大変か、という問いに関しては
「土曜の昼日中に2chやってる奴」が答え

**デフォルトの名無しさん** · 2015/05/16(土) 18:30:16.32

ソースのコピペを肯定する人多いのか
>>84状態なんだがw

**デフォルトの名無しさん** · 2015/05/16(土) 19:03:32.28

同じ処理が複数箇所に書いてあるなんぞあり得ない。

**デフォルトの名無しさん** · 2015/05/16(土) 20:12:39.41

>>87
アフリカの猿とアマゾンの猿は、同じに見えるかもしれませんが、別物です。
初心者にはわからないかもしれませんが。

**デフォルトの名無しさん** · 2015/05/16(土) 20:18:53.90

そこでOOPですよ

**デフォルトの名無しさん** · 2015/05/16(土) 20:31:58.78

oop厨ってアホだな。
共通化してはいけないものをなんとか共通化したいのならアスペクトだから。

**デフォルトの名無しさん** · 2015/05/16(土) 20:54:20.49

>>88
「同じに見える」んじゃなくて「同じもの」なら共通化して当たり前。

**デフォルトの名無しさん** · 2015/05/16(土) 21:03:55.92

その理屈でいうとforで0から順にインクリメントするものも関数化しないとな。
DoForZero2(100,print)

おー、短くなった。
流石プロですねー

**デフォルトの名無しさん** · 2015/05/16(土) 21:13:10.81

つーか毎回同じコードコピペしてんの？　まじで？

**デフォルトの名無しさん** · 2015/05/16(土) 21:18:26.81

それ、IT業界の中でも特定の業種だよね。
メーカーの内製でそんなことしないだろ？

**デフォルトの名無しさん** · 2015/05/16(土) 21:19:35.97

>>92
アホは喋るの止めた方がいいよ
帯域も容量も地球の資源ももったいない

**デフォルトの名無しさん** · 2015/05/16(土) 21:23:40.37

というか、pythonのrangeをmapで使おうとすると
まさにそんな感じのコードですわ。
視野の狭い人ね。

**デフォルトの名無しさん** · 2015/05/16(土) 22:42:32.71

>>82

**デフォルトの名無しさん** · 2015/05/17(日) 02:35:55.03

>>85
＞このスレの誰と仕事すると大変か、という問いに関しては
＞「土曜の昼日中に2chやってる奴」が答え

18:21:40.06 ID:UflOEuuo

書き込めるまで我慢してたんだろな～www

**デフォルトの名無しさん** · 2015/05/17(日) 04:53:31.15

>>86
>>71 だと実装の継承しようとしてるので、OOP 的には >>72 >>74 や >>83 みたいな反論は普通出てくる
ただ >>76 見るに相当ひどいみたいだし、何か特別な理由があるんじゃないかと

頻繁に特定の日付時の状態にバイナリ互換で戻すとか？あるかそんな事

**デフォルトの名無しさん** · 2015/05/17(日) 09:05:05.05

例えばヤフーのようなIDでログインさせるAPIを他社に提供することを考える
この場合ログイン処理をクラスとして切り出ししそのインスタンスのログイン処理をコールしてもらえるように設計するだろう
つまりヤフーも他社も同じクラスのインスタンスを持ちそのログイン処理を共有する

他社がヤフーのログイン処理を実装したりヤフーが提供先ごとにログイン処理を実装することがあるだろうか？
1つの処理もまた1つのオブジェクトだよ

**デフォルトの名無しさん** · 2015/05/17(日) 09:18:21.06

>>99
スレタイ嫁
特別な理由はそういうコード書く人物が存在する事だろ

**デフォルトの名無しさん** · 2015/05/17(日) 09:56:39.55

>>100
>つまりヤフーも他社も同じクラスのインスタンスを持ちそのログイン処理を共有する

そのやり方だとセキュリティがザルになるけどね。

**デフォルトの名無しさん** · 2015/05/17(日) 09:59:30.81

コピペするに足る理由がある時はいいんだよね？
例えばFlashがリテンションしてプログラムの一部が読めなくなった時でも
最低限人が死なない程度に制御するためにバイナリレベルで多重化するような場合とか

**デフォルトの名無しさん** · 2015/05/17(日) 10:30:23.26

>>99
一昔前のCOBOLでなら、聞いたことはある。

**デフォルトの名無しさん** · 2015/05/17(日) 11:57:01.86

>>103
それをソースコードのコピペと呼ぶのか

**デフォルトの名無しさん** · 2015/05/17(日) 12:12:04.44

>>61
逆にこういう例ってどうするのが正解なの？

**デフォルトの名無しさん** · 2015/05/17(日) 15:27:46.20

絶望を与えるのに間違ってる必要性はないんだ

**デフォルトの名無しさん** · 2015/05/17(日) 16:14:09.30

すごく納得した

**デフォルトの名無しさん** · 2015/05/17(日) 20:54:56.74

変数の取り違えに気づき、途中で値の交換を行ってたコード

**デフォルトの名無しさん** · 2015/05/17(日) 20:58:00.05

if文でどんどん分岐し、重複ロジックを大量に書いてあったプログラム。

**デフォルトの名無しさん** · 2015/05/17(日) 23:28:50.20

>>107
≫≫110とリンクするな（笑）

**デフォルトの名無しさん** · 2015/05/18(月) 00:27:44.33

派遣先で部下というか新しいメンバーにコード書かせたら恐ろしいコードよこしやがった。

ifのネストじゃなくて、同じifの判定をこれでもかってくらい繰り返してんの。

もうそこやめるからテスト通して放置。

**デフォルトの名無しさん** · 2015/05/18(月) 00:59:25.35

てーへんっすね
(大変ですねと、そんなレベルの低い人間と働かなきゃいけないなんて
あなたもやはり底辺の住人ですね、を掛けている)

**デフォルトの名無しさん** · 2015/05/18(月) 04:23:14.79

>>112
飛ぶ鳥跡を濁さず
知ってて放置は良くないぞ

>>113
同じ会社でも個々の能力はピンキリだろ
とくに大きい会社になるほど色んな奴と出会う

**デフォルトの名無しさん** · 2015/05/18(月) 06:10:30.81

コードソムリエ。

**デフォルトの名無しさん** · 2015/05/18(月) 06:12:37.96

>>114
> 飛ぶ鳥跡を濁さず

濁してるのは、飛ぶ鳥の方じゃないですからｗ

**デフォルトの名無しさん** · 2015/05/18(月) 06:44:46.92

飛ぶ鳥を落とすため、最高のライフルをご用意いたしました。

**デフォルトの名無しさん** · 2015/05/18(月) 07:08:35.97

>>114
ことわざ間違えるのは恥ずかしいから

**デフォルトの名無しさん** · 2015/05/18(月) 12:17:01.94

同じifの判定ってどういうこと？
if(a == 1) print "1";
if(a == 1) print "1";
if(a == 1) print "1";
なわけないよな？

if(a == 1) print "1";
if(a == 2) print "2";
if(a == 3) print "3";
って感じか？

**デフォルトの名無しさん** · 2015/05/18(月) 12:21:28.58

>>118
知って飛び去るのはよくないっていうことでは？逃げるが勝ちの方がふさわしいと思うが。

**デフォルトの名無しさん** · 2015/05/18(月) 14:17:46.16

>>120
http://kotowaza-allguide.com/ta/tatsutoriatowonigosazu.html

**デフォルトの名無しさん** · 2015/05/19(火) 20:48:19.66

お前らが管理しているソースで一番古いのっていつ頃のソース？

俺は1997年作成って言うソースが一番古い（C++)
かなりいろんな人がバグ修正、機能追加していて3K程のソース。

それ以外にも2004年以前のソースが２０本位ある（これもC++)

仕様書もソースも追加修正だらけで、作り直したいけど、動いているソースなんで勝手に修正するなって
言われていて作り直すわけにもいかなくて困ったもんだ状態だわ

**デフォルトの名無しさん** · 2015/05/20(水) 00:15:49.66

>>122
ボトルネックなんかが明確なら再作成ありだが、ただコードを綺麗にしたいってのはただのマスターベーションだしな

**デフォルトの名無しさん** · 2015/05/20(水) 07:37:13.08

まあ動作実績のあるコードは触らないのが鉄則だからな
仕様変更を待つしかないね

**デフォルトの名無しさん** · 2015/05/20(水) 08:43:49.75

>>122
そのコードが今でも頻繁にメンテされているならありだが、特に問題がなく、今後も大きな拡張が見込めないなら何もしないほうがいい。きもちは痛いほど分かるけどね。

**デフォルトの名無しさん** · 2015/05/20(水) 17:56:58.18

念の為に言っておくと、開発効率が悪いから
不具合が発生するリスクが高いから
という理由でコードを綺麗するのは重要なこと

**デフォルトの名無しさん** · 2015/05/20(水) 18:25:43.60

開発側の100%負担なら誰も反対しないだろうが、オナニしたいから開発費用を払って！
と言われて金を出す奴がいるとすれば、ただの鴨と鵜飼

**デフォルトの名無しさん** · 2015/05/20(水) 18:41:48.75

ムダに高い金を払っているってことを
知らせない、知らないほうがお互い幸せだからなｗ

金を払う方は金を払って、作る方は残業して。
こんなに苦しい物が実は無駄なんてｗ

**デフォルトの名無しさん** · 2015/05/21(木) 00:32:12.22

この手の議論は請負系と自社開発とで考え方が変わってくるな。

**デフォルトの名無しさん** · 2015/05/21(木) 08:11:01.01

請負なら検収さえ通ればゴミ納品してもいいしな