一旦检索到所有数据，如何解析内部带有foreach循环的嵌套promise

Question

我在函数中有一个很大的承诺，在函数内部我进行了两次调用。第一个是从组织中获取成员。然后我对它进行迭代，并为每个成员获取详细信息+将其填充到一个数组中。我只想在最后解决，一旦我有了每个成员的详细数据。我通过比较两个数组的长度来实现这一点。下面的代码是有效的，但我如何拆分它或用更好的方式编写它？

export const getTeamMembers = (firebaseContext, user) => {
const { activeOrganization } = user;
return new Promise((resolve, reject) => {
firebaseContext.db
.collection("organizations")
.doc(activeOrganization)
.get()
.then((snapshot) => {
if (!snapshot.empty) {
const members = snapshot.data().members;
const memberProfiles = [];
members.forEach((m) => {
firebaseContext.db
.collection("users")
.doc(m.uid)
.get()
.then((snapshot) => {
const profile = snapshot.data();
memberProfiles.push(profile);
console.log("person added");
console.log(memberProfiles);
if (members.length === memberProfiles.length) {
resolve(memberProfiles);
}
})
.catch((err) => reject(err));
});
} else {
reject([]);
}
})
.catch((err) => reject(err));
});
};

Answer 1

我建议使用async/await语法来清理此问题。此外，您可以将代码映射到promise和await的数组，而不是在forEach:中进行检查

export const getTeamMembers = async (firebaseContext, user) => {
try {
const { activeOrganization } = user;
const snapshot = firebaseContext.db
.collection("organizations")
.doc(activeOrganization)
.get();
const members = snapshot.data().members;
const memberPromises = members.map(m => {
return firebaseContext.db
.collection("users")
.doc(m.uid)
.get();
});
return (await Promise.all(memberPromises)).map(s => s.data());
} catch (e) {
// throw some error to cause promise to reject
}
};

Answer 2

你可以试试这个

声明const memberProfiles = []; 后

let membersPromises = [];
members.forEach((m) => { 
membersPromises.push(firebaseContext.db .collection("users") .doc(m.uid) .get());
Promise.all(membersPromises).then(profiles=>{
profiles.forEach(profile=>{
memberProfiles.push(profile.data());
});

resolve(memberProfiles);
}).catch((err) => reject(err)); });