Node.js+Mongoose用户资料编辑：空密码时graduated字段不更新求助

咱先捋清楚你遇到的核心坑点：你在findOneAndUpdate之后手动修改了user.graduated，但只有当密码不为空时才调用了user.save()，空密码的分支里啥都没做——这就导致graduated的修改只停留在内存里，根本没同步到数据库！另外代码里还有几个小细节需要调整，我给你一步步修复。

问题分析 #

1. graduated修改未持久化：你用findOneAndUpdate更新了一批字段后，手动修改了user.graduated，但空密码时没执行save()，这个修改直接丢了。
2. pre save钩子逻辑有漏洞：直接判断this.password会导致每次save()都重复加密已加密的密码，而且你写的console.log是语法错误（没加括号）。
3. 笔误问题：你代码里写的minor: req.body.mino应该是minor: req.body.minor，这个会导致专业辅修字段无法正确更新。

最优解决方案：一次性完成所有更新 #

推荐把所有要更新的字段都整合到findOneAndUpdate的参数里，不用后续手动修改再save()，效率更高也更不易出错：

修复后的编辑接口代码 #

router.post('/edit', isAuthenticated, function (req, res, next) {
  // 构建要更新的字段集合
  const updateData = {
    email: req.body.email,
    name: req.body.name,
    phone: req.body.phone,
    classc: req.body.classc,
    major: req.body.major,
    minor: req.body.minor, // 修复笔误：mino改为minor
    linkedin: req.body.linkedin,
    bio: req.body.bio,
    // 直接在这里处理graduated的逻辑，不用后续手动修改
    graduated: typeof req.body.graduated !== 'undefined' && req.body.graduated === 'string'
  };

  // 只有密码不为空时，才添加到更新列表中
  if (req.body.password) {
    updateData.password = req.body.password;
  }

  // 一次性完成所有字段更新
  User.findOneAndUpdate(
    { _id: req.user._id },
    { $set: updateData },
    { 
      new: true, // 返回更新后的用户对象
      runValidators: true // 确保更新时触发Schema的验证规则（如果有的话）
    },
    function (err, user) {
      if (err) {
        return next(err); // 把错误交给Express的错误处理中间件
      }
      res.redirect('/profile');
    }
  );
});

修复后的pre save钩子 #

userSchema.pre('save', function(next) {
  // 只有当密码字段被修改时才加密，避免重复加密已加密的密码
  if (this.isModified('password')) {
    this.password = bcrypt.hashSync(this.password, bcrypt.genSaltSync(8));
  }
  next();
});

备选方案：保留手动修改逻辑（不推荐） #

如果你坚持想用原来的手动修改方式，核心是不管密码是否为空，最后都要调用user.save()，把内存中的修改同步到数据库：

router.post('/edit', isAuthenticated, function (req, res, next) {
  User.findOneAndUpdate(
    { _id: req.user._id },
    { $set: {
        email: req.body.email,
        name: req.body.name,
        phone: req.body.phone,
        classc: req.body.classc,
        major: req.body.major,
        minor: req.body.minor,
        linkedin: req.body.linkedin,
        bio: req.body.bio
      } },
    { new: true },
    function (err, user) {
      if (err) {
        return next(err);
      }

      // 处理graduated字段
      user.graduated = typeof req.body.graduated !== 'undefined' && req.body.graduated === 'string';

      // 处理密码字段
      if (req.body.password) {
        user.password = req.body.password;
      }

      // 关键：不管密码是否为空，都要执行save！
      user.save(function(saveErr) {
        if (saveErr) {
          return next(saveErr);
        }
        res.redirect('/profile');
      });
    }
  );
});

核心总结 #

• 优先用findOneAndUpdate一次性更新所有字段，减少数据库操作次数，避免手动修改的遗漏。
• 如果要手动修改查询返回的对象，必须确保所有情况下都调用save()，否则内存中的修改不会同步到数据库。
• pre save钩子要判断密码字段是否被修改，避免重复加密已加密的密码。

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