在我的诺言/功能回来之前等待一个forEach完成

Question

我使用的是Firebase Cloud Firestore，然而，我认为这可能更像是JavaScript异步与同步promise返回的问题。

我正在执行一个查询以从一个集合中获取ID，然后循环该查询的结果以根据该ID从另一个集合中查找单个记录。

我怎么能等到forEach完成后才从外部promise和外部函数本身返回呢？

         getFacultyFavoritesFirebase() {
            var dbRef = db.collection("users").doc(global.user_id).collection("favorites");
            var dbQuery = dbRef.where("type", "==", "faculty");
            var dbPromise = dbQuery.get();
            var results = [];
            return dbPromise.then(function(querySnapshot) {
                querySnapshot.forEach(function(doc) {
                  var docRef = db.collection("faculty").doc(doc.id);
                  docRef.get().then(function(doc) {
                    if (doc.exists) {
                        results.push(doc);
                    }
                  })
                });
                console.log(results.length);
                return results;
            })
            .catch(function(error) {
                console.log("Error getting documents: ", error);
            });
          }

Answer 1

这里的诀窍是用promise而不是结果填充结果。然后可以对该promise数组调用promise.all()并获得所需的结果。当然，在推动promise之前不能检查是否doc.exists，所以在promise.all()解析后需要处理这个问题。例如：

function getFacultyFavoritesFirebase() {
    var dbRef = db.collection("users").doc(global.user_id).collection("favorites");
    var dbQuery = dbRef.where("type", "==", "faculty");
    var dbPromise = dbQuery.get();
    // return the main promise
    return dbPromise.then(function(querySnapshot) {
        var results = [];
        querySnapshot.forEach(function(doc) {
            var docRef = db.collection("faculty").doc(doc.id);
            // push promise from get into results
            results.push(docRef.get())
        });
        // dbPromise.then() resolves to  a single promise that resolves 
        // once all results have resolved
        return Promise.all(results)
    })
    .catch(function(error) {
        console.log("Error getting documents: ", error);
    });
}

getFacultyFavoritesFirebase
.then(results => {
    // use results array here and check for .exists
}