質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

89.13%

【springboot】バリデーションがエラーを検知してくれない

受付中

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 1,290

mitsuki_

score 20

前提・実現したいこと

spring初心者です。
エンティティクラスにバリデーションを作成し、ブラウザで入力された値のバリデーションチェックを行い、エラーを赤文字で表示するという学習内容です。

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

バリデータが作動(?)しません。
例えば、名前の入力項目は@MptEmptyをつけているのですが、空欄のままフォームを送信すると下記ホワイトラベルが表示されます。

Whitelabel Error Page
This application has no explicit mapping for /error, so you are seeing this as a fallback.

Thu Feb 07 17:26:48 JST 2019
There was an unexpected error (type=Bad Request, status=400).
Validation failed for object='formModel'. Error count: 1

  

該当のソースコード

1.controller
※エンティティのCRUDの学習を進めていたため、delete系などのメソッドが残ったままです。

package com.tuyano.springboot;

import java.util.Optional;

import javax.annotation.PostConstruct;

import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.stereotype.Controller;
import org.springframework.transaction.annotation.Transactional;
import org.springframework.validation.BindingResult;
import org.springframework.validation.annotation.Validated;
import org.springframework.web.bind.annotation.ModelAttribute;
import org.springframework.web.bind.annotation.PathVariable;
import org.springframework.web.bind.annotation.RequestMapping;
import org.springframework.web.bind.annotation.RequestMethod;
import org.springframework.web.bind.annotation.RequestParam;
import org.springframework.web.servlet.ModelAndView;

import com.tuyano.springboot.repositories.MyDataRepository;

@Controller
public class HeloController {
    @Autowired
    MyDataRepository repository;

    @RequestMapping(value = "/", method = RequestMethod.GET)
    public ModelAndView index(@ModelAttribute("formModel") MyData mydata, ModelAndView mav) {
        mav.setViewName("index");
        mav.addObject("msg", "this is sample content.");
        mav.addObject("formModel", mydata);
        Iterable<MyData> list = repository.findAll();
        mav.addObject("datalist", list);
        return mav;
    }

    @RequestMapping(value = "/", method = RequestMethod.POST)
    @Transactional(readOnly = false)
    public ModelAndView form(@ModelAttribute("formModel") @Validated MyData mydata, ModelAndView mov,
            BindingResult result) {
        ModelAndView res = null;
        if (!result.hasErrors()) {
            repository.saveAndFlush(mydata);
            res = new ModelAndView("redirect:/");
        } else {
            mov.setViewName("index");
            mov.addObject("msg", "sorry.error is occured...");
            Iterable<MyData> list = repository.findAll();
            mov.addObject("datalist", list);
            res = mov;
        }

        return res;
    }

    @RequestMapping(value = "/edit/{id}", method = RequestMethod.GET)
    public ModelAndView edit(@ModelAttribute MyData mydata, @PathVariable int id, ModelAndView mav) {
        mav.setViewName("edit");
        mav.addObject("title", "edit mydata.");
        Optional<MyData> data = repository.findById((long) id);
        mav.addObject("formModel", data.get());
        return mav;
    }

    @RequestMapping(value = "/edit", method = RequestMethod.POST)
    @Transactional(readOnly = false)
    public ModelAndView update(@ModelAttribute MyData mydata, ModelAndView mav) {
        repository.saveAndFlush(mydata);
        return new ModelAndView("redirect:/");
    }

    @RequestMapping(value = "/delete/{id}", method = RequestMethod.GET)
    public ModelAndView delete(@PathVariable int id, ModelAndView mav) {
        mav.setViewName("delete");
        mav.addObject("title", "delete my data");
        Optional<MyData> data = repository.findById((long) id);
        mav.addObject("formModel", data.get());
        return mav;
    }

    @RequestMapping(value = "/delete", method = RequestMethod.POST)
    @Transactional(readOnly = false)
    public ModelAndView remove(@RequestParam long id, ModelAndView mav) {
        repository.deleteById(id);
        return new ModelAndView("redirect:/");
    }

    @PostConstruct
    public void init() {
        //1つめのダミーデータ
        MyData d1 = new MyData();
        d1.setName("tuyano");
        d1.setAge(123);
        d1.setMail("tuyano@gmail");
        d1.setMemo("this is my data!");
        repository.saveAndFlush(d1);
        //2つめのダミーデータ
        MyData d2 = new MyData();
        d2.setName("hanako");
        d2.setAge(45);
        d2.setMail("hanako@gmail");
        d2.setMemo("this is my flower!");
        repository.saveAndFlush(d2);
        //3つめのダミーデータ
        MyData d3 = new MyData();
        d3.setName("sachiko");
        d3.setAge(67);
        d3.setMail("sachiko@gmail");
        d3.setMemo("this is my friend!");
        repository.saveAndFlush(d3);

    }
}


2.entity

package com.tuyano.springboot;

import javax.persistence.Column;
import javax.persistence.Entity;
import javax.persistence.GeneratedValue;
import javax.persistence.GenerationType;
import javax.persistence.Id;
import javax.persistence.Table;
import javax.validation.constraints.Email;
import javax.validation.constraints.Max;
import javax.validation.constraints.Min;
import javax.validation.constraints.NotEmpty;
import javax.validation.constraints.NotNull;

@Entity
@Table(name = "mydata")
public class MyData {

    @Id
    @GeneratedValue(strategy = GenerationType.AUTO)
    @Column
    @NotNull
    private long id;

    @Column(length = 50, nullable = false)
    @NotEmpty
    private String name;

    @Column(length = 200, nullable = true)
    @Email
    private String mail;

    @Column(nullable = true)
    @Min(0)
    @Max(200)
    private Integer age;

    @Column(nullable = true)
    private String memo;

    public long getId() {
        return id;
    }

    public void setId(long id) {
        this.id = id;
    }

    public String getName() {
        return name;
    }

    public void setName(String name) {
        this.name = name;
    }

    public String getMail() {
        return mail;
    }

    public void setMail(String mail) {
        this.mail = mail;
    }

    public Integer getAge() {
        return age;
    }

    public void setAge(Integer age) {
        this.age = age;
    }

    public String getMemo() {
        return memo;
    }

    public void setMemo(String memo) {
        this.memo = memo;
    }
}


3.thymeleaf

<!DOCTYPE html>
<html xmlns:th="http://www.thymeleaf.org">
<head>
<title>top page</title>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
<style>
h1 {
    font-size: 18pt;
    font-weight: bold;
    color: gray;
}

body {
    font-size: 13pt;
    color: gray;
    margin: 5px 25px;
}

tr {
    margin: 5px;
}

th {
    padding: 5px;
    color: white;
    background: darkgray;
}

td {
    padding: 5px;
    color: black;
    background: #f0f0f0;
}

.err {
    color: red;
}
</style>
</head>
<body>
    <h1 th:text="#{content.title}">helo page</h1>
    <p th:text="${msg}"></p>
    <table>
        <form method="post" action="/" th:object="${formModel}">
            <ul>
                <li th:each="error : ${#fields.detailedErrors()}" class="err"
                    th:text="${error.message}" />
            </ul>
            <tr>
                <td><label for="name">名前</label></td>
                <td><input type="text" name="name" th:field="*{name}" /></td>
            </tr>
            <tr>
                <td><label for="age">年齢</label></td>
                <td><input type="text" name="age" th:field="*{age}" /></td>
            </tr>
            <tr>
                <td><label for="mail">メール</label></td>
                <td><input type="text" name="mail" th:field="*{mail}" /></td>
            </tr>
            <tr>
                <td><label for="memo">メモ</label></td>
                <td><textarea name="memo" th:field="*{memo}" cols="20" rows="5"></textarea></td>
            </tr>
            <tr>
                <td></td>
                <td><input type="submit"></td>
            </tr>
        </form>
    </table>
    <hr />
    <table>
        <tr>
            <th>ID</th>
            <th>名前</th>
        </tr>
        <tr th:each="obj : ${datalist}">
            <td th:text="${obj.id}"></td>
            <td th:text="${obj.name}"></td>
        </tr>
    </table>
</body>
</html>

補足情報

eclipse 4.8.0
springboot 2.1.1

おそれいりますが、どなたかご教示のほどよろしくお願いいたします。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

0

プリミティブ型に@NotNullを付けても正しく動作しません。

例えばこのようなラッパー側で扱うと、

    @Data
    public static class AgeModel {
        @NotNull
        @Max(value=30)
        @Min(value=20)
        private Integer age;
    }


未入力の時は NotNull が、数値の範囲外なら、MaxやMinに定義されたエラーメッセージが出力されます。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 89.13%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る