SQLバインド変数の話

Question

###ネタ
SQLのWHERE句を動的にする
INPUTパラメータsettleType,staffTypeがnullもしくはタブ区切りの複数要素が来る
IN句を使わずに複数要素分の条件を記述していく

以下抜粋（DB接続とかは省k）

// 前準備
String[] settleList = null;
String[] staffList = null;
StringBuilder sql = new StringBuilder();
sql.append(
"SELECT " +
	うんたらかんたら
" FROM " +
"       STM " +
"      , WSM " +
" WHERE " +
);
// settleTypeがnullか判定
// バインド変数を個別に指定する
if ( !settleTypeのnullチェック ) {
	sql.append("      AND ( STM.SETTLE_TYPE = ? ");
	settleList = settleTypeをカンマ区切り(settleType, ",");
	for(int settleSize=1; settleSize<settleList.length; settleSize++) {
		sql.append("      OR STM.SETTLE_TYPE = ? ");
	}
	sql.append("      ) ");
}
// staffTypeがnullか判定
// バインド変数を個別に指定する
if ( !staffTypeのnullチェック ) {
	sql.append("      AND ( WSM.STAFF_TYPE = ? ");
	staffList = staffTypeをカンマ区切り(staffType, ",");
	for(int staffSize=1; staffSize<staffList.length; staffSize++) {
		sql.append("      OR WSM.STAFF_TYPE = ? ");
	}
	sql.append("      ) ");
}
PreparedStatement pstmt = con.prepareStatement(sql.toString());
try {
	// パラメータ設定
	int  settleSz = 0;
	int  staffSz = 0;
	// settleTypeがnull以外の場合に値を設定
	if ( !StringUtil.isNull(settleType) ) {
		for(String settleParam : settleList){
			settleSz++;
			pstmt.setString(settleSz, settleParam);
		}
	}
	// staffTypeがnull以外の場合に値を設定
	if ( !StringUtil.isNull(staffType) ) {
		for(String staffParam : staffList){
			staffSz++;
			pstmt.setString(settleSz + staffSz, staffParam);
		}
	}
	// SQL実行
	ResultSet rs = pstmt.executeQuery();
}

###課題
汚い

バインド変数の設定とパラメータの設定を上手く紐付けできればもっとスリムにできそう

###ここに至るまで起きたバグ（戒め）
バインド変数の設定箇所が無い状態でパラメータを設定してしまうと
「列索引が無効です」オラクルエラーが発生

Answer 1

まず拡張for文とnullチェックをutilメソッド化すればよさげ。
そうなると
// バインド変数を個別に指定する
のループを拡張for文にすればいいのかな？
（最初の一個目を飛ばす制御が必要になるけど、それはそれでダサそう・・）

Comments

[momon-ga]

技術的なメモを残したいなら別のサービスを利用されてはいかがでしょうか。

[pilepierce]

すいません　閉じておきます