Question

我正在建立这个承诺链。目标是对DB中的字段进行唯一性的第一次操作检查，然后如果唯一，则保存对象。但是如果对象不是唯一的，它就不应该保存，并且应该返回错误响应。

function(request, reply) {
  var payload = request.payload;

  checkThatEmailDoesNotExist().then(saveUser)

  function checkThatEmailDoesNotExist() {
    return User.where({email: payload.email}).countAsync()
      .then(function(count) {
        if (count > 0) {
          throw Boom.badRequest('The email provided for this user already exists')
        }

        return null;
      })
      .catch(function(err) { // ~This catch should stop the promise chain~
        reply(err);
      })
  }

  function saveUser() {
    // ~But instead it is continuing on to this step~
    return User.massAssign(request.payload).saveAsync()
      .spread(function(user, numAffected) {
        return reply(user);
      })
      .catch(function(err) {
        server.log(['error', 'api', 'auth'], err);
        throw Boom.badRequest('Object could not be saved to database');
      });
  }
}

如果checkThatEmailDoesNotExist()中出现错误，则catch()应返回错误，并停止处理原始承诺链的其余部分。

而不是以这种方式行事，catch()会触发，然后继续转移到saveUser()功能。

Answer 1

你正在混合承诺和回调，这是一种可怕的反模式。来电者只会使用返回的承诺，没有必要手动将东西连接回回调。

function save(request) {
  var payload = request.payload;

    return User.where({email: payload.email}).countAsync()
      .then(function(count) {
        if (count > 0) {
          throw Boom.badRequest('The email provided for this user already exists')
        }

        return User.massAssign(request.payload).saveAsync()
      })
      .get(0)
      /* equivalent to 
      .spread(function(user, numAffected) {
        return user;
      }) */
      .catch(Promise.OperationalError, function(err) {
        server.log(['error', 'api', 'auth'], err);
        throw Boom.badRequest('Object could not be saved to database');
      });
}

用法：

save(request).then(function(user) {
    response.render(...)
}).catch(function(e) {
    response.error(...)
})

如果你想公开一个回调api，那么理所当然的方法是在现有的promise api结束时使用nodeify并在一天之内调用它：

function save(request, callback) {
  var payload = request.payload;

    return User.where({email: payload.email}).countAsync()
      .then(function(count) {
        if (count > 0) {
          throw Boom.badRequest('The email provided for this user already exists')
        }

        return User.massAssign(request.payload).saveAsync()
      })
      .get(0)
      /* equivalent to 
      .spread(function(user, numAffected) {
        return user;
      }) */
      .catch(Promise.OperationalError, function(err) {
        server.log(['error', 'api', 'auth'], err);
        throw Boom.badRequest('Object could not be saved to database');
      })
      .nodeify(callback);
}

save(request, function(err, user) {
    if (err) return response.error(...);
    response.render(...);
});

为什么我可以在链中间获得Bluebird捕获来停止链执行的其余部分

1 个答案: