如何简化Laravel控制器中的课程创建代码？

完全没必要手动逐个赋值请求参数，利用Laravel的请求数据方法配合数组合并，就能大幅简化代码，同时保留所有请求参数并添加你需要的三个字段。

核心优化思路 #

• 直接用$request->all()获取所有请求参数，再合并自定义字段，替代逐个赋值
• 用Laravel自带的$request->has()判断字段是否存在，比isset()更适配HTTP请求场景
• 修复原代码里的字段名笔误：is_is_highlighted应该是is_highlighted

优化后的完整代码 #

public function store(Request $request)
{
    $this->validate($request, []);
    
    try {
        // 合并请求参数与自定义字段
        $data = array_merge($request->all(), [
            'user_id' => Auth::user()->id,
            'is_active' => (int)$request->has('is_active'),
            'is_highlighted' => (int)$request->has('is_highlighted'),
        ]);

        // 图片上传逻辑
        $imageName = time() . '.' . $request->image->extension();
        $request->image->move(public_path('uploads/images/courses/'), $imageName);
        
        $image = Image::create([
            'path' => 'uploads/images/courses/' . $imageName,
            'alt_text' => $request->alt ?? $request->title,
        ]);
        
        $data['image_id'] = $image->id;

        // 处理发布状态
        if ($request->has('is_published')) {
            $data['published_at'] = Carbon::now()->toDateTimeString();
        }

        $course = Course::create($data);

        return redirect()->route('courses.index')->withMessage(trans('crud.record_created'));
    } catch (\Exception $e) {
        // 补充异常处理，避免白屏
        return redirect()->back()->withError($e->getMessage());
    }
}

额外实用建议 #

• 安全控制字段：如果不想接收所有请求参数，用$request->only(['name', 'objectives', 'requirements', ...])指定允许的字段，防止恶意参数传入
• 模型批量赋值：确保Course模型的$fillable属性包含所有需要批量赋值的字段，否则create()会抛出批量赋值异常
• 简化布尔转换：(int)$request->has('字段名')会自动将存在的字段转为1，不存在转为0，比三元运算符更简洁
• 文件存储规范：建议使用Laravel的文件存储系统（Storage类）替代直接操作public_path()，方便后续迁移或CDN配置

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