在forEach循环中调用MongoDB updateMany无报错但数据未保存如何解决

问题原因 #

你的代码有两个核心问题导致更新不生效：

1. 异步操作未等待执行
   Group.updateMany是异步方法，返回Promise对象，你没有加await等待执行就直接返回了响应；且forEach原生不支持async/await的等待逻辑，程序走到响应返回的时候，数据库更新操作甚至还没开始执行，自然看不到数据变化，也不会触发报错。
2. 循环调用更新的方式完全没必要，不仅性能差，还容易出现事务、时序类的问题。

修复方案 #

推荐直接用MongoDB自带的$push + $each语法，一次性批量插入所有元素，无需循环：

const addContactsToGroup = async (req, res) => {
  try {
    const { groupId, selectedRows } = req.body
    const group = await Group.findById(groupId)
    // 过滤已存在的联系人
    const Matches = selectedRows.filter(row => {
      return !group.contacts.some(groupContacts => groupContacts === row)
    })
    // 单个目标文档用updateOne即可，不需要用updateMany
    await Group.updateOne(
      { '_id': groupId },
      // $each参数可以一次性插入数组内所有元素
      { $push: { contacts: { $each: Matches } } }
    )
    res.status(201).json({message: "Successfully updated"})
  } catch (err) {
    // 新增错误捕获，方便后续排查问题
    console.error('更新分组联系人失败：', err)
    res.status(500).json({message: "更新失败", error: err.message})
  }
}

额外注意事项 #

• 如果你坚持要循环执行更新（非常不推荐），需要把forEach替换为支持异步等待的for...of语法，且每次更新都加await：

// 仅作错误修正演示，不推荐生产使用
for (const match of Matches) {
  await Group.updateOne({ '_id': groupId}, {$push: {contacts : match}})
}

• 如果查询时匹配不到文档，检查_id类型是否一致：如果数据库存储的_id是ObjectId类型，传入的groupId是字符串的话，需要先用mongoose.Types.ObjectId(groupId)做类型转换再查询。

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