将 Promises 与猫鼬一起使用的正确方法是什么？

Question

我是 Promises 的新手，两天内我已经看到了至少五种方法。

有些方式是旧版本，有些则专注于其他语言，所以我决定问一下今天应该使用什么。

我是这样做的，带有 'err' 和 'doc' 参数：

Proposals.findById({ ...body, user }, (err, doc) => 
    Advertisements.findOneAndUpdate(
        {_id:  body.advertisement}, {$push: {proposals: doc._id}}
    )
)
.then((Proposals) => Proposals.view())
.then(success(res, 201))
.catch(next)

还有这个：

Proposals.create({ ...body, user })
.then((Proposals) => {
    Advertisements.findOneAndUpdate(
        {_id:  body.advertisement}, {$push: {proposals: Proposals._id}}
    )
    .catch(Advertisements)

    return Proposals.view()
}) 
.then(success(res, 201))
.catch(next)

没有“catch”和“return”是行不通的，但我不知道究竟是为什么。 :x

答案 Using Promises in Mongoose Routes 看起来很漂亮，但我无法在上面的代码中重现。

return findUser(userId)
  .then((user) => findReceipt(user.bookName))
  .then((receipt) => res.status(200).json(receipt))
  .catch((err) => res.status(500).json(null))

这些代码是否更正确？ 你能帮我修复第二个代码吗？

谢谢。

Answer 1

让我详细说明第一块代码的语法：

Proposals.findById({ ...body, user }, (err, doc) => {
  return Advertisements.findOneAndUpdate(
    {_id:  body.advertisement}, {$push: {proposals: doc._id}}
  );
})
.then((Proposals) => { return Proposals.view() })
.then(success(res, 201))
.catch(next)

在 ES6 语法中，这个函数：

() => ('test');

等于

() => { return 'test' };

所以我相信你的主要问题是因为一开始没有完全理解 ES6 语法。

对于Promises，您应该遵循这些最佳做法：

1. 避免在Promises 中嵌套catch
2. 通过返回它们链接Promises 并在then 中使用它们

希望对你有帮助。

Answer 2

示例代码没有等待或使用更新的结果。

在 Promise 中 return 使用 Promise 并使用 .then 等待异步操作。然后，您可以根据需要检查结果：

Proposals.create({ ...body, user })
.then((proposal) => {
    return Advertisements.findOneAndUpdate(
        {_id:  body.advertisement}, {$push: {proposals: proposal._id}}
    )
}) 
.then((update) => {
  if (!update) throw new Error(`Could not find advertisement for "${user}"`)
  success(res, 201))
})
.catch(next)

或者等效的 ES2017 代码使用 await 来等待 Promise 解决，这应该有助于解释上述 Promise .then 代码流是如何工作的。

async function handler(){
  try {
    let proposal = await Proposals.create({ ...body, user })
    let update = await Advertisements.findOneAndUpdate(
      {_id:  body.advertisement}, {$push: {proposals: proposal._id}}
    )
    if (!update) throw new Error(`Could not find advertisement for "${user}"`)
    success(res, 201)
  }
  catch (err) {
    next(err)
  }
}

Answer 3

在 Mongoose 中有多种处理承诺的方法。您决定实施它的确切方式取决于个人喜好，但您需要确保您的执行顺序符合预期。在您的第一个示例中，您将回调与承诺混合在一起，我个人认为这不是最佳选择。第三个示例更简洁，因为它是通过then 调用传递的一系列承诺。

需要注意的重要一点是，如果您想在 then 调用中运行异步代码，您需要确保返回一个在正确时刻解析的 Promise。在您的第二个示例中，您可能在您的 findOneAndUpdate 调用完成执行之前返回。您可以通过多种方式避免这种情况。一种选择是将第二个调用包装在Promise 中。我通过混合第二个和第三个示例的部分内容并假设proposal.view() 的返回值不是异步的，在下面给你一个想法。为了清楚起见，我使用明确的 return 语句。

Proposals.create({ ...body, user })
  .then((proposal) => {
    // call returns a new promise
    return new Promise((resolve, reject) => {
      // make second async call to database
      Advertisements.findOneAndUpdate(
        { _id:  body.advertisement }, 
        { $push: { proposals: proposal._id } },
      )
      .then(() => {
        // promise resolves here and passes return
        // value of proposal.view() to next `then`
        return resolve(proposal.view())
      });
    });
  })
  // consume the value from proposal.view() and send it in response
  .then(proposal => res.status(200).json(proposal))
  .catch(next)

这意味着proposal.view() 的返回值只会在findOneAndUpdate 调用完成后传递给下一个then。