Question

我不知道promise.all解决方案中的promise.all是否是一个好习惯。我不确定。

我需要从用户数组中获取信息，然后通过此信息响应，我需要发送消息通知。

let userList = ['key1', 'key2', 'key3']; //More data can arrive
let promises = userList.map((userKey,index)=>{
            return GetUserById(db.ref(`Users/${userKey}`));
});

Promise.all(promises).then(responses =>{
  let notificationPromises = responses.map((user)=>{
      sendNotification('message', user.token);
  });
  return Promise.all(notificationPromises)
}).then(()=>{
   //notifications were sent
   ...
};

使用Promise.all嵌套解决它是个好主意吗？

Answer 1

尽管这行得通，但很难理解为什么这比仅在第一组请求上调用then()更好的选择-请记住，then()还会返回一个承诺。这似乎不仅更短，而且对我而言更清楚。很明显，您正在向每个用户发送通知：

let userList = ['key1', 'key2', 'key3']; //More data can arrive
let promises = userList.map((userKey,index)=>{
            return GetUserById(db.ref(`Users/${userKey}`))
            .then(user => sendNotification('message', user.token) )
});

Promise.all(promises)
.then(()=>{
    //notifications were sent
    // ...
});

p.s。在您的代码中，您需要从map()回调中返回一些内容，否则notificationPromises将是一个空值数组：

Promise.all(promises).then(responses =>{
    let notificationPromises = responses.map((user)=>{
        return sendNotification('message', user.token); //<< add return
    });

Promise.All内部承诺。这对Promises来说是一个好习惯吗？

1 个答案: