Laravel更新模型记录时如何保留默认图片避免误删

需求回顾 #

• 上传新图片场景
  • 当前关联图片非默认图images/default.png：删除旧图后上传新图完成关联
  • 当前关联图片为默认图：不得删除默认图，直接上传新图关联即可
• 未上传新图片场景：保留原有图片（含默认图），正常更新其余字段

原代码问题 #

• 条件判断使用||逻辑运算符，逻辑完全错误：只要旧图不是默认图，哪怕用户没上传新图也会进入分支执行删除，后续调用空文件对象的方法会直接报错
• 未校验文件是否真实上传、是否合法，未上传文件时执行文件处理逻辑会触发致命错误
• 图片上传逻辑重复冗余
• 删除旧文件前未做存在性校验，文件缺失时会触发unlink报错

正确实现代码 #

控制器update方法 #

public function update(StoreMovieRequest $request, Movie $movie)
{
    $validated = $request->validated();

    // 仅当存在合法上传的新图片时，才执行图片处理逻辑
    if ($request->hasFile('image_path') && $request->file('image_path')->isValid()) {
        // 旧图非默认图时，删除已存在的旧图文件
        if ($movie->image_path !== 'images/default.png') {
            $oldImageFullPath = public_path($movie->image_path);
            if (file_exists($oldImageFullPath)) {
                unlink($oldImageFullPath);
            }
        }

        // 存储新图片
        $uploadedImage = $request->file('image_path');
        $newImageName = date('YmdHis') . '.' . $uploadedImage->getClientOriginalExtension();
        $uploadedImage->move(public_path('images'), $newImageName);
        $validated['image_path'] = 'images/' . $newImageName;
    }

    $movie->update($validated);

    // 可根据自身路由调整跳转逻辑
    return redirect()->back()->with('status', '电影信息更新成功');
}

其余代码说明 #

现有数据库迁移、Movie模型、表单验证规则无需修改即可正常运行：

• 迁移中image_path字段默认值配置正确
• 模型$fillable已包含所需字段
• 验证规则中image_path未设置required规则，符合更新时可不上传图片的需求

逻辑说明 #

• 优先判断是否存在合法上传的文件，从入口避免无文件时执行图片处理逻辑的报错
• 删除旧文件前增加存在性校验，兼容旧文件意外丢失的场景
• 未上传新图片时，验证后的数据中不包含image_path字段，模型更新时会自动保留数据库中原有的图片路径，无需额外赋值
• 新图片存储路径拼接使用Laravel自带的public_path()助手函数，避免手动拼接斜杠导致的路径错误

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