teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

1

微修正

2015/05/10 04:33

投稿

argius
argius

スコア9396

answer CHANGED
@@ -2,7 +2,7 @@
2
2
  列の順序(1から始まる)を指定する`ResultSet#getInt(int)`を使用した方が確実です。
3
3
 
4
4
 
5
- ご質問の点ではありませんが、他にも少なとも2点ほど改善した方が良い箇所があります。
5
+ また、ご質問のポイント以外にもつか改善した方が良いと思われる箇所があります。
6
6
  ・`con`,`ps`,`rs`はフィールドにする意味が無いので、必要になった時点でローカル変数を宣言するようにした方が良いでしょう。
7
7
  ・ Java7以降ではtry-with-resources文を使用した方が簡潔に書けます。
8
8
  ・`con`はメソッドを呼び出している側で接続を切断するようにした方が良いです。(開けた人が閉じる)