Node.js 中未处理的承诺拒绝

Question

我正在尝试拨打DELETE 并正在实现以下功能。我知道在一个承诺中，需要有一个“解决”和一个“拒绝”状态，但我收到了一个未处理的承诺拒绝错误：

UnhandledPromiseRejectionWarning：未处理的承诺拒绝（拒绝 id：1）：[object Object]

我真的不喜欢在 Promise 中使用条件语句，因为它会变得混乱，但我在这里要做的是检查组织是否经过验证，如果是，则不应发生删除操作并且会拒绝。

function deleteOrg(id) {
    return new Promise((resolve, reject) => {
        // A helper function that returns an 'org' object
        findById(id)
        .then((orgObject) => {
            if (orgObject.verified_at !== null) {
                throw new Error(422, 'Unable to delete organization')
            }

            //Organization is not verified, so proceed to delete
            new Organization().where({'id': id}).destroy()
            .then(() => {
                return resolve() //return 200 upon deletion
            })
            .catch((err) => {
                return reject(new Error(500, 'Unable to delete organization'))
            })
        })
        .catch((err) => {
            const m = `Unable to delete organization: ${err.message}`
            return reject(new Error(500, m))
        })
    })
}

我很确定我在处理 if 中的拒绝错误。

Answer 1

在 Promise 构造函数中创建 Promise 是 known anti-pattern。尝试将您的 Promise 模块化为单独的函数：

function deleteOrg(id) {

  const verifyOrg = (orgObject) => {
    if (orgObject.verified_at !== null) {
      throw new Error(422, 'Unable to delete organization')
    }
  };

  const destroyOrg = () => new Organization().where({
    'id': id
  }).destroy();

  return findById(id)
    .then(verifyOrg)
    .then(destroyOrg);
}

您可以让错误通过承诺链传播并在外部处理它们：

deleteOrg(id)
  .catch((err) => {
    const m = `Unable to delete organization: ${err.message}`;
    // ...
  });

Answer 2

由于 findById 和 .destroy 返回 Promises，所以不需要 Promsie 构造函数

然后您的代码被简化为

function deleteOrg(id) {
    return findById(id)
    .then((orgObject) => {
        if (orgObject.verified_at !== null) {
            throw new Error(422, 'Unable to delete organization')
        }

        //Organization is not verified, so proceed to delete
        return new Organization().where({'id': id}).destroy()
        .catch((err) => {
            throw (new Error(500, `Unable to delete organization: ${err.message}`));
        });
    });    
}