Question

我有一个首先需要查询数据库的路由，然后使用结果查询另一个Web服务，然后使用该结果呈现页面。我已经解决了这个问题，并试图找出错误处理方法。鉴于我与多个服务进行了交谈，我试图在将它们返回表达之前按摩错误。

以下是路线代码的结构：

Models.Episode.findById(request.params.episodeID)
    .catch(function (error) {
        throw (throwjs.notFound());
    })
    .then(function (episode) {
        if (episode.getUser().id !== request.user.href) {
            return next(throwjs.unauthorized("You do not have access to this podcast"));
        }
        return doSomeOtherAsyncStuff();
    })
    .then(function (queryResponse) {
        renderPage();
    })
    .catch(function (error) {
        next(error);
    });

我的问题在于第一次捕获。我抓住这个问题的目的是重新打包错误并停止执行并发送错误来表达中间件。

按照上面所述的方式，执行会停止，但我的快速错误处理程序不会被调用。

我尝试将第一个捕获重写为

.catch(function(error){
     return next(error);
})

但这并没有解决问题。我发现的唯一解决方案就是将捕获量移到最后。但后来我失去了失败地点的背景。

关于我做错了什么的任何线索？谢谢，奥利维尔

Answer 1

我建议采用不同的方法，这样您就不必依赖长期运行的承诺链。通过以下方法，您已将授权和验证分离到单独的中间件，因为它们不一定是实际情节处理程序本身的关注点。此外，这种方法更具惯用性。

额外的好处是，您可以将错误传递给错误处理程序，以便进一步将错误与路由处理程序分离。

function validateEpisode(req, res, next) {
  Models.Episode
    .findById(req.params.episodeID)
    .then(function(episode) {
      req.yourApp.episode = episode;
      next() // everything's good
    })
    .catch(function(error) {
      // would be better to pass error in next
      // so you can have a general error handler
      // do something with the actual error
      next(throwjs.notFound());
    });
}

function authUserByEpisode(req, res, next) {
  if (req.yourApp.episode.getUser().id !== req.user.href) {
    next(throwjs.unauthorized("You do not have access to this podcast"));
  }

  next(); // authorized
}

function episodeController(req, res) {
  // do something with req.yourApp.episode
}

app.get('/episode/:id', validateEpisode, authUserByEpisode, episodeController)

Answer 2

毕竟，这与我使用的throwjs框架以及我使用不正确的事实有关

    throw (throwjs.notFound());

应该是

    throw (new throwjs.notFound());

...

expressJS承诺和错误处理

2 个答案: