forEachループ内での非同期findOne（）操作

Question

JavaScriptの約束を理解するのに苦労しています。私は特定の条件を満たすオブジェクトのMongooseモデルを検索しています。存在する場合は、そのオブジェクトをプレーンなJSオブジェクトにしてプロパティを追加したいと考えています。

残念ながら、私の約束が解決し終わる前に私のforEachループが完全に実行されることを確実にするために頭を抱えることができません。私のコードを見てください。

// Called to check whether a user has participated in a given list of challenges 
participationSchema.statics.getParticipation = function(user, challenges) { 
    return new Promise((resolve, reject) => { 
    challengesArray = []; 
    challenges.forEach((challenge) => { 
     // Model#findOne() is Async--how to ensure all these complete before promise is resolved? 
     Participation.findOne({user, challenge}) 
     .then((res) => { 
     if (res) { 
      var leanObj = challenge.toObject(); 
      leanObj.participation = true; 
      challengesArray.push(leanObj); 
     } 
     }) 
     .catch(e => reject(e)); 
    }) 
    console.log("CHALLENGES ARRAY", challengesArray); // Challenges Array empty :(
    resolve(challengesArray); 
    }); 
} 

私は同様の質問を見てきましたが、答えに達することができませんでした。ヘルプをよろしくお願いいたします。

Answer 1

だから、何あなたがgetParticipationを呼び出すときに起こっていることはforEach心と体がすべての方法を実行し、Participation.findOneのためのすべての個々の約束がが作成されますが、まだを解決していないということです。実行は解決するのを待つことなく、forEachの後に続き、トップレベルの約束challengesArrayを解決します。その時点でまだ空です。いつの間にか、forEachで作成された約束は解決し始めますが、結果は失われます。

また、Bergiがコメントで述べたように、入れ子の約束はanti-patternとみなされます。約束は、ネストされているのではなく、連鎖されるべきである。

Promise.allのようなものを使用して、すべての約束が終わるのを待ってから、存在しないすべての結果を除外して最後に配列を返すことをお勧めします。

participationSchema.statics.getParticipation = function(user, challenges) { 
    return Promise.all(challenges.map(challenge => { 
    return Participation.findOne({user, challenge}).then(result => { 
     if (result) { 
     var leanObj = challenge.toObject(); 
     leanObj.participation = true; 
     return leanObj; 
     } 
    }); 
    }) 
    // at this point, results contains an array of `leanObject` and `undefined` depending if the `findOne` call returned anything and the code withing the `if` above was run 
    .then((results) => { 
    return results.filter(result => !!result) // filter out `undefined` results so we only end up with lean objects 
    }); 
}