Node.js中间件返回响应后代码未停止执行是什么原因？

执行顺序说明 #

你对中间件执行顺序的理解是正确的：Express 中 router.param 注册的参数处理中间件，会在路由包含对应参数时，优先于该路由挂载的所有普通中间件（包括isSignedIn）执行，顺序为 getUserById → isSignedIn → 路由处理函数。

异常返回后代码仍运行的原因 #

主要有两个常见场景：

• 请求头校验缺失导致提前抛出同步错误
  你的isSignedIn第一行直接对req.headers.authorization调用split方法，如果请求没有携带Authorization请求头，req.headers.authorization的值为undefined，此时调用split会直接抛出同步错误，根本不会进入jwt.verify的回调逻辑，你写在回调里的return自然不会生效，错误会被Express默认错误处理机制捕获，看起来就像是返回响应后代码仍在执行。
• 重复返回响应触发报错
  如果getUserById中已经调用res.json返回了响应，后续isSignedIn如果再次尝试调用res.json返回响应，会抛出Cannot set headers after they are sent to the client错误，你可能会把这个报错误认为是return之后代码仍在运行。

修复方案 #

修改isSignedIn逻辑，提前做请求头和token的存在性校验，同时所有错误响应统一加上对应HTTP状态码：

const isSignedIn = (req,res,next)=>{
   // 先校验Authorization请求头是否存在
   if(!req.headers.authorization) {
       return res.status(401).json({
           status: false,
           error: "未提供授权凭证"
       })
   }
   const token = req.headers.authorization.split(" ")[1]
   // 校验Bearer token格式是否正确
   if(!token) {
       return res.status(401).json({
           status: false,
           error: "授权凭证格式错误"
       })
   }
   jwt.verify(token, process.env.SECRET_KEY, (err, decoded)=>{
        if(err){
            return res.status(401).json({
                status: false,
                error: "无效的授权凭证"
            })
        }
        req.auth=decoded
        next()
   })
};

同时建议给getUserById里的错误响应也加上对应的状态码，比如查询错误加400，用户不存在加404，避免前端收到200状态码的错误响应，提升接口规范性。

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