Express商品评论开发遇类型转换及验证错误，需实现重复评论更新

核心问题分析 #

1. 类型不匹配导致Cast错误：你的评论Schema中user字段定义为Number类型，但req.user.id是MongoDB生成的ObjectId（字符串格式），存入时会触发类型转换失败。
2. 未实现“更新旧评论”逻辑：当前代码直接向数组push新评论，完全没处理用户已评论的场景。
3. 缺失基础校验与错误处理：未判断商品/用户是否存在，也没有捕获异常分支。

步骤1：修正评论Schema #

将user字段类型改为匹配MongoDB用户ID的ObjectId，同时可关联User模型提升数据规范性：

const mongoose = require('mongoose');

// 修正后的Schema片段
review: [
  {
    user: { type: mongoose.Schema.Types.ObjectId, ref: 'User', required: true },
    name: { type: String, required: true },
    comment: { type: String, required: true }
  }
]

步骤2：修复路由逻辑（实现更新/新增二选一） #

router.post("/review", verifyTokenAndAuthorization, async (req, res) => {
  try {
    const { comment, productId } = req.body;
    // 基础参数校验
    if (!comment || !productId) {
      return res.status(400).json({ message: "评论内容和商品ID不能为空" });
    }

    // 查询当前用户（简化写法，同时处理用户不存在的情况）
    const user = await User.findById(req.user.id);
    if (!user) {
      return res.status(404).json({ message: "用户不存在" });
    }

    // 查询目标商品
    const product = await Product.findById(productId);
    if (!product) {
      return res.status(404).json({ message: "商品不存在" });
    }

    // 检查当前用户是否已有评论（ObjectId需转字符串对比）
    const existingReviewIndex = product.review.findIndex(
      (r) => r.user.toString() === req.user.id.toString()
    );

    if (existingReviewIndex !== -1) {
      // 存在旧评论：更新内容（可选同步最新用户名）
      product.review[existingReviewIndex].comment = comment;
      product.review[existingReviewIndex].name = user.username;
    } else {
      // 无旧评论：新增评论
      const newReview = {
        user: req.user.id,
        name: user.username,
        comment
      };
      product.review.push(newReview);
    }

    // 保存修改并返回结果
    const updatedProduct = await product.save();
    res.status(200).json(updatedProduct);
  } catch (err) {
    // 捕获类型转换、验证等所有异常
    res.status(500).json({ message: "评论处理失败", error: err.message });
  }
});

关键修复点说明 #

• 类型匹配：将Schema的user字段改为ObjectId，彻底解决Cast转换错误。
• 逻辑实现：通过findIndex定位用户已有评论，存在则更新、不存在则新增，符合需求。
• 健壮性提升：添加参数校验、用户/商品存在性判断，以及全局异常捕获，避免崩溃并返回明确错误信息。

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