for文内部のasync/awaitで記述した処理に対してPromise.allで処理したい

Question

聞きたいこと

下のような配列内のオブジェクトの値(name)を、バックエンドから取得した値を元に更新したいと思っています。

// nameは初期値は''です
const users = [
  { id: 'xxxx', name: '' },
  { id: 'yyyy', name: '' },
  { id: 'zzzz', name: '' },
]

そこで下のような処理を書いています。

:

for (const user of users) {
  const response = await getUserDetail(user.id) // responseにはキーがid, nameのオブジェクトが入ります。
  user.name = response.name
}

:

処理自体は上で期待通りになっているのですが、Promise.allを使用する方法に置き換えたいと思っています。

そこで下のように置き換えましたが、エラーになってしまっています。

await Promise.all(users.forEach(async (user) => {
  const response = await getUserDetail(user.id)
  user.name = response.name  
})

エラーは下のとおりです。

undefined is not iterable (cannot read property Symbol(Symbol.iterator))

Comments

[maisumakun]

> 処理自体は上で期待通りになっているのですが、Promise.allを使用する方法に置き換えたいと思っています。

どのような目的があってでしょうか？

Answer 1

結果としてどういう状態を想定しているのでしょうか？
書き換わったusersを得たいならこんな感じで

Promise.all(users.map(async(x)=>(x.name=await getUserDetail(x.id)))).then(()=>{
  console.log(users);
});