SpringBootでバリテーションでのエラーメッセージが表示できない

Question

やりたい事

現在入力フォームを空で入力した際にmessage.propertiesに記述したエラーメッセージを出力するプログラムを書いています。

問題

Entityクラスのフィールド変数の中で、@NotNull private Date register;のエラーメッセージが意図しないメッセージが出ます。
解決策として、@DateTimeFormat(pattern="yyyy/MM/dd")も付与したところ意図したエラーメッセージが出たので解決したと思いきや、入力して登録した時にエラーが出ます。
入力フォームの値はyyyy/MM/ddです。
@DateTimeFormat(pattern="yyyy/MM/dd")を付与してない時は登録が○
@DateTimeFormat(pattern="yyyy/MM/dd")を付与した場合登録が×
@DateTimeFormat(pattern="yyyy/MM/dd")を付与してない時の空入力時は意図しないエラーメッセージ
@DateTimeFormat(pattern="yyyy/MM/dd")を付与している時の空入力時は意図したエラーメッセージ
という状況です。

@DateTimeFormat(pattern="yyyy/MM/dd")を付与して登録した時のエラー

Failed to convert property value of type java.lang.String to required type java.sql.Date for property register; nested exception is org.springframework.core.convert.ConversionFailedException: Failed to convert from type [java.lang.String] to type [@javax.validation.constraints.NotNull @org.springframework.format.annotation.DateTimeFormat java.sql.Date] for value 2019-09-01; nested exception is java.lang.IllegalArgumentException: Parse attempt failed for value [2019-09-01]

@DateTimeFormat(pattern="yyyy/MM/dd")を付与しないで空入力登録した時のエラー

Failed to convert property value of type java.lang.String to required type java.sql.Date for property register; nested exception is org.springframework.core.convert.ConversionFailedException: Failed to convert from type [java.lang.String] to type [@javax.validation.constraints.NotNull java.sql.Date] for value '; nested exception is java.lang.IllegalArgumentException

解決方法をご教授ください・・・。

大まかに流れとしては、
❶Controllerの@GetMapping("new")newBookメソッドからnew.html（入力フォーム)へ。
❷入力もしくは空のまま作成ボタンを押し、Controllerの@PostMapping createメソッドで入力チェックし、処理を分ける。
です。
以下コードになります。

Controller

@Controller
@RequestMapping("/books")
public class BookController {
	@Autowired
	private BookService bookService;
	
	@GetMapping
	public String index(Model model) {
		List<Book>books = bookService.findAll();
		model.addAttribute("books", books);
		return "books/index";
	}
    private Map<String, String> selectCategory;
    public Map<String, String> getSelectCategory(){
    	Map<String, String> category = new LinkedHashMap<String, String>();
    	category.put("A", "教育・自己啓発");
    	category.put("B", "小説");
    	category.put("C", "コミックス");
    	category.put("D", "ビジネス・経済");
    	category.put("E", "暮らし・健康・料理");    	
    	category.put("F", "ライトノベル");
		return category;
    }
    
	@GetMapping("new")
	public String newBook(@ModelAttribute Book book,Model model) {
		selectCategory = getSelectCategory();
		model.addAttribute("selectCategory", selectCategory);
		return "books/new";
	}
	
	@GetMapping("{id}/edit")
	public String edit(@PathVariable Long id, Model model) {
		Book book = bookService.findOne(id);
		model.addAttribute("book", book);
		return "books/edit";
	}
	
    @GetMapping("{id}")
    public String show(@PathVariable Long id, Model model) {
        Book book = bookService.findOne(id);
        model.addAttribute("book", book);
        return "books/show";
    }

    @PostMapping
    public String create(@ModelAttribute @Validated Book book, BindingResult bindingResult, Model model) {
    	if(bindingResult.hasErrors()) {
			return newBook(book,model);
		}
    	bookService.save(book);
    	return "redirect:/books";
    	
    }
    
    @PutMapping("{id}")
    public String update(@PathVariable Long id, @ModelAttribute Book book) {
        book.setId(id);
        bookService.save(book);
        return "redirect:/books";
    }

    @DeleteMapping("{id}")
    public String destroy(@PathVariable Long id) {
        bookService.delete(id);
        return "redirect:/books";
    }
}

Entity

@Getter
@Setter
@Entity
public class Book {
	
	@Id
	@GeneratedValue(strategy = GenerationType.IDENTITY)
	private Long id;
	@NotBlank
	private String title;
	@NotBlank
	private String person;
	@NotNull
	private Integer page;
	@NotNull
	private String selectCategory;
	@NotNull
	private Date register;
	
	@Override
    public String toString() {
        return "Player [id=" + id + ", title=" + title + ", pseron=" + person + ", page=" + page + ", selectCategory=" + selectCategory + "]";
    }
}

massages.properties

#タイトル
book.title=タイトル
NotBlank.book.title={0}を入力してください
#著者
book.person=著者
NotBlank.book.person={0}を入力してください
#ページ数
book.page=ページ数
NotNull.book.page={0}を入力してください
#カテゴリ
book.selectCategory=カテゴリ
NotNull.book.selectCategory={0}を選択してください
#登録日
book.register=登録日
NotNull.book.register={0}を記入してください

new.html

<!DOCTYPE html>
<html xmlns:th="http://www.thymeleaf.org">
<head>
<meta charset="utf-8" />
<title>New title - Book</title>
<link rel="stylesheet" href="/css/bootstrap.css" />
<script src="/js/jquery.js"></script>
<script src="/js/bootstrap.js"></script>
</head>
<body>
	<div class="container">
		<h1>New Book</h1>
		<form th:action="@{/books}" th:method="post" th:object="${book}">
			<!-- エラー用のCSS -->
			<div class="form-group"
				th:classappend="${#fields.hasErrors('title')}?'has-error'">
				<label class="control-label">タイトル</label> 
				<input class="form-control" type="text" name="title" /> <span
					class="text-danger" th:if="${#fields.hasErrors('title')}"
					th:errors="*{title}">error</span>
			</div>
			<!-- エラー用のCSS -->
			<div class="form-group"
				th:classappend="${#fields.hasErrors('person')}?'has-error'">
				<label class="control-label">著者</label> 
				<input class="form-control"	type="text" name="person" /> <span class="text-danger"
					th:if="${#fields.hasErrors('person')}" th:errors="*{person}">error</span>
			</div>
			<div class="form-group"
				th:classappend="${#fields.hasErrors('page')}?'has-error'">
				<label class="control-label">ページ数</label>
				<input class="form-control" type="number" name="page" /> <span
					class="text-danger" th:if="${#fields.hasErrors('page')}"
					th:errors="*{page}">error</span>
			</div>
			<!-- プルダウン -->
			<div class="form-group"
				th:classappend="${#fields.hasErrors('selectCategory')}?'has-error'">
				<select class="form-control" id="singleSelect" name="selectCategory">
					<option value="">カテゴリ</option>
					<option th:each="category : ${selectCategory}"
						th:value="${category.value}" th:text="${category.value}"
						th:selected="${category.value} == *{selectCategory}">singleSelect</option>
				</select> <span class="text-danger"
					th:if="${#fields.hasErrors('selectCategory')}"
					th:errors="*{selectCategory}">error</span>
			</div>
			<div class="form-group"
				th:classappend="${#fields.hasErrors('register')}?'has-error'">
				<label class="form-control-label">登録日</label> 
				<input class="form-control" type="date" name="register" /> <span
					class="text-danger" th:if="${#fields.hasErrors('register')}"
					th:errors="*{register}">error</span>
			</div>
			<button class="btn btn-default" type="submit">作成</button>
		</form>
		<div class="pull-right">
			<a class="btn btn-link" href="/books">一覧画面へ</a>
		</div>
	</div>
</body>
</html>

Answer 1

入力フォームの値はyyyy/MM/ddです。

とのことですが

<input class="form-control" type="date" name="register" />
htmlからControllerに値を渡した時点でコレの中身をコンソール出力してみてください。
yyyy-MM-dd形式になっていませんか？

Comments

[退会済みユーザー]

ご回答ありがとうございます。
コンソールで確認したところ、おっしゃる通りyyyy-MM-ddになっていました。
そのため、@DateTimeFormat(pattern="yyyy-MM-dd")に変更して試そうと思いましたが、htmlの方でtype="date"にしているためyyyy-MM-dd形式で入力ができない状態です
解決策としては、yyyy/MM/ddで受け取るようにする。
もしくはtype="text"に変更し、Entityのregisterの型をStringに変更し、カラムの型の変更・・・とまでしないといけないのでしょうか？そうなるとDate型で扱う必要がなくなってしまうので避けたいところですが、ご教授いただきたいです。

[tetutetu]

全体像が把握できていないので的確な助言は致しかねますが
Validate用のクラスを自前で作成して、[-]→[/]への置換を行った後にValidate処理等の実装は難しいでしょうか。

[退会済みユーザー]

いえ、とても助かります。一度試してみます。
あと初歩的な質問になってしまいますが、Dateのimportをimport java.sql.Date;に無意識にしていましたが、このような使い方をする時はimport java.util.Date;が適切なのでしょうか？

[退会済みユーザー]

今、根本的な改善には繋がっていないですが、
@NotNullと@DateTimeFormat(pattern="yyyy-MM-dd")を消し、
messages.propertiesにtypeMismatch.java.sql.Date={0}エラーメッセージを追記したところ、意図している動作は実現できました。
どうやらnullを型のミスマッチとして受け取っていたようなのでこのような改善策をとってみました。

[q_sane_q]

横から失礼します。
見当違いの話であれば申し訳ございません
> @DateTimeFormat(pattern="yyyy-MM-dd")に変更して試そうと思いましたが、htmlの方でtype="date"にしているためyyyy-MM-dd形式で入力ができない状態
とありますが、サーバにわたっている値はyyyy-MM-ddなので、@DateTimeFormat(pattern="yyyy-MM-dd")　で判定してもいいと思うのですが、どうなのでしょうか？

[tetutetu]

＞このような使い方をする時はimport java.util.Date;が適切なのでしょうか？
前述の通り全体像が把握できていないので
・時分秒を扱わない
・Date date = new Date()で現在時刻を取得しようとしない
のであれば、このままでも良いのでは？程度の認識です。
（その他いくつかのメソッドに違いがあったはずなので、このままの利用で問題がないかどうかは一度調べてみたほうが良いかと思われます。）
@DateTimeFormatについては、q_sane_q様のコメントの方が適切かなと思います。

[退会済みユーザー]

ありがとうございます。
私もそう思い試みてみましたが、なぜかタイプミスマッチエラーメッセージが表示されてしまいました。。。

[退会済みユーザー]

tetutetu様　ありがとうございます！