如何根据店铺ID和商品ID更新Shop模型products数组内的指定商品

错误点说明 #

1. 你的写法存在两个核心逻辑问题：要操作Shop集合的products数组，却去查询Product集合，方向完全错误；且不能链式调用两次find查询，不符合mongoose语法规范。
2. 你提供的Shop模型中products字段定义存在语法错误，根据你的业务场景，通常有两种正确定义方式：

// 方案A：products存储关联的商品ID数组
const Shop = mongoose.Schema({
  _id: mongoose.Schema.Types.ObjectId,
  shop_name: { type: String },
  products: [mongoose.Schema.Types.ObjectId],
});

// 方案B：products存储嵌套的商品子文档
const Shop = mongoose.Schema({
  _id: mongoose.Schema.Types.ObjectId,
  shop_name: { type: String },
  products: [Product.schema],
});

对应解决方案 #

场景1：Shop的products是商品ID数组，需要更新对应商品的详情

直接操作Product集合更新即可：

// 入参：shopId=店铺ID，productId=商品ID，updateFields=要更新的字段对象
const updatedProduct = await Product.findOneAndUpdate(
  { _id: productId, shop: shopId },
  { $set: updateFields },
  { new: true } // 配置项：返回更新后的文档
);

场景2：Shop的products是嵌套商品子文档，需要更新Shop内的子文档内容

原子更新方案（无需先查询，性能更高）

使用MongoDB位置操作符$定位数组匹配项：

// 入参：shopId=店铺ID，productId=商品ID，updateFields=要更新的子文档字段对象
const updateParam = {}
Object.keys(updateFields).forEach(key => {
  updateParam[`products.$.${key}`] = updateFields[key]
})
const updatedShop = await Shop.findOneAndUpdate(
  { _id: shopId, "products._id": productId },
  { $set: updateParam },
  { new: true }
);

先查后改方案（适合需要先做逻辑判断的场景）

// 查询对应店铺
const shop = await Shop.findById(shopId);
// 定位目标商品
const targetProduct = shop.products.find(item => item._id.toString() === productId.toString());
if (!targetProduct) throw new Error('目标商品不存在')
// 自定义修改逻辑
targetProduct.title = '新商品标题'
targetProduct.description = '新商品描述'
// 保存修改
await shop.save()

内容的提问来源于stack exchange，提问作者murat.karaca