MongoDB调用save()后User文档未更新的问题求助

嘿，我遇到过一模一样的坑！这其实是Mongoose变更追踪机制的小细节问题，我来给你拆解下原因和解决方案：

问题根源 #

你通过forEach修改数组里todo的completed属性时，Mongoose默认不会深度监听数组内嵌套对象的属性变化。也就是说，虽然你在内存里改了这个值，但Mongoose没察觉到todos数组有变更，调用save()时不会把这个更新同步到MongoDB——这就是为什么重启服务器后数据库里的数据没变化的原因。

解决方案（三种方式，按改动成本排序） #

1. 最小改动：手动标记数组为已修改

在修改完todo属性后，调用user.markModified('todos')告诉Mongoose：“这个数组我改了，记得同步到数据库”。修改后的代码如下：

User.findById(req.user._id)
 .then(user => {
 user.todos.forEach(function(todo) {
 if (todo._id == req.params.todoId) {
 todo.completed = !todo.completed;
 console.log(todo.completed);
 }
 })
 user.markModified('todos'); // 关键新增的一行
 return user.save();
 })
 .then(result => {
 console.log(result);
 res.json(result);
 })
 .catch(function(err) {
 res.send(err);
 })

这个方法几乎不用调整现有代码，加一行就能解决问题。

2. 更高效：直接用数据库更新操作（推荐）

上面的方法需要先把整个用户文档拉到内存再修改，其实可以用Mongoose的findOneAndUpdate直接在数据库层面修改指定todo项，减少一次数据库查询，还能避免并发修改的问题：

User.findOneAndUpdate(
 // 查询条件：找到当前用户，且todos数组包含指定id的项
 { _id: req.user._id, 'todos._id': req.params.todoId },
 // 更新操作：把匹配到的todo项的completed取反（MongoDB 4.2+支持$expr语法）
 { $set: { 'todos.$.completed': { $expr: { $not: '$todos.$.completed' } } } },
 // 配置：返回更新后的最新文档
 { new: true }
)
.then(result => {
 console.log(result);
 res.json(result);
})
.catch(err => res.send(err));

这里的$是MongoDB的位置操作符，用来匹配数组中第一个符合条件的元素，非常适合修改数组里的指定项。

3. 精准定位：通过索引修改数组元素

如果你不想遍历整个数组，可以用findIndex找到目标todo的位置，直接修改并标记：

User.findById(req.user._id)
 .then(user => {
 // 找到目标todo的索引（注意要把_id转成字符串比较）
 const todoIndex = user.todos.findIndex(todo => todo._id.toString() === req.params.todoId);
 if (todoIndex !== -1) {
 user.todos[todoIndex].completed = !user.todos[todoIndex].completed;
 user.markModified('todos');
 }
 return user.save();
 })
 .then(result => {
 console.log(result);
 res.json(result);
 })
 .catch(function(err) {
 res.send(err);
 })

这种方法比forEach更高效，尤其是当todos数组很大的时候。

总结 #

如果只是临时解决问题，第一种方法最快捷；如果要优化代码性能和稳定性，第二种方法是最优选择——直接操作数据库比先拉取再修改高效得多，还能避免潜在的并发冲突。

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