Laravel 9 博客数据更新失败问题求助

核心问题：错误覆盖数据库模型实例 #

你的控制器代码存在致命逻辑错误：先通过Anime::findOrFail($id)获取到了要更新的数据库记录实例，但紧接着用$update_data = new Anime;创建了全新的空模型，直接覆盖了之前的实例。这导致后续所有赋值操作都针对新模型而非目标更新记录：

• 调用update()时，新模型未绑定任何数据库主键，执行失败并进入dd('Error')分支
• 调用save()时，新模型因无主键值会被当作新数据插入数据库，出现“新增而非更新”的情况

修正步骤与优化建议 #

1. 修复控制器核心逻辑 #

删除$update_data = new Anime;这行代码，直接对查询到的模型实例进行赋值：

public function update(ValidateAnimeBlogRequest $request, $id)
{
    $request->validated();

    // 获取要更新的模型实例（不要覆盖它）
    $update_data = Anime::findOrFail($id);

    // 直接对目标实例赋值
    $update_data->anime_title = $request->input('anime_title');
    $update_data->blog_title = $request->input('blog_title');
    $update_data->user_id = auth()->user()->id;
    $update_data->description = $request->input('description');
    $update_data->slug = Str::slug($request->input('blog_title'));

    // 处理图片更新逻辑
    if($request->hasFile('anime_image_profile')) {
        // 修正路径：用public_path确保路径准确性
        $old_image_path = public_path('images/anime_image_profile/' . $update_data->anime_image_profile);
        
        if(File::exists($old_image_path)) {
            File::delete($old_image_path);
        }

        // 修正$request->name的笔误（表单无name字段，改为blog_title）
        $new_file_name = '9anime-' . time() . '-' . $request->blog_title . '.' . $request->anime_image_profile->extension();
        
        $request->anime_image_profile->move(public_path('images/anime_image_profile'), $new_file_name);
        $update_data->anime_image_profile = $new_file_name;
    }

    // 执行更新并返回重定向（必须加return）
    if($update_data->save()) {
        return redirect('/');
    }
    
    dd('Error');
}

2. 修正验证规则：更新时图片字段不应强制必填 #

你的ValidateAnimeBlogRequest里将anime_image_profile设为required，但更新场景下用户可能不需要更换图片，改为可选规则：

public function rules()
{
    return [
        'anime_title' => ['required', 'min:2', new nameRegex],
        'blog_title' => ['required','min:5', new nameRegex],
        'description' => ['required','min:1000'],
        'premiered' => ['required'],
        'genre' => ['required', new nameRegex],
        'licensors' => ['required', new nameRegex],
        'studio' => ['required', new nameRegex],
        // 将required改为nullable，允许用户不提交图片
        'anime_image_profile' => 'nullable | mimes:jpeg,jpg,png | max:5408'
    ];
}

注：规则写法尽量统一用数组格式，避免空格导致的解析问题。

3. 补充：重定向必须使用return #

原代码里的redirect('/');未加return，即使更新成功也不会触发跳转，会继续执行到dd('Error')，务必加上return关键字。

额外细节说明 #

• Laravel模型的update()方法需要实例绑定数据库主键（即从数据库查询出的实例）才能执行更新；save()方法在模型有主键值时执行更新，无主键值时执行插入，这就是之前用save()会新增数据的原因。
• 处理文件路径时优先使用public_path()辅助函数，避免因环境差异导致的路径错误。

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