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

안녕하세요. jeong정재인님, 공식 서포터즈 y2gcoder입니다.

findById()라는 메서드는 사실 결과 없음 / 결과 하나 라는 두 가지 케이스의 응답을 가정한다고 생각합니다. 메서드의 만들어진 목적으로 볼 때, 결과가 없을 때 반환하는 EmptyResultDataAccessException 는 findById() 입장에서 쿼리 결과가 없을 때 나올 수 있는 일반적인 상황이므로 Optional.empty()를 이용해 명시적으로 처리해주는게 맞다고 생각합니다.

반면에 둘 이상일 경우 발생하는 IncorrectResultSizeDataAccessException 는 findById 입장에서는 전혀 정상적인 상황이 아닙니다. 뭔가 개발자가 짠 로직에 문제가 있는 것이고, 데이터의 무결성이 깨진 상황이라 볼 수 있습니다. 이러한 경우는 디버깅과 수정이 필요한 상황이기 때문에 예외를 try catch문에서 먹지 않고 던져주어 프로그램의 진행을 막아주는 것이 좋다고 생각합니다. 그래야 개발자가 문제 상황을 인지할 수 있기 때문입니다.

물론 그럼에도 불구하고 비즈니스 로직 상황에 따라 IncorrectResultSizeDataAccessException 를 명시적으로 처리해줘야 한다면 catch 블록을 추가해서 처리해줄 수는 있습니다. 다만 이번 findById()에서는 이렇게 처리하는 것이 좀 더 맞지 않나 조심스럽게 생각해봤습니다.

감사합니다.