- 1 名前：仕様書無しさん [2006/12/26(火) 16:18:46 ]
- この会社辞めようと思ったソースコード。
プログラマとして幻滅するソースコード。
プログラマを悩ませるソースコード。
をつらつらと綴っていって頂戴。
ちなみにここは質問スレじゃないので
技術的な質問がしたいならム板に逝って。
前スレ
この会社辞めようと思ったソースコード＃14
pc8.2ch.net/test/read.cgi/prog/1158190165/
- 269 名前：仕様書無しさん mailto:sage [2007/01/07(日) 12:06:02 BE:149769465-2BP(294)]
- >>261
間違ってるだろ
- 270 名前：仕様書無しさん mailto:sage [2007/01/07(日) 13:24:36 ]
- >>269
具体的に。
- 271 名前：仕様書無しさん mailto:age [2007/01/14(日) 11:56:01 ]
- age
- 272 名前：仕様書無しさん [2007/01/14(日) 13:57:38 ]
- Ｃ言語で system() 関数呼びまくりのコードを引き継いでしまった。
一部分を紹介。
ディレクトリを再帰的に処理していきたいのは分かるんだが
１． 「dir /s ... path > output.txt」 と記述したバッチファイル makelist.bat を事前に準備。
２． それを system("makelist.bat") で呼び出し。
３． その出力結果 output.txt を fgets() & str 系でチマチマと処理。
str系の使い方も粗悪。
ファイル名が表示される位置を狙い撃ちでコーディング。
1バイト目が '1' か '2'、 かつ "DIR" が含まれていればｘバイト目からはファイル名という具合。
ファイル名のバッファも100バイトしかとってないし。
これがドキュメントになっていて、ＰＭやその上司の承認のハンコまで付いている。
「これ作った人って前の方が最初ですか？」と聞くと
「そうだよ。いまは別ＰＪのチーフＰＧやってる人だよ」と教えてくれた。
上司の評価は高いようだ。他に能力があるのかも。
- 273 名前：仕様書無しさん [2007/01/14(日) 14:21:17 ]
- >>272
遅くても問題ないならそれでもいいんじゃない。
工数かけずに作れて効果が同じならその方がいい。
内容がヘボであっても。
きちんと作れないのは問題外だが、かける工数と
その効果がわかってる人は評価が高い。
必要に応じて選べるからね、手段を。
「全て最高のレベルで美しく、無駄なく」なんて言ってると評価下がるよ。
大学じゃあないんだから、ね。
- 274 名前：仕様書無しさん [2007/01/14(日) 14:26:27 ]
- >>267
待ちの処理だなｗ
- 275 名前：272 [2007/01/14(日) 14:53:25 ]
- >>273
>遅くても問題ないならそれでもいいんじゃない。
>工数かけずに作れて効果が同じならその方がいい。
>内容がヘボであっても。
>
>きちんと作れないのは問題外だが、かける工数と
>その効果がわかってる人は評価が高い。
>必要に応じて選べるからね、手段を。
>
>「全て最高のレベルで美しく、無駄なく」なんて言ってると評価下がるよ。
>
>大学じゃあないんだから、ね。
まあそうだけどね。
後のメンテナンスのことを考えるとどうなんでしょうね。
出所不明の実行ファイルをsystem()で呼んでその結果を加工して使ってたり。
メンテナンスできないわな。
- 276 名前：仕様書無しさん [2007/01/14(日) 14:56:16 ]
- >>272
それは次の担当者の仕事だから問題ない。
そうやって責任を押しつけていかないと出世できないんだろ。
押しつけられた奴が負け。
- 277 名前：仕様書無しさん mailto:sage [2007/01/14(日) 15:14:22 ]
- >>276
本人乙。
多少なりとも良心のあるマならそんな糞コード書かない。
- 278 名前：仕様書無しさん [2007/01/14(日) 15:18:15 ]
- だれが本人だよ。
俺のコードはelegantだっつの。
大体、良心あるようでは、とてもとても。
- 279 名前：仕様書無しさん [2007/01/14(日) 15:28:41 ]
- >>272
（ﾟдﾟ） .....
9年前にC言語の勉強ということで、入社2ヶ月目に作った
ツールの仕様と死ぬほど似ているのは気のせいかな・・・・
・・・今は俺、確かにリーダーとかサブPMやってるけど。。
- 280 名前：279 mailto:sage [2007/01/14(日) 15:30:51 ]
- つか、あれは一時しのぎのツールということで、
先輩が２週間ぐらい後にきちんとしたのを作るから、
動けばいいってことでC言語を勉強しながら作ったんだけどな(´・ω・｀)
まさか、まだ使われている・・・？？
>>275
引用符つけておきながら全文引用って、２ちゃんねる初心者みたいなことするなよｗｗ
- 281 名前：仕様書無しさん mailto:sage [2007/01/14(日) 15:33:58 ]
- >>280 アンカーのこと引用符と呼ぶお前もどうかと
- 282 名前：HELPきーが差し押さえられました mailto:sage [2007/01/14(日) 15:37:40 ]
- >>281
アンカーじゃなくて、「>」一個のことじゃないかな。
- 283 名前：仕様書無しさん mailto:sage [2007/01/14(日) 15:42:25 ]
- >>272
サブディレクトリまで掘っていくことを考えると、楽な方法とは思えないな。
- 284 名前：ソースはソニー mailto:sage [2007/01/14(日) 15:42:44 ]
- ( ´∀`)＜ぬるぽ
- 285 名前：仕様書無しさん mailto:sage [2007/01/14(日) 15:44:42 ]
- >>280
>２ちゃんねる初心者みたいなことするなよｗｗ
お前こそ、まだ「フューザ」入れてねーんだろ？
すぐ分かるよｗｗｗ
- 286 名前：仕様書無しさん mailto:sage [2007/01/14(日) 15:46:55 ]
- せめて dir /b /s ... だったらな。
駄目な奴は手抜きでもセンスがないという見本。
- 287 名前：仕様書無しさん mailto:sage [2007/01/14(日) 16:17:37 ]
- >272
もう一回出てきて一部のコードをそのまま書いてくれwww
そうすれば、>279-280が自分の事だって確信持てるだろうからwww
- 288 名前：仕様書無しさん mailto:sage [2007/01/14(日) 16:36:09 ]
- さすがに何年も書いたソースを詳細には覚えてないんじゃないか？
やるなら会社名を晒すくらいしてくれないと
- 289 名前：仕様書無しさん mailto:sage [2007/01/14(日) 16:47:47 ]
- PHASERの読みって、フェイザーじゃないの？
- 290 名前：272 [2007/01/14(日) 17:14:43 ]
- >>280
全文引用すまん。ゆるせ。
>>287
ソースをそのまま書くのはマズイだろ〜
第一、詳細まで覚えてないし。
>>287さんがＨ社の仕事をしている（いた）のなら確率高そうだ。
これ以上は勘弁してくれ。
- 291 名前：仕様書無しさん mailto:sage [2007/01/14(日) 17:26:37 ]
- >>272
system()呼び出しまくってる時点でセンスがないと思われ
- 292 名前：仕様書無しさん mailto:sage [2007/01/14(日) 17:29:52 ]
- >>291
だな。
全くC言語の勉強になってないもんな
- 293 名前：仕様書無しさん mailto:sage [2007/01/14(日) 17:40:57 ]
- >>290
H系やった時変な規約があったりコードのルールとかすごくうるさかったんだが
仕様部分はそんなんで通るのか・・・さすがH
- 294 名前：仕様書無しさん mailto:sage [2007/01/14(日) 17:56:29 ]
- 普通にbat処理でいいでねーの？わざわざC使ってまでsysytem()呼び出すぐらいなら
- 295 名前：272 [2007/01/14(日) 18:18:57 ]
- >>293
うるさいのは、官公庁系じゃないのか？うちは、コーディングルールがないぞ。
つうか、かき集められた主任技師、技師、主任たちのもとで汗水たらしてジメジメと。
はじめのうちは、承認をもらっていたが、すぐに意味をなさなくなってきたのでダラダラ。
馬鹿になりそうだ。
- 296 名前：仕様書無しさん [2007/01/14(日) 18:32:19 ]
- ところで>>22って何？
ちょっとカワイソな人に見えるんだけど。
- 297 名前：仕様書無しさん mailto:sage [2007/01/14(日) 18:38:30 ]
- なんで掘り返すの？
あなたも頭がピンチなひとですか？
- 298 名前：仕様書無しさん mailto:sage [2007/01/14(日) 18:45:21 ]
- あらら、荒んでますね。
- 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
「行数」と「ステップ数」は違うぜ。
|
 |
