Laravel Worker存储逻辑代码优化及关联数据处理方案咨询

当前实现不符合Laravel开发最佳实践，存在以下明确问题：

• 存在语法错误：事务闭包内return $worker语句末尾缺少分号，会直接触发语法解析错误
• 未对可选关联字段做存在性判断：验证规则中course_name/doc_name/expiry_date均为sometimes可选字段，直接传入create方法时，若字段缺失会向数据库非空字段写入null，触发SQL错误
• 未校验关联模型创建结果：如果上一步关联模型（比如Course）创建失败返回null，后续调用$course->media()会直接触发「在null对象上调用成员函数」的致命错误
• 文件上传逻辑放在数据库事务中存在数据不一致风险：如果事务后续步骤出错回滚，已经上传到磁盘的文件不会被自动清理，会产生无效垃圾文件
• 硬编码文件类型为pdf：和验证规则中允许的jpeg/doc/csv等多文件类型冲突，会导致文件类型存储错误
• 无异常兜底处理：报错时直接抛出系统异常给前端，没有友好的错误返回

控制器Store方法优化 #

核心调整点：文件上传提前到事务外执行、可选关联先判断字段存在性再创建、增加异常兜底和垃圾文件清理、移除硬编码逻辑

public function store(StoreWorkerRequest $request)
{
    $attributes = $request->validated();
    $uploadedFiles = [];

    try {
        // 提前处理文件上传，IO操作不放在事务内，减少事务锁持有时间
        $courseFilePath = null;
        if ($request->hasFile('doc_file')) {
            $courseFilePath = fileUpload($attributes['doc_file'], 'Course');
            $uploadedFiles[] = $courseFilePath;
        }

        $worker = DB::transaction(function () use ($attributes, $courseFilePath) {
            // 创建主Worker记录
            $worker = Worker::create($attributes);

            // 仅当课程名称和课程文件都存在时，创建Course关联及对应媒体记录
            if (!empty($attributes['course_name']) && $courseFilePath) {
                $course = $worker->course()->create([
                    'course_name' => $attributes['course_name']
                ]);
                // 确认Course创建成功后再创建关联媒体，避免空对象调用
                if ($course) {
                    $course->media()->create([
                        "filename" => $courseFilePath,
                        "filetype" => $attributes['doc_file']->extension()
                    ]);
                }
            }

            // 仅当有效期存在时创建体检记录关联
            if (!empty($attributes['expiry_date'])) {
                $worker->medical_detail()->create([
                    'expiry_date' => $attributes['expiry_date']
                ]);
            }

            // 仅当证件名称存在时创建证件记录关联
            if (!empty($attributes['doc_name'])) {
                $worker->document()->create([
                    'doc_name' => $attributes['doc_name']
                ]);
            }

            return $worker;
        });

        return success(new WorkerResource($worker), __('Workers created successfully'));

    } catch (\Exception $e) {
        // 出错时自动清理所有已上传的文件，避免产生垃圾文件
        foreach ($uploadedFiles as $filePath) {
            // 替换为项目实际的文件删除方法，比如Storage::delete($filePath)
            fileDelete($filePath);
        }
        return error($e->getMessage(), 500);
    }
}

表单验证规则补充 #

在StoreWorkerRequest中增加关联字段依赖校验，避免关联数据不完整：

public function withValidator($validator)
{
    $validator->after(function ($validator) {
        // 传了课程名必须上传对应课程文件
        if ($this->filled('course_name') && !$this->hasFile('doc_file')) {
            $validator->errors()->add('doc_file', '填写课程名称时必须上传对应课程文件');
        }
        // 上传了课程文件必须填写对应课程名
        if ($this->hasFile('doc_file') && !$this->filled('course_name')) {
            $validator->errors()->add('course_name', '上传课程文件时必须填写对应课程名称');
        }
        // 其余关联字段（比如体检有效期和体检文件、证件名和证件文件）可按相同逻辑补充依赖校验
    });
}

• 原有实现针对单条Worker新增场景，关联数据量极小，串行写入本身不会产生明显性能瓶颈
• 优化后将文件IO移出事务，减少了事务持有数据库连接的时间，在并发请求场景下性能表现优于原有实现
• 新增的字段存在性判断均为内存逻辑，不会带来额外的性能开销

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