Array .push 在 Promise 函数中不起作用

Question

我有一个这样的 nodejs Mongodb 代码。我试图推入一个全局数组并返回我所有的结果，但该数组只包含一个值。知道为什么吗？

var s = [];
var p = function(){
return new Promise(function(resolve, reject) {
    MongoClient.connect(url, function (err, db) {

   db.listCollections().forEach(function(collname) {

   db.collection(collname.name).find().sort( { duration_in_seconds: 1 }).limit(1).toArray(

  function(err, docs) {

        assert.equal(null, err);
        s.push(docs);
        resolve();
      });


});

 });

});

}


p().then(function() {

  console.log(s);

});

Answer 1

当第一个集合返回其文档时，您正在解决承诺。您需要等待所有这些。而不是将所有内容都包装在一个大的 new Promise、promisify 每个异步函数中，并让返回的 Promise 实现 with 结果值，而不是将其放入全局变量中：

function connect(url) {
    return new Promise(function(resolve, reject) {
        MongoClient.connect(url, function (err, db) {
            if (err) reject(err);
            else resolve(db);
        });
    });
}
function getArray(cursor) {
    return new Promise(function(resolve, reject) {
        cursor.toArray(function(err, docs) {
            if (err) reject(err);
            else resolve(docs);
        });
    });
}

现在您可以使用这些帮助程序编写您的 p，将每个集合的 Promise 放入一个数组中，然后使用 Promise.all 等待它们，这会为所有结果的数组生成一个 Promise：

function p() {
    return connect(url).then(function(db) {
        return getArray(db.listCollections()).then(function(collections) {
            var promises = collections.map(function(collname) {
                return getArray(db.collection(collname.name).find().sort({duration_in_seconds: 1 }).limit(1));
            });
            return Promise.all(promises);
        });
    });
}

p().then(function(s) {
    console.log(s);
});