MongoDB购物车添加商品报错：Cannot read properties of undefined (reading 'find')

错误根源 #

你遇到的报错核心原因是：

1. 使用Carts.find(req.session.cartId)查询购物车时，Mongoose的find()方法返回的是文档数组而非单个文档。如果数据库中无对应ID的购物车，cart会是空数组，此时访问cart.items会因数组无items属性抛出错误。
2. 代码还存在几处逻辑漏洞，会导致后续功能失效。

分步修复方案 #

1. 修正购物车查询方法

将Carts.find()替换为Carts.findById()，这是Mongoose中根据_id查询单个文档的标准方法：

// 原错误代码
cart = await Carts.find(req.session.cartId);
// 修正后
cart = await Carts.findById(req.session.cartId);

2. 处理查询不到购物车的边界情况

即使存在req.session.cartId，也可能出现session残留但数据库数据已被删除的情况，需补充判断重新创建购物车：

else {
  cart = await Carts.findById(req.session.cartId);
  // 若查询不到，重新创建购物车并更新session
  if (!cart) {
    cart = await Carts.create({ items: [] });
    req.session.cartId = cart._id;
  }
}

3. 修复商品数量递增逻辑

原代码中existingItem++错误，existingItem是包含quantity属性的对象，需递增对应属性：

// 原错误代码
if (existingItem) {
  existingItem++;
}
// 修正后
if (existingItem) {
  existingItem.quantity++;
}

4. 统一商品ID字段名

Schema中商品项用的是_id，但push时用的是id，字段不统一会导致匹配失败，需修正：

// 原错误代码
cart.items.push({ id: req.body.productId, quantity: 1 });
// 修正后
cart.items.push({ _id: req.body.productId, quantity: 1 });

5. 优化购物车更新方式

使用Mongoose文档自带的save()方法，比updateOne()更直观且能自动处理字段验证：

// 原错误代码
await Carts.updateOne(cart._id, {
  items: cart.items
});
// 修正后
await cart.save();

完整修复后的路由代码 #

const express = require('express');
const Carts = require('../repo/carts');

const router = express.Router();

router.post('/cart/products', async (req, res) => {
  let cart;

  if (!req.session.cartId) {
    cart = await Carts.create({ items: [] });
    req.session.cartId = cart._id;
  } else {
    cart = await Carts.findById(req.session.cartId);
    if (!cart) {
      cart = await Carts.create({ items: [] });
      req.session.cartId = cart._id;
    }
  }

  // 查找购物车中已存在的商品
  const existingItem = cart.items.find(item => item._id === req.body.productId);
  if (existingItem) {
    existingItem.quantity++;
  } else {
    cart.items.push({ _id: req.body.productId, quantity: 1 });
  } 

  await cart.save();
  res.send('product added to cart!!');
});

module.exports = router;

额外建议 #

• Schema中无需手动指定_id: String，Mongoose会自动生成ObjectId类型的_id；若需自定义字符串ID，确保生成逻辑统一。
• 添加错误捕获逻辑，避免未处理的异步错误导致服务器崩溃：

router.post('/cart/products', async (req, res) => {
  try {
    // 现有逻辑...
  } catch (err) {
    console.error(err);
    res.status(500).send('添加商品到购物车失败');
  }
});

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