Node.js全局错误处理器未生效，求助问题原因

1. 404路由未匹配的问题根源 #

你当前写的全局错误中间件（带error, req, res, next四个参数的）只负责处理被主动抛出的错误（通过next(error)传递的），而404是请求完全没匹配到任何已定义的路由，不属于错误类型，所以这个中间件不会触发它。

你在每个路由文件里加的Router.all('*')只能处理该路由前缀下的未匹配路径，没法全局覆盖所有未命中的请求。

解决方法 #

在所有业务路由注册完成后、全局错误中间件之前，加一个专门的404处理中间件：

// 放在所有 App.use('/xxx', 路由实例) 之后
App.use((req, res) => {
  res.status(404).json({
    error: 'Invalid route or missing required parameter(s)'
  });
});

2. 全局500错误处理器不生效的原因 #

如果你的业务代码里出现错误，但没通过next(error)把错误传递给全局中间件，而是直接返回了响应，或者错误被路由内部的逻辑吞了，就会导致全局错误处理器没反应。

举个正确的错误抛出例子：

// 比如accounts路由里的接口
Router.get('/:id', (req, res, next) => {
  try {
    // 业务逻辑出错时
    throw new Error('获取账户信息失败');
  } catch (err) {
    // 把错误传给全局处理器
    next(err);
  }
});

异步代码也要注意捕获错误并传递：

Router.get('/:id', async (req, res, next) => {
  try {
    const account = await AccountService.getById(req.params.id);
    res.json(account);
  } catch (err) {
    next(err);
  }
});

另外要保证全局错误中间件的位置必须在所有路由和普通中间件之后，你当前的位置是对的，但如果中间有其他中间件提前结束了响应，也会导致它不生效。

3. 调整后的完整代码示例 #

修改后的src/index.js如下：

const express = require('express');
const App = express();
const PORT = 3000;

import Accounts from './application/accounts/routes.js';
import AccountTypes from './application/account-types/routes.js';
// ...其他路由导入

// 注册所有业务路由
App.use('/accounts', Accounts);
App.use('/account-types', AccountTypes);
// ...其他路由注册

// 全局404处理：捕获所有未匹配的路由
App.use((req, res) => {
  res.status(404).json({
    error: 'Invalid route or missing required parameter(s)'
  });
});

// 全局错误处理：捕获所有通过next(error)抛出的错误
App.use((error, req, res, next) => {
  console.error(error.stack);
  res.status(500).json({ message: error.message || '服务器内部错误' });
});

App.listen(PORT, () => {
  console.log(`Node is listening on ${PORT}`);
});

这样调整后，就能全局处理404未匹配路由和500服务器错误，不用再给每个路由文件单独加Router.all('*')了。

内容的提问来源于stack exchange，提问作者Mr.Singh