如何正确处理 Express 中的错误？

Question

我开始使用 Express JS 并遇到了一个问题。我似乎无法找出处理错误的正确方法。

例如，我有一个 Web 服务 API，它为一个名为“事件”的对象提供服务。当用户提交未找到的事件 ID 时，我想返回一个简单的“找不到事件”字符串。以下是我目前构建代码的方式：

app.get('/event/:id', function(req, res, next) {
    if (req.params.id != 1) {
        next(new Error('cannot find event ' + req.params.id));
    }

    req.send('event found!');
});

当我提交 1 以外的 id 时，Node 崩溃并显示以下输出：

http.js:527
   throw new Error("Can't set headers after they are sent.");
         ^
Error: Can't set headers after they are sent.
    at ServerResponse.<anonymous> (http.js:527:11)
    at ServerResponse.setHeader (/usr/local/kayak/node_modules/express/node_modules/connect/lib/patch.js:62:20)
    at /usr/local/kayak/node_modules/express/node_modules/connect/lib/middleware/errorHandler.js:72:19
    at [object Object].<anonymous> (fs.js:107:5)
    at [object Object].emit (events.js:61:17)
    at afterRead (fs.js:878:12)
    at wrapper (fs.js:245:17)

据我所知，通过使用 node.js 调试器< /em>，在调用 next() 后继续执行代码块，这意味着 req.send('event found!') 尝试运行。我不希望这种事发生。

我发现的唯一解决方法是简单地抛出一个 new Error() 而不是“next-ing”它，但这会导致生成默认的 Express HTML 错误页面。我想要更多一点的控制权。

我花时间阅读了 Express 文档的错误处理部分，但是我无法理解它。

Answer 1

您需要查看快速错误处理。从那里：

app.param('userId', function(req, res, next, id) {
    User.get(id, function(err, user) {
        if (err) return next(err);
        if (!user) return next(new Error('failed to find user'));
        req.user = user;
        next();
    });
});

您缺少的最佳点是 return next(...)

Answer 2

那是因为你做错了：你已经抛出了一个错误（该错误将由 Express 处理并为用户返回 500 - 错误页面或类似的内容），但你也尝试将你自己的响应发送给客户端： res.send('发现事件！');

您确实应该在此处查看有关错误处理的 Express 指南： http://expressjs.com/guide /error-handling.html

在你的例子中我会做的是：

function NotFound(msg){
  this.name = 'NotFound';
  Error.call(this, msg);
  Error.captureStackTrace(this, arguments.callee);
} 

app.get('/event/:id', function(req, res, next){
  if (req.params.id != 1) {
    throw new NotFound('Cannot find event ' + req.params.id);
  } else {
    res.send('event found!');
  }
});

app.error(function(err, req, res, next){
    if (err instanceof NotFound) {
        res.render('404.ejs');
    } else {
        next(err);
    }
});

Answer 3

您的代码中存在一些问题：

• 响应客户端时，您需要使用响应对象（res 而不是比 req）。

• 当向 next 发送错误时，您应该返回，这样函数的其余部分就不会运行。

这是修复这些错误后的代码：

app.get('/event/:id', function(req, res, next) {
    if (req.params.id != 1) {
        return next(new Error('cannot find event ' + req.params.id));
    }

    res.send('event found!'); // use res.send (NOT req.send)
});