JAVAからSQLiteのテーブル作成

Question

###前提・実現したいこと
JAVAのアプリケーションからSQLiteのテーブル3つの存在確認を行い、
存在しなければテーブルを作成しようとしています。

###発生している問題・エラーメッセージ
テーブルの存在確認は正しくできているようですが、
その後テーブルを作成する際に、3つのうち一つしか作成されません。
何か制約があるのでしょうか？
何かやり方がおかしいのでしょうか？

下記コード内のSQL文をコマンドラインから直接発行すると、テーブル作成は可能です。

なお、本来複数ファイルに分けているものを、
実行に必要な分だけ持ってきているので、使っていない変数などは気にしないでください。
（このコードで再現することは確認済みです。）

###ソースコード

import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;

public class Test {
	public static String SELECT_TABLE  = "SELECT COUNT(*) FROM sqlite_master WHERE type='table' AND name='__TABLENAME__'";
	public static String CREATE_INFO   = "CREATE TABLE INFO(ID INTEGER PRIMARY KEY AUTOINCREMENT,NAME,IMAGE,ATTRIBUTE,OBTAINING_METHOD,TAG);";
	public static String CREATE_TAG    = "CREATE TABLE TAG(ID INTEGER PRIMARY KEY AUTOINCREMENT,TAG_NAME,TAG_DETAIL);";
	public static String CREATE_NAME   = "CREATE TABLE NAME(ID INTEGER PRIMARY KEY AUTOINCREMENT,NAME_NAME,NAME_DETAIL);";
	private static Connection con = null;
	private int batchCount = 0;
	private static final String dbFilePath = "C:/Test.db";

	public static void main(String[] args) {
		try {
			Class.forName("org.sqlite.JDBC");
			con = DriverManager.getConnection("jdbc:sqlite:" + dbFilePath);
			Statement stmt = con.createStatement();
			isTable(stmt);
		} catch (SQLException | ClassNotFoundException e) {
			// TODO 自動生成された catch ブロック
			e.printStackTrace();
			System.out.println("テーブル存在確認の失敗");
		}
	}

	//テーブル確認(なければ作る)
	public static void isTable(Statement stmt) throws SQLException{
		String[] tableName = {"INFO", "NAME", "TAG"};
		int queryCount = 0;
		for(int i = 0; i < tableName.length; i++){
			String sql = SELECT_TABLE.replace("__TABLENAME__", tableName[i]);
			ResultSet res = stmt.executeQuery(sql);
			int result = 0;
			result = res.getInt(1);

			if(result == 0){
				stmt.addBatch(getCreateQuery(tableName[i]));
				queryCount++;
			}
		}
		if(queryCount != 0){
			executeBatch(stmt, queryCount);
		}
	}

	public static boolean executeBatch(Statement stmt, int queryCount){
		boolean ret = false;
		try {
			int[] updateCounts = stmt.executeBatch();
			int successCnt = 0;
			for(int i = 0; i < updateCounts.length; i++){
				if(updateCounts[i] >= 0){
					//クエリ実行回数
					successCnt++;
				}
			}
			if(successCnt == queryCount){
				ret = true;
			}
		} catch (SQLException e) {
			// TODO 自動生成された catch ブロック
			e.printStackTrace();
		}
		return ret;
	}

	public static String getCreateQuery(String tableName){
		if(tableName == null){
			return "";
		}
		String upperTableName = tableName.toUpperCase();
		if(upperTableName.equals("INFO")){
			return CREATE_INFO;
		}else if(upperTableName.equals("TAG")){
			return CREATE_TAG;
		}else if(upperTableName.equals("NAME")){
			return CREATE_NAME;
		}
		return "";
	}
}

###補足情報(言語/FW/ツール等のバージョンなど)
環境は、
java 1.8.0_60
eclipse 4.5
sqlite 3.8.6
です。

Answer 1

Statementはexecute系メソッドを実行するとaddBatchで追加したSQLは破棄されます。
そのため、isTableメソッドのstmt.addBatch(getCreateQuery(tableName[i]));の代わりにListに追加しておき、executeBatch(stmt, queryCount);の直前でaddBatchすれば3つのテーブルが作成されます。

詳しくはStmt.javaのexecute系メソッドとinternalCloseメソッドを確認してください。

Comments

[moredeep]

回答ありがとうございます！解決できました。

[shiena]

ついでにもう一言。
create table if not existsにするとテーブルが存在しないときだけテーブルが作成されます。
こっちを使うと処理がすっきりすると思います。
https://www.sqlite.org/lang_createtable.html
http://www.dbonline.jp/mysql/table/index1.html

[moredeep]

そんな便利なクエリがあるんですね。。。
使ってみます！

Answer 2

こんにちわ。

isTableメソッド内で、executeBatchメソッドが呼び出される場所がforループの外だからでは
ないでしょうか?該当箇所をforループ内に入れてみてください。

Comments

[moredeep]

回答ありがとうございます！解決できました。