select文の結果を変数に取得したい

現在、SQL文の発行によりデータベースアクセスを行うdaoクラスを作成中です。
このクラスの中に、ID自動採番を行うメソッド(呼び出し元に戻り値integer型のIDが代入された変数を返す)を作っているのですが、select文の問い合わせ結果をどのように変数に取得すればよいのかがわからず困っています。自分でこうではないかというソースコードを作成したのですが、何か違うような気もしています。
宜しければどうかご助力いただけませんでしょうか？
以下、該当メソッド部分のソースコードになります。

/* フィールド変数 */
private Connection con;        // 接続オブジェクト
private PreparedStatement ps;  // ステートメント
private ResultSet rs;          // SELECT文の結果を受け取る


    /* param con 接続オブジェクト */
public Integer getId(Connection con){
	
	/* SQL文を作成する */
	StringBuilder sb = new StringBuilder();
	sb.append("SELECT ");
	sb.append("MAX(ID) + 1 ");
	sb.append("AS ID ");
	sb.append("FROM ");
	sb.append("TABLE_LIST ");

//IDを格納する変数numを初期化
Integer num = 0;

	try {
		
		// プリペアードステートメントを生成する
		ps= con.prepareStatement(sb.toString());
		
		// SQL文発行
		rs = ps. executeQuery();

  num = rs.getInt("ID"); //"ID"という文字列から果たして結果が取得できるのでしょうか？

	}catch(Exception e){
		e.printStackTrace();

	}finally{
		try{

			// 後処理を行う
			if(ps!=null) ps.close();
            if(ps!=null) ps.close();
			if(con!=null) con.close();

		}catch(SQLException e){
			e.printStackTrace();
		}
	}

//呼び出し元へ返す
return num;
}

以上になります。
宜しくお願い致します。

行動規範の内容に同意します

回答2件

SELECT MAX(ID) + 1
は止めた方が良いかも。

取得された MAX(ID) + 1 のデータがINSERTされる前に、同時ユーザがいれば、重複してしまうことになります。最大値の管理テーブルを作り、以下の様に処理をした方が良いのでは？

主処理のsession(トランザクション)開始
↓
最大値取得用のsession(トランザクション)開始
↓
最大値管理テーブルをロック
↓
最大値を取得、カウントアップしてコミット(ロック解除)
↓
主処理
↓
主処理のsessionをコミット(またはロールバック)

※ 主処理でこけたら飛び番ができます。

投稿2015/05/11 00:52

kantomi

総合スコア295

フィールド名で取得しようとすると、データベースの種類によっては大文字、小文字のどちらかしか使えないケースが出てきます。
列の順序（1から始まる）を指定するResultSet#getInt(int)を使用した方が確実です。

また、ご質問のポイント以外にもいくつか改善した方が良いと思われる箇所があります。
・con,ps,rsはフィールドにする意味が無いので、必要になった時点でローカル変数を宣言するようにした方が良いでしょう。
・ Java7以降ではtry-with-resources文を使用した方が簡潔に書けます。
・conはメソッドを呼び出している側で接続を切断するようにした方が良いです。（開けた人が閉じる）

以上を踏まえると、こんな感じになります。

lang
1// Java7以降
2
3// import java.sql.*;
4
5/* param con 接続オブジェクト */
6public Integer getId(Connection con) {
7
8    /* SQL文を作成する */
9    StringBuilder sb = new StringBuilder();
10    sb.append("SELECT ");
11    sb.append("MAX(ID) + 1 ");
12    sb.append("AS ID ");
13    sb.append("FROM ");
14    sb.append("TABLE_LIST ");
15
16    //IDを格納する変数numを初期化
17    Integer num = 0;
18
19    // プリペアードステートメントを生成する
20    try (PreparedStatement ps = con.prepareStatement(sb.toString())) {
21        // SQL文発行
22        try (ResultSet rs = ps.executeQuery()) {
23            num = rs.getInt(1);
24        }
25    } catch (Exception e) {
26        e.printStackTrace();
27    }
28
29    //呼び出し元へ返す
30    return num;
31}