我试图使一个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
类。
如果这让我感到困惑,或者如果你有任何其他问题,请告诉我为什么我认为这种方法是朝着错误的方向迈出的一步。