Laravel 8中重构重复图片上传代码的最佳实践是什么

核心思路是把重复的上传、重命名、旧文件清理逻辑抽成统一的可复用单元，通过配置映射自动匹配不同上传位的文件名前缀，不需要为每个上传字段写重复的判断代码。

快速落地（控制器内复用，改造成本最低） #

适合当前仅单个控制器需要复用逻辑的场景，改造成本极低：

1. 先在控制器内维护统一的字段映射配置，把前端上传input的name值和对应的文件名前缀做绑定：

// 图片字段映射配置：key是前端传参的字段名，value是文件名对应的前缀
protected $vehicleImageMap = [
    'vehicleImageRight' => 'Right',
    'vehicleImageLeft' => 'Left',
    'vehicleImageFront' => 'Front',
    'vehicleImageBack' => 'Back',
    // 后续新增上传位只需要在这里加配置即可，比如'vehicleImagePanorama' => 'Panorama'
];

2. 在控制器内写一个私有通用方法，统一处理所有图片位的校验、旧文件删除、重命名、文件移动、模型赋值全流程：

/**
 * 统一处理车辆违章图片上传
 * @param \Illuminate\Http\Request $request 请求实例
 * @param mixed $violation 违章模型实例
 * @param string $uploadDir 图片存储目录路径
 * @return void
 */
private function processVehicleImages($request, $violation, string $uploadDir)
{
    foreach ($this->vehicleImageMap as $field => $prefix) {
        // 没有上传对应字段直接跳过
        if (!$request->hasFile($field)) {
            continue;
        }

        $file = $request->file($field);
        // 先删除当前字段对应的旧文件，避免冗余文件占存储
        $oldFileFullPath = $uploadDir . DIRECTORY_SEPARATOR . $violation->$field;
        if ($violation->$field && \File::exists($oldFileFullPath)) {
            \File::delete($oldFileFullPath);
        }

        // 按规则生成新文件名
        $extension = $file->getClientOriginalExtension();
        $filename = sprintf(
            '%s-%s-%s.%s',
            $prefix,
            $violation->plateNumber,
            $violation->violationType,
            $extension
        );

        // 移动文件到目标目录，给模型属性赋值
        $file->move($uploadDir, $filename);
        $violation->$field = $filename;
    }
}

3. 原来写5段重复if判断的位置，直接替换成一行方法调用即可：

// 替换掉原来5段重复的上传判断代码
$this->processVehicleImages($request, $violation, $uploadsDir);
// 后续正常执行模型保存逻辑
$violation->save();

进阶优化（跨控制器复用） #

如果项目里多个控制器都有类似的图片上传需求，可以把通用上传逻辑抽成Trait，所有需要上传功能的控制器直接引入Trait即可，不用重复写相同逻辑：

// app/Traits/HandleFileUpload.php
namespace App\Traits;

use Illuminate\Http\UploadedFile;
use Illuminate\Support\Facades\File;

trait HandleFileUpload
{
    /**
     * 通用单文件上传处理
     * @param UploadedFile $file 上传的文件实例
     * @param string $targetDir 存储目标目录
     * @param string $fileName 生成的新文件名
     * @param string|null $oldPath 旧文件完整路径，传值则自动删除旧文件
     * @return string
     */
    public function uploadFile(UploadedFile $file, string $targetDir, string $fileName, ?string $oldPath = null): string
    {
        if ($oldPath && File::exists($oldPath)) {
            File::delete($oldPath);
        }
        $file->move($targetDir, $fileName);
        return $fileName;
    }
}

注意：原示例代码存在严重逻辑bug：直接执行File::delete($uploadsDir)会删除整个上传目录，导致目录下所有已上传的图片全部丢失，上述复用代码已经修正该问题，仅删除当前字段对应的旧文件，不会影响其他资源。

额外实用优化建议：

• 文件名建议增加时间戳/随机字符串后缀，避免同车牌、同违章类型的文件重名被意外覆盖
• 上传文件的格式、大小校验可以统一写在Laravel表单请求类中，不需要在上传逻辑里重复写判断
• 推荐使用Laravel自带的Storage门面处理文件存储，后续切换本地存储、云存储时不需要修改业务上传逻辑

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