작성
·
239
0
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);
}
답변 1
0
안녕하세요. 유승현님
1. 이런방식으로 밖에 안떠오르는데 혹시 다른 방법이 더 좋다거나 한지 알려 주실 수 있을까요?
-> 반환 타입을 ResponseEntity<Object>로 하시면 상황별로 원하는 타입을 반환할 수 있습니다.
2. 추가적으로 로그인에서 에러가 발생할 경우, (미인증사용자)
네 해당 방법을 사용하시거나 뒤에서 학습할 @ControllerAdvice(@RestControllerAdvice)를 사용하시면 됩니다.
3. 이건 마이너 한것일 수 있는데, 저는 이렇게 응답 객체 네이밍을 했는데 (SignUpResponseDTO) 영한님은 어떻게 네이밍을 하시는지 또한 궁금합니다.
-> 네 괜찮다고 생각합니다.
추가 질문
네 맞습니다.
감사합니다.