Express路由处理函数中调用next()是否为正确实现方式？

在路由处理函数中调用next()是完全合法且符合Express设计逻辑的标准用法，你当前的实现思路没有问题，代码能正常运行是符合框架运行规则的，但存在两处可以优化的细节：

• 第一处是末尾404中间件的冗余逻辑：你在执行res.status(404).render('error')已经向客户端返回响应后，不应该再调用next()。如果后续中间件栈还有其他逻辑，很容易触发Cannot set headers after they are sent to the client这类重复响应的报错，完成响应后直接结束处理即可。
• 第二处是语义层面的优化空间：你当前把「title参数无效」和「路由不存在」两种场景统一交给404中间件处理，功能上没有问题，但如果后续需要区分错误类型（比如参数无效返回400状态码、路由不存在返回404状态码），这种写法扩展性不好。更规范的做法是参数校验不通过时，向next()传入错误对象，走专门的错误处理中间件分流逻辑。

修正后的参考代码 #

路由逻辑 #

app.get('/:title', (req, res, next) => {
  const title = req.params.title;

  if (title.isValid()) {
    res.render('post', { title });
  } else {
    // 若不需要区分错误类型，直接调用next()交给404中间件完全可行
    next();

    // 若需要区分错误类型，可传入错误对象走错误处理逻辑
    // const invalidErr = new Error('无效的title参数');
    // invalidErr.statusCode = 400;
    // next(invalidErr);
  }
});

404处理中间件 #

app.use((req, res) => {
  res.status(404).render('error');
  // 已完成响应，无需调用next()
});

可选：统一错误处理中间件 #

如果需要区分不同错误类型，可以在所有路由、普通中间件的最后添加四参数格式的错误处理中间件（Express会自动识别四个参数的中间件为错误处理逻辑）：

app.use((err, req, res, next) => {
  const status = err.statusCode || 500;
  res.status(status).render('error', {
    errorMsg: err.message || '服务器内部错误'
  });
});

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