valid와 ExceptionHandler를 동시에 사용해도 되나요?

Question

안녕하세요! spring 설계가 너무 어려운 뉴비입니다!ㅠㅠ현재 채팅룸을 생성하는 코드를 작성하고 있는데요. valid와 ExceptionHandler를 동시에 사용하면 안되는건지 궁금해서 여쭤봅니다현재 코드 형식은 이러합니다. Controller에서 RequestBody DTO를 받아 validation 한다.1-1. Controller에서 validation에 충족하지 못하면 서버 메시지를 받는다.Service에 createChatRoom 메소드로 생성 요청을 보낸다.Service에서 3-1. Service에서 예외처리 코드를 통과하지 못하면 new Exception을 생성한다.3-2. Service에서 예외처리 코드를 통과하면 repository에 Entity를 저장한다. Controller에서는 새로운 ResponseEntity 객체를 만들어 클라이언트에게 보내준다.문제Swagger로 방 인원을 11명으로 DTO를 보낸면 1-1. @Valid에 의해 Server Response를 400 Bad Request를 받게 되는데요. 저는 3-1 방법으로 메시지를 받길 원합니다. 그렇다면 @Valid를 빼는게 맞는걸까요? 아니면 @Valid로도 원하는 메시지를 받는 방법이 있을까요? Controller에서 ResponseEntity 타입을 받는 것도 맞게 코드를 짠 건지 잘 모르겠습니다 ㅠㅠ, 왜냐하면 형식에 맞지 않는 DTO를 Swagger에 찍어도 Server Response는 Bad Request를 받지만 Response는"채팅룸이 생성되었습니다." 라는 도 같이 오기 때문입니다.답변 주시면 정말 감사하겠습니다..ㅠㅠCAPACITY_WRONG(HttpStatus.BAD_REQUEST, "정원은 최소 5명 이상 10명 이하 입니다."),Dto @Max(value=10, message = "방 인원은 10명 이하이여야 합니다.") @Min(value=5, message = "방 인원은 5명 이상이여야 합니다.") @NotNull(message="방 인원은 필수입니다.") @Schema(description = "정원") private Long capacity;Controller @PostMapping(value = "/chatrooms") public ResponseEntity createChatRoom(@Valid @RequestBody ChatRoomReqDTO chatRoomReqDTO) { ResponseCode responseCode = ResponseCode.CHATROOM_CREATE; return new ResponseEntity<>(ResResult.builder() .responseCode(responseCode) .code(responseCode.getCode()) .message(responseCode.getMessage()) .data(chatRoomService.createChatRoom(chatRoomReqDTO)) .build(), HttpStatus.OK); }ResResultpublic class ResResult { private ResponseCode responseCode; private String code; private String message; private T data; @Override public String toString() { return "ResResult{" + "responseCode=" + responseCode + ", code='" + code + '\'' + ", message='" + message + '\'' + ", data=" + data + '}'; } }GlobalExceptionHandler@RestControllerAdvice @Slf4j public class GlobalExceptionHandler extends ResponseEntityExceptionHandler { @ExceptionHandler(CustomException.class) protected ResponseEntity handlerCustomException(CustomException e) { log.error("CustomException: " + e.getErrorCode().getMessage()); return new ResponseEntity<>(new ErrorResponse(e.getErrorCode()), e.getErrorCode().getStatus()); } @ExceptionHandler(Exception.class) protected ResponseEntity handlerException(Exception e) { log.error("Unexpected_Exception : " + e.getMessage()); return ResponseEntity.status(500).body("UNEXPECTED_EXCEPTION: " + e); } }ErrorCode@Getter @ToString public enum ErrorCode { CAPACITY_WRONG(HttpStatus.BAD_REQUEST, "정원은 최소 5명 이상 10명 이하 입니다."), }Service public ChatRoom createChatRoom(ChatRoomReqDTO chatRoomReqDTO) { // 인원이 범위를 벗어나는 경우 Long capacity = chatRoomReqDTO.getCapacity(); if (capacity < 5 || capacity > 10) throw new CustomException(ErrorCode.CAPACITY_WRONG); return chatRoomJpaRepository.save(createToEntity(chatRoomReqDTO)); }

KJ9988 · Accepted Answer

@Valid는 파라미터 검증하는 인터셉터에서 처리하는 것이고, 문제 없다면 controller 메소드로 들어오겠죠. 문제가 있다면 GlobalExceptionHandler의 @ExceptionHandler(Exception.class) 부분으로 들어올 듯 한데, 나이스해보이지는 않습니다. GlobalExceptionHandler 제거해서 그냥 파라미터 오류 예외 반환하게 하는게 상세한 결과가 반환되고 좋겠네요.Controller의 오류는 controller 클래스에 @ExceptionHandler 메소드에서 처리하는게 좋겠습니다. 이 부분이 여러 클래스에 복붙해서 넣어야 한다면 @RestControllerAdvice를 이용하거나 super controller 클래스에 넣어서 상속받으면 되겠네요.

valid와 ExceptionHandler를 동시에 사용해도 되나요?

1개의 답변

Q&A 목록