如何在 Laravel 5.1 上使请求类型动态化（验证）

Question

我正在尝试制作一个 crud 控制器基础，从那里我扩展它并设置模型基础然后我有一些基本的 crud 方法。我得到了所有动态工作。但是我不能创建动态请求类型，为了验证它，我有 ChannelRequest，它工作正常，如下所示，但我希望它是动态的：

这是我的 CrudController Class（我将扩展并设置一个模型）：

    public function store(ChannelRequest $request)
    {
        $this->save($request); // this method get the model instantiated in parent class and save the inputs

        return redirect('admin/' . $this->plural);
    }

在上面的这个例子中，我硬编码了依赖注入的请求类型，然后它验证了，但我想动态地改变请求类型，像这样：

    // i know it not being work
    public function store($this->model .'Request' $request)
    {
        $this->save($request);

        return redirect('admin/' . $this->plural);
    }

我试过这个：

    public function store()
    {
        $request = new ChannelRequest();
        $request->validate(); //hopping it runs like when dependency injection

        $this->save($request);

        return redirect('admin/' . $this->plural);
    }

这让我出错：

FatalErrorException in FormRequest.php line 75:
Call to a member function make() on null
in FormRequest.php line 75
at FatalErrorException->__construct() in HandleExceptions.php line 133
at HandleExceptions->fatalExceptionFromError() in HandleExceptions.php line 118
at HandleExceptions->handleShutdown() in HandleExceptions.php line 0
at FormRequest->getValidatorInstance() in ValidatesWhenResolvedTrait.php line 20
at FormRequest->validate() in CrudController.php line 67

Answer 1

首先，我想强调的是，为每个资源（模型）设置单独的控制器是一种很好的做法，可以防止单独的关注点过于混杂。使用动态请求 class 违背了明确定义请求 class 的初衷。

但是，为了尽我所能回答这个问题，我会告诉你如何解决这个问题。此代码未经测试，但概念应该是正确的。

我在这里所做的是使用 SmartRequest class 扩展标准请求 class 并覆盖 __construct 以允许我运行预加载器对于给定请求类型的正确请求 class。

这将允许您定义单独的请求 classes，然后根据 resourceType 请求参数将它们加载到 SmartRequest::$subRequest 属性（这可以如果你想为最后一个修改一些代码，请成为 POST、GET 或 URL 参数的一部分。

代码：App\Http\Requests\SmartRequest

<?php 

use App\Http\Requests\Request;

class SmartRequest extends Request {

    /**
     * Holds sub request class
     * @var Request
     */
    protected $subRequest;

    /**
     * Default constructor
     * @param array             $query      
     * @param array             $request    
     * @param array             $attributes 
     * @param array             $cookies    
     * @param array             $files      
     * @param array             $server     
     * @param string|resource   $content    
     */
    public function __construct(array $query = array(), array $request = array(), array $attributes = array(), array $cookies = array(), array $files = array(), array $server = array(), $content = null)
    {
        // make sure standard constructor fires
        parent::__construct($query, $request, $attributes, $cookies, $files, $server, $content);

        // instantiate the sub request object, we must also pass through all the data for the base
        // request since the sub class requires this data.
        $this->loadSubRequest($query, $request, $attributes, $cookies, $files, $server, $content);
    }

    /**
     * Default constructor
     * @param array             $query      
     * @param array             $request    
     * @param array             $attributes 
     * @param array             $cookies    
     * @param array             $files      
     * @param array             $server     
     * @param string|resource   $content    
     */
    public function loadSubRequest($query, $request, $attributes, $cookies, $files, $server, $content)
    {   
        // get resource type off the request data to generate the class string
        $class = $this->getRequestClassName();

        $this->subRequest = new $class($query, $request, $attributes, $cookies, $files, $server, $content);
    }

    /**
     * Get the sub class name with namespace
     * @return string
     */
    public function getRequestClass()
    {
        return '\<path>\<to\<namespace>\' . studly_case($this->resourceType) . 'Request';
    }

    /**
     * Returns rules based on subclass, otherwise returns default rules
     * @return array
     */
    public function rules()
    {
        // return the default rules if we have no sub class
        if (empty($this->subRequest)) return [];

        // return the rules given by the sub class
        return $this->subRequest()->rules();
    }
}

同样，这不是真正的代码（因为我没有测试过）但这可能是完成您的请求的一种方式。这也依赖于在请求中发送了一些标识符（在这种情况下，一个 requestType 参数），因为除了请求发送到哪里以及使用什么参数之外，你不会知道任何关于请求的信息。

不过，我认为这完全违背了此功能的初衷。有明确的请求并在需要它们的方法中明确地使用它们要好得多。为什么？自记录代码。只需阅读操作中的 ChannelRequest $request 之类的内容，人们就会知道您在何处使用什么。像上面这样的东西 (SmartRequest) 会导致某种魔法发生，任何其他开发人员在撕开 SmartRequest class.

之前都不会理解

如果这让我感到困惑，或者如果您对为什么我认为这种方法是朝着错误方向迈出的一步有任何其他疑问，请告诉我。

如何在 Laravel 5.1 上使请求类型动态化（验证）

How to make Request type dynamic (validation) on Laravel 5.1

php

laravel

laravel-5.1