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

**デフォルトの名無しさん** · 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

よかったね

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

>>933
残念やったねwww

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

なんでお前ら揉めてんねん

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

みんな糞コードが悪いんや。

**デフォルトの名無しさん** · 2018/10/15(月) 04:30:42.78

25行っていつの話だω
縦400しかない頃か

90年代初頭でも480のハイレゾで30行くらいは行けたぞ

**デフォルトの名無しさん** · 2018/10/15(月) 04:50:52.67

目糞鼻糞

**デフォルトの名無しさん** · 2018/10/15(月) 07:29:09.96

どっちがおじいちゃんかわからないねwww

**デフォルトの名無しさん** · 2018/10/15(月) 08:16:17.00

インデックスが一つも無いオラクルのシステム
しかもパッケージだからイジるなと・・・

検索に何分かかってるんだよ

**デフォルトの名無しさん** · 2018/10/15(月) 08:23:23.46

>>919
voidが無かった時代もあるんだわ
え、そんな問題じゃない？

**デフォルトの名無しさん** · 2018/10/15(月) 10:35:12.32

ハイレゾにモードチェンジするコマンド名忘れたな

**デフォルトの名無しさん** · 2018/10/15(月) 10:36:16.16

>>941
voidが無くてもva_listはあっただろ

**デフォルトの名無しさん** · 2018/10/15(月) 11:38:50.62

↓経験１０年以上の奴のプログラムがこれｗｗｗｗ

bool Hoge::isExistElement() const
{
　　int cnt = 0;
　　for (int i = 0; i < m_ElemList.size(); i++) {
　　　　++cnt;
　　}
　　return (cnt > 0)? true : false;
}

データ量が増えるとめちゃ遅くなるので、原因を調べたら
この関数がループの中で繰り返し呼ばれてたｗｗｗｗ

**デフォルトの名無しさん** · 2018/10/15(月) 17:39:03.89

return m_ElemList.size() > 0;
あるいは
for(int i = m_ElemList.size(); --i >= 0;) return true;
return false;

なんかの罠か？
全力で釣られてやったぞ

**デフォルトの名無しさん** · 2018/10/15(月) 20:01:02.54

不要な行は消せば良い言う人いるが、
消したくても消せない環境があるのよ…
クソコード × クソ規約でお腹いっぱいぱい

**デフォルトの名無しさん** · 2018/10/16(火) 03:33:36.46

トンキン試合ぶち壊しω最低だなωω

**デフォルトの名無しさん** · 2018/10/20(土) 16:35:46.84

wをωかくのキモいね

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

もまいらが優秀だと思う著名なプログラマーは誰？

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

おれが昨年作った糞コードを張りたいけど、
死ね！と言わない？

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

死んでる物体に死ねとは言わないでしょ普通

◆QZaw55cn4c · 2018/10/21(日) 21:37:32.52

>>950
いわないから見せて！
https://ideone.com

2018/10/22(月) 09:23:43.91

ムチャクチャだとされるCのコード

#include <stdio.h>
#include <stdlib.h>

// 構造体の宣言
typedef struct {
int num;
char *str;
} strct;

int main(void) {
// 実体を生成
strct *entity;

// 動的メモリの確保。確保したメモリをstrct型ポインタにキャスト。
entity = (strct*)malloc(sizeof(strct));

// メンバの初期化
entity->num = 0;
entity->str = (char*)malloc(sizeof(32));

// メモリに文字列を代入
sprintf(entity->str, "%s %s!", "Hello", "World");
printf("%s\n", entity->str);

// メモリの解放
free(entity->str);
free(entity);

return 0;
}

2018/10/22(月) 09:39:27.73

>>953
siseof(32)ってなんだろ？

2018/10/22(月) 09:53:43.00

数値リテラルは大抵intかな

2018/10/22(月) 10:05:30.30

>>953
sizeof(32) が怪しいくらいじゃね？
個人的には main(void) も気になるけど、どの辺がムチャクチャ？

2018/10/22(月) 10:33:20.99

typedefで小文字とstlctって名前も少々気持ち良くない

**デフォルトの名無しさん** · 2018/10/22(月) 23:15:17.28

sprintf で思いっきりバッファオーバーフローしとる

**デフォルトの名無しさん** · 2018/10/23(火) 01:03:56.45

>>958
そりゃsizeof(32) の指摘に含まれてるからあえて言う必要は無いんじゃね

**デフォルトの名無しさん** · 2018/10/23(火) 12:33:21.37

なぜ誰もfreeに突っ込まない？おまえら初心者ばかりか？

**デフォルトの名無しさん** · 2018/10/23(火) 14:07:56.51

えっ…

**デフォルトの名無しさん** · 2018/10/23(火) 14:25:23.38

freeなんて飾りです

**デフォルトの名無しさん** · 2018/10/23(火) 16:20:31.77

おいおい

**デフォルトの名無しさん** · 2018/10/23(火) 23:23:13.96

初心者さん、言葉がでない

**デフォルトの名無しさん** · 2018/10/24(水) 09:09:08.71

free不要論争勃発

**sage** · 2018/10/24(水) 15:23:23.13

sejuku が話題になってるスレがあったけど見つからねぇ
長野透とかいうひとのソースを分析してみた
最近の C というか ideon のメモリ割り当てって安全なのか？
#include <stdio.h>
#include <string.h>

int main() {
char s1[7] = "hello "; // taken 0x10 bytes
char s2[19] = "world! long length"; // taken 0x20 bytes
char s4[16] = "ABCDEFGHIJKLMOP";
char *s3;

s3 = strcat(s1, s2);
printf("%08x[%s]\n", s1, s1);
printf("%08x[%s]\n", s2, s2);
printf("%08x[%s]\n", s4, s4);
printf("%08x[%s]\n", s3, s3);
return 0;
}
https://ideone.com/58GHMk

**デフォルトの名無しさん** · 2018/10/28(日) 12:22:39.71

>>966
auto の話ならたまたま

◆QZaw55cn4c · 2018/10/28(日) 15:14:32.11

>>966
＞strcat(s1, s2);
これは決定的にあかん奴やね…

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

memcpyだよな

**デフォルトの名無しさん** · 2018/10/30(火) 14:36:53.65

解読不能

今起こり猿馬見れんだろ大会なの小倉の
日のお風呂中日本中４代目の枕にメール
お願い泊なっている心も離れてない練習
ご紹介私の命の言葉としての勉強

**デフォルトの名無しさん** · 2018/12/07(金) 02:37:05.58

昔の翻訳ソフトってこんな感じだった

**デフォルトの名無しさん** · 2019/04/23(火) 18:25:02.94

>>881
会社名を知りたい

**デフォルトの名無しさん** · 2019/06/19(水) 04:46:05.82

【出資】松本卓朗人工知能詐欺【注意】
https://rio2016.5ch.net/test/read.cgi/rikei/1560859403/

**デフォルトの名無しさん** · 2019/06/23(日) 21:33:39.90

熱い麺と冷たい麺

**デフォルトの名無しさん** · 2020/09/02(水) 03:47:42.55

画像データをコピーするとき愚直にforでコピーじゃなくて
memcpyにしたらめっさ早くなった

**デフォルトの名無しさん** · 2020/09/02(水) 03:57:59.43

てか、sizeof(32)て32byteだから別にオーバーフローしてなくね

**デフォルトの名無しさん** · 2020/09/02(水) 04:39:12.76

すみません普通にsizeof(32)は4を返しました
治すならこんな感じ？
entity->str = (char*)malloc(sizeof(32));
↓
entity->str = (char*)malloc(sizeof(char) * 32);

**デフォルトの名無しさん** · 2020/09/02(水) 09:13:25.17

sizeof(32)はsizeof(int)と等価やね。

**デフォルトの名無しさん** · 2020/09/02(水) 21:10:42.03

可読性高いコードかけた
https://ideone.com/h0rfiy

**デフォルトの名無しさん** · 2020/10/03(土) 20:46:23.99

　　　　　 | . . . . .　. │. .　.＼ . . ＼ . . . . . . ＼ . . . . . .　.＼.∨| . .
　　　　 │. . . . . . . .|.| . . . . |＼ . . ＼. . ／ .　.＼. . . . . . . .|│| . .　　　こんな話を知っているかしら。
　　　　 │. . . . . . . .|.|ｉ . . . . 、＼ . . Х＞― .斗＼ . .　.　|│| . .
　　　　 │. . . . . . . .|八 . . . . ＼　｀　‐---／　弌Y^ . . . . .ト|人厶　　ある中国人が、
　　　　　 |. .| . .　.l. .│　＼＼ . . ＼　　〈/Ｖ＼　ヽ　| . . . |├くヘ　　私の国には報道の自由がありません。
　　　　　 |. .| . .　.| . .|＿___,＼＼.　. ＼　　＼く___ﾉ　| . |. ∧|. . . . . 〉　マスコミは党の命令で反日報道ばかり
　　　　　 |. .| . .　.|／| 　　 .斗≧=‐----　　　　　　ﾉイ|/　人＿／〉　していますって言ったの。
　　　　　 |. . . . . ∧. .'. 　Ｙ＾Vj弌　　　　　　　　　　 __／ . j. . .イ
　　　　　 . . . . . . . . . .V人　＼__〉　　　　　　　　　　　 |/. . ／＼イ　　すると、それを聞いた日本人が言ったそうよ。
　　　　　＼.〈＼ . . ＼＼ヽ.　　　　 ′　　 ┐ 　　 ∧. ∧. . . .ｿ　　　奇遇ですね、私の国のマスコミも反日報道
　　　　　　　＼　＼ . . ＼トヘ.　　　　く　　ﾉ　　　　　∨__,＞く　　　　ばかりしていますって、
　　　　　　　　　　　｀　‐--- T＼　　　　　　／　　 V￣＼
　　　　　　　　　　　　　　乂__≧=‐-----rく　　　／　　　＼
　　　　　　　　　　　　　　（　＼＿ﾉ､__,ﾉ／|　　／　　　　　ト
　　　　　　　　　　　　　　　　　　　　　　　／　∧ _／　　　　　　　 |　＼

**デフォルトの名無しさん** · 2020/10/04(日) 09:32:48.58

日本自治区しか知らん