Есть ли более простой способ перевести обещания из результатов MongoDB?

Question

В этой функции перечислены все коллекции в базе данных MongoDB с количеством документов в каждой коллекции (bluebird).

function listMongoCollections(db) { 
    var promises = [] 
    db.listCollections().toArray().then((docs) => { 

     docs.forEach((doc) => { 

      promises.push(
       new Promise((resolve) => { 
        db.collection(doc.name).count().then((count) => { 
         doc.count = count 
         resolve() 
        }) 
       }) 
      ) 
     }) 

     return Promise.all(promises) 
    }) 
} 

Есть ли более простой способ сделать это? Используя этот метод, вы наполните приложение кодом, и я даже не включил обработку ошибок.

Answer 1

Вы можете сделать несколько вещей:

• использования .map для преобразования массива коллекций в массив обещаний, вместо того, чтобы .push ИНГОВ в массив вручную.
• избегать обертывания обещаний в новых обещаниях.
• гнездо .then как можно меньше. В следующем примере обратите внимание, как первая часть просто возвращает массив обещаний. Только перед возвращением основной функции мы завершаем ее в обещании .all.

function listMongoCollections(db) { 
    const docs = db.listCollections().toArray().then(docs => { 
     return docs.map(doc => { 
      return db.collection(doc.name).count().then(count => { 
       doc.count = count 
       return doc; 
      }); 
     }); 
    }); 
    return Promise.all(docs); 
} 

Вы можете «упростить» его дальше, удалив return с и промежуточной docs постоянной.

function listMongoCollections(db) { 
    return Promise.all(
     db.listCollections().toArray().then(docs => 
      docs.map(doc => 
       db.collection(doc.name).count().then(count => { 
        doc.count = count 
        return doc; 
       }) 
      ) 
     ) 
    ); 
} 

И, возможно, с async/await мы можем сделать его еще более удобным для чтения (хотя я не слишком хорошо знакомы с ним):

async function listMongoCollections(db) { 
    let docs = await db.listCollections().toArray(); 
    docs = docs.map(async doc => { 
     doc.count = await db.collection(doc.name).count(); 
     return doc; 
    }); 
    return Promise.all(docs); 
} 

Это только примеры, Монго может предложить даже лучшие решения.