- 1 名前：仕様書無しさん [2006/12/26(火) 16:18:46 ]
- この会社辞めようと思ったソースコード。
プログラマとして幻滅するソースコード。
プログラマを悩ませるソースコード。
をつらつらと綴っていって頂戴。
ちなみにここは質問スレじゃないので
技術的な質問がしたいならム板に逝って。
前スレ
この会社辞めようと思ったソースコード＃14
pc8.2ch.net/test/read.cgi/prog/1158190165/
- 299 名前：仕様書無しさん mailto:sage [2007/01/14(日) 18:59:23 ]
- 頭ピンチなひとはスルーで。
- 300 名前：仕様書無しさん [2007/01/14(日) 19:03:32 ]
- 掘り返したらなんかまずいらしい
- 301 名前：仕様書無しさん mailto:sage [2007/01/14(日) 19:10:57 ]
- 必死ですね＾＾；
- 302 名前：仕様書無しさん mailto:sage [2007/01/14(日) 19:12:46 ]
- >>297
本人乙
- 303 名前：仕様書無しさん mailto:sage [2007/01/14(日) 19:15:00 ]
- >>296、>>300 おまえ空気読めないって陰で言われてるから直した方がいいよ
- 304 名前：仕様書無しさん mailto:sage [2007/01/14(日) 20:13:06 ]
- はいはい荒らし荒らし
他のスレ逝け
- 305 名前：仕様書無しさん mailto:age [2007/01/14(日) 20:40:51 ]
- 図星か…
本人ｶﾜｲｿｽ（´・ω・`）
- 306 名前：仕様書無しさん mailto:sage [2007/01/14(日) 21:02:29 ]
- ねぇ
荒らしたいの？
何がしたいの？
- 307 名前：仕様書無しさん mailto:sage [2007/01/14(日) 21:23:10 ]
- キミのソコに挿れたい...
- 308 名前：仕様書無しさん mailto:sage [2007/01/14(日) 21:51:17 ]
- ハッハッハ、おれは１５００Stepの関数を作った男だ。
※ メインルーチン、かつデータ移行ツールです。
スペース：コード ３：１
コメント：コード １：１
なので実際は６００Stepぐらい？
別システムの参考にしようとした人には「すいません、理解できませんでした」と言われた（こちらこそすいません）。
でも、後悔はしてない。一応現場は「現場市場まれにみる有用なツールである」と言わしめたから。
- 309 名前：308 mailto:sage [2007/01/14(日) 21:52:47 ]
- すま、計算しなおしたら２００Step程度じゃん。
ってことは迫害をうけるいわれはない、わけ？
（まぁコード：コメント １：１だけでも指摘ありそうだけどね）
- 310 名前：仕様書無しさん mailto:sage [2007/01/14(日) 22:07:51 ]
- ほとんどコメントナシ派の俺が颯爽と登場！！！
- 311 名前：仕様書無しさん [2007/01/14(日) 22:42:44 ]
- ステップってなに？
行数と違うの？
- 312 名前：308 mailto:sage [2007/01/14(日) 22:53:23 ]
- >>311
おいらがいったのはステップ＝行数ね。
- 313 名前：仕様書無しさん mailto:sage [2007/01/14(日) 22:55:10 ]
- >>306
>>22はどう見ても煽りだろ
- 314 名前：仕様書無しさん mailto:sage [2007/01/14(日) 23:02:44 ]
- >>310
専門学校へお帰りください
- 315 名前：仕様書無しさん mailto:sage [2007/01/14(日) 23:07:33 ]
- どう見てもマンガ道場出演者だろ
- 316 名前：仕様書無しさん mailto:sage [2007/01/14(日) 23:18:28 ]
- グローバル変数が「i」
- 317 名前：仕様書無しさん mailto:sage [2007/01/14(日) 23:27:02 ]
- >>316
「おれはねセイバー、１文字変数なんてなかったらよかったと思うんだ」
i,j,k,lとかやる香具師が許せない。
おれはnLoop、nL2、nL3〜でやってる（まぁこれもちょっとアレだが）
- 318 名前：仕様書無しさん mailto:sage [2007/01/14(日) 23:32:34 ]
- 俺は、変数名や関数名に連番を打つ奴が許せない
いったい何のためにプログラミングの世界に
"シンボル" という概念が導入されたのか
解っていないんじゃないの？
そんなに決め打ちが好きなのなら、マシン語の世界に逝け。
- 319 名前：仕様書無しさん mailto:sage [2007/01/14(日) 23:37:36 ]
- i,j,k,l,m,nあたりについては、
ループ用の変数として固定してる人がほとんどだから
そういうものとして扱っておけばいいと思う
- 320 名前：仕様書無しさん mailto:sage [2007/01/14(日) 23:52:04 ]
- >>319
k 辺りでアルゴリズム見直せと思うけどな．
何の疑問も無く l,m,n まで使うヤツはセンスがないと思う．
- 321 名前：仕様書無しさん mailto:sage [2007/01/14(日) 23:58:28 ]
- >>320
そればっかりはケースバイケースでしょ。
- 322 名前：仕様書無しさん mailto:sage [2007/01/15(月) 00:06:47 ]
- 単純な３重ループくらいまでならごく稀に使うことあるけど
さすがにl,m,nまで行ったら何を基準で回ってるか分からなくなってくるな。
シンボルの話だがローカルなループカウンタで
iやjを読みづらいと思ったことはあまりない。
メンバ変数とかでやった日には人格疑うが。。。
- 323 名前：仕様書無しさん mailto:sage [2007/01/15(月) 01:52:52 ]
- ローカル変数とメンバ変数は別者だろう。
- 324 名前：仕様書無しさん mailto:sage [2007/01/15(月) 12:14:05 ]
- 昨今のプログラミング言語では狭いスコープが使えて、本当にうれしい。
ただ、こんなことを言うと、自分はもうおっさんなんだと感じる。
会社辞めたい。
- 325 名前：仕様書無しさん mailto:sage [2007/01/15(月) 12:55:18 ]
- いや どうしても3重以上のループを必要とする場合もあるんだが
l だけは使うのやめてくれ
画面でも紙でも見づらいんだよ
あと ループカウンタにも一文字変数使うなっていう香具師は
一度 プログラミング作法 読んでおけ
- 326 名前：仕様書無しさん mailto:sage [2007/01/15(月) 17:01:20 ]
- >>325
同意
oと0,lとi等、ぱっと見でわかり辛いものは使わないほうがいいって言うしね。
- 327 名前：仕様書無しさん mailto:sage [2007/01/15(月) 20:31:42 ]
- iとjならいいのか？
- 328 名前：仕様書無しさん mailto:sage [2007/01/15(月) 22:26:54 ]
- >>325-326
お前の使っているフォントが悪い。
それだけ。
- 329 名前：仕様書無しさん mailto:sage [2007/01/16(火) 00:38:35 ]
- 謎コード
public function getInt(num as string) as integer
getint = cint(num)
end function
- 330 名前：仕様書無しさん mailto:sage [2007/01/16(火) 01:24:50 ]
- >>329
つ CInt()
- 331 名前：仕様書無しさん mailto:sage [2007/01/16(火) 08:53:32 ]
- >>329
オレは、if文で変換可能かどうか判定しろと習った。
あらゆるコードをresume nextした上エラー処理をしない奴らが、
後から、if文を撤去したのでは？
- 332 名前：仕様書無しさん mailto:sage [2007/01/19(金) 13:58:52 ]
- >327
まぁ、I J 辺りは FORTRAN から続いてるだろうからなぁ
今更見慣れ過ぎてループカウンタ以外に見えない
…逆にループ以外で i 使われると焦るｗ
- 333 名前：仕様書無しさん mailto:sage [2007/01/19(金) 21:03:39 ]
- >>332
いやいや 上の方で「区別がつきづらいのはやめろ」と書いているが
伝統的なiとjも区別がつきづらいだろ！ということだろう
- 334 名前：仕様書無しさん mailto:sage [2007/01/19(金) 23:04:26 ]
- > 伝統的なiとjも区別がつきづらいだろ！ということだろう
これ、、、特定されたら困るけど、、、やっぱりｱﾌｫだと思う。
そして、辞めますた。
- 335 名前：仕様書無しさん mailto:sage [2007/01/19(金) 23:10:17 ]
- iとjはあまり見間違えないと思うがな...
- 336 名前：仕様書無しさん mailto:sage [2007/01/19(金) 23:12:27 ]
- >>334
意味不明
- 337 名前：仕様書無しさん mailto:sage [2007/01/20(土) 00:27:46 ]
- >>326
俺時々ADOをADDと見間違うお
- 338 名前：仕様書無しさん mailto:sage [2007/01/20(土) 01:06:31 ]
- 俺はADOとDAOを(ry
- 339 名前：仕様書無しさん mailto:sage [2007/01/20(土) 01:35:45 ]
- 失せろ。ブイバー。
- 340 名前：仕様書無しさん mailto:sage [2007/01/20(土) 01:44:39 ]
- >337
AD/HD乙
- 341 名前：仕様書無しさん mailto:sage [2007/01/20(土) 02:43:11 ]
- >>335
[i][j]
フォントによっては結構わかりづらい
- 342 名前：仕様書無しさん mailto:sage [2007/01/20(土) 02:50:24 ]
- わかりやすいように適宜スペース入れとけ
わかりやすいフォント使え
- 343 名前：仕様書無しさん mailto:sage [2007/01/20(土) 03:12:22 ]
- >>342
じゃー l だって別にいいじゃん
- 344 名前：仕様書無しさん mailto:sage [2007/01/20(土) 03:21:48 ]
- うん、いいよ。
- 345 名前：仕様書無しさん mailto:sage [2007/01/20(土) 23:40:28 ]
- 全部読んだ。
>>22みたいな事をいう奴と仕事をすることになるときつそうだな…
- 346 名前：仕様書無しさん mailto:sage [2007/01/20(土) 23:58:46 ]
- いちいち終わった話を蒸し返そうとする奴と同じくらいきつそうだな。
- 347 名前：仕様書無しさん [2007/01/21(日) 00:35:39 ]
- 大変ですね（＾Д＾）
- 348 名前：仕様書無しさん mailto:sage [2007/01/21(日) 14:03:18 ]
- 俺的にはきれいにかけてれば
1関数1000行でも許すよ
- 349 名前：仕様書無しさん mailto:sage [2007/01/21(日) 14:57:21 ]
- >>348
蒸し返すなってーの。
空気嫁ないって陰で言われてるから、気をつけろ。
- 350 名前：仕様書無しさん mailto:sage [2007/01/21(日) 15:04:05 ]
- 定期的に蒸し返し厨が現れるな・・・
- 351 名前：348 mailto:sage [2007/01/21(日) 15:12:31 ]
- すみませんでした（藁）
- 352 名前：仕様書無しさん mailto:sage [2007/01/21(日) 15:41:18 ]
- ん？俺には恥かしい事を書いてしまった本人と、
全部読んだという人の構図にしか見えないんだが。
話題もないし、べつにいいんじゃね？
- 353 名前：仕様書無しさん mailto:sage [2007/01/21(日) 16:03:07 ]
- 話題がないから蒸し返す〜♪
不毛な議論を蒸し返す〜♪
- 354 名前：仕様書無しさん [2007/01/21(日) 16:08:31 ]
- >>351
>（藁）
ああ、そういうことか（藁）
- 355 名前：仕様書無しさん mailto:sage [2007/01/21(日) 16:21:16 ]
- VBAでいちいち下記のようにやってるんだけど、
なんで1行にまとめないんだと言いたい
bFlag=someFunc()
if Not(bFlag) then goto errHdr
bFlag=someFunc2()
if Not(bFlag) then goto errHdr
・・・
- 356 名前：仕様書無しさん mailto:sage [2007/01/21(日) 16:23:51 ]
- C++で
return (0);
とか
return (-1);
とかやってるソース見たんだけど、（）いるの？
- 357 名前：仕様書無しさん mailto:sage [2007/01/21(日) 16:26:47 ]
- >>355
>>356
冗長ではあるが、その程度で辞めようとは思わないけどな
- 358 名前：仕様書無しさん mailto:sage [2007/01/21(日) 16:28:42 ]
- >>355
デバッグのときに、何が戻ってきているか分かりやすいから。
速度や、メモリの消費が気になるようなことなら、VBAでやらないでしょ？
- 359 名前：仕様書無しさん mailto:sage [2007/01/21(日) 16:29:27 ]
- 辞めようとまでは思わないよ。ソースコードごときで
辞めようと思うのはやっぱ体制とかスケジュールとかそのあたり。。
- 360 名前：仕様書無しさん mailto:sage [2007/01/21(日) 16:32:52 ]
- >>358
え、それってVBの常套テクニックなの?
確かにフラグの値変わったときにブレークできるかもしれんけど、
普通にステップ実行で良いような気が･･･
- 361 名前：仕様書無しさん mailto:sage [2007/01/21(日) 16:34:24 ]
- >>39
- 362 名前：仕様書無しさん mailto:sage [2007/01/21(日) 16:35:59 ]
- >>361
納得！
- 363 名前：仕様書無しさん mailto:sage [2007/01/21(日) 16:37:46 ]
- 小粒だなぁ。できれば>>22みたいな良質な燃料が欲しいぞ？
- 364 名前：仕様書無しさん mailto:sage [2007/01/21(日) 16:41:18 ]
- >>363
俺的には、
一目見てきれいであれば1000行オーダーは許す
SQL文とかでどうしても長くなる場合があるし。
でも、10000行はどう考えてもやりすぎだと思う。
まあ、どっちかっていうと、長さってよりは読みやすさって気はするが
- 365 名前：仕様書無しさん [2007/01/21(日) 16:44:32 ]
- ＶＢコード読まされてる時点でおわっとる
- 366 名前：仕様書無しさん mailto:sage [2007/01/21(日) 16:50:50 ]
- >>364
きれい、がまた難しいけど概ねそうだね。
どっちかというと単に長いコードを書いて自慢したがるような輩と
一緒に仕事すると、会社辞めたく…いや、会社辞めさせたくなるよ。
- 367 名前：仕様書無しさん mailto:sage [2007/01/21(日) 16:53:37 ]
- >>365
悪かったなｗｗｗｗ
しかもエクセルのVBAでつよｗｗｗ
- 368 名前：仕様書無しさん mailto:sage [2007/01/21(日) 16:56:09 ]
- エクセルって何気にすごいからな
あの技術は実はMSの中でも最高峰だろうｗ
- 369 名前：仕様書無しさん mailto:sage [2007/01/21(日) 16:57:54 ]
- >>364
「行数」と「ステップ数」は違うぜ。
- 370 名前：仕様書無しさん mailto:sage [2007/01/21(日) 17:02:44 ]
- >>369
うーん、まぁ、それでも同じかな。言いたいことは。
でも、{ }で囲まれた部分が１０００ステップとかだと嫌
- 371 名前：仕様書無しさん mailto:sage [2007/01/21(日) 18:01:32 ]
- >>365
すまん、この前javaでそんなコード書いたのだが
どうやっったら一行になるの？
日曜プログラマだから許して
- 372 名前：仕様書無しさん mailto:sage [2007/01/21(日) 18:06:05 ]
- こうしろってことじゃねーの
if Not( someFunc() ) then goto errHdr
- 373 名前：仕様書無しさん mailto:sage [2007/01/21(日) 18:54:58 ]
- >>371
boolean flag = isHoge();
if(flag){ /* 処理 */ }
じゃなくて
if(isHoge()){ /* 処理 */}
って書けばいいと思う。
- 374 名前：仕様書無しさん mailto:sage [2007/01/21(日) 19:03:31 ]
- if( !someFunc() || !someFunc2() ){ /* エラーに飛ぶ処理 */}
- 375 名前：仕様書無しさん mailto:sage [2007/01/21(日) 19:08:51 ]
- >>374
if(someFunc() && someFunc2()){ /* 正常系 */}else{/* エラーに飛ぶ処理 */}
両方notの条件をorで繋げるのは嫌い。
- 376 名前：仕様書無しさん mailto:sage [2007/01/21(日) 19:33:33 ]
- >>355
おれもそうやるよ。コーディング時の効率、行数削減よりも
デバッグ時ちゃんとエラー値がハンドルできてることを見れるコードの方が
単体でバグ取りちゃんとできてあとで問題になりにくいと思ってる。
>>355的な１つにまとめてってのはどう書くの？
if(func1() <> x ) or (func2() <> y)
とか
bFlag1 = func1();
bFlag2 = func2();
if(bFlag1 <> x) or (bFlag2 <> y)
？後ろのやつはfunc1()がエラーのときもfunc2()を通っちゃうから実装自体に問題ありだけど
- 377 名前：仕様書無しさん mailto:sage [2007/01/21(日) 20:12:41 ]
- >>375
>>355だけ見たら>>374が一番近いな
>>375のは厳密には意味が変わってくるからな
- 378 名前：355 mailto:sage [2007/01/21(日) 21:40:16 ]
- そんな深いこと書いたつもり無かったんだけどｗ
おいらが思ったのは>>372
>>374は微妙な感じ。。。
>>375はsomeFunc, someFunc2両方とも実行されるから意味が異なる
- 379 名前：355 [2007/01/21(日) 21:42:10 ]
- うそでした
>>375もsomeFuncがfalseとなった時点でthen以下が実行されるな
すまそ
- 380 名前：仕様書無しさん mailto:sage [2007/01/21(日) 21:46:04 ]
- >>374, 375とも微妙なのは
たとえば、
func1をconnectDB()
func2をexecSQL
とすると分かりやすい。
って、んなこと書いてねーじゃん。すまん
- 381 名前：仕様書無しさん mailto:sage [2007/01/21(日) 21:56:35 ]
- VBだとORは〜ってのを思い出すなぁ
- 382 名前：仕様書無しさん mailto:sage [2007/01/21(日) 22:50:49 ]
- >>380
微妙じゃないじゃん。
- 383 名前：仕様書無しさん mailto:sage [2007/01/22(月) 00:19:24 ]
- >>382
javaならそうね。
VBだと、両方評価されるんだ。
- 384 名前：仕様書無しさん mailto:sage [2007/01/22(月) 01:09:32 ]
- なにその糞言語。
- 385 名前：仕様書無しさん mailto:sage [2007/01/22(月) 01:15:00 ]
- マジだから。
- 386 名前：仕様書無しさん mailto:sage [2007/01/22(月) 01:22:09 ]
- マジだね。
ショートサーキット利かない言語って意外とあるよ。
それで何度ブチ切れたことか。
# 言語仕様を把握せずに書いてる俺が悪いんだが。
- 387 名前：仕様書無しさん mailto:sage [2007/01/22(月) 02:15:01 ]
- support.microsoft.com/kb/817250/ja
msdn.microsoft.com/library/ja/default.asp?url=/library/ja/vbcn7/html/vaconbooleanoperatorchanges.asp
これをみると、VB6はショートサーキット評価は効かない
2005もしくは.NETの場合でも明示的にAndAlsoかOrElseを使わないと効かない
ということだ。355のおかげで勉強になりました。ありがトン
- 388 名前：仕様書無しさん mailto:sage [2007/01/22(月) 12:59:37 ]
- 言語仕様がうんぬんに左右されず，読みやすいってことで
>>373の書き方が一番じゃないのかなぁ
- 389 名前：仕様書無しさん mailto:sage [2007/01/22(月) 16:22:04 ]
- そんな程度のことにこだわっても良いこと無さげ。
- 390 名前：仕様書無しさん mailto:sage [2007/01/22(月) 19:42:24 ]
- そもそも、VB6以前は、C言語の&&や||に相当する演算子が無かった希ガス。
(And は & の意味だし、Or は | の意味だったと思う）
- 391 名前：仕様書無しさん mailto:sage [2007/01/22(月) 21:28:57 ]
- ねーよ
速度を考えると多段IFとかになる
だからもうね
- 392 名前：仕様書無しさん mailto:sage [2007/01/22(月) 21:56:07 ]
- なんでVBって
dim i as integer = 0
とかって書けないの？いちいち2行使うのうざいんですけど
あと、! も使えなくてNot 3文字も打つ羽目になる
- 393 名前：仕様書無しさん mailto:sage [2007/01/22(月) 22:01:38 ]
- そうそう
for dim i=0 to 5
と書けないのがうざい
- 394 名前：仕様書無しさん mailto:sage [2007/01/22(月) 22:09:03 ]
- >392
上は .NET で解消したはず
オレもVBやってるときにイラついた仕様だ
- 395 名前：仕様書無しさん mailto:sage [2007/01/22(月) 22:49:16 ]
- dim って dimension じゃなかったっけ？
配列以外の変数を宣言するのにも dim を使うのってなんか違和感ある。
- 396 名前：仕様書無しさん mailto:sage [2007/01/22(月) 22:51:17 ]
- そーっすか
- 397 名前：仕様書無しさん mailto:sage [2007/01/22(月) 23:03:59 ]
- >>395
BASIC時代に配列をDIMで宣言していた名残ですな
いまさら変更するわけにもいかんって感じじゃない?
- 398 名前：仕様書無しさん mailto:sage [2007/01/23(火) 00:24:30 ]
- VBで一番イラつくのは ; がないこと。 といってみる。
- 399 名前：仕様書無しさん mailto:sage [2007/01/23(火) 00:36:05 ]
- . があるCOBOLならOKですか？
|
 |
