如何处理递归承诺拒绝？

Question

我有一个关于异步等待代码的概念性问题：这是我的代码中的一个 sn-p：

async function uploadFileBackup(fileId, versionNumber, filePath, sync) {
//...some irrelevant code
 const toReturn = new Promise((accept, reject) => {
    s3.upload(params, async function (err, data) {
      if (err) {
        reject(err)
      } else {
        //here is the relevant code
        file.backupStatus = 'success'
        await file.save()
        accept(data)
      }
    })
  })

  if (sync)
    return toReturn
//...some other irrelevant code
}

现在如果行

等待文件.save()

（在承诺成功时发生）失败，然后我的应用抛出 未处理的承诺拒绝，这正是我的问题。

另外，如果传递的 accept 方法抛出异常，promise 将有一个未处理的拒绝。

我可以自然地用 try catch 块包围 else 语句中的代码，但这无济于事，更糟糕的是，它会误导开发人员，因为他会认为错误来自承诺代码 s3.upload，而该部分将成功，而接受代码将是失败的部分：这肯定会产生不同的后果：

• s3 上传失败意味着文件没有上传
• 但是 file.save() 失败意味着只有我们的 db 中的文件状态没有更新，但物理文件在 s3 系统中

这两个错误不能在同一个 try catch 块中处理，但是在 accept 方法中发生的事情也不清楚（关于幕后发生的事情）。最后，在回调-承诺桥中添加 try catch 块使代码和可读性变得复杂

那么在这种情况下最好的做法是什么

Answer 1

你需要避免Promise constructor antipattern！并且不能将 async 函数作为普通回调传递给忽略返回值的函数，因此不会处理返回的承诺中的错误。

您应该单独使用promisify s3.upload

function upload(params) {
  return new Promise((resolve, reject) => {
    s3.upload(params, (err, data) => {
      if (err) reject(err)
      else resolve(data)
    })
  })
}

然后像这样使用它

async function uploadFileBackup(fileId, versionNumber, filePath, sync) {
  … // some irrelevant code
  if (sync) {
    const result = await upload(params)
    file.backupStatus = 'success'
    await file.save()
    return result
  }
  … // some other irrelevant code
}

此外，您应该避免将 toReturn 创建为仅有条件等待 (if (sync)) 的承诺，否则将创建 forgotten about without handling errors。

如果您现在想要处理来自await upload 或await file.save 的错误，您可以将它们包装在try/catch 或whatever is appropriate 中。如果你不处理它们，它们将拒绝uploadFileBackup 返回的承诺，需要由它的调用者在某个地方处理，但它们不会导致流浪承诺的未处理拒绝。