Mongoose findOneAndUpdate返回更新文档但数据库未更新问题求助

你遇到的这个问题，核心原因是嵌套异步操作的闭包陷阱，加上没有正确控制异步流程的执行顺序，导致更新操作没有按预期作用到目标文档上。

具体原因 #

你的代码里用var声明循环变量（d和i），而var是函数级作用域的变量，所有异步回调会共享同一个变量引用。当Vote.find和findOneAndUpdate的回调执行时，循环已经跑完了，变量已经变成循环结束后的数值——哪怕控制台输出看起来正确，也只是回调捕获到了最后一次循环的变量值，实际操作的可能根本不是你预期的那个文档。

另外，所有数据库操作都是异步并行执行的，没有等待前一个操作完成就开始下一个，这也可能引发意想不到的竞态条件。

修复后的代码 #

我们用let替代var解决作用域问题，再用async/await理顺异步流程，让代码更易读且确保操作按顺序执行：

async function processDelegations(propId) {
  try {
    const voteNumber = await Vote.count({ propId, delegation: true });
    const maxDelegation = voteNumber;

    // 用let声明循环变量，确保每次迭代的作用域独立
    for (let d = 1; d <= maxDelegation; d++) {
      const specWeightVotes = await Vote.find({ propId, delegation: true, weight: d });
      
      for (let i = 0; i < specWeightVotes.length; i++) {
        const currentVote = specWeightVotes[i];
        console.log(currentVote.content);

        // 给被委托者增加对应权重
        await Vote.findOneAndUpdate(
          { propId, voter: currentVote.content },
          { $inc: { weight: currentVote.weight } }
        );

        // 将委托者的权重设为0，同时启用验证确保字段合法
        const updatedVote = await Vote.findOneAndUpdate(
          { _id: currentVote._id },
          { $set: { weight: 0 } },
          { new: true, runValidators: true }
        );

        console.log("更新后的委托者文档：", updatedVote);
        // 检查是否真的匹配并更新了文档
        if (!updatedVote) {
          console.warn(`未找到ID为${currentVote._id}的投票文档`);
        }
      }
    }

    console.log("所有委托投票处理完成");
  } catch (err) {
    console.error("处理委托过程中出错：", err);
  }
}

// 调用处理函数
processDelegations(propId);

额外注意事项 #

1. 优先用let/const替代var：let是块级作用域，每次循环迭代都会创建新的变量实例，彻底避免闭包陷阱。
2. 用async/await管理异步流程：既避免回调地狱，又能确保每个数据库操作完成后再执行下一个，消除竞态条件。
3. 启用runValidators选项：确保更新操作符合Mongoose模型的字段验证规则（比如weight必须是数字类型），如果模型定义有误，这个选项会帮你抛出明确的错误。
4. 完善错误处理：用try/catch捕获所有异步操作的错误，方便快速定位问题。
5. 检查更新结果：如果findOneAndUpdate返回null，说明没有匹配到目标文档，这时候可以针对性排查文档ID是否正确、查询条件是否有误。

内容的提问来源于stack exchange，提问作者Adrien Merlier