今までみた絶望的なソースコード [転載禁止]©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/07/26(日) 20:26:36.68

なんでJQueryの実装をそんなにマンセーしたいのか理解できない。
ライブラリなんて小規模で作るもんだから、読みやすさという基準は自分の主観で決めるもんだし。
後付けの解釈を止めろよ。

**デフォルトの名無しさん** · 2015/07/26(日) 20:59:57.20

逆でしょ？
読みやすさという基準は自分の主観で決めるもんなのに
jQueryの実装をそんなに馬鹿にしたいのか理解できない。

**デフォルトの名無しさん** · 2015/07/26(日) 21:26:25.41

>>250
hoistingはコード規約で予防しているという事かな？
それでも、宣言はブロックの先頭と制約するだけで十分な気がする
関数スコープに広げて宣言する理由がわからない

**デフォルトの名無しさん** · 2015/07/26(日) 21:31:57.10

ああ、意味がわかった
関数スコープで使用していた上に、ブロック内でも使用した場合か
それならバグになる可能性あるな

**デフォルトの名無しさん** · 2015/07/29(水) 01:58:34.13

やっぱVBの配列(0)で1要素あるってバグの元だろ
引継いだコード前任者が全員間違えたまま使ってて戦慄した

**デフォルトの名無しさん** · 2015/07/29(水) 09:27:40.97

まあ、多い分にはいいんじゃないの？
行番号付きだった頃から配列は１要素を予備とすることが多かったからねー。
C とかから入ると気になるだろうけど。
自分も今は気になるけどｗ

**デフォルトの名無しさん** · 2015/07/30(木) 11:48:36.63

DBに予備項目を作って空を保つのならわかるけど
配列の予備って意味若ランナー

**デフォルトの名無しさん** · 2015/07/30(木) 21:27:47.71

配列変数を空の状態またはその逆にするのが面倒なので0番目は使わずに取っておく

**デフォルトの名無しさん** · 2015/07/31(金) 16:17:53.20

動的配列の扱いが内蔵とVariantに代入する配列(Variant配列ではなく)で挙動が違うから
Variantで統一するとvbsと扱い同じにできて捗る

**デフォルトの名無しさん** · 2015/07/31(金) 21:11:17.11

絶望的なプログラマたち

**デフォルトの名無しさん** · 2015/07/31(金) 21:48:20.19

main(){
int* p=1;
...

**デフォルトの名無しさん** · 2015/08/01(土) 14:23:13.10

*(int*)0=0;

**デフォルトの名無しさん** · 2015/08/01(土) 14:44:38.33

printf("AとBの積は593です");
もうアホかとなんのためのCなんだか

◆QZaw55cn4c · 2015/08/01(土) 19:29:40.69

593 は素数なんだが‥

**デフォルトの名無しさん** · 2015/08/01(土) 21:31:08.62

そうすっうすね。

**デフォルトの名無しさん** · 2015/08/01(土) 21:39:39.36

1*593

**デフォルトの名無しさん** · 2015/08/01(土) 22:19:25.72

>>266
掛け算作ってみろって言ったらこれが出てきた

**デフォルトの名無しさん** · 2015/08/02(日) 00:25:01.79

>>269
最適化出来てる
100点

**デフォルトの名無しさん** · 2015/08/02(日) 01:45:42.48

掛け算作ってみろってのもおかしな物言いだな
そんなコミュ力だからおかしな物が出てくる

**デフォルトの名無しさん** · 2015/08/02(日) 02:10:40.49

>>270
あれだと最適化しないコード作るほうが頭いるだろ
一文字づつあたらしいプロセス作って動かしてくとか？

**デフォルトの名無しさん** · 2015/08/02(日) 07:33:48.36

>>269
模範解答じゃん

**デフォルトの名無しさん** · 2015/08/02(日) 17:23:58.64

絶望って程でもない

**デフォルトの名無しさん** · 2015/08/02(日) 18:37:05.66

無駄な計算してないとか優秀賞だな

**デフォルトの名無しさん** · 2015/08/31(月) 12:38:09.95

>>12
なんで先頭じゃダメなの？
蟹飯本では当たり前のように先頭でまとめて宣言してるからそれに倣って先頭で宣言してるけど

**デフォルトの名無しさん** · 2015/08/31(月) 13:10:34.00

>>276
変数は使用する直前で初期化して使うべきだから
初期化と宣言が別だと必ず未初期化で使うバグが出る
また関数が長く分岐が多くなると初期化されてるか判断が難しくなる
変数のスコープは小さければ小さいほどいい
関数先頭に宣言するのは昔のC言語の制約だったと記憶してるが今は上記理由からやめるべき

**デフォルトの名無しさん** · 2015/08/31(月) 15:56:13.40

>>277
なるほどサンクス

**デフォルトの名無しさん** · 2015/09/01(火) 20:57:53.90

>>277
どこの説？

**デフォルトの名無しさん** · 2015/09/12(土) 03:17:33.09

for ループから条件でgotoでforの外に飛んで帰ってもこないプログラム

**デフォルトの名無しさん** · 2015/09/12(土) 04:01:54.66

それ普通ですが
breakがまさにそういう動作だし、breakはgotoの一種と言われているわけだが
二重のforを一気に抜けるのにgoto使うとか普通のテクニック
return出来るならそっちのがいいけど

むしろforからgotoで抜けてまたgotoで戻ってくる方が恐ろしいが
そういうサブルーチン的な動作にgoto使われるとかなり鬱陶しいし
それでgotoが禁止されたわけだが

**デフォルトの名無しさん** · 2015/09/12(土) 08:00:00.87

for使ってるの？甘いね
プロは全部gotoしか使わないよ

**デフォルトの名無しさん** · 2015/09/12(土) 09:09:13.39

柔軟に環境に合わせられるのがプロ

**デフォルトの名無しさん** · 2015/09/12(土) 09:59:18.43

真のプロには、製品の方が合わせてくる。

企画原案に意見を求められるし、監修を頼まれる。

**デフォルトの名無しさん** · 2015/09/12(土) 10:01:01.89

>>284
それで真のプロのプログラマだと
どうなるわけ？

**デフォルトの名無しさん** · 2015/09/12(土) 10:11:11.52

>>285
その問題については既に解いてあるんだが、2chは数式を張り付けられないからなあ。

**デフォルトの名無しさん** · 2015/09/12(土) 10:11:50.87

>>286
つまり解いてないってことかｗ

**デフォルトの名無しさん** · 2015/09/12(土) 10:56:04.07

>>286
ほれ
http://formula.s21g.com/

**デフォルトの名無しさん** · 2015/09/12(土) 11:30:03.64

一休「虎が屏風から出てくれば捕まえられるのにな～ああ～残念だなあ～」
将軍「出したよ」
って状況？

**デフォルトの名無しさん** · 2015/09/12(土) 11:38:50.74

>>285
達人ともなれば、むしろタンスが小指に恐れをなし、というアレだよ。

**デフォルトの名無しさん** · 2015/09/12(土) 12:07:10.77

>>286
zipでくれ

**デフォルトの名無しさん** · 2015/09/12(土) 18:16:31.99

>>289
https://www.youtube.com/watch?v=sXtYIgqKobc

**デフォルトの名無しさん** · 2015/09/12(土) 22:51:55.44

971 名前：名無しさん＠七周年[] 投稿日：2006/11/11(土) 00:56:10 ID:M8+ahUZV0
自称スーパープログラマー某

強烈加齢臭
メール見ない　メール無視
ドキュメント見ない　ドキュメント無視
説明聞かない　説明無視
話聞かない　話無視
ガム→むっちゃーむっちゃーむっちゃーむっちゃー
煎餅→バリバリバリバリバリバリ
菓子パン→モグモグモグモグモグモグモグモグモグモグ
カップアイス→むっちゃーむっちゃーペロペロペロペロ

フロアすべてに響き渡るばかでかいくしゃみ当然手でおさえない
半径10ｍで耳鳴り、ひどいと頭痛誘発

異音と異振動
ドスーン！ドスーン！ドカーン！ドカーン！
バコーン！バコーン！ズドン！　　ズドン！
机が近いとマウスポインターが飛ぶ
誤操作誘発
書類だしたりしまうだけでドカンドカン

口と鼻から異音
ふんっっ
ふんっーーーーー　　
マジでうっとおしい30秒ごと

備品破壊　貸与P C 破壊　他人のもの破壊

奴の作ったスクリプトとプログラムは使えない
バグりすぎ

**デフォルトの名無しさん** · 2015/09/13(日) 00:10:23.04

ストレス貯まるなそれは

**デフォルトの名無しさん** · 2015/10/16(金) 11:13:43.44

結論：そんな奴はいない

**デフォルトの名無しさん** · 2015/10/17(土) 10:29:18.04

コメントが一切なくて、ものすこく複雑な処理をやっている。

**デフォルトの名無しさん** · 2015/10/18(日) 01:32:01.64

ものすごく」の程度は人によるからなあ
複雑そうに見えてイディオム使ってるだけのもあるし
自動生成された物を見て複雑だなあって思ってる場合もあるし

**デフォルトの名無しさん** · 2015/10/18(日) 08:42:48.35

他人のコード見てコメント無いだの少ないだの騒ぐ奴
本人はクソポエムみたいなコメント書く上にコメントの整備はしないし何がしたいのやら

**デフォルトの名無しさん** · 2015/10/18(日) 22:58:23.93

>>298
お前の環境にはそんな奴しか居ないってだけ。

**デフォルトの名無しさん** · 2015/10/19(月) 03:05:21.12

更新履歴すら書くの忘れる
面倒というレベルじゃなく、忘れる

**デフォルトの名無しさん** · 2015/10/19(月) 12:29:11.07

ああー俺もコメントポエム書きてえぇーーー
本当はポエマーなんだよな俺
俺だってソースにポエム埋め込みてーんだよ！

**デフォルトの名無しさん** · 2015/10/19(月) 13:51:30.68

>>300
最近思うが、更新履歴、コメントはいらないと思う。
その代わりバージョン管理だけは、キッチリ機械的にやって欲しい。

更新履歴は忘れる奴が必ずいるから、更新履歴で探すと見つからずにハマる。
コメントもプログラマの現状の理解と感情で書いてあるので、変な先入観で見てしまいハマる時がある。

自分で書いていても、10年位経つと何書いているか解らない。
また、自分の名前が書いてあると、忘れた頃に質問が来る。

更新履歴、コメントは百害あって一利無しだと思う。

**デフォルトの名無しさん** · 2015/10/20(火) 10:03:44.13

>>298
うちにも居るわ、コメント赤ペン先生

**デフォルトの名無しさん** · 2015/10/20(火) 15:31:58.74

>>302
コメントの付け方が下手なだけだろ

**デフォルトの名無しさん** · 2015/10/20(火) 21:25:15.03

/*
此処に入って来たらマジやばい。
誰も周りは助けてくれないし。

あのリーダの下は、本当に最悪。
もう、2度と一緒に仕事をしたく無い。
*/

当時、こんなコメントを読んで大変なやぁ…って思った。

**デフォルトの名無しさん** · 2015/10/21(水) 01:07:37.37

/*
そのリーダーなわけだが。 ↑そりゃないだろー
引継ぎも満足にできねーで去りやがって
自身の無能を知ってゆっくり野垂れ死んでね
*/
みたいなしょーもない愚痴コメントって実際少ないよね

**デフォルトの名無しさん** · 2015/10/21(水) 01:09:50.98

だってそんなコメント無駄だし、むしろコメントの質を下げる要因にしかならないじゃん

**デフォルトの名無しさん** · 2015/10/21(水) 10:26:45.72

こういうコードが入る時点で、
コードレビューも行われていないような
ところだろうなってわかるねｗ

**デフォルトの名無しさん** · 2015/10/21(水) 10:33:35.51

コードレビューに夢見過ぎ…

**デフォルトの名無しさん** · 2015/10/21(水) 12:47:50.82

夢じゃなくて品質を上げるのは
コードレビューしかないよ。

人によって生産性が100倍違うのも当たり前なのは
コードが違うからなんだし。

**デフォルトの名無しさん** · 2015/10/21(水) 13:05:30.14

コードレビューで品質が上がる…
ますます、夢見る夢子ちゃん確定の発言だな…

**デフォルトの名無しさん** · 2015/10/21(水) 13:08:00.35

質を上げるんじゃない
質の低いコードの混入を防ぐんだ

**デフォルトの名無しさん** · 2015/10/21(水) 13:20:09.67

質の低いコード混入するのは室の低いマしか雇えないブラックだからだよ…

**デフォルトの名無しさん** · 2015/10/21(水) 13:21:26.33

>>311
否定するなら理由を言えよｗ
何も言えないから説得力0なんだぞ

**デフォルトの名無しさん** · 2015/10/21(水) 13:22:22.09

>>313
> 質の低いコード混入するのは室の低いマしか雇えないブラックだからだよ…

じゃあどうするかって話。

コードレビューしかないね。

**デフォルトの名無しさん** · 2015/10/21(水) 13:24:43.24

コードレビューするのはどの業界でも当たり前の話なんだけどね。
新聞社でも新人が書いた記事をそのまま出すわけ無い。

**デフォルトの名無しさん** · 2015/10/21(水) 13:36:50.19

いや、そんなに必死になって連投して否定しなきゃならないようなことじゃないから…
コードレビューは、より良いコードを生み出す為でなく、室の悪いマのリカバリをする
だけだって、言ってるだけだよ…

**デフォルトの名無しさん** · 2015/10/21(水) 14:11:29.69

質が悪いのを直せば、質が良くなるのは当たり前ですね。

あんたコードレビューして、あんたが直してるの？
質の悪いプログラマは成長しないの？

**デフォルトの名無しさん** · 2015/10/21(水) 14:23:16.59

だ・か・ら、質が良くなるんじゃなく、悪い質じゃなくなる＝極普通、になるだけだよ…
未成熟なマなら成長したりするかもしれないけど、まあ、質の悪いマは多分成長
できなかったから、質の悪いマになったんであって、それが成長するというのは
見通し的に暗いと思うよ…

**デフォルトの名無しさん** · 2015/10/21(水) 14:31:53.30

えとなぁ、どんな優れた奴でも普通の時代がなかったわけじゃないんだよ。
駄目なやつがいきなり優秀になるわけないんだよ。
極普通になって、それから成長すれば良くなるだろ。

**デフォルトの名無しさん** · 2015/10/21(水) 14:32:32.69

だいたいな、悪いやつがなんでずっと悪いかって言うと、
それが悪いってわかってないからなんだよ。
それを教えるのもコードレビューなわけで。

**デフォルトの名無しさん** · 2015/10/21(水) 15:01:27.32

すまんな、あんた俺と違って幸せな環境にいるみたいだから、どこまで行っても平行線だわ…

**デフォルトの名無しさん** · 2015/10/21(水) 15:36:18.25

相手が単に嫌いとか理解できないってコードへの無意味な批判が無くなれば、コードレビューもありだとは思うんだけどな。

俺はテストパターン増やしたくないから、ダメなら途中でreturn false
最後まで到達出来ればreturn trueする関数書くけど、
returnは必ず一つ、って人となかなか言い合いになる。

**デフォルトの名無しさん** · 2015/10/21(水) 16:58:05.40

>>323
MISRAで決まってるから仕方あるまい。
俺はその辺resultにNG入れておいて
resultがOKじゃなかったら残りの処理はスルーして最後にreturnで返すようにしてる。
業界によって作法が違うからいろいろあるよな。

**デフォルトの名無しさん** · 2015/10/21(水) 18:10:39.28

>>322
コードレビューしないのが幸せってなんだそりゃｗ
お前の所はブラックなんだろうな。

**デフォルトの名無しさん** · 2015/10/21(水) 20:13:39.10

>>323
途中returnだからってパターン減るわけないだろ

**デフォルトの名無しさん** · 2015/10/21(水) 20:17:59.29

教えて解るのなら誰も苦労していない。
解る人も少数ながら、もちろんいるが。

**デフォルトの名無しさん** · 2015/10/21(水) 20:54:58.96

>>326
早期returnっていうのは、パターンを減らすのが
目的じゃなくて可読性を上げるのが目的。
可読性っていうのは人間にとっての読みやすさだから。

人間が読まないコードであるなら別にどんな書き方でもいいが、
そうじゃないだろう？

**デフォルトの名無しさん** · 2015/10/21(水) 21:16:36.85

そうじゃないだろう？って俺に言われてもｗ

**デフォルトの名無しさん** · 2015/10/21(水) 22:38:13.09

>>328
確かに可読性は人間にとっての読みやすさだけど、
理由じゃなくてただ単語の意味を説明してるだけなのが笑える

**デフォルトの名無しさん** · 2015/10/21(水) 23:31:28.17

returnが増えるほど可読性は下がる、という文化もある

**デフォルトの名無しさん** · 2015/10/22(木) 00:05:08.39

可読性というか、一見上から下に素直に流れる関数のように見えて実は
文の最後にちょろっとreturnがあるのを見落とす、なんてのはあるね。

**デフォルトの名無しさん** · 2015/10/22(木) 00:12:05.24

ねーよｗ

**デフォルトの名無しさん** · 2015/10/22(木) 00:17:58.38

いや、俺はあるw

**デフォルトの名無しさん** · 2015/10/22(木) 00:36:51.97

その理屈だと、そのreturnと同じ位置にi++とかあった場合、
それを見落とすんだろうなｗ

**デフォルトの名無しさん** · 2015/10/22(木) 07:19:28.30

>>326
減らないけど、記載するマトリックスは減るんだよ。
通るifとかswitchを列挙して、真の場合偽の場合を分けないといかんから、
頭でフラグ持ったとしても、通過する限り「処理なし」と書く必要がある。
処理なしと書く限り、処理なしのエビデンス取らなきゃならん。
それが嫌なら、通過もしないようネストを深くしなきゃならん。

>>328
可読性ももちろん上がるな。対応する括弧を追う必要も無いし。

**デフォルトの名無しさん** · 2015/10/22(木) 07:45:32.98

俺はすぐにreturnしちゃうけど、可読性は上がるよね
I/Fの仕様が変わった場合、戻り値の変更漏れとかたまにあるけど

**デフォルトの名無しさん** · 2015/10/22(木) 07:58:12.26

>>336

if (...) {
}
// 関数の終わり

だと管理しなきゃならなくて if (...) return; だと書かなくていいの？妙な規則だね。
ひとごとながら、if (...) return 関数(); とか大丈夫なのかと思ってしまった。

**デフォルトの名無しさん** · 2015/10/22(木) 08:03:40.68

MISRA準拠のときに困るからreturnは複数書かないな

**デフォルトの名無しさん** · 2015/10/22(木) 08:41:43.01

(関数){
hage
}
と
(関数)
{
hage
}
だとどっちがいいの？

**デフォルトの名無しさん** · 2015/10/22(木) 08:50:50.09

コーディングルールにある通りでいい
なければ多数派に合わせるようにコーディングルールを決めてしまえばいい

**デフォルトの名無しさん** · 2015/10/22(木) 09:17:53.18

>>338
一つならどちらでも書く必要があるよ。

条件1と条件2と条件3がある場合、
if 条件1
　処理1
else
　不処理フラグ
endif
if !不処理フラグ、条件2
　処理2
else
　　不処理フラグ
endif
if !不処理フラグ、条件3
　処理3
endif
なら管理必要。

if 条件1
　処理1
　if 条件2
　　処理2
　　if 条件3
　　　処理3
　　endif
　endif
endif
は、パターンは減るけど可読性悪すぎ。

**デフォルトの名無しさん** · 2015/10/22(木) 09:49:22.97

>>325
コードレビューしないことじゃなく、コードレビューして
「成長可能かもしれない奴が周りにいる」という
環境が幸せなんだよ…

**デフォルトの名無しさん** · 2015/10/22(木) 09:59:20.88

>>342
それ（後半のコード）を早期returnパターンにすると更にわかりやすくなる。

if ! 条件1
　return
endif

処理1
if ! 条件2
　return
endif

処理2
if ! 条件3
　return
endif

処理3

言語によっては、更に短くすることができる。わかりやすさがぜんぜん違うね。

unless 条件1 then return
処理1
unless 条件2 then return
処理2
unless 条件3 then return
処理3

**デフォルトの名無しさん** · 2015/10/22(木) 15:28:35.78

>>344
そうそう。だから割と俺は早期リターンでネスト深めずに処理出来るものはしたいんだよね。

**デフォルトの名無しさん** · 2015/10/22(木) 22:05:36.79

>>342
等価なのは後者なんだからやっぱりテストパターンは関係ないでしょ。
あとマトリクスと言ってるけど、もしかして前者の形で書いた場合は処理1を通過せずに
処理2を実行するケースも考慮しないとならないとか？

ツール使わずに手作業でC1カバレッジ管理しなきゃならないプロジェクトなんて
やったことないからよくわからないけど、大変だねw

**デフォルトの名無しさん** · 2015/10/22(木) 23:53:09.22

>>344
こっちの方が絶望的なコードに見える

**デフォルトの名無しさん** · 2015/10/23(金) 00:40:43.08

>>347
お前の目が腐ってるのでは？

**デフォルトの名無しさん** · 2015/10/23(金) 00:41:38.26

基本的に腐ってる人っていうのは
ルールで決まってるから、そうなんだ。みたいに
ルールのためのルールを愛する。
自分で考える脳を持たない。

**デフォルトの名無しさん** · 2015/10/23(金) 01:25:49.95

>>346
だから後者はネスト深くて嫌いなの。
当たり前だろ。
通る限り、例え結果がわかりきってようがその分のパターンも書かなきゃならんし、
スタック壊れた時のことも考えにゃならん。

ツールがどうのとかMISRAがどうのいってるお偉方には意味ないの。
机の上で紙でデバッグできなければ意味がない、検算は手計算、の人らだよ。

**デフォルトの名無しさん** · 2015/10/23(金) 02:16:55.96

こういう派閥争いの結果、複数の形式が混ざった糞コードが生まれるのであった

**デフォルトの名無しさん** · 2015/10/23(金) 06:14:19.52

モーゼ<俺は中道を行くぞー！ﾊﾞﾘﾊﾞﾘﾔﾒﾃ！>

**デフォルトの名無しさん** · 2015/10/23(金) 07:59:46.43

>>350
じゃあまとめると、>>323で途中returnを使わないとテストパターンが増えるかのように書いて
いたのは、おまえが深いネストが嫌い+おまえのところの妙な管理の合わせ技ってことね。