제대로 한게 맞는건가요?

스프링 MVC 2편 - 백엔드 웹 개발 활용 기술

@ControllerAdvice

해결된 질문

작성

315

@Transactional
@Override
public void updatePassword(String UUID, PasswordUpdateDto passwordUpdateDto) {
    User user = userRepository.findByUuid(UUID).orElseThrow(() -> new IllegalArgumentException("UUID정보 없음 = " + UUID));
    user.hashPassword(passwordUpdateDto.getPassword());
}

이게 서비스 코드인데 여기서 예외가 터지면 컨트롤러로 넘어가서 catch (IllegalArgumentException ex) { throw ex } 을 해서 처리하는게 맞게 하는건가요?

@PutMapping("/user/update-password/{UUID}")
public ResponseEntity<String> updatePassword(@PathVariable String UUID, @RequestBody PasswordUpdateInfo passwordUpdateInfo) {
    try {
        ModelMapper modelMapper = new ModelMapper();
        PasswordUpdateDto passwordUpdateDto = modelMapper.map(passwordUpdateInfo,PasswordUpdateDto.class);
        userService.updatePassword(UUID,passwordUpdateDto);
        return ResponseEntity.ok("Password updated successfully");
    } catch (IllegalArgumentException ex) {
        throw ex;
    }

}

@Slf4j
@RestControllerAdvice(annotations = RestController.class)
public class ExceptionController {

    @ExceptionHandler
    public ResponseEntity<ErrorResult> userExHandle(IllegalArgumentException e) {
        ErrorResult errorResult = new ErrorResult("USER-NOT-FIND", e.getMessage());
        return new ResponseEntity<>(errorResult, HttpStatus.BAD_REQUEST);
    }
}

포스트맨 쓰니까 일단 뜨긴뜨닙다

spring api restapi

답변 1

y2gcoder

안녕하세요. 박정민님, 공식 서포터즈 y2gcoder입니다.

만약 서비스단에서 발생한 IllegalArgumentException을 던져 @RestControllerAdvice의 @ExceptionHandler에서 잡고 싶으시다면

컨트롤러에서의 try-catch 문을 제거하셔도 동작할 것으로 보입니다! 그리고 좀 더 정확하게 IllegalArgumentException을 잡고 싶으시다면

@ExceptionHandler(IllegalArgumentException.class)

와 같이 @ExceptionHandler 의 속성값으로 예외 클래스를 지정해서 줄 수 있습니다 :)

감사합니다.

ㅇㅇㅇ

질문자

그러면 추가적으로 궁금한게 있습니다 다른 메서드에서 같은 IllegalArgumentException을 하는데 안에 코드가 다른경우에는 어떻게 하나요? USER-NOT-FIND 부분을 말하는겁니다.

@ExceptionHandler(IllegalArgumentException.class)
public ResponseEntity<ErrorResult> userExHandle(IllegalArgumentException e) {
    ErrorResult errorResult = new ErrorResult("USER-NOT-FIND", e.getMessage());
    return new ResponseEntity<>(errorResult, HttpStatus.BAD_REQUEST);
}

y2gcoder

throw new IllegalArgumentException("A");
throw new IllegalArgumentException("B");

의 형식이라면

만드신 @ExceptionHandler 가 붙은 메서드에서 파라미터로 받은 IllegalArgumentException의 getMessage() 메서드로 불러올 수 있습니다. 지금 하고 계신 것처럼 하시면 그 exception에 직접 설정하신 메시지를 담고 있는 것입니다.

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

제대로 한게 맞는건가요?