我需要等待每个嵌套的承诺数组吗？

Question

在下面的代码中，我为每个异步函数使用了await Promise.all，然后最终返回一个值。所以我想问一下：是否有必要为每个异步函数使用await Promise.all，还是一个等待（放在最后）await Promise.all 来完成这项工作？

async sendEmailNotifications() {
  const users = await User.find({  })
  const promises = users.map(async(user) => {
    const _promises = user.appId.map(async(app) => {
      const myApp = await App.findOne({ _id: app })
      if (myApp) {
        const sendNotification = await emailService.analyticsNotification(emailObj)
      }
    })
    await Promise.all(_promises)
  })
  await Promise.all(promises)
  return 'done'
}

Answer 1

是的，Promise.alls 对于您当前的实施都是必需的。异步函数返回一个 Promise。如果该函数内没有awaits，并且该函数没有显式返回 Promise，则 Promise 将立即解析。所以，没有

await Promise.all(_promises)

或

return Promise.all(_promises)

，您的const promises = users.map 数组将产生一系列承诺，所有承诺都会立即解决。您需要内部 Promise.all 才能使整个 sendEmailNotifications 函数的 Promise 仅在所有 .findOnes 和 analyticsNotifications 完成后才解析。

请记住，在您当前的代码中，所有请求都是一次性发出的；没有迭代取决于最后一次迭代的 Promise 首先完成，如果那是你担心的。唯一的代码阻塞行为是在

  const myApp = await App.findOne({ _id: app })
  if (myApp) {
    const sendNotification = await emailService.analyticsNotification(emailObj)
  }

.findOne 必须在 analyticsNotification 运行之前解析，但该逻辑看起来是必需的，并且没有办法绕过它。

我想技术上可能将您的代码更改为一个巨大的 Promise 数组，您在 一次上调用 Promise.all，每个 user.appId item 被转换为 Promise 并推送到外部数组，但这功能较少，导致代码更丑陋，并且没有任何好处，IMO。