SQLへのINSERT文に関して

Question

お世話になります。JAVA初学者のものです。
現在ECLIPSEからMySQLへのINSERT文を実行したく試行錯誤しているのですが、
コード含めどうすれば良いのか分からないため質問をさせて頂ければ幸いです。

現在WEBアプリ上で入力された値をInsert_Servletで各々受け取っており、
birthのみ、本来登録先のSQLではDate型なため、java.sql.DateのvalueOfを使ってDate型に変換し、受け取った全ての値をUserBeanクラス（インスタンスの変数はub）にセット。
daoをインスタンス化してaddInfoメソッドに引数ubを渡して呼びだし。

その後Daoの方で実際にSQL分に値を詰めて実行。
…とこれでINSERTできるかなと思っていたのですが、データベースに反映されず…

どうすればINSERTを実行できるのでしょうか？
returnが変な位置にあるとかでしょうか？

昨日の朝からinsert分に悩まされておりまして、そろそろ解決したいです…（泣）
初学者なものでうまく全編を理解出来ておりませんが、どうかよろしくお願いいたします。

__**「①Insert_Servlet」**__


protected void doPost(HttpServletRequest request, HttpServletResponse response)
			throws ServletException, IOException {

		request.setCharacterEncoding("UTF-8");

		String loginId = request.getParameter("login_id");
		String password = request.getParameter("login_password");
		String rePassword = request.getParameter("login_repassword");
		String userName = request.getParameter("login_username");
		Date birth = Date.valueOf(request.getParameter("login_birth"));

		UserBean ub = new UserBean();
		ub.setLoginId(loginId);
		ub.setPassword(password);
		ub.setName(userName);
		ub.setBirthDate(birth);

		UserDao dao = new UserDao();
		dao.addInfo(ub);

	}

}



__**「②Insertを実行するDaoのコード」**__


public void addInfo(UserBean ub) {

		Connection conn = null;
		PreparedStatement ps = null;

		try {

			conn = DBManager.getConnection();

			String sql = "INSERT INTO UserManagement.user (login_id,name,birth_date,password)VALUES(?,'?','?','?')";

			ps = conn.prepareStatement(sql);

			ps.setString(1, ub.getLoginId());
			ps.setString(2, ub.getName());
			ps.setDate(3, (Date) ub.getBirthDate());
			ps.setString(4, ub.getPassword());

			ps.executeUpdate();

			return;

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

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

Answer 1

以下は、先のご質問 teratail#218157 - SQLへのINSERT文に関して でご提示されたコードの一部で、
「②Insertを実行するDaoのコード」 の部分ですが

          PreparedStatement ps = null;

            try {

                conn = DBManager.getConnection();

                String sql = "INSERT INTO UserManagement.user(login_id,name,birth_date,password)VALUES(?,'?','?','?')";


                ps.setString(1, ub.getLoginId());
                ps.setString(2, ub.getPassword());
                ps.setDate(3, (Date) ub.getBirthDate());
                ps.setString(4, ub.getName());

                ps.executeUpdate();

何か、おかしいと思いませんでしょうか。

• PreparedStatement psの初期化は。。。？
• DBManager.getConnection()で取得したコネクションとの関連は？
• Exceptionがthrowされていませんか？

ググるなどして、PreparedStatementの使い方を再度確認してみると良いと思います。

Comments

[beginner_7]

ご回答いただき有難う御座います！

PreparedStatement psの初期化はtry内で行う必要がある様ですね。
PreparedStatement ps = conn.prepareStatement(sql);
という風に改変してみました。

①PreparedStatement psの初期化は。。。？
②DBManager.getConnection()で取得したコネクションとの関連は？
③Exceptionがthrowされていませんか？

これで①②は解決しましたでしょうか？
また③はdaoには記載はありませんがServletの方には記載があります。
これはなにかまずいのでしょうか？

完全独学で学習を進めているため、細かな理解が出来ておらずで申し訳御座いませんが、何卒よろしくお願い致します。

[dodox86]

> これで①②は解決しましたでしょうか？
いや、それはコードを書いた、質問者の本人たるbeginner_7さんが実際に実行して、確認すべきものでは。。。？

> また③はdaoには記載はありませんがServletの方には記載があります。
>
> これはなにかまずいのでしょうか？

実際のところ、まずくするか、try～catchがなくてもうまく動くようするかはプログラマー次第です。try～catch の目的を理解しないとダメだと思います。例えば正しく初期化されていない変数valがあったとして、値がnullのままアクセスしたらどうなりますか？

[beginner_7]

ご回答いただき有難うございます。
①②を直し、Date型で渡していたbirthをString型で渡したら無事INSERT出来ました。

ご指摘ただいた通り、今後のためにも今一度Try〜CatchやpreparedStatementの理解を深められる様に勉強し直します。

ご丁寧なご回答を頂きどうも有難うございました！
また何かありましたらよろしくお願い致します！

[dodox86]

解決でき、お役に立ててよかったです。引き続きがんばってください。