Mongoose更新集合需执行2次才能生效的问题排查求助

问题根本原因 #

1. findByIdAndUpdate 默认返回更新前的文档：调用该方法时没有加{ new: true }配置项，第一次执行时拿到的category对象是修改前的旧数据，用旧的分类名去更新Item自然不会有任何变化。第二次执行时数据库里的分类已经被第一次操作更新了，此时拿到的category对象才是更新后的新数据，所以第二次才会生效。
2. 代码存在拼写错误：遍历Item时声明的遍历变量是item，但保存时写的是await single.save()，single是未定义变量，会导致保存逻辑执行异常。
3. 冗余操作：findByIdAndUpdate已经直接将修改写入数据库，后续调用category.save()属于完全多余的操作，没有任何作用。
4. 性能问题：全量查询所有Item再遍历判断属于极低效的写法，数据量大时会严重拖慢接口速度。

优化建议 #

从数据结构设计层面来看，更合理的方案是Item存储分类的_id而非分类名称，修改分类名称时根本不需要更新Item，查询时做关联查询即可，从根源避免这类同步问题。如果需要保留现有存储逻辑，可使用下方修复后的代码：

正确修复代码 #

module.exports.updateCtg = async (req, res) => {
  try {
    const { id } = req.params;
    // 新增new: true配置，直接返回更新后的分类对象
    const updatedCategory = await CategoryModel.findByIdAndUpdate(
      id,
      { ...req.body.category },
      { new: true }
    );
    // 直接用updateMany批量更新符合条件的Item，不需要遍历全表
    await ItemModel.updateMany(
      { facility: updatedCategory.facility },
      { $set: { category: updatedCategory.name } }
    );
    res.render('dashboard/ctgview', { category: updatedCategory._id });
  } catch (err) {
    // 新增错误处理逻辑，避免服务无响应崩溃
    console.error('更新分类失败：', err);
    res.status(500).send('更新分类操作失败');
  }
}

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