인프런 커뮤니티 질문&답변

BeanValidation 에서 에러메세지 출력해줄 때,

아래와 같이 검증 대상 객체 뒤에 BindingResult를 추가해 주고 bindingResult로부터 새로 에러 메세지를 작성한 뒤,

응답에 넣어 주면 될까요?

-----

public ResponseEntity<SignUpResponseDTO> signUp(

@Validated @ResponseBody SignUpRequestDTO requestDTO, BindingResult bindingResult ....... )

if(bindingResult.hasError()){

// new SignUpResponseDTO 생성 후

// 에러메세지를 위에 담아 응답

}

//정상로직

정상로직 응답 =  new  SignUpResponseDTO();

return   정상로직 응답

----

그렇다면 정상로직일 경우와, 에러의 경우 응답 DTO가 같은데

정상 로직의 경우 message에 정상응답 메세지를 추가로 담아서 응답해주면 되는거겠죠?

1. 이런방식으로 밖에 안떠오르는데 혹시 다른 방법이 더 좋다거나 한지 알려 주실 수 있을까요?

2. 추가적으로 로그인에서 에러가 발생할 경우, (미인증사용자)

로그인 인터셉터 내에서 에러메세지를 출력해주고자 하면

response.getWriter()를 통해 에러메세지를 전달해주면 되겟죠?

3. 이건 마이너 한것일 수 있는데, 저는 이렇게 응답 객체 네이밍을 했는데 (SignUpResponseDTO)
영한님은 어떻게 네이밍을 하시는지 또한 궁금합니다.

------

추가로 ExceptionHandler 강의 듣고왔는데

컨트롤러에서는 exception을 터트리고, handler에서 예외를 처리하는데

여기서 RestEntity에 에러객체를 담아, json 형태로 에러 결과를 응답해줄수 있는건 알겠습니다.

그러면 위와 같은 validation에서 에러가 터질 때 ExceptionHandler를 통해 예외처리를 하고자 한다면,

입력값 4개중 2개의 필드에 벨리데이션에러가 터졌고, 각각의 필드에 대해 biningResult에 에러가 있을 경우,

A. 어떤 익셉션을 터뜨리고

B. 그 익셉션 메세지에 bindingResult에 있는 에러들을 취합해서 넣는다

이런 스텝으로 예외 처리를 해야 하는건가요?

@ExceptionHandler({MethodArgumentNotValidException.class})
public ResponseEntity<ErrorResult> validationExHandler(MethodArgumentNotValidException e) throws JsonProcessingException {
    BindingResult bindingResult = e.getBindingResult();
    List<FieldError> fieldErrors = bindingResult.getFieldErrors();

    ValidationErrorMessage validationErrorMessage = new ValidationErrorMessage();
    List<ValidError> validationErrors = validationErrorMessage.getValidationErrors();

    for (FieldError fieldError : fieldErrors) {
        String defaultMessage = fieldError.getDefaultMessage();
        String field = fieldError.getField();
        validationErrors.add(new ValidError(field, defaultMessage));
    }
    String errorMessage = objectMapper.writeValueAsString(validationErrors);

    ErrorResult errorResult = new ErrorResult("BAD_REQUEST", errorMessage);
    return new ResponseEntity<>(errorResult, HttpStatus.BAD_REQUEST);
}

@Data
@AllArgsConstructor
@NoArgsConstructor
public class ValidationErrorMessage {

    private List<ValidError> validationErrors= new ArrayList<>();

    public void addError(ValidError error){
        validationErrors.add(error);
    }
}

@Data
@NoArgsConstructor
@AllArgsConstructor
public class ValidError{
    private String field;
    private String message;
}

@RestController
@RequestMapping("/api/v1/accounts")
@RequiredArgsConstructor
public class AccountController {

    private final AccountService accountService;

    @PostMapping("/new")
    public ResponseEntity<SignUpResponseDto> signUp(
            @Validated @RequestBody SignUpRequestDto signUpRequestDto,
            HttpServletRequest request, HttpServletResponse response){

        Account account = new Account(
                signUpRequestDto.getNickname(),
                signUpRequestDto.getPhone(),
                signUpRequestDto.getMail(),
                signUpRequestDto.getPassword());

        Account result = accountService.signUp(account);

        SignUpResponseDto signUpResponseDto = new SignUpResponseDto(
                result.getNickname(),
                result.getPhone(),
                result.getMail());

        return new ResponseEntity<>(signUpResponseDto, HttpStatus.OK);
    }