PreparedStatementについて質問

質問

すみません。PreparedStatementについて勉強中です。

現在、下記の形で更新処理をしようとしています。

{
for(int i ; i<500;i++){
　　　　　//PreparedStatementを使って更新処理
　　　　　classA.updateData();
　　}
}finally{
//PreparedStatementやResultSetのclose処理
　classA.close();
}

実行すると、[ORA-1000 最大オープン・カーソル数を超えました 。]が発生してしまいました。

ネットで探してみると、close処理が抜けていたりすると発生すると書いてありました。
close位置が悪いのでしょうか。（for文の中に入れる？）
オラクルの設定が悪いのでしょうか。

分かりづらい質問で申し訳ないです。

ご指導、また参考になるサイト様がありましたら教えていただきたいです。
よろしくお願いいたします。

回答No.1

root139 · ベストアンサー

classA.updateData() の中身が分からないとハッキリしたことは言えないのですが、エラーメッセージからしてOracleに設定されている最大カーソル数より多くの ResultSet を同時にオープンしているのでしょう。
classA.updateData() の中で PreparedStatement/ResultSet オブジェクトを取得していて、かつ close をしていなければ、少なくとも500個は同時にオープンしていることになるかと。(参照が外れていてGCが走ればそうならない可能性も有りますが)

また、classA.close() の中では classA.updateData() で開いた分(500個？)の close を行っているでしょうか？
たとえ、一つの PreparedStatement 変数/フィールドしか使っていなくても、複数のオブジェクトを生成している場合はそれぞれのオブジェクトに close が必要となります。

悪い例) --------------------------------------------

PreparedStatement pstmt = null;

try {

// 1個目の PreparedStatement オブジェクト
pstmt = connection.prepareStatement(sql1);
・
・
・

// 2個目の PreparedStatement オブジェクト
pstmt = connection.prepareStatement(sql2);
・
・
・

} finally {
pstmt.close(); // 2個目の PreparedStatement しかcloseされない！
}

----------------------------------------------------
※ 簡単にする為に例外処理などは省略しています。

メモリ以外のリソースに関わる PreparedStatement などについては、原則的に一つのメソッド内でオープンとクローズを行うと良いです。どうしても分けなければならない場合も有るでしょうが。

お礼

分かりにくい説明にご回答ありがとうございます。

classA.updateData()のなかは、端よって書くと

PreparedStatement pstm = con.prepareStatement("UPDATE テーブルA SET 項目A=?");

pstm.setInt(1, 更新値);
pstm.executeUpdate():

という感じです。

PreparedStatement,ResultSetで勉強不足で分らないことも多いのですが・・・

確かに、ループの分だけ「pstmt」が作られます。

ということは、同じSQL文で更新値が違うだけだと

if(pstmt!=null){
pstmt = connection.prepareStatement(sql);
}

とすれば大丈夫ということでしょうか。