Laravel控制器重写store方法参数不匹配报错及架构方案咨询

这是预期情况吗？ #

是的，这是PHP的预期行为。PHP的方法重写要求参数类型必须与父类兼容：当父类方法指定了具体的参数类型（比如PostStoreRequest），子类重写时不能使用另一个同层级的子类类型（比如AdvertStoreRequest）作为参数——尽管二者都继承自FormRequest，但这种参数类型"变窄"的写法不符合PHP的方法重写规则，因此会触发参数不匹配的错误。

合理解决方案 #

结合你基于多态Post表的设计场景，推荐以下几种实用方案：

方案1：抽象请求类获取逻辑，复用核心Store方法 #

在父类中定义一个动态获取验证请求类的方法，子类仅需重写该方法返回对应的请求类，核心的store逻辑完全复用，避免参数冲突：

// 父类 PostController
class PostController extends Controller
{
    protected $postableType;

    public function __construct()
    {
        $this->postableType = PostType::GENERIC;
    }

    // 子类重写该方法返回对应请求类
    protected function getStoreRequestClass(): string
    {
        return PostStoreRequest::class;
    }

    // 核心store方法，使用基类Request接收请求
    public function store(Request $request): RedirectResponse
    {
        // 动态解析并验证请求
        $requestInstance = app($this->getStoreRequestClass());
        $validated = $requestInstance->validate();

        // 抽象模型创建逻辑，子类可按需重写
        $post = $this->createPost($validated, $request->user());
        $post->save();

        return Redirect::route('feed');
    }

    protected function createPost(array $validated, User $user): Post
    {
        $post = new Post();
        $post->message = $validated['message'];
        $post->user_id = $user->id;
        $post->postable_type = $this->postableType;
        return $post;
    }
}

// 子类 AdvertController
class AdvertController extends PostController
{
    public function __construct()
    {
        $this->postableType = PostType::ADVERT;
    }

    // 重写返回广告专属请求类
    protected function getStoreRequestClass(): string
    {
        return AdvertStoreRequest::class;
    }

    // 重写模型创建逻辑，生成Advert实例
    protected function createPost(array $validated, User $user): Advert
    {
        $advert = new Advert();
        $advert->message = $validated['message'];
        $advert->user_id = $user->id;
        $advert->postable_type = $this->postableType;
        return $advert;
    }
}

方案2：使用动态验证规则的基类请求 #

如果不同类型的验证规则差异可以通过条件判断区分，可以定义一个基类请求，根据postable_type动态返回验证规则，父类和子类共用同一个store方法：

// 基类请求
class BasePostStoreRequest extends FormRequest
{
    public function rules(): array
    {
        // 从路由或请求中获取postable类型
        $postableType = $this->route('type') ?? PostType::GENERIC;

        return match($postableType) {
            PostType::ADVERT => [
                'message' => 'required|max:200',
                // 广告专属验证规则
            ],
            PostType::POLL => [
                'message' => 'required|max:100',
                'answers' => 'required|array|min:2',
            ],
            default => [
                'message' => 'required|max:500',
                'allow_comments' => 'boolean',
            ]
        };
    }
}

然后父类store方法直接使用这个基类请求，子类仅需处理postableType和模型创建的差异，无需重写store方法。

方案3：使用Trait复用通用逻辑，避免继承限制 #

放弃控制器继承，将通用CRUD逻辑封装到Trait中，每个控制器独立定义自己的store方法（使用专属FormRequest），同时复用Trait中的通用代码：

// 通用逻辑Trait
trait PostCrudTrait
{
    protected function savePostBaseData(array $validated, User $user, string $postableType): Post
    {
        $post = new Post();
        $post->message = $validated['message'];
        $post->user_id = $user->id;
        $post->postable_type = $postableType;
        return $post;
    }

    // 其他通用方法（如index、show等）
}

// PostController
class PostController extends Controller
{
    use PostCrudTrait;

    public function store(PostStoreRequest $request): RedirectResponse
    {
        $validated = $request->validated();
        $post = $this->savePostBaseData($validated, $request->user(), PostType::GENERIC);
        $post->save();
        return Redirect::route('feed');
    }
}

// AdvertController
class AdvertController extends Controller
{
    use PostCrudTrait;

    public function store(AdvertStoreRequest $request): RedirectResponse
    {
        $validated = $request->validated();
        $advert = new Advert();
        $advert->message = $validated['message'];
        $advert->user_id = $request->user()->id;
        $advert->postable_type = PostType::ADVERT;
        $advert->save();
        return Redirect::route('feed');
    }
}

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