如何使用Mongoose更新用户文档中数组内的Todo项？

看起来你遇到了典型的MongoDB嵌套数组更新不同步的问题，我来帮你拆解原因并给出可行的解决方案：

问题分析 #

1. 你的第一种方法只更新了独立的Todo集合，但用户文档里的todos数组是单独存储的副本，自然不会自动同步变化；
2. 第二种方法虽然找到了用户并遍历修改了todos数组，但你忘记将修改后的用户对象保存回数据库，而且直接用==比较ObjectId类型的_id也可能存在隐式转换的问题。

解决方案 #

方案1：用MongoDB数组更新操作符直接修改（推荐） #

MongoDB提供了专门的数组操作符，可以直接定位嵌套数组中的元素进行更新，无需先查询整个用户文档，效率更高：

// 直接更新用户文档中指定的todo项
User.findOneAndUpdate(
  // 查询条件：匹配用户ID，同时匹配todos数组中对应todo的ID
  { _id: req.user._id, "todos._id": req.params.todoId },
  // 更新操作：用$占位符定位匹配到的数组元素，修改completed字段
  { $set: { "todos.$.completed": !req.body.completed } },
  // 配置项：返回更新后的文档
  { new: true }
)
.then(updatedUser => {
  // 从更新后的用户文档中找到对应的todo返回
  const updatedTodo = updatedUser.todos.find(todo => todo._id.toString() === req.params.todoId);
  res.json(updatedTodo);
})
.catch(err => res.send(err));

• 这里的"todos.$"是MongoDB的特殊占位符，代表查询条件中匹配到的todos数组元素的位置；
• 这种方式只需要一次数据库操作，比先查后改更高效。

方案2：先查询用户再修改（适合复杂逻辑场景） #

如果你需要在更新前做一些额外的业务逻辑处理，可以先查询用户文档，修改后再保存：

User.findById(req.user._id)
.then(user => {
  // 用find方法定位目标todo，注意用equals()比较ObjectId
  const targetTodo = user.todos.find(todo => todo._id.equals(req.params.todoId));
  
  if (!targetTodo) {
    return Promise.reject(new Error("指定的Todo不存在"));
  }
  
  // 修改todo状态
  targetTodo.completed = !targetTodo.completed;
  // 必须调用save()才能将改动持久化到数据库！
  return user.save();
})
.then(updatedUser => {
  const updatedTodo = updatedUser.todos.find(todo => todo._id.toString() === req.params.todoId);
  res.json(updatedTodo);
})
.catch(err => res.send(err));

• 重点：修改内存中的对象后，一定要调用user.save()才能把变化写入数据库；
• 用todo._id.equals()代替==，避免ObjectId和字符串比较时的隐式转换问题。

方案3：自动同步Todo集合与用户数组（可选） #

如果希望修改独立Todo集合时自动同步用户文档中的todos数组，可以在Todo模型中添加post钩子：

// 在Todo模型中添加更新后的钩子
Todo.post('findOneAndUpdate', function(updatedTodo) {
  // 自动更新所有包含该todo的用户文档
  User.updateMany(
    { "todos._id": updatedTodo._id },
    { $set: { "todos.$": updatedTodo } }
  ).catch(err => console.error("同步用户todos数组失败：", err));
});

这样你第一次写的更新Todo集合的代码就能自动同步用户数组了，但要注意这种方式可能会有性能开销，适合数据量不大的场景。

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