在 Laravel 中将其他函数放在 FormRequest 上
Put other function on FormRequest in Laravel
我正在构建一个 Laravel 6 应用程序,我担心 "best practices." 我有一个名为 CustomerController 的控制器。在我的控制器中,我想更新 Customer 模型,因此我将具有如下功能。
public function update(UpdateCustomer $request, Customer $customer){
//
}
UpdateCustomer 是我的表单请求,我将在其中进行验证。在我的 update() 方法中,我有经典验证。
public function rules()
{
$validationArray = [];
$validationArray['customer.name'] = 'string|required';
$validationArray['customer.vat'] = 'string|required';
$validationArray['customer.email'] = 'email|required';
return $validationArray;
}
现在我必须做一些经典以外的特殊验证。
假设我的模型中有更多数据,并且我不想更改这些值。
例如,我有以下内容:address, cap, locality.
我可以验证 UpdateCustomer 请求的第二种方法。
public function validateForDataCantChange()
{
$data = $this->input("customer");
$customer = $this->route("customerID");
$validator = Validator::make([], []); // Empty data and rules fields
$arrayDataThatCantChange = [
'address' => $data['address'] ?? NULL,
'cap' => $data['cap'] ?? NULL,
'locality' => $data['locality'] ?? NULL
];
foreach ($arrayDataThatCantChange as $key => $v) {
if ($customer->{$key} !== $v) {
$validator->errors()->add($key, __("messages.the field :field can't be changed", ['field' => $key]));
}
}
if ($validator->errors()->any()) {
throw new ValidationException($validator);
}
}
然后在我的控制器中,我添加了以下内容。
public function update(UpdateCustomer $request, Customer $customer){
$request->validateForDataCantChange();
}
这是一种不好的做法吗?我应该创建一个新的 FormRequest 吗?在这种情况下(两个表单请求),我如何才能对一个控制器使用两个不同的请求?
为了省事,我会亲自创建一个新的表单请求。
如果您希望使用相同的表单请求,您可以执行以下操作:
public function rules()
{
$rules = [
'title' => 'required:unique:posts'
];
// when editing i.e. /posts/2/edit
if ($id = $this->segment(2)) {
$rules['title'] .= ",$id";
}
return $rules;
}
但是,我总是为每个动作使用单独的 class。
我正在构建一个 Laravel 6 应用程序,我担心 "best practices." 我有一个名为 CustomerController 的控制器。在我的控制器中,我想更新 Customer 模型,因此我将具有如下功能。
public function update(UpdateCustomer $request, Customer $customer){
//
}
UpdateCustomer 是我的表单请求,我将在其中进行验证。在我的 update() 方法中,我有经典验证。
public function rules()
{
$validationArray = [];
$validationArray['customer.name'] = 'string|required';
$validationArray['customer.vat'] = 'string|required';
$validationArray['customer.email'] = 'email|required';
return $validationArray;
}
现在我必须做一些经典以外的特殊验证。 假设我的模型中有更多数据,并且我不想更改这些值。
例如,我有以下内容:address, cap, locality.
我可以验证 UpdateCustomer 请求的第二种方法。
public function validateForDataCantChange()
{
$data = $this->input("customer");
$customer = $this->route("customerID");
$validator = Validator::make([], []); // Empty data and rules fields
$arrayDataThatCantChange = [
'address' => $data['address'] ?? NULL,
'cap' => $data['cap'] ?? NULL,
'locality' => $data['locality'] ?? NULL
];
foreach ($arrayDataThatCantChange as $key => $v) {
if ($customer->{$key} !== $v) {
$validator->errors()->add($key, __("messages.the field :field can't be changed", ['field' => $key]));
}
}
if ($validator->errors()->any()) {
throw new ValidationException($validator);
}
}
然后在我的控制器中,我添加了以下内容。
public function update(UpdateCustomer $request, Customer $customer){
$request->validateForDataCantChange();
}
这是一种不好的做法吗?我应该创建一个新的 FormRequest 吗?在这种情况下(两个表单请求),我如何才能对一个控制器使用两个不同的请求?
为了省事,我会亲自创建一个新的表单请求。
如果您希望使用相同的表单请求,您可以执行以下操作:
public function rules()
{
$rules = [
'title' => 'required:unique:posts'
];
// when editing i.e. /posts/2/edit
if ($id = $this->segment(2)) {
$rules['title'] .= ",$id";
}
return $rules;
}
但是,我总是为每个动作使用单独的 class。