Node.js使用AsyncHandler包装函数捕获错误：异常后控制器仍返回响应问题

1. AsyncHandler未正确处理异步函数错误：
   当前的asyncHandler直接调用异步函数但未使用await，异步函数返回的Promise拒绝无法被try/catch捕获，错误处理逻辑失效。同时，next(err)仅将错误传递给全局处理器，不会终止当前函数执行，导致控制器后续的successResponse依然运行。

2. 服务层触发错误后未中断控制器流程：
   服务层调用next(err)后返回undefined，控制器的await操作会继续执行，最终同时触发错误响应和成功响应，不符合预期。

方案一：修复AsyncHandler并调整服务层返回Rejected Promise #

1. 修正AsyncHandler #

改为正确捕获异步函数的Promise拒绝：

const asyncHandler = (func) => (req, res, next) => {
  func(req, res, next).catch(next);
};

或async/await版本：

const asyncHandler = (func) => {
  return async (req, res, next) => {
    try {
      await func(req, res, next);
    } catch (error) {
      next(error);
    }
  };
};

2. 修改服务层代码 #

调用next(err)后返回rejected Promise，中断控制器后续代码：

const categoryService = {
  createCategory: async (data, x, next) => {
    if (data === "") {
      console.log("condition");
      const err = new CustomError(404, "Wrong input");
      next(err);
      return Promise.reject(err);
    } else {
      return data;
    }
  },
};

此时控制器的await会捕获rejected Promise，AsyncHandler将错误传递给全局处理器，successResponse不会执行。

方案二：服务层直接抛错，AsyncHandler统一处理（更简洁） #

这种方式更符合Express错误处理最佳实践，无需在服务层调用next：

1. 服务层代码修改 #

移除next参数，直接抛出错误：

const categoryService = {
  createCategory: async (data, x) => {
    if (data === "") {
      console.log("condition");
      throw new CustomError(404, "Wrong input");
    } else {
      return data;
    }
  },
};

2. 控制器代码保持不变 #

const categoryCtrl = {
  create: asyncHandler(async (req, res, next) => {
    const data = await CategoryService.createCategory("", 4);
    successResponse({res, data:data, msg:"Category created Successfully"});
  }),
  getAll: async (req, res, next) => {},
  getById: async (req, res, next) => {},
  delete: async (req, res, next) => {},
  update: async (req, res, next) => {},
};

AsyncHandler会自动捕获服务层抛出的错误，调用next(error)交给全局处理器，控制器后续的成功响应代码会因错误被抛出而终止执行。

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