如何使用Mongoose修改Inventory中指定物品的amount并保存至MongoDB

原代码存在的问题 #

1. 变量顺序错误：先计算newAmount再查找picked，会触发picked is not defined的报错
2. 错误替换整个数组：用$set: { Inventory: newItem }会直接覆盖整个Inventory数组，导致"Sword"丢失
3. 重复操作：同时调用findOneAndUpdate和user.save()，可能引发数据冲突
4. 类型不一致："Sword"的amount是字符串类型，后续计算容易出现异常（比如字符串拼接而非数值相加）

可行解决方案 #

方案一：直接在数据库层面精准更新（推荐）

利用MongoDB的$inc操作符结合arrayFilters，直接定位数组中指定元素并更新，无需拉取整个文档，性能更优：

User.findOneAndUpdate(
  { ID: interaction.user.id },
  {
    // 对name为Axe的元素，amount加1
    $inc: { "Inventory.$[item].amount": 1 },
    // 可选：统一Sword的amount为数字类型（修正原数据的类型问题）
    $set: { "Inventory.$[sword].amount": 2 }
  },
  {
    arrayFilters: [
      { "item.name": "Axe" },
      { "sword.name": "Sword" }
    ],
    upsert: true,
    useFindAndModify: false,
    new: true // 配置后返回更新后的文档，方便后续处理
  }
)
.then(updatedDoc => console.log("更新完成：", updatedDoc))
.catch(err => console.error("更新失败：", err));

方案二：先查询文档再修改数组（适合需要额外内存逻辑的场景）

如果需要先将文档拉取到内存做更多处理，需注意仅修改目标元素而非替换整个数组：

User.findOne({ ID: interaction.user.id })
.then(user => {
  if (!user) {
    // 若用户不存在，创建新文档（对应原upsert逻辑）
    return User.create({
      ID: interaction.user.id,
      XP: 0,
      Level: 1,
      Cash: 700,
      Bank: 1000,
      Inventory: [{ name: "Axe", amount: 1 }, { name: "Sword", amount: 2 }]
    });
  }

  // 定位Inventory中的Axe元素
  const axeItem = user.Inventory.find(item => item.name === "Axe");
  if (axeItem) {
    // 确保amount为数字类型后加1
    axeItem.amount = Number(axeItem.amount) + 1;
  } else {
    // 若Axe不存在，新增该元素
    user.Inventory.push({ name: "Axe", amount: 1 });
  }

  // 可选：修正Sword的amount类型为数字
  const swordItem = user.Inventory.find(item => item.name === "Sword");
  if (swordItem) swordItem.amount = Number(swordItem.amount);

  return user.save();
})
.then(updatedUser => console.log("更新完成：", updatedUser))
.catch(err => console.error("更新失败：", err));

注意事项 #

• 优先选择方案一，减少数据库交互次数，避免并发场景下的数据冲突
• 确保Inventory数组中所有元素的amount均为数字类型，避免后续计算异常
• 不要同时使用findOneAndUpdate和user.save()，二者选其一即可

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