編集画面でパスワード未入力の場合はバリデーションエラーが出ないようにしたい

Question

Spring Bootで編集画面を作成しています。
編集画面において、パスワードが未入力の場合はそのまま値が保持される（SQL上で値が変更されない）ようにしたく、Controllerクラスに以下のようにコードを記述したのですが、依然として「6～20文字で入力してください」というエラーが出てしまいます。

// 方法１
@Controller
public class UsersController {
	@PostMapping("{id}")
	public String edit(@PathVariable("id") Long id, @Valid @ModelAttribute("editForm") EditForm editForm, BindingResult result, Model model) {

（中略）

		// パスワードが未入力の場合
		if (editForm.getPassword().equals("") || editForm.getPassword() == null) {
			if (!editForm.getPassword().matches("^(?=.*[.?/-])[a-zA-Z0-9-/:-@\[-\`\{-\~]{6,20}$")) {
				model.addAttribute("error", "変更する場合、パスワードは6～20文字の記号を含む半角英数字で入力してください");
				editForm.setPassword(editForm.getPassword());
				return "users/edit";
			}
		}
		// バリデーションチェック
		if (result.hasErrors()) {
			return "users/edit";
		}
		editService.update(editForm);
		return "redirect:/users";
	}

@Service
public class EditService {

	@Autowired
	private UserMapper mapper;
	@Transactional
	public int update(EditForm editForm) {
		User entity = new User();

（中略）

		BCryptPasswordEncoder encoder = new BCryptPasswordEncoder();
		entity.setPassword(encoder.encode(editForm.getPassword()));

（中略）

		return mapper.update(entity);
	}

そこで下記の動画を参考に、serviceクラスに以下のようにコードを記述し、またcontrollerにてInitBinderで未入力がnullとして返ってくるようにしたのですが、それだとパスワードのみ未入力で更新した場合、更新自体はうまくいってもSQL上でパスワードが「null」となってしまいます。

【参考動画 1:42:32～】
https://www.youtube.com/watch?v=n6s_24Ph944&t=6316s

// 方法２
@Service
public class EditService {
		// パスワードが未入力の場合
		if (form.getPassword() == null && form.getConfirmPass() == null) {
			BCryptPasswordEncoder encoder = new BCryptPasswordEncoder();
			entity.setPassword(encoder.encode(form.getPassword()));
		}
}

@InitBinder
	public void initBinder(WebDataBinder binder) {
		binder.registerCustomEditor(String.class, new StringTrimmerEditor(true));
	}

どちらの方法をとるにしても、上記のほかに記述しないといけないコードがあるのでしょうか？
どのようにすればいいのかご教示いただけますと幸いです。

Answer 1

パスワードが未入力の場合に以下のどちらかの機能と実装すればよいと思います。

• DBに保存されたパスワードを使って更新する
• パスワードの更新をしない MyBatis の定義を書く

DBに保存されたパスワードを使う場合

String password = form.getPassword();
if (Objects.isNull(password) && Objects.isNull(form.getConfirmPass())) {
  User user = mapper.findById(id);  // id でUserが取得できるものとして
  entity.setPassword(user.getPassword());  
} else {
  BCryptPasswordEncoder encoder = new BCryptPasswordEncoder();
  entity.setPassword(encoder.encode(password));
}

MyBatis の定義を書く場合

<update id="updateUserWithoutPassword">
  UPDATE
    User
  SET
    name = #{name},
    // あと何か更新したものがあれば
  where id = #{id} 
</update>

User entity = new User();
user.setId(id);
user.setName(form.getName());

String password = form.getPassword();
if (Objects.isNull(password) && Objects.isNull(form.getConfirmPass())) {
  mapper.updateUserWithoutPassword(entity)
} else {
  BCryptPasswordEncoder encoder = new BCryptPasswordEncoder();
  entity.setPassword(encoder.encode(password));
  mapper.update(entity)
}

Comments

[an_yuka09]

ご丁寧に回答ありがとうございました！
こちらを参考にさせていただきます！