- 1 名前：仕様書無しさん [2006/12/26(火) 16:18:46 ]
- この会社辞めようと思ったソースコード。
プログラマとして幻滅するソースコード。
プログラマを悩ませるソースコード。
をつらつらと綴っていって頂戴。
ちなみにここは質問スレじゃないので
技術的な質問がしたいならム板に逝って。
前スレ
この会社辞めようと思ったソースコード＃14
pc8.2ch.net/test/read.cgi/prog/1158190165/
- 560 名前：仕様書無しさん [2007/02/03(土) 02:58:26 ]
- 眠ったままブログを更新するとは、流石、スーパーコディネーター！
- 561 名前：仕様書無しさん mailto:sage [2007/02/03(土) 03:13:43 ]
- d.hatena.ne.jp/y_aoi/20061221
これも、よくよく内容みてみたらぐだぐだやね
- 562 名前：仕様書無しさん mailto:sage [2007/02/03(土) 03:14:48 ]
- ちゅーかヲチ板逝くか別スレ立ててやってくれんか
- 563 名前：仕様書無しさん [2007/02/03(土) 03:23:58 ]
- >>562
必要を感じないな。
- 564 名前：仕様書無しさん [2007/02/03(土) 03:30:02 ]
- >>561
→>>427とかな
- 565 名前：仕様書無しさん mailto:sage [2007/02/03(土) 03:33:30 ]
- ＞当然のようにファイル一個だけの open/close の HP は良く見かけるが
＞二つ以上となると余り書かれていません。ていうか 0 です。
書くまでもないからじゃないの？
- 566 名前：仕様書無しさん mailto:sage [2007/02/03(土) 04:08:34 ]
- > C が 0.0000… になるが、これは main(){printf("%c %d %f", A, B, 2);} この場合と同じ事。型の優先度が enum よりも高い。
ﾜﾛﾀ
- 567 名前：仕様書無しさん [2007/02/03(土) 04:21:40 ]
- >>558
ああ、そうか、
「教授のスキルが10とすれば、学校を出た時点でオレは5か6」
とか言うのが、その日の分、丸ごとなくなっているんだな。
- 568 名前：仕様書無しさん [2007/02/03(土) 04:36:48 ]
- 心臓が凍りつくのはこれだな。
d.hatena.ne.jp/y_aoi/20061101
>(char *)malloc() コレなんでキャスト必要なの？ malloc って void 型じゃん
- 569 名前：仕様書無しさん mailto:sage [2007/02/03(土) 05:32:43 ]
- ;:ﾞ;｀(;ﾟ;ж;ﾟ; )ﾌﾞﾌｫ
- 570 名前：仕様書無しさん mailto:sage [2007/02/03(土) 06:39:17 ]
- >>568
それがHigh Performance Programmingクオリティって
やつだから勘弁してやってくれ。
>今日 C しか打てないプログラマの為のキャストだと知りました(意外に本当かも)
じゃなくてCも使えないプログラマなんだからそっとしといてあげてくださいデス。
- 571 名前：仕様書無しさん mailto:sage [2007/02/03(土) 09:41:58 ]
- >>511
「このユーザーのページにはアクセスできません」って言われちゃったよ。
- 572 名前：仕様書無しさん mailto:sage [2007/02/03(土) 12:26:14 ]
- 今起きたらhatenaがメンテ・・・
- 573 名前：葉猫 ◆Jz.SaKuRaM mailto:sage [2007/02/03(土) 13:11:40 ]
- コーディネーターが現れるとスピードが違うな (｀･ω･´) ｼｬｷｰﾝ
- 574 名前：仕様書無しさん mailto:sage [2007/02/03(土) 13:36:22 ]
- いっそもの凄い勢いでOSを書き換えてみてくれｗ
- 575 名前：仕様書無しさん mailto:sage [2007/02/03(土) 13:40:30 ]
- いい加減死ねよ糞コテ
- 576 名前：仕様書無しさん mailto:sage [2007/02/03(土) 13:55:08 ]
- 粘着しすぎ
- 577 名前：仕様書無しさん [2007/02/03(土) 13:58:31 ]
- >>576
だって、面白過ぎ、と言うか、
オモコワ！
- 578 名前：仕様書無しさん mailto:sage [2007/02/03(土) 14:04:17 ]
- でもさすがにネタ切れ感は否めないね
- 579 名前：仕様書無しさん [2007/02/03(土) 14:19:47 ]
- >>578
hatenaがメンテ中だからね。
そこで、
www.ogaki-tv.ne.jp/~freedom/
のソース内の"The Prince Of Darkness"だが、
ジョン・カーペンターの映画は"The"が就かない様だ。
"The"がつく作品を探してみた結果は以下にたどり就いた。
www.b-ch.com/cgi-bin/contents/ttl/det.cgi?ttl_c=444
これや、プロファイルのイラストを見ると、URLのfreedomも違った意味に取れる。
- 580 名前：仕様書無しさん mailto:sage [2007/02/03(土) 14:30:40 ]
- 要はｵﾀってこと？
ｵﾀは別にいいや・・・。俺もそうだしｗｗ
- 581 名前：仕様書無しさん [2007/02/03(土) 14:43:10 ]
- >>580
そう、アニオタ。そして自称モデル。
- 582 名前：仕様書無しさん mailto:sage [2007/02/03(土) 15:14:00 ]
- こいつのはオタだから面白いってわけじゃないだろ。
- 583 名前：仕様書無しさん mailto:sage [2007/02/03(土) 15:21:28 ]
- ｽｰﾊﾟｰﾊｶｰとかそういうジャンルの面白さだよねw
- 584 名前：仕様書無しさん mailto:sage [2007/02/03(土) 15:34:57 ]
- 結城 葵
- 585 名前：仕様書無しさん mailto:sage [2007/02/03(土) 15:41:57 ]
- >>584
新ジャンル？
- 586 名前：仕様書無しさん mailto:sage [2007/02/03(土) 16:19:23 ]
- >>556
晒すほどのもんでもないね
- 587 名前：仕様書無しさん mailto:sage [2007/02/03(土) 20:05:41 ]
- スレ汚しすま（自分と同じトラブルにあってサポートに「そんな事例はありません」といわれ
泣き寝入りしている人の為に以下の情報を残します。）
J-COMが提供しているマイシールドがバージョンアップされたがそれを適用した所、
・マシン終了時、ハングアップして終了しない
・次のマシン起動以降、３～５分でsvchost.exeが死にマシンがまともに動かなくなる。
（マウスカーソル以外動かない状態になる。）
またIEなどでインターネットを見ていたとして、見えなくなってしまう。
なんとかマイシールドをアンインストールしたが、それでも挙動はおかしいまま変わらない。
WindowsXPHomeを再インストールしまともに動いている所に、またマイシールドをインストールしたら
また同じ挙動になった。
※ マイシールドが入っている状態で、無効化すると５分以上過ぎてもまともに動いていた。
こんな現象に出会った方、J-COMにちゃんとクレームしましょう。
（クレームが入っていない、と言い張っているが、少なくとも私はクレームしました。）
- 588 名前：仕様書無しさん mailto:sage [2007/02/03(土) 20:17:20 ]
- >>587
市ね
- 589 名前：仕様書無しさん mailto:sage [2007/02/03(土) 21:05:42 ]
- 固定小数点の四捨五入
a = ((b & 0xFFC0) + (((b & 0x3F) / 0x20) * 0x40));
- 590 名前：仕様書無しさん mailto:sage [2007/02/04(日) 06:46:47 ]
- 後半がｗ
- 591 名前：仕様書無しさん mailto:sage [2007/02/04(日) 10:57:09 ]
- >>472
mixi.jp/show_friend.pl?id=6414557
- 592 名前：仕様書無しさん mailto:sage [2007/02/05(月) 19:30:28 ]
- ほげ
- 593 名前：仕様書無しさん mailto:sage [2007/02/05(月) 22:51:41 ]
- なんか新しいネタないの？
- 594 名前：仕様書無しさん mailto:sage [2007/02/05(月) 23:00:11 ]
- >25
ゲーム業界じゃ当たり前。
- 595 名前：仕様書無しさん mailto:sage [2007/02/06(火) 00:44:46 ]
- じゃあ、アンケート
www.atmarkit.co.jp/im/carc/serial/redge51/redge51.html
仕事で上下どっちに近いコードを書いている？
上 1
- 596 名前：仕様書無しさん mailto:sage [2007/02/06(火) 01:17:38 ]
- >>595
下１
- 597 名前：仕様書無しさん mailto:sage [2007/02/06(火) 01:38:33 ]
- >>595
目くそ鼻くそで選ぶ気になれねえ
- 598 名前：仕様書無しさん mailto:sage [2007/02/06(火) 02:08:44 ]
- 下の方が多少いいんだろうけど、コメントない時点でうんこだわ。
- 599 名前：仕様書無しさん mailto:sage [2007/02/06(火) 02:21:08 ]
- 五十歩百歩。
だけど綺麗なコードを提示しろと言われると
面倒で面倒で
- 600 名前：仕様書無しさん mailto:sage [2007/02/06(火) 03:35:35 ]
- 下っぽく書こうとして気づいたら上になってる １
- 601 名前：仕様書無しさん mailto:sage [2007/02/06(火) 08:55:11 ]
- コメントはここでは問題ないんじゃ
そういう融通のきかない人がいる会社もどうなんですかね
- 602 名前：仕様書無しさん mailto:sage [2007/02/06(火) 10:43:11 ]
- >>601
一行目：
正論
二行目：
人が融通がきかないのではなく、
属している組織自体の考え方が融通がきかない
のではないかと思われる
- 603 名前：仕様書無しさん mailto:sage [2007/02/06(火) 10:50:22 ]
- 個人的には、上は有り得ない。
- 604 名前：仕様書無しさん mailto:sage [2007/02/06(火) 10:51:49 ]
- leadersGamesなどに代入した値を、再びgamesWonからみる理由が分からない。
- 605 名前：仕様書無しさん mailto:sage [2007/02/06(火) 10:59:04 ]
- >>603
決定表をIF文の羅列に落とすことはあるが、
上の例は、上手く出来ていない。
わざと上手く出来ていないので、仕方ない。
- 606 名前：仕様書無しさん mailto:sage [2007/02/06(火) 11:34:02 ]
- if( ( game[0] == 7 ) ||
( game[0] == 6 && game[1] < 5 ) ) {
return "Player1 wins the set " + games[0] + " - " games[1];
}
if( ( game[1] == 7 ) ||
( game[1] == 6 && game[0] < 5 ) ) {
return "Player2 wins the set " + games[1] + " - " games[0];
}
if( games[0] > games[1] ) {
return "Player1 leads " + games[0] + " - " + games[1];
}
if( games[0] < games[1] ) {
return "Player2 leads " + games[1] + " - " + games[0];
}
return "Set is tied at " + games[0] + "games";
上の方式でif列挙にするとしたらこうだよな。
あとは数値の定数化。
ついでにgamesWon関数で数値の範囲例外処理行わないとNG。
引数のPlayerが4とか来た場合とか、
呼ばれすぎて勝利数が両方8とかになったらどうするつもりかと小一時間(ry
- 607 名前：仕様書無しさん mailto:sage [2007/02/06(火) 12:20:01 ]
- >>602
598も今の会社をやめればまともになる
＝
悪いのは全て会社の体質というのがこのスレの主旨
ということですかね？
- 608 名前：仕様書無しさん [2007/02/06(火) 12:28:44 ]
- >>601のほうが記事の意図に合わせようとして
逆に融通が利いてない気がするが
両方とも×という答えのほうがいいと思う
俺も正直上も下もどっちもどっちだと思う｡
game[0]とか意味がわからん
- 609 名前：仕様書無しさん mailto:sage [2007/02/06(火) 12:49:25 ]
- >>608のおかげで、会社をやめたくなるようなコードが生成される理由を
垣間見れた気がする
- 610 名前：仕様書無しさん mailto:sage [2007/02/06(火) 14:32:56 ]
- >>609
「意味がわからん」って、業務知識の無さを誇られても困るよね・・・
テニスではセット内での得点のことをゲームといいます、とか
コメントに書けってか？
- 611 名前：仕様書無しさん mailto:sage [2007/02/06(火) 14:38:57 ]
- >>610
ずれてる、ずれてるｗ
- 612 名前：仕様書無しさん mailto:sage [2007/02/06(火) 15:25:16 ]
- >>611
あながちズレてないんじゃない？
- 613 名前：仕様書無しさん mailto:sage [2007/02/06(火) 15:29:37 ]
- 609ではないが、テニスがわからないとメンテできないプログラムというのはどうか。
業務で使うソースにはもっとコメントを書いて欲しいところだね。
- 614 名前：仕様書無しさん mailto:sage [2007/02/06(火) 15:37:58 ]
- 609や613は、例えば携帯電話のモックアップを見せられて、
「モックAとモックBのどちらのデザインがいいですか？」
と聞かれて
「どちらのモックアップも通話機能が搭載されてないからゴミです」
とか答えるんですか？
- 615 名前：仕様書無しさん [2007/02/06(火) 16:04:02 ]
- デザインと通話機能は全く関係ないんだから
そんなこと答えるわけないだろｗ
- 616 名前：仕様書無しさん mailto:sage [2007/02/06(火) 16:09:50 ]
- 俺的には下だと思うんだが、同僚にいわせると上らしい。納得いかない。
気分悪いからもう帰る。
- 617 名前：仕様書無しさん mailto:sage [2007/02/06(火) 16:16:38 ]
- >>613
作る人、メンテする人が業務知識を持ってることは前提じゃないか？
テニスに関係したものならテニスに関係した知識を持っておくのが当然でしょ。
普段できない子扱いされてない？大丈夫？イ㌔
- 618 名前：仕様書無しさん mailto:sage [2007/02/06(火) 16:30:33 ]
- なんかズレまくってるようだけど、
595のやつって要は論理演算ちゃんとわかってますか？
ってことでしょ
サンプルコードが悪いけど、
少ない分岐に場合わけできるのにそれをしないでダラダラIF文書いても
冗長になって汚くなるよってことがいいたいんじゃないの？
- 619 名前：仕様書無しさん [2007/02/06(火) 17:41:41 ]
- 上のほうが、直感的にわかりやすいんじゃね
下は若干頭をひねる必要があるが
どっちがクリーンかっていってもなー
下がせめて3行くらいにまとまってれば下がいいと思うかも知れんが｡
まあこれ作った人も、最初はもっときれいになると思ってたんだと思うが
- 620 名前：仕様書無しさん mailto:sage [2007/02/06(火) 17:46:07 ]
- オレならこんな感じかな
テニスの事は良く知らんから、間違ってるかも
if ( gamesWon[0] == gamesWon[1] ) {
return "Set is tied at " + gamesWon[0] + "games";
}
String leader = null;
int[] point = null;
if ( gamesWon[0] > gamesWon[1] ) {
leader = "Player1"; point[0] = gamesWon[0]; point[1] = gamesWon[1];
} else {
leader = "Player2"; point[0] = gamesWon[1]; point[1] = gamesWon[0];
}
if ( point[0] == 7 ) {
return leader + " wins the set 7 - " + point[1];
}
return leader + " leads " + point[0] + " - " + point[1];
- 621 名前：620 mailto:sage [2007/02/06(火) 17:48:00 ]
- ×int[] point = null;
○int[] point = new int[2];
- 622 名前：仕様書無しさん mailto:sage [2007/02/06(火) 17:52:49 ]
- ネタにしても酷いな。
- 623 名前：620 mailto:sage [2007/02/06(火) 18:07:04 ]
- うわあ変数名間違ってる
gamesWon じゃなくて全部 games ね
あと、
if ( point[0] == 7 ) { ？ }
は
if ( point[0] == 7 || (point[0] == 6 && point[1] < 5) ) {
return leader + " wins the set " + point[0] + " - " + point[1];
}
て事で
見直すと >>606 と大差無いな
- 624 名前：仕様書無しさん mailto:sage [2007/02/06(火) 18:52:57 ]
- ﾔﾚﾔﾚ
お前らほんっっとバカ！！！
- 625 名前：仕様書無しさん mailto:sage [2007/02/06(火) 20:00:36 ]
- 盛り上がってんなー
無意味に
- 626 名前：仕様書無しさん [2007/02/06(火) 20:31:36 ]
- 誰か任天堂でテニス作ったやつはいないのか？？？
- 627 名前：仕様書無しさん mailto:sage [2007/02/06(火) 20:35:08 ]
- 俺的にはpoint[0]だのpoint[1]で終わってると思う
配列使ってる時点で、point[2]とかの存在も想像できるから
pointの意味がまったく意味不明となってしまう
すなわち元記事のやつらにえらそうなこと言われる筋合いは無いということだ
- 628 名前：仕様書無しさん mailto:sage [2007/02/06(火) 20:48:52 ]
- >>610
テニスが業務知識かよ
って、業務と全然関係ないんですけど
的外れもいいところ
- 629 名前：仕様書無しさん mailto:sage [2007/02/06(火) 20:52:38 ]
- >>628
対象のメタテクストは要求されて然るべき。
例文のプログラムの対象はテニス。
現状で、お前の業務がテニスで無いだけ。
- 630 名前：仕様書無しさん mailto:sage [2007/02/06(火) 20:54:31 ]
- String setScoreMessage = null;
エエー
- 631 名前：仕様書無しさん mailto:sage [2007/02/06(火) 20:56:11 ]
- あはは
- 632 名前：仕様書無しさん mailto:sage [2007/02/06(火) 20:56:33 ]
- 結論。
どっちもどっち。
- 633 名前：仕様書無しさん mailto:sage [2007/02/06(火) 20:57:11 ]
- >>628
＞テニスが業務知識かよ
＞って、業務と全然関係ないんですけど
えー。要件も読めないんですかー。
コーディングスタイル以前の話だな・・・
元の記事より引用
＞ ここで、同じ処理を行う2つの短いコードを見てみたい。これは、テニスのセットカウントを数えるコードだ［注3］。
- 634 名前：仕様書無しさん mailto:sage [2007/02/06(火) 21:03:00 ]
- 趣味：お琴とテニス
おとことペニス
- 635 名前：仕様書無しさん mailto:sage [2007/02/06(火) 21:03:15 ]
- >>630
String setScoreMessage = "";
が良いと言うこと？
- 636 名前：仕様書無しさん mailto:sage [2007/02/06(火) 21:03:59 ]
- テニスごときで一生懸命になるなよアホか
- 637 名前：仕様書無しさん mailto:sage [2007/02/06(火) 21:07:16 ]
- ステップ数論争のときも思ったけど、
おかしな人がいるね。
- 638 名前：仕様書無しさん [2007/02/06(火) 21:13:19 ]
- long a,b;
・・・
a = (long)b;
- 639 名前：仕様書無しさん mailto:sage [2007/02/06(火) 21:34:06 ]
- ここの連中みてたら、昨日までこの会社やめたいって思ってたコードが
ましにみえてきた
- 640 名前：仕様書無しさん mailto:sage [2007/02/06(火) 21:51:37 ]
- やめたいって思ってたコードを詳しく
- 641 名前：仕様書無しさん mailto:sage [2007/02/06(火) 22:41:44 ]
- 上の方がいい点は引数に意味ある名前ついてるとこだな。
というわけでどっちもウンコ。
まあ英語が普通に読めるなら下は問題ないのかもしれんが。
VBAの変数が日本語なのと大差ない感じだと思うがね。
- 642 名前：595 mailto:sage [2007/02/07(水) 01:48:27 ]
- おー、仕事にゆとりある人けっこう多いのね。
「どちらとも言えない」って、力みすぎな気がする。
>>595でも言ったけど、いちおうコード書いてご飯食べてる人対象ってことで。
- 643 名前：仕様書無しさん mailto:sage [2007/02/07(水) 02:12:33 ]
- どっちのコードもまずgameWon()メソッドがおかしいし
引数がそのまま配列のインデックスとして扱われるなんて・・・
- 644 名前：仕様書無しさん mailto:sage [2007/02/07(水) 02:28:12 ]
- >>643
playerだからさほど気にならなかった。
言われてみれば、入力値チェックくらい欲しいかも。
- 645 名前：仕様書無しさん mailto:sage [2007/02/07(水) 02:33:15 ]
- >>643
なんでおかしいの？テニスではプレイヤー（ダブルスではチーム）は絶対に２つしかないから
スコアをintの配列で扱ってもなんら問題は無いと思うけども。。。。
で、どっちがいいかって言うとコメントなしで考えるなら上のソースのほうが読んで
仕様が理解しやすい。
んでも拡張性で考えると、どっちもどっちな気がする。
たとえば、>>595のソースは７ゲーム先取したほうが勝ちっていう仕様だけど
実際のテニスみたいにタイブレークが永遠に続くようにって仕様変更される
ことを考えるとどっちも修正は容易じゃないきがするね。
勝利判定のメソッドをもうひとつ挟み込めば結構すっきりするし拡張性も
あがるんじゃないかとなんとなく思った。
- 646 名前：仕様書無しさん mailto:sage [2007/02/07(水) 02:38:42 ]
- >>644
チェックしてどうすんの？ｗ
1か2以外がきたら明らかなバグだからException投げてもらわないと困るだろｗ
チェックしても結局Exception投げるしかないんだからチェックの意味なす
- 647 名前：606 mailto:sage [2007/02/07(水) 10:22:11 ]
- >>623
いんや、根本が違う。
漏れのは上方式。
おまいさんのは下方式。
>>645
そのへんの指摘は606で書いてる。
勝利条件を6以上かつ差が2以上あいたとき、
にしておけばタイブレークは問題ないと思われ。
- 648 名前：606 mailto:sage [2007/02/07(水) 10:24:12 ]
- 補足すると606でああいう勝利条件判定にしたのは
オリジナルの勝利条件判定との差異を入れたくなかったから。
バカ要求仕様（コメント無し勝利条件おかしいエラーチェック無し）どおりに作ったうえで
ソースコードの見易さを比較するのが今回の話のキモだからな。
- 649 名前：仕様書無しさん mailto:sage [2007/02/07(水) 12:04:24 ]
- > 実際のテニスみたいにタイブレークが永遠に続くようにって仕様変更される
タイブレークがどんだけ続いても、最終的には7-6か6-7のはずだが…
- 650 名前：仕様書無しさん mailto:sage [2007/02/07(水) 13:56:11 ]
- おぉ。こんな単純であるはずの問題にも
仕様の解釈誤りの問題が。
- 651 名前：仕様書無しさん mailto:sage [2007/02/07(水) 14:23:32 ]
- C#2.0にて
List<ArrayList>
- 652 名前：仕様書無しさん mailto:sage [2007/02/07(水) 19:21:21 ]
- 引き継いだVCプログラムで、グリッドに列を追加して欲しいって
要望があったので修正しようとコード見たら列番号が
全部マジックナンバーで指定してある・・・
しかたなしに'4'とか'5'で検索してみたら
m_flg = 4;とかm_flg = 5;らしきコードがコメントも無く・・・
見なきゃ良かったorz
- 653 名前：仕様書無しさん mailto:sage [2007/02/09(金) 12:54:29 ]
- >>650
たとえ問題が単純でも、ついでに頭まで単純だとこういうことになる。
- 654 名前：仕様書無しさん mailto:sage [2007/02/09(金) 19:01:56 ]
- ここの連中、馬鹿ばっかりだもんな
- 655 名前：仕様書無しさん mailto:sage [2007/02/14(水) 17:29:37 ]
- 馬鹿なほうが生きていくのが楽だとか聞いた事がある。
>>654はどうなの？
- 656 名前：仕様書無しさん mailto:sage [2007/02/14(水) 21:13:00 ]
- 馬鹿ばっかりで困ってます
- 657 名前：仕様書無しさん mailto:sage [2007/02/23(金) 14:29:55 ]
- 左のパーツと右のパーツの動きが同じ仕様のある製品。
プロジェクトの途中から加わりソースを眺めていると
ほぼ同じような構成の、左右2つ分のファイルの組が沢山あった
- 658 名前：仕様書無しさん mailto:sage [2007/02/24(土) 15:24:54 ]
- うちの会社のバカのコードをここに無造作に張ると、たいてい30は罵倒のレスが付く。
場合によっては荒れる。
- 659 名前：仕様書無しさん mailto:sage [2007/02/24(土) 16:07:54 ]
- よし貼れ
- 660 名前：仕様書無しさん mailto:sage [2007/02/24(土) 16:27:54 ]
- ﾜｸﾜｸﾃｶﾃｶ
|
 |
