Mongoose-Express服务器更新对象数组指定索引的问题

问题代码中的错误点 #

• User.find()返回的是匹配文档的数组，直接执行equipment.coins=50是给数组对象添加属性，而非修改目标用户文档的coins字段
• 错误直接访问equipments.purchased：equipments是数组类型，必须通过索引（equipments[0]）定位到第一个元素后，才能修改其purchased属性
• 未处理查询阶段的错误，也未判断是否找到目标文档，导致无反应时无法排查原因

修正方案1：修正原查询后修改的写法 #

router.put("/update/:identifier", function(req, res) {
  User.find({ identifier: req.params.identifier }, function(err, users) {
    // 处理查询错误
    if (err) {
      return res.status(400).send({
        message: errorHandler.getErrorMessage(err)
      });
    }
    // 判断是否找到目标用户
    if (!users || users.length === 0) {
      return res.status(404).send({
        message: "未找到指定用户"
      });
    }
    const targetUser = users[0];
    // 更新coins数值
    targetUser.coins = 50;
    // 更新equipments数组第一个元素的purchased属性
    if (targetUser.equipments && targetUser.equipments.length > 0) {
      targetUser.equipments[0].purchased = true;
    }
    // 保存修改
    targetUser.save(function(err) {
      if (err) {
        return res.status(400).send({
          message: errorHandler.getErrorMessage(err)
        });
      } else {
        res.json(targetUser);
      }
    });
  });
});

修正方案2：推荐使用findOneAndUpdate（更高效） #

直接通过MongoDB更新操作符在数据库层面修改，减少交互次数，避免并发覆盖问题：

router.put("/update/:identifier", function(req, res) {
  User.findOneAndUpdate(
    { identifier: req.params.identifier },
    {
      $set: {
        coins: 50,
        "equipments.0.purchased": true // 用点语法定位数组第一个元素的属性
      }
    },
    { new: true } // 返回更新后的文档，而非更新前的旧文档
  )
  .then(updatedUser => {
    if (!updatedUser) {
      return res.status(404).send({ message: "未找到指定用户" });
    }
    res.json(updatedUser);
  })
  .catch(err => {
    res.status(400).send({ message: errorHandler.getErrorMessage(err) });
  });
});

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