Zwracanie error kodu jako ResponseEntity po walidacji DTO w REST API

Question

Korzystając z poradnika https://www.leveluplunch.com/[...]ring-rest-webservice-request/ chcę dokonać walidacji formularza za pomocą java bez przeładowywania strony. Robię to w REST API.
Mój kontroler

@PutMapping(value = "/changeEmail", consumes = MediaType.APPLICATION_JSON_VALUE)
    public HttpEntity<ChangeEmailDTO> showChangeEMail(
            @RequestBody @Valid ChangeEmailDTO changeEmailDTO,
            BindingResult result
    ) {
        System.out.println("Email: " + changeEmailDTO.getEmail());
 
        if(result.hasErrors()) {
            System.out.println("Error: " + changeEmailDTO.getEmail());
            return ResponseEntity.badRequest().body(changeEmailDTO);
        }
 
        System.out.println("Success: " + changeEmailDTO.getEmail());
        return ResponseEntity.ok(changeEmailDTO);
    }

przyjmuje adres email, waliduje w adnotacjach

 @NotEmpty
    @Getter @Setter private String email;

Według podanego wyżej poradnika, podobno powinno zwrócić error code w postaci json. Natomiast jedyny efekt jaki otrzymuje to https://zapodaj.net/81a6db4635168.png.html
Czyli zwraca mi sam obiekt, a nie tak jak np. w poradniku

{"timestamp":1417379464584,"status":400,"error":"Bad Request","exception":"org.springframework.web.bind.MethodArgumentNotValidException","message":"Validation failed for argument at index 0 in method: public org.springframework.http.ResponseEntity<demo.AgencyResource> demo.AgencyController.saveAgency(demo.AgencyResource), with 2 error(s): [Field error in object 'agencyResource' on field 'name': rejected value [null]; codes [NotNull.agencyResource.name,NotNull.name,NotNull.java.lang.String,NotNull]; arguments [org.springframework.context.support.DefaultMessageSourceResolvable: codes [agencyResource.name,name]; arguments []; default message [name]]; default message [may not be null]] [Field error in object 'agencyResource' on field 'id': rejected value [50]; codes [Max.agencyResource.id,Max.id,Max.java.lang.Integer,Max]; arguments [org.springframework.context.support.DefaultMessageSourceResolvable: codes [agencyResource.id,id]; arguments []; default message [id],20]; default message [must be less than or equal to 20]] ","path":"/agencies"}

W jaki sposób zwrócić takie dane?

Comments

Samo throw httpexception nie wystarczy?

---

Nie udało mi się zwrócić takich danych ja w poradniku, ale bardzo podobny.

Zmieniłem kontroler:

@PutMapping(value = "/changeEmail", consumes = MediaType.APPLICATION_JSON_VALUE)
    public HttpEntity<ChangeEmailDTO> showChangeEMail(
            @RequestBody @Valid ChangeEmailDTO changeEmailDTO
    ) {
        System.out.println("Email: " + changeEmailDTO.getEmail());

        return ResponseEntity.ok(changeEmailDTO);
    }

i stworzyłem exception handler dla MethodArgumentNotValidException

@ExceptionHandler(MethodArgumentNotValidException.class)
    @ResponseStatus(HttpStatus.BAD_REQUEST)
    @ResponseBody
    public ValidationErrorDTO processValidationError(MethodArgumentNotValidException ex) {
        BindingResult result = ex.getBindingResult();
        List<FieldError> fieldErrors = result.getFieldErrors();

        return processFieldErrors(fieldErrors);
    }

    private ValidationErrorDTO processFieldErrors(List<FieldError> fieldErrors) {
        ValidationErrorDTO dto = new ValidationErrorDTO();

        for (FieldError fieldError: fieldErrors) {
            String localizedErrorMessage = resolveLocalizedErrorMessage(fieldError);
            dto.addFieldError(fieldError.getField(), localizedErrorMessage);
        }

        return dto;
    }

    private String resolveLocalizedErrorMessage(FieldError fieldError) {
        Locale currentLocale =  LocaleContextHolder.getLocale();
        String localizedErrorMessage = messageSource.getMessage(fieldError, currentLocale);

        //If the message was not found, return the most accurate field error code instead.
        //You can remove this check if you prefer to get the default error message.
        if (localizedErrorMessage.equals(fieldError.getDefaultMessage())) {
            String[] fieldErrorCodes = fieldError.getCodes();
            localizedErrorMessage = fieldErrorCodes[0];
        }

        return localizedErrorMessage;
    }

dzięki temu zwraca mi ładną listę błędów https://zapodaj.net/7e359361ed05a.png.html

Tylko jest mały szczególik, że podczas gdy ten wyjątek ma być obsłużony przez ExceptionHandler to wyrzuca mi go również w konsoli

org.springframework.web.bind.MethodArgumentNotValidException: Validation failed for argument at index 0 in method: public org.springframework.http.HttpEntity<com.movie.database.app.dto.ChangeEmailDTO> com.movie.database.app.rest.controller.UserController.showChangeEMail(com.movie.database.app.dto.ChangeEmailDTO), with 2 error(s): [Field error in object 'changeEmailDTO' on field 'email': rejected value []; codes [IsValidEmail.changeEmailDTO.email,IsValidEmail.email,IsValidEmail.java.lang.String,IsValidEmail]; arguments [org.springframework.context.support.DefaultMessageSourceResolvable: codes [changeEmailDTO.email,email]; arguments []; default message [email]]; default message [Please enter a valid e-mail address.]] [Field error in object 'changeEmailDTO' on field 'email': rejected value []; codes [NotEmpty.changeEmailDTO.email,NotEmpty.email,NotEmpty.java.lang.String,NotEmpty]; arguments [org.springframework.context.support.DefaultMessageSourceResolvable: codes [changeEmailDTO.email,email]; arguments []; default message [email]]; default message [may not be empty]] 

Jak go obsłużyć, aby nie wyświetlał się w konsoli.

---

Chciałbym zauważyć, że samo podanie w endpoincie "/changeEmail" deklasyfikuje to API jako REST.

Answer 1

BindingResult result - bezsensu w to iść.

@Adnotacje na encjach

Zrób sobie jakiegoś checkState(T t) - i rzucaj wyjątkiem, prościej, ładniej :)

 

ps żeby ten checkState ładniej wyglądał, niech przyjmuję jakieś wyrażenie lambda.