为何asyncHandler中调用next()后代码仍继续执行？

为什么调用next()后函数不会停止执行？ #

next()是Express框架中用来将请求控制权传递给下一个中间件/路由处理函数的工具，但它本身并没有终止当前函数执行的能力。调用next()之后，当前函数里后续的代码依然会按顺序执行——这就是你遇到的问题：在返回200响应并调用next()后，代码继续走到了res.status(401)的逻辑，导致重复设置响应头，触发Cannot set headers after they are sent to the client错误。

代码问题拆解 #

你的代码存在两个关键问题：

1. 混用await和.then()：已经用await等待UserService.findUserByEmail的结果，没必要再嵌套.then()，这会让逻辑混乱，也容易遗漏终止条件。
2. 缺少分支终止逻辑：当用户验证通过时，没有阻止后续代码执行，导致错误的响应被重复发送。

优雅的修复方案 #

方案1：用return终止执行（这其实是Express的标准做法） #

在调用res.send()和next()后添加return，是终止当前处理逻辑的常规手段，完全符合规范：

const login = asyncHandler(async (req: Request, res: Response, next: NextFunction) => {
  const credentials: Credentials = req.body as Credentials;
  const user = await UserService.findUserByEmail(credentials.email);
  
  if (user != null && user.password === credentials.password) {
    res.status(200).send('Token will be here in future');
    return next(); // return终止当前函数，避免后续代码执行
  }

  res.status(401).send('Email or password is wrong');
  next();
})

方案2：重构为纯async/await的清晰分支 #

去掉冗余的.then()，用if-else明确分支逻辑，让代码可读性更强：

const login = asyncHandler(async (req: Request, res: Response, next: NextFunction) => {
  const credentials: Credentials = req.body as Credentials;
  const user = await UserService.findUserByEmail(credentials.email);
  
  if (user && user.password === credentials.password) {
    res.status(200).send('Token will be here in future');
    next();
  } else {
    res.status(401).send('Email or password is wrong');
    next();
  }
})

额外安全提醒：不要明文存储密码！ #

你的代码里直接比较明文密码，这是严重的安全漏洞。应该在用户注册时用哈希算法（比如bcrypt）加密密码，验证时比对哈希值，永远不要存储或传输明文密码。

内容的提问来源于stack exchange，提问作者Juan Cruz Carrau