- 1 名前：仕様書無しさん [2006/12/26(火) 16:18:46 ]
- この会社辞めようと思ったソースコード。
プログラマとして幻滅するソースコード。
プログラマを悩ませるソースコード。
をつらつらと綴っていって頂戴。
ちなみにここは質問スレじゃないので
技術的な質問がしたいならム板に逝って。
前スレ
この会社辞めようと思ったソースコード＃14
pc8.2ch.net/test/read.cgi/prog/1158190165/
- 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 ]
- ﾜｸﾜｸﾃｶﾃｶ
- 661 名前：仕様書無しさん mailto:sage [2007/02/24(土) 17:04:39 ]
- ごめん、いまちょっと俺にひらめきがない。
- 662 名前：仕様書無しさん mailto:sage [2007/02/24(土) 17:35:03 ]
- (･∀･)ｶｴﾚ!
- 663 名前：仕様書無しさん mailto:sage [2007/02/24(土) 23:35:12 ]
- 不発ﾜﾛｽｗ
- 664 名前：仕様書無しさん mailto:sage [2007/02/25(日) 04:37:34 ]
- >>22
- 665 名前：仕様書無しさん mailto:sage [2007/02/25(日) 20:22:21 ]
- この流れで思い出したのは
学生 「あれあれ？怒らせていいんですか？使いますよ。イオナズン。」
面接官「いいですよ。使って下さい。イオナズンとやらを。それで満足したら帰って下さい。」
学生 「運がよかったな。今日はMPが足りないみたいだ。」
面接官「帰れよ。」
- 666 名前：仕様書無しさん mailto:sage [2007/02/26(月) 15:43:02 ]
- これは嫌だ。
182 名前：仕様書無しさん[sage] 投稿日：2007/02/23(金) 17:08:35
何を考えたか、クラスごとに（VSの）プロジェクトを分割したプロジェクト。
クラスといっても全ては画面。互いに参照設定して開発をするらしい。
なんか、バカが頭を使うとろくなことにならんなとしみじみ思った。
- 667 名前：仕様書無しさん mailto:sage [2007/02/26(月) 20:14:01 ]
- >>666
VBで開発してた頃は、跳び先ごとにactivex.exeにしたりしてたけど？
- 668 名前：仕様書無しさん mailto:sage [2007/02/26(月) 20:52:30 ]
- オーメン！
- 669 名前：仕様書無しさん mailto:sage [2007/02/26(月) 23:39:16 ]
- iCount As Long 'Integer
- 670 名前：仕様書無しさん mailto:sage [2007/02/27(火) 00:28:01 ]
- >669
この程度見飽きたわ!! (TT)
- 671 名前：仕様書無しさん mailto:sage [2007/02/27(火) 16:04:57 ]
- 中身はIntegerの範囲しかはいらないけど、ループ変数だから処理速度の関係で
あえてLongにしてるんじゃないの？
- 672 名前：仕様書無しさん mailto:sage [2007/02/27(火) 16:07:18 ]
- >>669
かっこいいスペルだな！
- 673 名前：仕様書無しさん mailto:sage [2007/02/27(火) 16:07:18 ]
- ｌCount As Long 'Integer
って書けっつーこと？
- 674 名前：仕様書無しさん mailto:sage [2007/02/27(火) 17:22:47 ]
- >>673
そういう話だとしたら、、、
くっだらねえな。
- 675 名前：仕様書無しさん mailto:sage [2007/02/27(火) 17:25:38 ]
- なんだ、「私はカウントする者」という自己主張なのかと思った。
- 676 名前：仕様書無しさん mailto:sage [2007/02/27(火) 17:37:41 ]
- １．"iCount As Long 'Integer"って書いた人は、ハンガリアン記法が間違ってる
２．>>669がVBの最適化を知らなかった
３．両方
３ってこと？
- 677 名前：仕様書無しさん mailto:sage [2007/02/27(火) 17:38:56 ]
- >>675はﾜﾛｽ
- 678 名前：仕様書無しさん mailto:sage [2007/02/27(火) 20:25:48 ]
- >>676
始めはこれ
iCount As Integer
↓
後で桁数が足りないことに気づいたが、
変数名を置換するのが面倒で型のみを変更した。
iCount As Long 'Integer
答えは４．手抜き
- 679 名前：仕様書無しさん mailto:sage [2007/02/27(火) 21:05:07 ]
- 君はこの地獄に生きる我々にその程度のネタを開陳してどうしたかったのだ？
- 680 名前：仕様書無しさん mailto:sage [2007/02/27(火) 21:36:30 ]
- 開ﾁﾝ？
- 681 名前：仕様書無しさん mailto:sage [2007/02/27(火) 21:44:26 ]
- 半刈り案なんか強要される莫迦な職場は辞めたくなるな。
- 682 名前：仕様書無しさん mailto:sage [2007/02/28(水) 00:50:48 ]
- ラベルに限って日本語変数名はいがいと便利だと思うようになってきたり…
- 683 名前：仕様書無しさん mailto:sage [2007/02/28(水) 02:21:10 ]
- ラベルにコメントをつけるんだ。
1度坂道を転がり落ちたら下まですぐだぞ！
- 684 名前：仕様書無しさん mailto:sage [2007/02/28(水) 02:29:11 ]
- CP/M使ってたころならともかく
もう識別子に日本語使えるようになってもいいんぢゃね？
- 685 名前：仕様書無しさん mailto:sage [2007/02/28(水) 02:35:17 ]
- 書きにくい
- 686 名前：仕様書無しさん mailto:sage [2007/02/28(水) 02:37:09 ]
- >678
ぶっちゃけその程度は
このスレ的には「ふーん。よくあるね。で？」ってなレベル
- 687 名前：仕様書無しさん mailto:sage [2007/02/28(水) 07:26:48 ]
- >>684
最近のJavaでは、使えるらしいが、
幸い使った事も使われてるのをみた事も無いな。
- 688 名前：仕様書無しさん mailto:sage [2007/02/28(水) 12:07:55 ]
- 昔Lispで試したことがあるが
関数とグローバル変数が日本語だと物凄く読み易くなることが分かった
入力が面倒なのが難点（ローマ字で打っておいてまとめて置換ｗ）
- 689 名前：仕様書無しさん mailto:sage [2007/02/28(水) 20:44:15 ]
- 俺のコメントが669のものだと勘違いしている奴多すぎｗ
- 690 名前：仕様書無しさん mailto:sage [2007/02/28(水) 21:11:09 ]
- まぁ、君はそれほどつまらんオチをつけたということだよ。
誇っていい。
- 691 名前：仕様書無しさん mailto:sage [2007/02/28(水) 22:55:42 ]
- >687
まさにそれが横溢しているプロジェクトにいた
で、日本語クラス名変数名メソッド名使ってないと思ったら、一転、謎識別子のオンパレード……
(GYMKYK0500132A.javaとか。汎用機のCOBOLバリバリなシステムをリプレースしたもんだからって……なぁ)
どうにか半年強で抜けてきた……
- 692 名前：仕様書無しさん mailto:sage [2007/03/01(木) 12:00:15 ]
- ソースコードじゃないが、ダイアログで
△更新が完了しました
OK →キャンセル
一体何がキャンセルされるのかと…。
何も起こらないけど。
- 693 名前：仕様書無しさん mailto:sage [2007/03/01(木) 12:12:10 ]
- ループってIntegerよりLong使ったほうが速くなるもんなの？
- 694 名前：仕様書無しさん mailto:sage [2007/03/01(木) 19:38:46 ]
- 32bit環境ならlongのが早い
- 695 名前：仕様書無しさん mailto:sage [2007/03/01(木) 21:41:26 ]
- 今、調べて知ったんだがVB6の整数型って16bit環境みたいな名前なんだな
今までもこれからも使うことない知識だろうが
- 696 名前：仕様書無しさん mailto:sage [2007/03/01(木) 21:56:57 ]
- 旧資産の超拡張でしかないからな
- 697 名前：仕様書無しさん mailto:sage [2007/03/01(木) 23:30:41 ]
- >>684
VC++2005なら。
- 698 名前：仕様書無しさん mailto:sage [2007/03/02(金) 14:41:19 ]
- VBって最適化とかないの？
- 699 名前：仕様書無しさん mailto:sage [2007/03/02(金) 14:57:16 ]
- VBを選択しないことが最適化
- 700 名前：仕様書無しさん mailto:sage [2007/03/02(金) 15:51:37 ]
- 奇才現る
- 701 名前：仕様書無しさん mailto:sage [2007/03/02(金) 16:15:55 ]
- >>698-699
VBの開発は、まずこういう奴らを黙らせることから始まる。
- 702 名前：仕様書無しさん mailto:sage [2007/03/02(金) 16:38:00 ]
- やっぱ、拳と拳の語り合いですか
- 703 名前：仕様書無しさん mailto:sage [2007/03/02(金) 22:13:58 ]
- おまえもマならコードで語るんだな。
- 704 名前：仕様書無しさん mailto:sage [2007/03/03(土) 11:27:37 ]
- VB以外のコードで語ってもVB厨には読めません。
- 705 名前：仕様書無しさん mailto:sage [2007/03/03(土) 20:44:07 ]
- 逆に考えるんだ。VB厨はマじゃないんだ。そう考えるんだ。
- 706 名前：仕様書無しさん mailto:sage [2007/03/03(土) 22:25:46 ]
- アクセス使えます！さらにVBでカスタマイズも出来るんですっ！！
ODBC経由のDBならなんでも操作できますぅっっっっｖ！！！１
データリンク貼るだけぇえええええええええええええええええええええ
- 707 名前：仕様書無しさん mailto:sage [2007/03/05(月) 15:00:00 ]
- 辞めようと思ったほどじゃないけど
for( i=0; i<5; i++ ){
Switch(i) {
case 0: str = "なんとか１"; break;
case 1: str = "なんとか２"; break;
case 2: str = "なんとか３"; break;
case 3: str = "なんとか４"; break;
case 4: str = "なんとか５"; break;
default:break;
}
}
こういうのをそこら中で見ると、イライラするｗ
ループにする必然性ないだろｗｗｗ
- 708 名前：仕様書無しさん mailto:sage [2007/03/05(月) 15:57:12 ]
- >>707
こんな風に書きたかった動機はなんだろう。これ。
さっぱり真意が読めない。
- 709 名前：仕様書無しさん mailto:sage [2007/03/05(月) 16:36:48 ]
- >>707
これは変数strを上書きしてるぞｗ
ループ以前の問題だな。
真意はコードの水増しが考えられるけど。
- 710 名前：仕様書無しさん mailto:sage [2007/03/05(月) 16:56:24 ]
- str(i)になっていてもちょっとひく。
- 711 名前：仕様書無しさん [2007/03/06(火) 00:08:47 ]
- >>707
このコードってしょっちゅう俺が使うコードだよ。
ステップ数も少なくて済むし、なによりも解りやすいのが利点だと思うが。
ループにする必然性が無いとか言う奴笑える。
己の理解できないものを否定する奴こそしんの馬鹿なり。
- 712 名前：仕様書無しさん mailto:sage [2007/03/06(火) 00:13:26 ]
- >>711
>>711
>>711
- 713 名前：仕様書無しさん mailto:sage [2007/03/06(火) 00:16:53 ]
- >>711
>>711
>>711
>>711
>>711
>>711
>>711
>>711
>>711
- 714 名前：仕様書無しさん mailto:sage [2007/03/06(火) 00:31:42 ]
- >>711は何を釣りたいのだろうｗ
- 715 名前：仕様書無しさん mailto:sage [2007/03/06(火) 00:35:11 ]
- >>714
<<712-714
- 716 名前：仕様書無しさん mailto:sage [2007/03/06(火) 00:36:15 ]
- >>711
sage忘れないでくださいね。
- 717 名前：仕様書無しさん mailto:sage [2007/03/06(火) 00:43:07 ]
- >>715
「<<」は何ですか？
- 718 名前：仕様書無しさん mailto:sage [2007/03/06(火) 00:45:32 ]
- >>717
左のShiftキーを5回押す
- 719 名前：仕様書無しさん mailto:sage [2007/03/06(火) 00:48:03 ]
- Ctrl+Shift+Escだろ
- 720 名前：仕様書無しさん mailto:sage [2007/03/06(火) 00:49:57 ]
- >>711
ループや分岐やマジックナンバーといった人間が理解しにくい
構造をわざわざ使わなくても、
std::cout
<< "なんとか1" << std::endl
<< "なんとか2" << std::endl
<< "なんとか3" << std::endl;
とか、
outputNantoka("なんとか1");
outputNantoka("なんとか2");
outputNantoka("なんとか3");
ストレートに書けばいいんじゃ・・・？
- 721 名前：仕様書無しさん mailto:sage [2007/03/06(火) 00:51:00 ]
- >>717
左ビットシフト演算
- 722 名前：仕様書無しさん mailto:sage [2007/03/06(火) 00:54:26 ]
- >>720
意味が変わってる
- 723 名前：仕様書無しさん mailto:sage [2007/03/06(火) 00:55:33 ]
- >ループや分岐やマジックナンバーといった人間が理解しにくい
>構造をわざわざ使わなくても、
>>720
>>720
>>720
>>720
>>720
>>720
- 724 名前：仕様書無しさん mailto:sage [2007/03/06(火) 01:09:14 ]
- ぶっちゃけマジックナンバーは人間にわかりやすい
- 725 名前：仕様書無しさん mailto:sage [2007/03/06(火) 01:43:12 ]
- str = "なんとか５";
1行で書けた
俺ってすげえ
- 726 名前：仕様書無しさん mailto:sage [2007/03/06(火) 04:04:44 ]
- 奇才現る
- 727 名前：仕様書無しさん mailto:sage [2007/03/06(火) 09:56:00 ]
- >>725
それに気づかない奴が沢山釣られたなｗ
- 728 名前：仕様書無しさん mailto:sage [2007/03/06(火) 11:11:19 ]
- >>727
ha?
- 729 名前：仕様書無しさん mailto:sage [2007/03/06(火) 12:38:21 ]
- と思ったら = がオーバーロードされてて例外投げるんだぜ
- 730 名前：仕様書無しさん mailto:sage [2007/03/06(火) 15:50:14 ]
- 同じところかもしれんが
for( i=0; i<200; i++ ){
switch(i) {
case 0: str[i] = "なんとか１"; break;
case 1: str[i] = "なんとか２"; break;
case 2: str[i] = "なんとか３"; break;
（中略）
case 199: str[i] = "なんとか２００"; break;
default: str[i] = "なんとかかんとか"; break;
}
}
3個くらいdefaultに飛んでた・・・
- 731 名前：仕様書無しさん [2007/03/06(火) 16:17:53 ]
- >>730
defaultに飛ぶ処理が3つもあるなら、意味はあるのかも。
でも、書き方は変えたほうが良い。
- 732 名前：仕様書無しさん mailto:sage [2007/03/06(火) 16:19:59 ]
- なんで動的に代入するんだ
テーブルにしとけよ
- 733 名前：仕様書無しさん mailto:sage [2007/03/06(火) 17:48:15 ]
- 書き忘れが3つくらいあったって話？
|
 |
