トップページ⇒tech
36コメント10KB

あなたの同僚(糞PG)が書いた糞コード

■ このスレッドは過去ログ倉庫に格納されています
0001デフォルトの名無しさんNGNG
を暴露してください(反面教師)。
そしてこのスレにあるような糞コードを書かないようにきおつけよう。
0002デフォルトの名無しさんNGNG
>>1
だからまず自分がだせよ。
00031NGNG
だって職業PGじゃないもん。
いや、現場の声を聞きたいと思ってね。
0004デフォルトの名無しさんNGNG
>>1 既出

あなたのまわりの糞コード
http://piza2.2ch.net/test/read.cgi?bbs=tech&key=981185737
00052NGNG
>>1
なら自分の糞コードを公開すべき。人に何かしてほしければまず
自分からやるべき。
00061NGNG
>>5
今コードは手元にないっす。
それに反面教師になるような、
換言すれば、良い意味にせよ悪い意味にせよ
学べる部分があるほどのコードはもってないです(初心者なので)。
ただ、早いうちから、どういうコードがマズいのかを知っておきたかったです。
>>4
ありがとうございま

＠＠＠＠＠＠＠終了ってことで＠＠＠＠＠＠＠＠
0007デフォルトの名無しさんNGNG
ついでに１の人生も


＠＠＠＠＠＠＠終了ってことで＠＠＠＠＠＠＠＠
0008弁護人NGNG
異議あり！
0009デフォルトの名無しさんNGNG
ageんなヴォケ
0010デフォルトの名無しさんNGNG
Delphiにて、必ずTrueを返すだけの意味不明理解不能な関数。
(関数中の演算も意味不明)

Function fnMST_CHK(pTokCode:Integer):Boolean;
var
wInt:Integer;
begin
wInt:=0;

if (wInt<>0) then
begin
wInt:=wInt-1000;
Result:=False;
end else begin
wInt:=wInt+1000;
Result:=True;
end;

end;
0011デフォルトの名無しさんNGNG
>>10
なあに、プログラム全体が意味のない計算で出来ていた
新メモリ最適化ツールに比べたらなんてことはないよ。
0012デフォルトの名無しさんNGNG
>>11

それって、Mターボ2の事ですか？
0013デフォルトの名無しさんNGNG
>>12
CompJapanとかで検索してください。
001412NGNG
>>13
をぉ、こんな事があったのか、長年生きてきたが全くしらなんだ。
是非とも入手しておきたかった一品だが既に残骸すら見当たらない、知るのが遅すぎたようだ。（ｗ
0015デフォルトの名無しさんNGNG
>>10
なあ、>4みてageてんの？
なら史ネ
0016３０代後半NGNG
特殊な場合（画面表示以外に使用されない、等）を
除いて、金額を表すデータは固定小数点型（あるいは
文字型）を使用するのは常識かと思っていたのですが、
以前、発注先ソフトウェアハウスのＰＧ連中に
「桁が大きいから浮動小数点使って書きました」と
言われて、顔面蒼白になりました。（納期前２週間）
ちゃんとCurrency型かlong long型使えって文書で
ガイドして口頭でも説明したのに。
会計系のシステムなのに、誤差どうすんだよー。

当然やりなおしを命じましたが、全然深刻度がわかってもらえ
ず不満顔。同僚までもが「なんで？」って奴がほとんど
だし、鬱だ。

みんなもうそんなこと気にしないのか？それでいいのか？
0017デフォルトの名無しさんNGNG
>>16
　Currency は　内部で1万倍された long double であったりしますよ

逆にCurrencyなら大丈夫って訳ではないし

加算時に必ず少数点が無いとか、除算を丸めの直前にやってるだけとか
ちゃんと気を付けて使えば問題はないでしょう。

まあ気を付けて使わない人が殆どなのが問題なんですが・・・・
0018デフォルトの名無しさんNGNG
>>16
納期2週間前まで進捗をろくに聞いてないおまえが全て悪い。
と思うのは俺だけか？
0019デフォルトの名無しさんNGNG
処理が終わった後どうしていいか判らなかった馬鹿のコード。(Z80)

stop:　　jp　　stop
0020デフォルトの名無しさんNGNG
IRQではなくNMIにしてくれと強く主張した馬鹿の割りこみルーチン。(Z80)

　　　exx
　　　割りこみ処理
　　　exx
0021デフォルトの名無しさんNGNG
馬鹿の書いたレジスタ退避サブルーチン。（Z80）

push_reg:
　　　push　　af
　　　push　　bc
　　　push　　de
　　　push　　hl
　　　ret　　
　　　　　　
0022デフォルトの名無しさんNGNG
>>21
う・・。
0023デフォルトの名無しさんNGNG
float使っても、要求仕様の範囲内で誤差が出ないように作っていれば
問題ないのでは？

誤差が出てるのだとすれば、それはコードレビューをしなかったり、
テスト内容のレビュー、結果のチェックををしなかった方も悪い。
0024デフォルトの名無しさんNGNG
>>23
金が絡んでるのに誤差ってなんだよ。
まあ、給料が1円減ったからって気にはならないかもしれないけど。

昔ねぇ、利息の計算上の端数（1 cent 以下）を全部自分の講座に振り込んだやつが
いるらしいよ。
0025デフォルトの名無しさんNGNG
>>24
よほど給料が少なかったんだろ。許してやれよ。
0026デフォルトの名無しさんNGNG
>>24
オチがあって（本当かどうかしらんが）、銀行が
大口預金者を招待したパーティーを企画して、
招待客をリストアップしたところ、このプログラマーが
載っていて、「こいつの給料でこんなにも預金がある
のは絶対おかしい」ということになって発覚したらしい。
ゴミがつもって大金になってたんだろうなぁ。
0027デフォルトの名無しさんNGNG
ところで、利息の計算て複利だとしても　〆日毎に小数点以下を処理するのが普通じゃないの？
0028まあ、ネタですが。NGNG
ああ、そんなことで困らないために Cobol があるのに...
0029デフォルトの名無しさんNGNG
>>21
それで馬鹿というのは…
分かるけどサ
0030デフォルトの名無しさんNGNG
あるファイルを排他で開くのに、時間切れまでロックを繰り返していた。
0031デフォルトの名無しさんNGNG
マへ帰れ>all
0032デフォルトの名無しさんNGNG
一瞬、dllへ帰るのかと思った。
0033デフォルトの名無しさんNGNG
>>26
最近見たドイツ映画のネタで同じのがあった。
全欧中の銀行で使用されるシステムで切り捨てられるコンマいくらか以下を
銀行口座に入れるってやつ。
0034 NGNG
>>26
「太陽にほえろ」でもそのネタあったなあ。石原兄弟の一人が「マイコンデカ」を演じて解いてたなあ。
0035デフォルトの名無しさんNGNG
つか、銀行のシステムで切り捨ての金額を自分の口座に振りこんだって
話は都市伝説なんでしょ？
0036デフォルトの名無しさんNGNG
>>35
あんたなにをいまさら。
■ このスレッドは過去ログ倉庫に格納されています