これはひどい

http://www.atmarkit.co.jp/fjava/rensai4/webjousiki08/webjousiki08_3.html

@ITってSI業界だと結構参考にされるサイトだと思うのですが(違っていたらごめんなさい)、こういうコードを鵜呑みのにして業務システムが作られていると思うとぞっとします。

つっこみどころとしては、

・PreparedStatement/ResultSetをCloseしていない

ConnectionをCloseした段階でPreparedStatementもCloseされるって?
コネクションプールをしている場合も同じ事が言えますか?
PreparedStatementCacheをしている場合も同じ事が言えますか?
「データベースに接続したら後始末は確実に」という記事を参照しろと書いてあるにもかかわらず、掲載されているコードは参照してかつ参考しているとは思えない。

 

・while(rs.next())の部分

これWhileである必要ないですよね。
Whileで回すって事はIdはユニークではないのでしょうか。
でもCount==1の時のみ処理しています。
何がしたいのか意味がわかりません。こんな冗長で分かりづらいコード書かないでIfで良いのでは。
if(rs.next()){ 一致データを取得 } else{ 認証失敗 }

 

こんなコード出してきたらグーパンチですよヽ(`Д´)ノ

新しいサイトもよろしくお願いします!