Nestjs-如何构造代码以优雅处理错误？

Question

我正在努力处理Nestjs应用程序中的错误，因为我在任何地方都在写尝试/捕捉。我也不确定在服务或解决方案/控制器中丢弃错误，还是在某些拦截器或异常过滤器中捕获并重新捕获它是清洁/最佳实践。还在用fn（）。然后编写代码。

这是我的代码的一个示例，其中有许多样板代码处理错误的错误处理。

您将如何更干净和干燥处理错误？

  async validateUser(
    email: string,
    plainTextPassword: string,
  ): Promise<User | null> {
    try {
      const user = await this.usersRepository.findOne({ email });
      if (!user) {
        throw new HttpException(
          "Wrong credentials provided",
          HttpStatus.BAD_REQUEST,
        );
      }

      const isMatch = await this.verifyPassword(
        plainTextPassword,
        user?.password,
      );
      if (isMatch) {
        await this.usersRepository.filter(user);
        return user;
      }
    } catch (err) {
      this.logger.error(err);
      throw new HttpException(
        "Something went wrong",
        HttpStatus.INTERNAL_SERVER_ERROR,
      );
    }
  }

  /** This function is used by the validateUser function
   *  Note: This could be written much much shorter in 1-2 lines...
   */
  async verifyPassword(plainTextPassword: string, hashedPassword: string) {
    try {
      const isMatch = await bcrypt.compare(plainTextPassword, hashedPassword);
      if (!isMatch) {
        throw new HttpException(
          "Wrong credentials provided",
          HttpStatus.BAD_REQUEST,
        );
      }
      return isMatch;
    } catch (err) {
      this.logger.error(err);
      throw new HttpException(
        "Something went wrong",
        HttpStatus.INTERNAL_SERVER_ERROR,
      );
    }
  }

Answer 1

以下是一些建议：

• verifypassword：如果唯一可能的值为true（false，函数抛出时），则返回布尔值的目的没有太大目的。您可以将功能的错误部分移动到验证器中，然后简单地返回密码是否匹配。
• 在这两个功能中，您都首先提出一个特定的错误，然后捕获并用另一个错误替换。如果有人输入一封没有这种方式的不存在电子邮件，他们将收到内部服务器错误，该错误不会提供有关为什么造成错误的信息。这样，您可以删除两个尝试/捕获条款。
• 在我的 shobm 意见中，当找不到用户电子邮件时，抛出的400应该是404（找不到），而当密码不匹配时，则应该是403（禁止）。
• 为了进一步简化，您可以使用Nest的特定异常，而不是httpexception，例如notfoundexception，forbidDenexception和internalSeralSeralSeralSeralSeralSeralSeralSeralSeralSeralSeralSeralRorexception。参见 this 。
• 无需投掷internalServererRorexception s，因为Nestjs的默认过滤器将处理任何未透露的错误并亲自抛出500个错误。
• 如果您有一些要不同的特定错误（主要是特定于图书馆的错误，您可能想通知自己或特别记录它们），则可以实现