- 1 名前：仕様書無しさん [2006/12/26(火) 16:18:46 ]
- この会社辞めようと思ったソースコード。
プログラマとして幻滅するソースコード。
プログラマを悩ませるソースコード。
をつらつらと綴っていって頂戴。
ちなみにここは質問スレじゃないので
技術的な質問がしたいならム板に逝って。
前スレ
この会社辞めようと思ったソースコード＃14
pc8.2ch.net/test/read.cgi/prog/1158190165/
- 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
>ローマ字の母音抜きされた
日本語のローマ字表記の母音抜きをかましてくるウチの職場より大分マシ。
- 782 名前：仕様書無しさん mailto:sage [2007/03/11(日) 13:54:58 ]
- wktk
- 783 名前：仕様書無しさん [2007/03/11(日) 16:12:44 ]
- ∧＿∧ +
（0゜・∀・） ﾜｸﾜｸﾃｶﾃｶ
（0゜∪ ∪ +
と＿_）__） +
つーかVIP語(?)使うな。
- 784 名前：仕様書無しさん mailto:sage [2007/03/11(日) 20:44:17 ]
- ○racleの30字制限のためそんなのはザラ。
あの制限さっさと外せやとはいつも思ってる。
- 785 名前：仕様書無しさん mailto:sage [2007/03/11(日) 20:48:02 ]
- あと、無理に英単語使って意味が間違ってるヤツとか。
例えば、最新○○番号->New○○No（既に存在するもののうち最新のモノを指す）
そこはNewじゃなくてLastでしょって、言ってもへらへら笑って誤魔化しやがる。
- 786 名前：仕様書無しさん mailto:sage [2007/03/12(月) 00:42:51 ]
- >>785
ちなみにそれだとどういう意味になるんだ？
誤解が生まれるレベルじゃない場合は目くじら立てる必要無いと思うが。
- 787 名前：仕様書無しさん [2007/03/12(月) 00:49:44 ]
- >>786
生理なんだよきっとｗｗ
つうか、>>785と一緒に仕事したくないなｗ
- 788 名前：仕様書無しさん mailto:sage [2007/03/12(月) 00:51:57 ]
- NoはNo.が正しい。じゃなきゃNumとかが妥当。
- 789 名前：仕様書無しさん mailto:sage [2007/03/12(月) 00:53:12 ]
- Newで始まってるとなにかのファクトリ(オブジェクト生成関数)
かと思ってしまうな。ところが実は既存のものから最新のを返す
んだと言われると当惑する。
- 790 名前：仕様書無しさん mailto:sage [2007/03/12(月) 00:54:14 ]
- 普通の英語では785の意味の最新はlatestだろう。
lastでも通じると思うがnewでは明らかに通じない。
- 791 名前：仕様書無しさん mailto:sage [2007/03/12(月) 01:01:36 ]
- だな
New の新しさはこれまで無かった新しさ
- 792 名前：仕様書無しさん mailto:sage [2007/03/12(月) 01:04:16 ]
- Newestならまだ分かるけど、ふつーそれより先にlatestが出てくるな。
- 793 名前：仕様書無しさん mailto:sage [2007/03/12(月) 01:33:09 ]
- // QAC対処
// exit -1;
って、それじゃ、終了できねえよ。ＱＡＣがすべてじゃねぇ。
- 794 名前：仕様書無しさん [2007/03/12(月) 07:31:29 ]
- ○ クラスを全部 Globalってマクロ作って定義してる。
○ キー入力ができたりできなかったりする。
○ 基本的にスパゲティ。どこにとんでいるかで動作が決まってる。
まぁマイコンだけどさぁ・・・。
ひどす。
- 795 名前：仕様書無しさん mailto:sage [2007/03/12(月) 17:42:59 ]
- >>794
無理矢理高級言語を使う意味あるのか？
普通は言語を使い分けてリンクするよな。
>キー入力ができたりできなかったりする。
昔、俺の後ろでそれやって修羅場ってるの見たことあるな。
Javaでそれをやるのは、普通なら難しいのに。w
- 796 名前：仕様書無しさん [2007/03/14(水) 23:37:45 ]
- // 値が10以上の場合
if (val > 10) {
- 797 名前：仕様書無しさん mailto:sage [2007/03/14(水) 23:40:05 ]
- あるあ…ねーよｗ
- 798 名前：仕様書無しさん mailto:sage [2007/03/14(水) 23:42:35 ]
- クライアントアプリにJava使って何のメリットがある？
Javaは通信は強いけどクライアントアプリとして使うにはあまりにも・・・
- 799 名前：仕様書無しさん [2007/03/14(水) 23:42:36 ]
- >>796
中２かよｗ
- 800 名前：仕様書無しさん mailto:sage [2007/03/14(水) 23:44:02 ]
-
//------------------↓ここから僕のちとちぃ〜〜〜〜〜〜〜〜〜
- 801 名前：仕様書無しさん mailto:sage [2007/03/14(水) 23:44:53 ]
-
//----------------あーずるいぞのびた！
// なら俺はここから俺の土地ぃーーーーーーーーーーーーーーーーーーーーーーー
- 802 名前：仕様書無しさん mailto:sage [2007/03/15(木) 00:48:20 ]
- >798
J2EEサーバと通信しやすく、セキュリティポリシーをスカスカにすればローカルマシンも弄り放題。
ってことで、MVCのVはほとんど素のHTML、Mは90%J2EEサーバのEJBに置いて
CをアプレットとJavaScriptの眩暈がするほど華麗な連携でこなすシステムに携わったことがある。
(ちなみにアプレットは基本的にUI持ってない)
ソースも大概だったがこの構造だけで1週間で辞めたくなったorz
# まぁ、各地の事務所が9600BPSのモデムで繋がってて、HTMLファイルの転送すら
# 勿体ないって事情はあったらしいが。
# (だから、VとなるHTMLは各クライアントマシンのローカルに置きっぱ)
- 803 名前：仕様書無しさん mailto:sage [2007/03/15(木) 00:54:52 ]
- >>802
金の掛け方が根本的に間違ってるな。w
そんな特殊ななアーキテクチャにする金でVPN引いたほうがいいのにな。
- 804 名前：仕様書無しさん mailto:sage [2007/03/15(木) 07:55:26 ]
- >>802
Vに素のHTMLは悪いと思えなかったり。
ダイナミックしてないんだから、ビューを果たしてるんでないかい？
- 805 名前：仕様書無しさん mailto:sage [2007/03/15(木) 18:55:31 ]
- 日付がNVARCHAR2(8)ってどうなのよ。
- 806 名前：仕様書無しさん mailto:sage [2007/03/15(木) 19:28:27 ]
- >>805
4+2+2に切り分けるわけ？
・・・遺体
- 807 名前：仕様書無しさん mailto:sage [2007/03/15(木) 20:15:43 ]
- >>805
そんなのしょっちゅうさ…
- 808 名前：仕様書無しさん [2007/03/15(木) 21:32:59 ]
- >>805
俺のいるところも日付の扱いがかなり適当だよ。
DATE、CHAR(6)、VARCHAR2(8)が混在してる
- 809 名前：仕様書無しさん mailto:sage [2007/03/15(木) 23:04:58 ]
- >>808
うちはvarchar2(6) だな。
2100年以降のことなんか考えてない。
|
 |
