首页 > 解决方案 > Laravel 5.6 可以轻松验证多个输入字段

问题描述

我是学习 Laravel 的新手,但仍在尝试学习漂亮的编码。

我有一个这样的代码,可以在提交更新表单时验证输入字段,并且它有许多不必要的代码可以修改以使其更短更好。

所以我的问题是,我应该采用什么方法或技术来重写我的代码以使其简短而美观。

public function update(Request $request, $id)
    {
        $product = Product::find($id);

        if ($product->name != $request->name AND $product->sku != $request->sku AND $product->description != $request->description) {
            $this->validate($request, [
                'name' => 'required|unique:products,name',
                'sku' => 'required|unique:products,sku',
                'description' => 'required'
            ]);
        } elseif ($product->name != $request->name AND $product->sku != $request->sku) {
            $this->validate($request, [
                'name' => 'required|unique:products,name',
                'sku' => 'required|unique:products,sku'
            ]);
        } elseif ($product->name != $request->name AND $product->description != $request->description) {
            $this->validate($request, [
                'name' => 'required|unique:products,name',
                'description' => 'required'
            ]);
        } elseif ($product->description != $request->description AND $product->sku != $request->sku) {
            $this->validate($request, [
                'description' => 'required',
                'sku' => 'required|unique:products,sku'
            ]);
        } elseif ($product->name != $request->name) {
            $this->validate($request, [
                'name' => 'required|unique:products,name'
            ]);
        } elseif ($product->description != $request->description) {
            $this->validate($request, [
                'description' => 'required'
            ]);
        } elseif ($product->sku != $request->sku) {
            $this->validate($request, [
                'sku' => 'required|unique:products,sku'
            ]); 
        } else {
            return redirect('products/' . $product->id)->with('info', 'Product does not changed!'); 
        }

    }

标签: laravelvalidationlaravel-5.6

解决方案


看起来您有两个问题需要解决以缩短代码:

  • 1) 产品名称和 sku 字段是必需的,但您希望避免与您计划更新的产品上存在的值发生冲突。
  • 2) description 字段实际上不是必需的,但是当它存在时,它必须不被验证。(不应该有min:1验证之类的吗?)

因此,要解决 #1,您需要编写自己的unique规则,以在确定唯一值时忽略$product您尝试更新的电流。请参阅强制使用唯一规则以忽略给定 ID -- Laravel Docs

要解决 #2,您可以使用sometimes验证条件添加规则 -- Laravel Docs

结合这些应该允许您使用这样的单个验证语句:

use Illuminate\Validation\Rule;

...

$this->validate($request, [
    'name' =>  [
         'required',
         Rule::unique('products')->ignore($product->id)
    ],
    'sku' => [
         'required',
         Rule::unique('products')->ignore($product->id)
    ],
    'description' => Rule::sometimes('description', 'required', function($input){
        return $input->description !== $product->description;
    });
]);

如果这不正确,请为下一个偶然发现的人更新此答案。希望这可以帮助。

[提示:] 这似乎是一个很长的镜头,但如果描述字段是客户端界面上的 WYSIWYG 字段,我已经让他们达到了文本字段的最大长度,因为他们可以将图像转换为 base64,所以即使这对大多数人来说不太可能我尝试max:65535在验证中加入一些情况,以便它优雅地失败。(65535 是 MySQL 文本字段的最大长度,其他数据库大小不同)


推荐阅读