如何使请求类型动态(验证)在Laravel 5.1


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

我试图使一个crud控制器基础,从我扩展它并设置模型基础,然后我有一些基本的crud方法。我让所有的工作都是动态的。但是我不能做一个动态的请求类型,为了验证它,我有ChannelRequest,它的工作如下,但我想要它动态:

这是我的CrudController类(我将扩展和设置一个模型):

    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);
    }

i tried this:

    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

首先,我想强调的是,为每个资源(模型)拥有单独的控制器是一种很好的实践,可以防止单独的关注点过于混杂。使用动态Request类首先会破坏显式定义请求类的目的。

然而,为了最好地回答这个问题,我将给你一个如何解决这个问题的想法。此代码未经测试,但概念应该是合理的。

我在这里所做的是用SmartRequest类扩展标准请求类,并覆盖__construct,以允许我为给定请求类型的适当请求类运行预加载器。

这将允许你定义单独的请求类,然后将它们加载到基于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类。

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