- 1 名前：仕様書無しさん [2006/12/26(火) 16:18:46 ]
- この会社辞めようと思ったソースコード。
プログラマとして幻滅するソースコード。
プログラマを悩ませるソースコード。
をつらつらと綴っていって頂戴。
ちなみにここは質問スレじゃないので
技術的な質問がしたいならム板に逝って。
前スレ
この会社辞めようと思ったソースコード＃14
pc8.2ch.net/test/read.cgi/prog/1158190165/
- 832 名前：仕様書無しさん mailto:sage [2007/03/17(土) 02:39:42 ]
- 3.5インチFDDのシャッターを「持つ所」と、のたまった御仁が居たな。
- 833 名前：仕様書無しさん mailto:sage [2007/03/17(土) 02:42:12 ]
- お願いです先輩、コメントアウトは行単位でしてください。
1つのステートメントの中で数文字コメントアウトとか
訳わかんない事しないでくださいorz
変数を全部ドキュメントに書かなければならないからって、
1つの変数を全然違う用途で使いまわすの止めてくださいorz
- 834 名前：仕様書無しさん mailto:sage [2007/03/17(土) 06:41:14 ]
- NVARCHAR2(8)とDATEってどっちが節約できるの？
- 835 名前：仕様書無しさん [2007/03/17(土) 09:37:50 ]
- >>834
OracleだとDATE型だな。7バイトだし。
- 836 名前：仕様書無しさん mailto:sage [2007/03/17(土) 10:21:55 ]
- そんなことは普通、考えません
- 837 名前：仕様書無しさん mailto:sage [2007/03/17(土) 13:51:28 ]
- データベースの良し悪しって結局は容量だろ？
- 838 名前：仕様書無しさん [2007/03/17(土) 15:09:49 ]
- 排他を全く考えてないマルチスレッドは勘弁してくれ。
もう総合テストに入って、原因不明（実は、競合が原因）の不具合が多発。
いまさら大きな修正はやりずらいし。
何も考えずに作るのは楽だろうが、保守させられる身にもなってくれって感じ。
たぶん、このプロジェクトは失敗確定だろうな。
- 839 名前：仕様書無しさん mailto:sage [2007/03/17(土) 15:38:45 ]
- >>837
あと使いやすさ
たとえばYYYYMMDDを入れる場合、
文字列だと入れる前と入れた後に日付の整合性チェックをしないといけない
（8桁で閏年の計算もして日付として正しいフォーマットになっているか）
DATEであれば入れるときは例外をキャッチすればいいし、出すときはbetweenで必要な期間だけ取得すればいい
- 840 名前：仕様書無しさん mailto:sage [2007/03/17(土) 16:31:40 ]
- >>838
>排他を全く考えてないマルチスレッドは勘弁してくれ。
それマルチスレッドて言わないw
さすがにDBの話じゃないよね。
- 841 名前：仕様書無しさん mailto:sage [2007/03/17(土) 16:47:04 ]
- 838 じゃないけど、きちんと排他処理できていない
マルチタスクなアプリはあるな。
アプリの性質上、データの編集はロングタイム トランザクションになることが
多くて、現在の編集状態を表すテーブルがあるんだけど、そのテーブルの
編集にトランザクション管理などの適切な処理をしていないから、
たまに、一つのデータを二箇所で同時に編集していたりする。
それで、いざ登録となったときに訳のわからんエラーになってる。
- 842 名前：仕様書無しさん mailto:sage [2007/03/17(土) 16:50:52 ]
- 共有ロックと占有ロック覚えておかないとDBプログラミングで痛い目にあう。
- 843 名前：仕様書無しさん mailto:sage [2007/03/17(土) 17:05:15 ]
- データベースはロックだぜぃ！！
- 844 名前：仕様書無しさん mailto:sage [2007/03/17(土) 17:25:57 ]
- 冗談じゃなく、ロック知らないPGが多いから困るな
- 845 名前：仕様書無しさん mailto:sage [2007/03/17(土) 17:35:11 ]
- ああ、そのためにも情報処理の資格試験はもうちょっと重視してもいいと思うんだ
- 846 名前：仕様書無しさん mailto:sage [2007/03/17(土) 17:51:34 ]
- >>845
基本情報処理の範囲はプログラマーとして最低限必要な知識に過ぎないけど、
その程度知らない人がプログラミングするのはさすがにやばすぎる
- 847 名前：仕様書無しさん mailto:sage [2007/03/17(土) 20:13:48 ]
- うちの職場は基本情報処理以上受かってるのは２割弱。
- 848 名前：仕様書無しさん [2007/03/17(土) 20:21:11 ]
- みんな日曜日がもったいないのだとポジティブシンキン
- 849 名前：仕様書無しさん mailto:sage [2007/03/17(土) 20:45:07 ]
- うちの職場は数％・・・
しかも資格の無い奴が偉そうにふんぞり返ってるよ。
この状況知ってたら入らなかったんだけどね。
さっさと上級資格揃えて辞めるかな。
- 850 名前：仕様書無しさん mailto:sage [2007/03/17(土) 21:33:18 ]
- 基本情報はアホでも受かるだろ…
- 851 名前：仕様書無しさん mailto:sage [2007/03/17(土) 22:06:10 ]
- アホだから仕事片付かないし受験しに行く暇が作れない
- 852 名前：仕様書無しさん mailto:sage [2007/03/17(土) 22:33:57 ]
- アホでも受かるから一応取っとくのか、それとも受験するまでもないのか
どっちだ？
- 853 名前：仕様書無しさん mailto:sage [2007/03/17(土) 22:35:07 ]
- 資格が役に立つと思ってるザコは3年くらい我慢して
自分の経歴にむやみに汚点残さないようにしましょう。
- 854 名前：仕様書無しさん mailto:sage [2007/03/17(土) 22:38:53 ]
- >>851
その言い訳は今まで嫌というほど聞いてきた
実際には受験するつもりがない人ばかり
- 855 名前：仕様書無しさん mailto:sage [2007/03/17(土) 23:06:27 ]
- 受験しに行く暇がないとか言ってるやつ絶対いるよな
どうせ休日は遊んでるんだろ
- 856 名前：仕様書無しさん mailto:sage [2007/03/17(土) 23:27:59 ]
- 誰か俺を呼んだか？
- 857 名前：仕様書無しさん mailto:sage [2007/03/17(土) 23:45:42 ]
- 呼んでないから帰れ
- 858 名前：仕様書無しさん mailto:sage [2007/03/17(土) 23:48:09 ]
- >>856-857
ﾜﾛﾀ
- 859 名前：仕様書無しさん mailto:sage [2007/03/17(土) 23:51:41 ]
- 基本情報を持ってるからと言って使い物になるわけでも無いが
最低限の知識の証明にはなるな。
基本情報も取れない奴は使い物にならん。
- 860 名前：仕様書無しさん mailto:sage [2007/03/18(日) 00:34:56 ]
- 足の裏についたご飯粒
- 861 名前：仕様書無しさん mailto:sage [2007/03/18(日) 00:45:23 ]
- ときめきメモリアルONLINE 94時間目 [大規模MMO]
- 862 名前：仕様書無しさん mailto:sage [2007/03/18(日) 03:19:03 ]
- >>859
確かに「取れない」ヤツは100%使いものにならんな。
「取らない」ヤツはたまに使えるけど。
- 863 名前：仕様書無しさん mailto:sage [2007/03/18(日) 12:57:07 ]
- 基本情報を履歴書に書いてくるのは使い物にならんな…
- 864 名前：仕様書無しさん mailto:sage [2007/03/18(日) 13:11:16 ]
- 履歴書に資格なしの方がもっと使えない確率が高いと思うが
- 865 名前：仕様書無しさん mailto:sage [2007/03/18(日) 13:43:09 ]
- 基本情報は持ってて当然だから資格無しは有り得ないだろ。
最初の足切りラインとしては使えるぞ>>863
- 866 名前：仕様書無しさん mailto:sage [2007/03/18(日) 14:57:23 ]
- 運転免許証と同じくらいかね
- 867 名前：仕様書無しさん mailto:sage [2007/03/18(日) 15:04:50 ]
- 毎年2万人受かってるから
- 868 名前：仕様書無しさん mailto:sage [2007/03/18(日) 15:14:07 ]
- ベテランが「資格とりにいきます！納期遅らせてください！！」なんて言ったら顰蹙買うだろ？
だからこっそり行ってるんだよ
落ちてもバレないし
- 869 名前：仕様書無しさん mailto:sage [2007/03/18(日) 15:29:11 ]
- IT関連企業なら情報処理資格取得者は普通もっといるべきなんだよな。
あまりにもレベルの低い技術者が多すぎる
- 870 名前：仕様書無しさん mailto:sage [2007/03/18(日) 15:32:53 ]
- 資格とっても評価されない会社多くね？
- 871 名前：仕様書無しさん mailto:sage [2007/03/18(日) 15:33:46 ]
- 資格は初心者をスタート地点まで連れて行ってくれるもの
すでにスタートした奴らに取らせても意味ない
- 872 名前：仕様書無しさん mailto:sage [2007/03/18(日) 15:37:36 ]
- 業務経験が長るなるほど、有保有者との知識の差が広がっていくのが現実
- 873 名前：仕様書無しさん mailto:sage [2007/03/18(日) 15:56:04 ]
- うちの会社では資格なしで落とすことはないが、資格ありだから
拾い上げるということもないな。
- 874 名前：仕様書無しさん mailto:sage [2007/03/18(日) 16:18:11 ]
- >>871
それは資格試験に落ちた奴の言い訳。
本当にスタート地点に立ってるのか怪しい口だけの奴が９割。
適当に誤魔化してるくせに態度だけはでかい若者が多いのは嘆かわしい限り。
- 875 名前：仕様書無しさん mailto:sage [2007/03/18(日) 16:39:40 ]
- おまいら資格スレでも立てて存分に語り合え。
俺なんか俺なんか
10年前のソースをメンテしてるんだが（当時の担当はすでに退職済
実行環境を変えたらアプリケーションエラーで落ちまくるようになったと言われたので、
調査してたら、ファイルIO関数のほぼ全部が
char cFile[40]; /* ファイル名 */
そりゃあ、D:\Program files\(長いフォルダ名)\（さらに長いフォルダ名） にインストールしなおせば
落ちるよな、そうだよな'`,､('∀`) '`,､ '` ,､ '` ,､ '`
- 876 名前：仕様書無しさん mailto:sage [2007/03/18(日) 16:47:44 ]
- >>875
マジックナンバー使いまくってそうだ・・・
一度整理した方がいいんじゃない？
- 877 名前：仕様書無しさん mailto:sage [2007/03/18(日) 16:58:58 ]
- そんなもん整理始めたら収拾が付かないぞ。
コメントもどこまで信用できるか判らん代物を整理するくらいなら
リプレースしたほうが早いｗ
- 878 名前：仕様書無しさん mailto:sage [2007/03/18(日) 17:08:48 ]
- 協力会社の新人が Ethenet ケーブルといって分からなくて、LAN ケーブルと言っても
分からなかったときには、基本情報取るまで出てくんなとは思ったけどな
4年前の話
- 879 名前：仕様書無しさん mailto:sage [2007/03/18(日) 17:36:59 ]
- >>878
そういう問題でも無さそうだがw
- 880 名前：仕様書無しさん mailto:sage [2007/03/18(日) 17:49:56 ]
- わんつーすりーふぉーかてごりぃ〜ふぁいぶっ！！
- 881 名前：仕様書無しさん mailto:sage [2007/03/18(日) 20:55:19 ]
- 古い
(･∀･)ｶｴﾚ!
- 882 名前：仕様書無しさん mailto:sage [2007/03/19(月) 01:10:02 ]
- >>875
ファイル名が長すぎるといってエラーになるのはいいけど、
「落ちる」ってのは、cFile[40] の 40 が問題なのではないんじゃ?
- 883 名前：仕様書無しさん mailto:sage [2007/03/19(月) 02:39:45 ]
- つまり存在チェックをしていないんだろう
- 884 名前：仕様書無しさん mailto:sage [2007/03/19(月) 02:58:22 ]
- ところで、40というのはどこから出てきた数字なんだ？
ライブラリ辺りから出てきた数字だったらかなり嫌だな。
- 885 名前：仕様書無しさん mailto:sage [2007/03/19(月) 04:00:51 ]
- long long i*=i; // 流れ星ぴゅー（＾。＾
- 886 名前：仕様書無しさん mailto:sage [2007/03/19(月) 10:46:49 ]
- 基礎情報受けに行く時間がもったいないならいきなりソフ開なり上級なり受ければいいじゃない
- 887 名前：仕様書無しさん mailto:sage [2007/03/19(月) 18:11:18 ]
- 32あれば充分で、それに８も余分を足しとけば
たりなくなるわけがない。
ぐらいの考えだったんじゃね。
- 888 名前：仕様書無しさん mailto:sage [2007/03/19(月) 19:08:42 ]
- FILENAME_MAX や MAX_PATH 辺りをつかっておけばいいのにね.
この二つがどう違うのか知らんけど(ぉ
- 889 名前：仕様書無しさん mailto:sage [2007/03/19(月) 20:02:59 ]
- >875
元がDOS版だったりとかしない？（；・∀・）
- 890 名前：仕様書無しさん mailto:sage [2007/03/19(月) 20:17:50 ]
- >>839
素朴な疑問だが、
「入れる前」でチェックしているのなら、
「入れた後」でチェックする意味あるの？
想定外の経路(直接Accessとかでレコード作成とか)が無い限りは
普通Input側で整合性取れば後は無視しない？
いや、マジで素朴な疑問なんだよ。
俺なら想定外の経路を許可する事をしないモノで。。。
- 891 名前：仕様書無しさん mailto:sage [2007/03/19(月) 20:20:19 ]
- 想定外の経路を許可してなくてもどっかのバカが入れるんだよ
仕様書にきちっと書いてあっても間違った試験データ投入してうだうだと言ってくる
- 892 名前：仕様書無しさん mailto:sage [2007/03/19(月) 20:21:50 ]
- >>890
updateのときとか？
# 入れるときに入るかなぁ？
入力側と出力側を別人が作っている場合は、
まあ表示前にチェックってのはアリと言えばアリ。
- 893 名前：仕様書無しさん mailto:sage [2007/03/19(月) 21:46:41 ]
- >>890
19000000はNULLのことです、とか言われたらどうする？
- 894 名前：仕様書無しさん mailto:sage [2007/03/19(月) 21:51:56 ]
- >>890
そういう考えが「サニタイズすればいい」ってところに通じるんじゃないのか？
想定外のデータが嫌なら、DBに制約つけるなりなんなりしてくれ。
そうすれば、構造が変わらない限りは平気になる。
- 895 名前：仕様書無しさん mailto:sage [2007/03/19(月) 22:43:03 ]
- Dim 消しフラグ
Dim 消し消しフラグ
（中略）
'------------------------------------
' 小細工
'------------------------------------
End Sub
- 896 名前：仕様書無しさん mailto:sage [2007/03/19(月) 23:20:18 ]
- 勝手にCSVつくってインポートしたがる奴はどこの世界にもいるもんだよ
- 897 名前：仕様書無しさん mailto:sage [2007/03/20(火) 00:50:49 ]
- それはダメなのか？
- 898 名前：仕様書無しさん mailto:sage [2007/03/20(火) 16:32:48 ]
- >>897
勝手に作ったCSVデータを勝手にインポートした挙句データの整合性が取れなくなり、
ソフトに不具合が発生したらデータ作成した本人（もしくは指示した人間）が
「データベースが壊れた！バグだ！！修正しる！！！」
と騒いで無料サポートをさせるってパターンが『よく』あるから
>>896のグチになるわけだ。
その対策が>>890みたいな意見になるわけだけど、そうすると今度は
「データを入れたいのに受け付けない！バグだ！！修正しる！！！」となって
最初に戻る。
- 899 名前：898 mailto:sage [2007/03/20(火) 16:37:15 ]
- ｽﾏｿ、上の890は894の間違い
- 900 名前：仕様書無しさん mailto:sage [2007/03/20(火) 17:30:20 ]
- 「勝手に作ったCSV」なんて恐ろしいものの入力許すなよ。
入力チェック側も「カンマ区切りのテキスト」しか想定してない場合が多いんだから…orz
もちろん何のエスケープもなしで。
- 901 名前：仕様書無しさん mailto:sage [2007/03/20(火) 18:52:20 ]
- >>900
importの話だよ
- 902 名前：仕様書無しさん mailto:sage [2007/03/20(火) 19:42:09 ]
- >>898
そんな馬鹿の相手はしないに限る。
- 903 名前：８９０ mailto:sage [2007/03/20(火) 19:45:01 ]
- なんか、色々有るんね。
みんなの苦労が良くわかったよ。
おれは幸せなんだね。
ごめんね＞みんな
- 904 名前：仕様書無しさん mailto:sage [2007/03/21(水) 04:48:34 ]
- その時出来る対処はその時やっておいた方があとで
記憶を取り戻してからやるより結果的に時間の節約になるから
やったほうがいいね。
やる時間がないなら、せめて警告コメントくらいは入れておくもんだよ。
- 905 名前：仕様書無しさん [2007/03/21(水) 08:04:20 ]
- catch(Exception e) {
// Exceptionをcatchしたらthrowする
throw e;
}
- 906 名前：仕様書無しさん mailto:sage [2007/03/21(水) 08:30:54 ]
- 逆に入力チェック完璧って前提だから、出力の際にチェックしなくていいっていうのは
どういう仕事なのか気になる。
- 907 名前：仕様書無しさん mailto:sage [2007/03/21(水) 09:46:47 ]
- 余裕の無いプロジェクトだとありがち。
- 908 名前：仕様書無しさん mailto:sage [2007/03/21(水) 09:47:56 ]
- え？
出力時にチェックいるの?
- 909 名前：仕様書無しさん mailto:sage [2007/03/21(水) 10:22:20 ]
- 常識的に考えて入力チェックしたものを出力時再度チェックする必要ないぞ？
- 910 名前：仕様書無しさん mailto:sage [2007/03/21(水) 10:30:24 ]
- どういうシステムなのかにもよるが
それは認識が甘いと言わざるを得ない
- 911 名前：仕様書無しさん mailto:sage [2007/03/21(水) 10:36:38 ]
- 入力時に正しいデータだったのに、出力するときに不正なデータになってたら、ただのバグだと思うが。。。
- 912 名前：仕様書無しさん mailto:sage [2007/03/21(水) 10:40:10 ]
- >>911
つまり、ただのバグを検知できる、という指摘もありだな。
- 913 名前：仕様書無しさん mailto:sage [2007/03/21(水) 10:41:25 ]
- >>911みたいな人ってswitchのcaseが仕様上の選択肢すべてを
網羅しているときにはdefaultを書かないんだろうな。
- 914 名前：仕様書無しさん mailto:sage [2007/03/21(水) 10:50:49 ]
- いや、バグの検出と、入力のチェックを区別つけないで議論してるヤツとかダメだろ。。。
- 915 名前：仕様書無しさん mailto:sage [2007/03/21(水) 10:52:45 ]
- defaultに到達したらバグだろう普通にｗ
その場合バグ解析コードだろdefaultは
- 916 名前：仕様書無しさん mailto:sage [2007/03/21(水) 10:55:26 ]
- 2chでよくいる、「バグがあっても動いてしまうコード」をいい書き方だって固く信じてるやつって、
バグ検出と入力時のチェックを概念的に混同してるんだと思われ。
- 917 名前：仕様書無しさん mailto:sage [2007/03/21(水) 11:29:22 ]
- バグの検出と入力のチェックの区別がつけられない奴って
アサーションも碌に使えないんだろうなきっと。
- 918 名前：仕様書無しさん mailto:sage [2007/03/21(水) 11:49:42 ]
- データなんて保存中に壊れることもよくある
- 919 名前：仕様書無しさん mailto:sage [2007/03/21(水) 11:50:13 ]
- 「assert で停止するのはデバッグ時のみ」であることを曲解すると
「リリース時はそのまま動き続けるのが正しい」となる。
…気持ちは解らんでもない。
- 920 名前：仕様書無しさん mailto:sage [2007/03/21(水) 12:01:39 ]
- この前、VB4からVB6に移植する仕事をやったけど、全サブルーチンにエラーハンドラがつけてあって、
多くは、Beepを鳴らして、ログにエラー番号みたいのを書き出して、そのまま処理を続行になってた。
現場じゃ、画面が明らかにおかしくなってるようなバグ以外は、そのまま操作を続行しちゃうんだろうなぁ。
- 921 名前：仕様書無しさん mailto:sage [2007/03/21(水) 13:35:25 ]
- 入力時のチェックは厳重・厳密に行い、出力時のチェックはゼロ。。。。
マスタ登録部はチェックが多重だが、読み出し部は「マスターが正」だから
チェックも糞もない。データとマスタとのチェックは厳密だろうけど。
これくらいしか思いつかない。
- 922 名前：仕様書無しさん [2007/03/21(水) 15:20:06 ]
- 日付・時間が
char(6)Char(8)
とか勘弁してください。
比較が面倒です
- 923 名前：仕様書無しさん mailto:sage [2007/03/21(水) 15:44:52 ]
- 西暦・和暦変換するクラスの名前が
WestanYearとなってる。笑い殺すつもりかよ
- 924 名前：仕様書無しさん [2007/03/21(水) 16:02:11 ]
- Seireki2Warekiだろ
- 925 名前：仕様書無しさん mailto:sage [2007/03/21(水) 16:05:02 ]
- CalenderConversがいいんでないか？
- 926 名前：仕様書無しさん mailto:sage [2007/03/21(水) 16:10:46 ]
- 28時とか入っててびびった(char(4)で2800）
というか仕様書に書いてもないのにそれが正常とか言わんでくれ・・・
- 927 名前：仕様書無しさん mailto:sage [2007/03/21(水) 16:14:53 ]
- >924
あるあるｗｗ
- 928 名前：仕様書無しさん mailto:sage [2007/03/21(水) 16:26:38 ]
- >>926
テーブルの制約で不可にするのが当たり前だろう
- 929 名前：仕様書無しさん mailto:sage [2007/03/21(水) 16:31:20 ]
- >>28時とか入っててびびった(char(4)で2800）
char(4)だと99時99分でオーバーフローになります、
もっと桁をとりましょう
- 930 名前：仕様書無しさん mailto:sage [2007/03/21(水) 16:46:01 ]
- >>922
正確には「時間」ではなくて「時刻」ではないかと。
多くの言語やDBで混同されているみたいだが。
>>929
四徹程度でオーバーフローしてしまうとは情けない・・・か。
- 931 名前：仕様書無しさん mailto:sage [2007/03/21(水) 16:52:56 ]
- >>930
VC(MFC)、C#, Oracleは区別あるな。
ほかのでもあるんじゃね?
- 932 名前：仕様書無しさん mailto:sage [2007/03/21(水) 16:54:32 ]
- >>905
それはよくやる。というか、結果的にそうなることが多い。
#if DEBUG で括ったMessageBoxをいれてるだけのとことか。
似たようなことをやろうとして、↓のようにするやつもいるが…。
catch(Exception e) {
// Exceptionをcatchしたらthrowする
new throw Exception();
}
|
 |
