예외 처리 위치

[thelightway24]

안녕하세요!

개발하다 보면 예외 처리를 자주 하게 되는데, 그 위치를 일관되게 통일하면 가독성이나 유지보수 측면에서 훨씬 좋을 것 같아 의견을 모아보고자 합니다.

현재 팀 내에서

• 예외는 try-catch로 감싸고
• CustomException으로 래핑해서 처리하자는 방향에는 대체로 합의된 것 같은데요,

문제는 이 예외를 어디서 처리하느냐에 대한 기준이 아직 명확하지 않아 보여요.

저는 개인적으로는 서비스 클래스는 최대한 깔끔하게 유지하고,
private 메서드 내부에서 예외를 처리하는 쪽이 낫다고 생각해요.

서비스 레이어는 비즈니스 흐름만 담당하고, 세부 예외 처리는 내부 로직에 맡기는 방식이 구조적으로 더 명확하다고 느꼈습니다.

---

🔍 비교 예시

① 서비스 메서드에서 try-catch로 처리하는 방식

@Service
public class UserService {

    public void registerUser(UserDto dto) {
        try {
            validateEmail(dto.getEmail());
            userRepository.save(dto.toEntity());
        } catch (DuplicateEmailException e) {
            throw new CustomException("이메일이 중복되었습니다", e);
        }
    }

    private void validateEmail(String email) {
        if (userRepository.existsByEmail(email)) {
            throw new DuplicateEmailException();
        }
    }
}

② private 메서드 내부에서 예외를 처리하는 방식

@Service
public class UserService {

    public void registerUser(UserDto dto) {
        validateEmail(dto.getEmail());
        userRepository.save(dto.toEntity());
    }

    private void validateEmail(String email) {
        if (userRepository.existsByEmail(email)) {
            throw new CustomException("이메일이 중복되었습니다");
        }
    }
}

[LEELISE]

동의 합니다 public 메서드에서는 예외를 처리하지 않고 private에서 예외를 잡아서 처리해주는 방식이 좋아보입니다

[HaechangLee]

코드 가독성 측면에서 제안해 주신 방법이 맞는 방향이라고 생각합니다.
최대한 신경써서 코드 작성해보겠습니다

[thelightway24]

예외는 public에서 처리하지 않고, private에서 모두 처리해주는 걸로 하겠습니다.

discussion은 코드 컨벤션으로 결정 되어 닫습니다.