注:ソースは私が作ったものではありませんw
1.hiddenがいっぱい
1画面に30個くらいhiddenコントロールが設置されていました。
hiddenとは?
いくら社内向けシステムと言えど、セキュリティ上危険なの(CSRF攻撃など)分かってるのかなぁ・・・(´-ω-`)
#この後、機能追加によりhidden増加...orz
2.プロパティの意味なし
【Sample1 クラス】
// この時点でオチが分かる罠
public string aaa;
public string bbb;
public string ccc;
public string Aaa{
get{
return aaa;
}
set{
aaa = value;
}
}
public string Bbb{
get{
return bbb;
}
set{
bbb = value;
}
}
public string Ccc{
get{
return ccc;
}
set{
ccc = value;
}
}
Aaa = "0001";
Bbb = "hogehoge";
Ccc = "2006/02/03";
【Sample2クラス】
public Sample1 clsSample1 = (Sample1) Context.Handler;
hogeID.Text = clsSample1.aaa;
hogeName.Text = clsSample1.bbb;
hogeDate.Text = clsSample1.ccc;
// ちょwwwプロパティの意味がn
//結論:【カプセル化について説明】
3.無意味なクライアントスクリプト(JavaScript)
親ウィンドウ上にあるボタンをクリック
↓
子ウィンドウ表示
↓
子ウィンドウのボタンクリック
↓
親ウィンドウをリロード(子ウィンドウから親ウィンドウへ値渡し)
↓
子ウィンドウ閉じる
という処理をしたかったらしいが、
window.close();
window.opener.document.forms[0].submit();
・・・いやいや、自分が先に閉じちゃったら意味ないからw
4.無限ループ
for(i=DataGrid1.Items.Count;i>4;i++)
{
//なんかの処理してる
}
DataGrid1.Items.Count>=5でこの処理に入ると
もれなく無限ループになるんですけど・・・
#そのうちメモリ60%越えして落ちる
作った本人に指摘したら大爆笑してた。
こんな常駐先で仕事してます...orz