DB更新処理の実装方式を教えてください。

Question

前提・実現したいこと

閲覧いただき、ありがとうございます。
Eclipse,Springを使ってJAVAの勉強をしている（超）初心者です。
現在、マスタメンテナンス画面の更新処理を作成中ですが、更新ボタン
押下時に下記のようなエラーが発生してしまいます。
RecordManager.javaのjdbcTemplate.update命令がうまくいっていないのと、
そのjdbcTemplate.updateの戻り値(int)をAddressBookController.java側で
Employee型で受けようとしているからうまくいかないのかなと思っている
のですが、ではどのように実装すべきなのかが検討つかない状況です。
ご教示いただけないでしょうか。

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

nested exception is org.springframework.dao.TransientDataAccessResourceException: PreparedStatementCallback; SQL [update unit02.addressbook set name = ? , tel = ?, address = ? where id = ? ]; Invalid argument value: java.io.NotSerializableException; nested exception is java.sql.SQLException: Invalid argument value: java.io.NotSerializableException

該当のソースコード

AddressBookController.java

	@RequestMapping(params = "reflection")
	public String reflection(AddressBookForm form, Model model) {
		recordManager.updateEmployee(form.getId(), form.getName(), form.getPhone(), form.getAddress());
		if (employee != null) {
			form.setName(employee.getName());
			form.setPhone(employee.getPhone());
			form.setAddress(employee.getAddress());
			model.addAttribute("message", UPDMSG2);
			return INIT;
		} else {
			model.addAttribute("message", ERRMSG2);
			return INIT;
		}
	}

RecordManager.java

	public int updateEmployee(String id,String name, String phone, String address){
		System.out.println("para1=" + id + name + phone + address);
		try{
			return jdbcTemplate.update("update unit02.addressbook set name = ? , tel = ?, address = ? where id = ? ",
				new Object[]{ name , phone , address, id },
				new RowMapper<Employee>() {
				@Override
				public Employee mapRow(ResultSet rs, int rowNum) throws SQLException {
					System.out.println("para2=" + rs  + rowNum);
					Employee employee = new Employee("","","","");
					employee.setId(rs.getString("id"));
					employee.setName(rs.getString("name"));
					employee.setPhone(rs.getString("tel"));
					employee.setAddress(rs.getString("address"));
					return employee;
				}
			});
		} catch (IncorrectResultSizeDataAccessException e) {
			System.out.println("data incorrect ERR");
			return 0;
		}
	}

試したこと

RecordManager.javaの
System.out.println("para1=" + id + name + phone + address);
は正常値が表示されています。

Answer 1

spring実は使ったことないので、的外れでしたらゴメンなさい。

えっと、やりたいことと使っているメソッドはあっているでしょうか？

https://docs.spring.io/spring/docs/current/javadoc-api/

https://docs.spring.io/spring/docs/current/javadoc-api/org/springframework/jdbc/core/JdbcTemplate.html

updateの引数で、RowMapperを使うのでしょうか？

単純に更新値だけ渡せばよいのではないでしょうか。

return jdbcTemplate.update("update unit02.addressbook set name = ? , tel = ?, address = ? where id = ? ",
                new Object[]{ name , phone , address, id });

Comments

[usako]

momon-ga様、コメントありがとうございます。
ご教授いただいたように、シンプルにしたらうまく動作するようになりました！

Answer 2

Invalid argument value

無効な引数値　とのことで、引数がおかしいとのことですね。

マニュアル見ますと

public int update(String sql,

              @Nullable
              Object... args)
       throws DataAccessException

んー合ってそうに見えますが・・

解決するか分かりませんが、ArrayListにしてみては？

あとidなど数値型は文字列から数値にパースしたほうが良さそうです。

ArrayList<Object> param = new ArrayList<>();
param.add(name);
param.add(phone);
param.add(address);
param.add(Integer.parseInt(id));

return jdbcTemplate.update("update unit02.addressbook set name = ? , tel = ?, address = ? where id = ? ",
    param ,

Comments

[asahina1979]

質問者の第3引数はRowMapだから

update(String sql, Object... args) が使われてる気がするんだが

[m.ts10806]

あ、確かに
でしたらつぎにかいてるstringで渡してるidが問題ですかね。

[usako]

mts10806様、asahina1979様、コメントありがとうございます。
ArrayListも試してみたのですが、私の理解（スキル）が及ばずうまくいきませんでした・・・

[m.ts10806]

回答は2つ案を書いています。2つ目（idをパースする）をお試しください。

[usako]

mts10806様、コメントありがとうございます。
タイミングが悪くてBAを決めてしまいました。こちらの案も試してご報告させていただきます。

[m.ts10806]

MySQLでしたらあまり関係ないかもしれませんが(PostgreSQLなどには必要)、型は合わせておくとトラブルは少ないと思います