NestJS中try-catch异常处理的正确实现方案咨询

你的代码问题核心在于：手动抛出的404异常会被try块的catch捕获，随后被重新包装成500异常返回。要实现预期逻辑，关键是区分「业务异常（如404）」和「系统异常（如数据库连接错误）」，以下是几种可行方案：

方案一：在catch块中判断异常类型 #

捕获异常后，先检查是否为自定义的HttpException实例，若是则直接抛出原异常，否则统一转为500：

async fetchItem(id: string): Promise<Item> {
  try {
    const item = await this.ItemsRepository.findOneBy({ id });
    if (!item) {
      throw new HttpException("Item wasn't found", HttpStatus.NOT_FOUND);
    }
    return item;
  } catch (err) {
    // 区分HTTP业务异常与系统异常
    if (err instanceof HttpException) {
      throw err; // 直接抛出原404异常
    }
    // 其他异常统一返回500
    throw new HttpException("Internal server error", HttpStatus.INTERNAL_SERVER_ERROR);
  }
}

方案二：使用NestJS内置的业务异常类 #

Nest提供了封装好的HTTP异常类（如NotFoundException），它们本身就是HttpException的子类，代码更简洁规范：

// 先引入内置异常类
import { NotFoundException, InternalServerErrorException } from '@nestjs/common';

async fetchItem(id: string): Promise<Item> {
  try {
    const item = await this.ItemsRepository.findOneBy({ id });
    if (!item) {
      throw new NotFoundException("Item wasn't found");
    }
    return item;
  } catch (err) {
    // 内置HTTP异常直接抛出，其他异常转500
    if (err instanceof NotFoundException) {
      throw err;
    }
    throw new InternalServerErrorException();
  }
}

方案三：拆分try-catch范围，仅包裹数据库操作 #

将数据库查询放在try块中，把404判断逻辑移出try块，避免业务异常被catch捕获：

async fetchItem(id: string): Promise<Item> {
  let item: Item | null;
  try {
    item = await this.ItemsRepository.findOneBy({ id });
  } catch (err) {
    // 仅处理数据库相关的系统异常
    throw new HttpException("Internal server error", HttpStatus.INTERNAL_SERVER_ERROR);
  }

  // 业务逻辑判断，抛出404不会被catch捕获
  if (!item) {
    throw new HttpException("Item wasn't found", HttpStatus.NOT_FOUND);
  }
  return item;
}

以上三种方案都能实现需求：数据不存在返回404，系统错误返回500。推荐方案二，符合NestJS最佳实践，代码可读性更高。

内容的提问来源于stack exchange，提问作者sl_89