예외처리가 더 효율적인 코드

Question

유저 로그인 관련 메서드를 구현하던 중, 예외처리 방법 두가지 중 어느 것이 더 효율적(가독성, 유지보수 측면 등등..)인지 여쭤보고 싶습니다

첫번째 방법: try문에 NotFoundException을 던지고, catch문에서 instanceof를 사용해서 예외의 타입을 확인하고 처리하기

async userLogin(nickname: string, password: string) {
    try {
      const user = await this.usersRepository.findOne({ where: { nickname } });
      // console.log(user)
      if (user && (await bcrypt.compare(password, user.password))) {
        return user;
      } else {
        throw new NotFoundException(아이디 또는 비밀번호가 일치하지 않습니다.);
      }
    } catch (e) {
      console.error(e);
      if (e instanceof NotFoundException) {
        throw e; // NotFoundException은 그대로 던지기
      } else {
        throw new InternalServerErrorException(알 수 없는 오류);
      }
    }
  }

두번째 방법: try문에서는 일반적인 Error객체를 던진 후 catch문에서 error.message를 확인하여 예외 유형을 판단하기

async userLogin(nickname: string, password: string) {
    try {
      const user = await this.usersRepository.findOne({ where: { nickname } });
      // console.log(user)
      if (user && (await bcrypt.compare(password, user.password))) {
        return user;
      } else {
        throw new Error(아이디 또는 비밀번호가 일치하지 않습니다.);
      }
    } catch (error) {
      if (error.message === 아이디 또는 비밀번호가 일치하지 않습니다.) {
        throw new NotFoundException(아이디 또는 비밀번호가 일치하지 않습니다.);
      } else {
        // 다른 예외 처리
        throw new InternalServerErrorException(알 수 없는 오류);
      }
    }
  }

김하림 · Answer

저는 1번 방법을 추천 드립니다. 2번의 경우 문자열을 기반으로 분기를 하기 때문에 사람의 실수로 (마지막에 .을 실수로 붙이지 않는다거나, 요구사항이 변경되어 메세지를 수정하다가 오타가 발생하는 등의 상황) 의도하지 않은 동작이 발생할 수 있습니다. 타입스크립트 관점에서도 1번이 좋은데, 에러 객체를 분기처리해서 처리하는 게 타입을 좁히는 효과도 있습니다. 그 외 가독성 면에서도 1번이 훨씬 좋다고 봅니다.