forEach 循环中的异步 findOne() 操作

Question

我很难理解 JavaScript Promises。我正在我的 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 中创建的 Promise 开始解析，但它们的结果现在丢失了。

另外，正如 Bergi 在 cmets 中提到的，嵌套 promise 被认为是 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
  });
}