• 카테고리

    질문 & 답변
  • 세부 분야

    백엔드

  • 해결 여부

    해결됨

제대로 한게 맞는건가요?

23.08.22 17:51 작성 조회수 210

0

 

@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);
    }
}

 

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

 

 

 

 

 

 

 

 

답변 1

답변을 작성해보세요.

0

y2gcoder님의 프로필

y2gcoder

2023.08.23

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

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

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

@ExceptionHandler(IllegalArgumentException.class)

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

감사합니다.

ㅇㅇㅇ님의 프로필

ㅇㅇㅇ

질문자

2023.08.23

그러면 추가적으로 궁금한게 있습니다 다른 메서드에서 같은 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님의 프로필

y2gcoder

2023.08.23

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

의 형식이라면

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