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

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

첫번째 방법: 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('알 수 없는 오류');
      }
    }
  }