- 1 名前：仕様書無しさん [2006/12/26(火) 16:18:46 ]
- この会社辞めようと思ったソースコード。
プログラマとして幻滅するソースコード。
プログラマを悩ませるソースコード。
をつらつらと綴っていって頂戴。
ちなみにここは質問スレじゃないので
技術的な質問がしたいならム板に逝って。
前スレ
この会社辞めようと思ったソースコード＃14
pc8.2ch.net/test/read.cgi/prog/1158190165/
- 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つくらいあったって話？
- 734 名前：仕様書無しさん mailto:sage [2007/03/06(火) 20:39:48 ]
- そんな馬鹿コードは問答無用で書き換えてしまえばスッキリ
- 735 名前：仕様書無しさん mailto:sage [2007/03/06(火) 20:55:40 ]
- しかしなぜかバグ発生
- 736 名前：仕様書無しさん mailto:sage [2007/03/06(火) 22:50:22 ]
- 実は組み込み系なんじゃね？
- 737 名前：仕様書無しさん mailto:sage [2007/03/07(水) 00:43:08 ]
- str = "なんとか１";
str = "なんとか２";
str = "なんとか３";
str = "なんとか４";
str = "なんとか５";
5行で書けた
俺ってすげえ
- 738 名前：仕様書無しさん mailto:sage [2007/03/07(水) 00:47:49 ]
- str = "なんとか１", str = "なんとか２", str = "なんとか３", str = "なんとか４", str = "なんとか５";
1行で書けた
俺ってすげえ
- 739 名前：仕様書無しさん mailto:sage [2007/03/07(水) 01:52:32 ]
- ここでカンマ演算子オーバーロードの登場です。
- 740 名前：仕様書無しさん mailto:sage [2007/03/07(水) 03:10:50 ]
- 次はプリプロセッサによる置換が来ると予想
- 741 名前：仕様書無しさん mailto:sage [2007/03/07(水) 03:21:00 ]
- /*
str = "なんとか１";
str = "なんとか２";
str = "なんとか３";
str = "なんとか４";
str = "なんとか５";
*/
str = "なんとか５";
挑発してやった。俺Tueeeee！
- 742 名前：仕様書無しさん mailto:sage [2007/03/07(水) 17:40:41 ]
- コメントアウトするなら理由と名前を入れろ！
といわれます...
- 743 名前：仕様書無しさん mailto:sage [2007/03/07(水) 18:14:00 ]
- そしてソースコード上のケンカへと発展していくわけですね。
- 744 名前：仕様書無しさん mailto:sage [2007/03/07(水) 18:31:07 ]
- >>743
客との喧嘩議事録をソースで作った同僚がイタｗ
- 745 名前：仕様書無しさん mailto:sage [2007/03/07(水) 18:33:41 ]
- 議事の流れに沿って制御文が書かれていたら嫌だ。
- 746 名前：仕様書無しさん mailto:sage [2007/03/07(水) 19:10:50 ]
- >>745
流れてたんだよ。ﾋﾞｯｸﾘ
- 747 名前：仕様書無しさん mailto:sage [2007/03/08(木) 00:29:54 ]
- while(1) {
customer.loudMouthed("なんで出来ないんだ！簡単だろ！");
programer.loudMouthed("じゃぁ自分でやってみろ！");
}
- 748 名前：仕様書無しさん mailto:sage [2007/03/08(木) 00:50:46 ]
- >>747
それは顧客クラスとプログラマクラスの
メッセージパッシングの無限連鎖で書いたほうが面白いかと
- 749 名前：仕様書無しさん mailto:sage [2007/03/08(木) 00:56:35 ]
- 言い出しっぺの法則を適用
- 750 名前：仕様書無しさん mailto:sage [2007/03/08(木) 01:00:46 ]
- すかしっぺで回避
- 751 名前：仕様書無しさん mailto:sage [2007/03/08(木) 03:00:13 ]
- そうはいかんざき
- 752 名前：仕様書無しさん mailto:sage [2007/03/08(木) 03:06:14 ]
- C++だと何が起きるかわからんからむやみに変更しないほうがいいという
教訓が。
- 753 名前：仕様書無しさん mailto:sage [2007/03/08(木) 07:24:14 ]
- >>752
普通にプログラムが保守を無視してコーディングしてるのがほとんどだから変更できないだけ。
JAVAやC#と同じオブジェクト指向言語としてコーディングされているなら多少は保守可能かな。
どっちにしろ、ポインタ使われると保守性が無視されてしまう可能性があるからね。
- 754 名前：仕様書無しさん mailto:sage [2007/03/08(木) 08:30:55 ]
- ポインタ使われると保守性が無視されてしまう
- 755 名前：仕様書無しさん mailto:sage [2007/03/08(木) 09:28:12 ]
- ボインちゃんを使われると理性が無視されてしまう
- 756 名前：仕様書無しさん mailto:sage [2007/03/08(木) 11:00:39 ]
- 保守不能であるとは
運用終了までの（おそらく複数回の）保守実施コストが
再構築のコストを上回る場合であることを意味する
- 757 名前：仕様書無しさん mailto:sage [2007/03/08(木) 11:04:49 ]
- >>755
自分で保守しるｗ
- 758 名前：仕様書無しさん mailto:sage [2007/03/08(木) 16:46:02 ]
- >>756
1年で作ったシステムを4年も使えば、
再構築より保守の方が金が掛かるが？
「保守」が違うのか．．．？
- 759 名前：仕様書無しさん mailto:sage [2007/03/08(木) 19:12:18 ]
- >>756
不能とはいわんだろう。
- 760 名前：仕様書無しさん mailto:sage [2007/03/08(木) 19:28:46 ]
- if( s == 1 ) {
if( ss > 3 ) {
tt = 12;
} else {
tt = 12;
}
} elseif( s == 2 ) {
if( ss < 2 ){
tt = 12;
} else {
tt = 12;
}
} else{
if( ss > 3 ){
tt = 12;
} else {
tt = 12;
}
}
十九万八千歩譲って、条件が後でいろいろ変更になったとしても、
せめて見直してリファクタリングぐらいしろとorz
（変数名はフェイクいれてあえて適当にしてます
- 761 名前：仕様書無しさん mailto:sage [2007/03/08(木) 20:41:01 ]
- >>760
ワロタｗｗｗ
- 762 名前：仕様書無しさん mailto:sage [2007/03/08(木) 22:15:59 ]
- if( s == 1 ) {
if( ss > 3 ) { // ?
tt = 12;
} else {
tt = 12;
}
} elseif( s == 2 ) {
if( ss < 2 ) { // ?
tt = 12;
} else {
tt = 12;
}
} else {
if( ss > 3 ){
tt = 12;
} else {
tt = 12; // ?
}
}
- 763 名前：仕様書無しさん mailto:sage [2007/03/08(木) 23:09:53 ]
- tt = 12;
だけで充分のような
- 764 名前：仕様書無しさん mailto:sage [2007/03/08(木) 23:25:00 ]
- >>763
帰れ！帰れ帰れっ！！
- 765 名前：仕様書無しさん mailto:sage [2007/03/09(金) 06:55:00 ]
- NOPのつもりなのでは？
- 766 名前：仕様書無しさん mailto:sage [2007/03/09(金) 14:27:07 ]
- ステップ数かせぎ？
- 767 名前：仕様書無しさん mailto:sage [2007/03/09(金) 18:21:23 ]
- >>766
waitみたいなものとか、前後のロジックを同時にプリフェッチやパイプラインに入れたくないとか、いろいろ
- 768 名前：仕様書無しさん mailto:sage [2007/03/09(金) 18:35:34 ]
- >>760
これってコンパイラが最適化してくれるのかな？
- 769 名前：仕様書無しさん mailto:sage [2007/03/09(金) 19:00:21 ]
- >>768
コンパイラ次第だろう。
- 770 名前：仕様書無しさん mailto:sage [2007/03/09(金) 19:36:41 ]
- >>768
こんなのは最適化もワーニングも出さないでエラーにして欲しいな。
- 771 名前：760 mailto:sage [2007/03/09(金) 20:11:47 ]
- 最初は、ｓとssの値に応じてttもそれぞれ違う値が代入されてたらしいです・・・・
度重なる修正や改造によって>760の状態になってしまったのなら、せめてコメント入れて欲しかったですよorz
敢えてこのままにしてあるのか単なる手抜きなのかわからーん！ヽ(`Д´)ﾉ
- 772 名前：仕様書無しさん mailto:sage [2007/03/10(土) 00:09:09 ]
- 中国製のソース全般。
さすがコピー大国、ソースもコピペだらけだぜ！
……orz
- 773 名前：仕様書無しさん mailto:sage [2007/03/10(土) 08:34:24 ]
- >>772
コピペは日本でも当たり前にあるから気にならならない。
コメントがｗ
- 774 名前：仕様書無しさん mailto:sage [2007/03/11(日) 01:11:26 ]
- 中華と言えば変数名で、stlLighteHandleh,stlLefteHandlehって何のことかと思ったよ。
要所要所で微妙に間違えやがってｗ
- 775 名前：仕様書無しさん mailto:sage [2007/03/11(日) 01:39:06 ]
- h→rに関しては余裕で補完できるレベル。
- 776 名前：仕様書無しさん mailto:sage [2007/03/11(日) 04:19:24 ]
- コメントならそういうのの宝庫だよね、中国製。
セマファだのセマフォアだのセマフォンだのｗ
- 777 名前：仕様書無しさん mailto:sage [2007/03/11(日) 08:20:31 ]
- 日本語でも、handol とかやる香具師いるしな。ｗ
- 778 名前：仕様書無しさん mailto:sage [2007/03/11(日) 11:55:13 ]
- つ[uwarite]
- 779 名前：仕様書無しさん mailto:sage [2007/03/11(日) 13:20:08 ]
- 出たｗ
- 780 名前：仕様書無しさん mailto:sage [2007/03/11(日) 13:27:02 ]
- 中国人がpinyinで変数つけてたときは直させたよ。
よめねぇし意味もわからん過ぎる。
と思ったら、今度はローマ字の母音抜きされた
例）一覧->Itiran->Itrn
そっちもそっちでわからんかった。
- 781 名前：仕様書無しさん mailto:sage [2007/03/11(日) 13:42:22 ]
- >>780
>ローマ字の母音抜きされた
日本語のローマ字表記の母音抜きをかましてくるウチの職場より大分マシ。
|
 |
