Laravel框架更新Product产品记录时异常创建新记录问题咨询

你的代码存在3个核心问题，按优先级修复即可解决异常：

核心问题说明 #

• 代码执行顺序完全错误
  你把「查询待更新产品实例」的逻辑放在了图片上传处理、参数验证之后，处理图片时用到的$product变量此时还未定义，如果你在代码其他位置提前实例化了新的Product对象来获取ID，就会直接产生一条新的产品记录。
  正确的逻辑顺序应为：先校验请求参数→再查询待更新的产品实例→最后处理图片上传、更新产品属性
• 模型更新方法调用错误
  你已经手动给$product实例的各个属性赋值，此时应该调用save()方法保存修改，而不是update()方法。update()方法是用于传入数组批量赋值的场景，如果你调用时没有传入参数且当前$product是未持久化的新实例，就会触发新增逻辑。
• 关联逻辑与表结构不匹配
  你当前有products、images、product_images三张表，说明Product和Image是多对多关联，images表本身不应该有product_id字段，你现在直接往Image模型写入product_id的逻辑不符合多对多关联的设计，应该在创建Image实例后用attach()方法关联到产品，写入中间表product_images。

修复后代码示例 #

public function updateproduct(Request $request, $id)
{
    // 第一步先做参数校验
    $validated = $request->validate([
        'name' => 'required',
        'description' => 'required',
        'images' => 'nullable',
        'price' => 'required|numeric',
        'quantity' => 'required|numeric',
    ]);

    // 第二步查询待更新的产品实例，找不到直接抛404
    $product = Product::findOrFail($id);

    // 第三步处理图片上传
    if($request->hasFile('images'))
    {
        foreach($request->file('images') as $file)
        {
            $name = time().'.'.$file->extension();
            if($file->move(public_path('/files/'), $name)){
                // 多对多关联使用这段逻辑
                $image = Image::create([
                    'images' => $name,
                ]);
                $product->images()->attach($image->id);

                // 若确认是一对多关联、images表存在product_id字段，替换为下面的代码
                // Image::create([
                //     'images' => $name,
                //     'product_id' => $product->id,
                // ]);
            }         
        }
    }

    // 第四步更新产品属性并保存
    $product->name = $validated['name'];
    $product->description = $validated['description'];
    $product->price = $validated['price'];
    $product->quantity = $validated['quantity'];   
    $product->save();

    return redirect()->back()->with('success', '产品更新成功');
}

补充检查项 #

• 检查路由定义，更新接口需要用PUT或PATCH方法注册，不要和新增产品的POST路由冲突
• 检查Product模型的$fillable属性是否包含name、description、price、quantity四个字段
• 如果自定义了Product模型的主键，需要在模型内声明protected $primaryKey = '你的主键字段名';，否则findOrFail会默认用id字段查询，找不到对应记录的情况下也会触发新增逻辑

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