java PreparedStatementをトランザクションで使いたい

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
新規登録の際、入力されたログインIDと名前とパスワードを基にInsertのsqlを3つ行いたいと考えています。
新規登録用のサーブレットで以下のDAOの中のsignupメソッドを使う感じです。
3つの引数は入力されたログインIDと名前とパスワードとなっています。

発生している問題・エラーメッセージ

sqlで問題が発生していて、データベースに反映ができません。
データベースに登録できるようにしたいです。

public void signup(String inputLoginId, String inputUserName, String inputPassword) {
		Connection con=null;
		con=DBManager.getConnection();

		try {
			con.setAutoCommit(false);
			String sql="BEGIN;";
			sql+=" INSERT INTO user (login_id,name,password) VALUES (?,?,?);";
			sql+=" INSERT INTO ranking_easy (user_id) select id from user where login_id=?;";
			sql+=" INSERT INTO ranking_hard (user_id) select id from user where login_id=?;";
			sql+=" COMMIT;";
			PreparedStatement st=con.prepareStatement(sql);
			st.setString(1, inputLoginId);
			st.setString(2, inputUserName);
			st.setString(3, inputPassword);
			st.setString(4, inputLoginId);
			st.setString(5, inputLoginId);
			st.executeUpdate();
			con.commit();
			st.close();

		}catch (SQLException e) {
			e.printStackTrace();
		} finally {
			if (con != null) {
				try {
					con.close();
				} catch (SQLException e) {
					e.printStackTrace();
				}
			}
		}

	}

```
エラーメッセージ
```

 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'INSERT INTO user (login_id,name,password) VALUES ('tanaka','田中','password');' at line 1

```ここに言語名を入力
ソースコード
```

### 試したこと

ググりましたがよくわからず・・・・

### 補足情報（FW/ツールのバージョンなど）
eclipse/tomcat使用
mySQL使用

データベースのuserテーブルについて
id	int(11) auto_increment
name	varchar(255)
login_id	varchar(255)
password	varchar(255)

データベースのranking_easyテーブルについて
id	int(11) auto_increment
user_id	int(11)
easy_high_score	int(11)

データベースのranking_hardテーブルについて
id	int(11) auto_increment
user_id	int(11)
hard_high_score	int(11)

Comments

[jimbe]

inset 毎に個別に実行しては如何ですか.
トランザクションも sql ではなく jdbc のレベルで行ってください.

Answer 1

SQLのバッチ実行

3つのSQLを一度に実行したいのならStatement#executeBatchを利用すると良いかもしれません。3つのSqlがそれぞれ異なるのでPreparedStatementは使えません。Statementになります。SQLインジェクション対策が必要。

public void signup(String inputLoginId, String inputUserName, String inputPassword) {
  try (Connection conn = DBManager.getConnection()) {
    conn.setAutoCommit(false);
    try (Statement stmt = conn.createStatement()) {
      stmt.addBatch("INSERT INTO user (login_id,name,password) VALUES ('"+inputLoginId+"','"+inputUserName+"','"+inputPassword+"')");
      stmt.addBatch("INSERT INTO ranking_easy (user_id) select id from user where login_id='"+inputLoginId+"'");
      stmt.addBatch("INSERT INTO ranking_hard (user_id) select id from user where login_id='"+inputLoginId+"'");
      int[] results = stmt.executeBatch();
      for (int cnt : results) {
        if (cnt != 1) {
          conn.rollback();
          throw new SQLException("signup('"+inputLoginId+"','"+inputUserName+"') failed.");
        }
      }
      conn.commit();
    }
  } catch (SQLException e) {
    e.printStackTrace();
  }
}

1つのPreparedStatementを大量に実行する場合はバッチ実行が可能です。

Answer 2

インジェクション対策のため；の後ろは空白文字以外許可されてないはずです。

五回に分けましょう。
現在の記述では出来たとしてもオートコミットと同じ処理です