Как сделать DTO с валидацией данных в Java Spring?

Question

[MockingAlchemist]

В общем, я руководствовался этой статьей - https://habr.com/ru/post/343960/ и еще несколькими другими, но валидация так и не срабатывает.

Вот что я сделал:
1. Контроллер

import ideaweb.sunny.web.DTO.ProxyDto;
import org.springframework.http.MediaType;
import org.springframework.validation.Errors;
import org.springframework.web.bind.annotation.*;

import javax.validation.Valid;

@RestController
public class MainController
{
    @PostMapping("/proxy/add")
    public void saveProxy(@Valid @RequestBody ProxyDto dto) {

        System.out.println(dto.toString());

    }
}

2. DTO:

import javax.validation.constraints.NotEmpty;
import javax.validation.constraints.NotNull;


public class ProxyDto {

    @NotNull(message = "123")
    @NotEmpty(message = "456")
    private String proxy;
    
}

Но если я посылаю запрос (json), в котором поле proxy отсутствует, то получаю ответ 200, и в консоль печатаются эти объекты как ни в чём не бывало.
Пожалуйста, подскажите, где я ошибаюсь.

Запрос посылаю следующим образом:

curl -v -X POST localhost:8081/proxy/add -H "Content-type:application
/json" -d "{\"proxy\":\"127.0.0.1\"}"

и ожидаю увидеть статус 400 в ответ на это:

curl -v -X POST localhost:8081/proxy/add -H "Content-type:application
/json" -d "{\"somethingElse\":\"127.0.0.1\"}"

Comments

[Сергей Горностаев]

Boot используете или контекст конфигурируете руками?

[MockingAlchemist]

Сергей Горностаев, насколько я понимаю, должен использоваться boot (он есть в зависимостях и в parent указан org.springframework.boot.spring-boot-starter-parent)
(это практически первый опыт использования spring, поэтому не могу с уверенностью ответить на ваш вопрос)

[Сергей Горностаев]

MockingAlchemist, тогда покажите конфигурацию сборщика.

P.S. Я бы рекомендовал начать с учебника, чтобы понимать как работает фреймворк, а не бродить в потёмках, после статьи, написанной джуниором по мотивам его же вопросов на Stackoverflow.

[MockingAlchemist]

Сергей Горностаев,

<build>
		<plugins>
			<plugin>
				<groupId>org.apache.maven.plugins</groupId>
				<artifactId>maven-compiler-plugin</artifactId>
				<version>3.8.0</version>
				<configuration>
					<source>1.8</source>
					<target>1.8</target>
					<verbose>true</verbose>
				</configuration>
			</plugin>
			<plugin>
				<groupId>org.springframework.boot</groupId>
				<artifactId>spring-boot-maven-plugin</artifactId>
			</plugin>
		</plugins>
	</build>

[Орхан Гасанлы]

MockingAlchemist, а что в зависимостях? Используется ли какой-нибудь стартер?

[MockingAlchemist]

Орхан Гасанлы, да - web, data-jpa и тесты:

<dependency>
			<groupId>org.springframework.boot</groupId>
			<artifactId>spring-boot-starter-web</artifactId>
		</dependency>
		<dependency>
			<groupId>org.springframework.boot</groupId>
			<artifactId>spring-boot-starter-data-jpa</artifactId>
		</dependency>
		<dependency>
			<groupId>org.springframework.boot</groupId>
			<artifactId>spring-boot-starter-test</artifactId>
			<scope>test</scope>
			<exclusions>
				<exclusion>
					<groupId>org.junit.vintage</groupId>
					<artifactId>junit-vintage-engine</artifactId>
				</exclusion>
			</exclusions>
		</dependency>

[Сергей Горностаев]

MockingAlchemist, целиком pom.xml покажите.

[MockingAlchemist]

<?xml version="1.0" encoding="UTF-8"?>
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
	xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd">
	<modelVersion>4.0.0</modelVersion>
	<parent>
		<groupId>org.springframework.boot</groupId>
		<artifactId>spring-boot-starter-parent</artifactId>
		<version>2.4.2</version>
		<relativePath/> <!-- lookup parent from repository -->
	</parent>
	<groupId>ideaweb.sunny</groupId>
	<artifactId>web</artifactId>
	<version>0.0.1-SNAPSHOT</version>
	<name>web</name>
	<description>Demo project for Spring Boot</description>
	<properties>
		<java.version>11</java.version>
	</properties>
	<dependencies>

		<dependency>
			<groupId>org.springframework.boot</groupId>
			<artifactId>spring-boot-starter-web</artifactId>
		</dependency>
		<dependency>
			<groupId>org.springframework.boot</groupId>
			<artifactId>spring-boot-starter-data-jpa</artifactId>
		</dependency>
		<dependency>
			<groupId>org.springframework.boot</groupId>
			<artifactId>spring-boot-starter-test</artifactId>
			<scope>test</scope>
			<exclusions>
				<exclusion>
					<groupId>org.junit.vintage</groupId>
					<artifactId>junit-vintage-engine</artifactId>
				</exclusion>
			</exclusions>
		</dependency>

		<dependency>
			<groupId>org.postgresql</groupId>
			<artifactId>postgresql</artifactId>
			<version>42.2.14</version>
		</dependency>

		<dependency>
			<groupId>org.springframework.boot</groupId>
			<artifactId>spring-boot-devtools</artifactId>
			<scope>runtime</scope>
			<optional>true</optional>
		</dependency>



		<dependency>
			<groupId>junit</groupId>
			<artifactId>junit</artifactId>
			<version>4.13</version>
		</dependency>

		<!-- https://mvnrepository.com/artifact/org.liquibase/liquibase-core -->
		<dependency>
			<groupId>org.liquibase</groupId>
			<artifactId>liquibase-core</artifactId>
			<version>4.2.2</version>
		</dependency>

		<!-- https://mvnrepository.com/artifact/javax.validation/validation-api -->
		<dependency>
			<groupId>javax.validation</groupId>
			<artifactId>validation-api</artifactId>
			<version>2.0.1.Final</version>
		</dependency>

		<dependency>
			<groupId>org.projectlombok</groupId>
			<artifactId>lombok</artifactId>
			<version>1.18.16</version>
			<scope>provided</scope>
		</dependency>

	</dependencies>

	<build>
		<plugins>
			<plugin>
				<groupId>org.apache.maven.plugins</groupId>
				<artifactId>maven-compiler-plugin</artifactId>
				<version>3.8.0</version>
				<configuration>
					<source>1.8</source>
					<target>1.8</target>
					<verbose>true</verbose>
				</configuration>
			</plugin>
			<plugin>
				<groupId>org.springframework.boot</groupId>
				<artifactId>spring-boot-maven-plugin</artifactId>
			</plugin>
		</plugins>
	</build>

</project>

[сергей кузьмин]

MockingAlchemist,

поменяйте на

@Validated 
import org.springframework.validation.annotation.Validated;

я отметил в своем проекте

public class Book {

	@Id
	private String id;
	@NotNull(message = "123")
	@NotEmpty(message = "456")
	private String isbn;
	

...

и вызвал

curl -XPOST http://localhost:8080/book/add -d "{}"  -H "Content-type:application/json

и будет вам

[Field error in object 'book' on field 'isbn': rejected value [null]; codes [NotNull.book.isbn,NotNull.isbn,NotNull.java.lang.String,NotNull]; arguments [org.springframework.context.support.DefaultMessageSourceResolvable: codes
 [book.isbn,isbn]; arguments []; 

default message [isbn]]; default message [123]]

как видите 123 оно сказало и еще кучу служебной информации

если помогло - отметьте решением

[MockingAlchemist]

сергей кузьмин, @Validated я уже тоже пробовал, и проблему решил подключением нужной зависимости

[сергей кузьмин]

MockingAlchemist, мне кажется правильная аннотация и есть решение

Answer 1

[Сергей Горностаев]

Похоже, что у вас в зависимостях нет библиотек валидатора, соответственно Boot валидацию не включает.

Comments

[MockingAlchemist]

для валидации вот что подключил:

<dependency>
			<groupId>javax.validation</groupId>
			<artifactId>validation-api</artifactId>
			<version>2.0.1.Final</version>
		</dependency>

[Сергей Горностаев]

MockingAlchemist, этого мало, это только API, нужна ещё реализация. Лучше spring-boot-starter-validation подключите, как советует документация.

[MockingAlchemist]

Сергей Горностаев, да, действительно, увидел. добавил, заработало. спасибо большое!

Answer 2

[Орхан Гасанлы]

Здравствуйте!
Если честно, то меня сама строка настораживает

@NotNull(message = "123")
    @NotEmpty(message = "456")
    private String proxy;

NotEmpty и так проверяет строку на NULL и на пустоту и соответственно, аннотация @NotNull не нужна.
Попробуйте удалить эту аннотацию и проверьте заново.

Comments

[MockingAlchemist]

попробовал убрать, всё то же самое - статус 200, ошибок нет

Answer 3

[сергей кузьмин]

у вас импорт неправильный был вместо

import javax.validation.Valid;
@RestController
public class MainController
{
    @PostMapping("/proxy/add")
    public void saveProxy(@Valid @RequestBody ProxyDto dto) {

надо

import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.validation.annotation.Validated;
import org.springframework.web.bind.annotation.PathVariable;
import org.springframework.web.bind.annotation.RequestBody;
@RestController
@RequestMapping("/proxy")

public class MainController
{
	@RequestMapping(value = "add", method = RequestMethod.POST)
	public Map<String, Object> addBook(@Validated @RequestBody ProxyDto dto ) {
....

и

import javax.validation.constraints.NotNull;
public class ProxyDto {

	@NotNull(message = "idbn is missing")
	@NotEmpty(message = "idbn is empty")


 private String proxy;

в зависимости от версии фрамеворка скажкт типа

{
  "timestamp": 1612029070201,
  "status": 400,
  "error": "Bad Request",
  "exception": "org.springframework.web.bind.MethodArgumentNotValidException",
  "errors": [
    {
      "codes": [
        "NotNull.book.isbn",
        "NotNull.isbn",
        "NotNull.java.lang.String",
        "NotNull"
      ],
      "arguments": [
        {
          "codes": [
            "book.isbn",
            "isbn"
          ],
          "arguments": null,
          "defaultMessage": "isbn",
          "code": "isbn"
        }
      ],
      "defaultMessage": "idbn is missing",
      "objectName": "book",
      "field": "isbn",
      "rejectedValue": null,
      "bindingFailure": false,
      "code": "NotNull"
    },
    {
      "codes": [
        "NotEmpty.book.isbn",
        "NotEmpty.isbn",
        "NotEmpty.java.lang.String",
        "NotEmpty"
      ],
      "arguments": [
        {
          "codes": [
            "book.isbn",
            "isbn"
          ],
          "arguments": null,
          "defaultMessage": "isbn",
          "code": "isbn"
        }
      ],
      "defaultMessage": "idbn is empty",
      "objectName": "book",
      "field": "isbn",
      "rejectedValue": null,
      "bindingFailure": false,
      "code": "NotEmpty"
    }
  ],
  "message": "Validation failed for object='book'. Error count: 2",
  "path": "/book/add"
}