Laravel 5现有标签的自定义验证规则

时间:2015-04-09 12:09:47

标签: php validation laravel customization laravel-5

我正在寻找一种更简洁的方法来存储帖子时验证标签。

所有输入验证都发生在我的自定义请求StorePostRequest中。问题是我需要检查数据库中是否存在给定的标签,只允许现有的标签。函数$request->input('tags')返回一个逗号分隔值的字符串,例如:Tag1,Tag2,Tag3

以下是代码:

/**
 * Store a newly created resource in storage.
 *
 * @param  StorePostRequest $request
 * @return Response
 */
public function store(StorePostRequest $request)
{
    //THIS PIECE OF CODE
    $tags = explode(',', $request->input('tags'));
    $tags = Tag::whereIn('title', $tags)->lists('id');

    if(count($tags) < 1)
    {
        return redirect()->back()->withInput()->withErrors([ trans('tags.min') ]);
    }
    else if(count($tags) > 5)
    {
        return redirect()->back()->withInput()->withErrors([ trans('tags.max') ]);
    }
    //TILL HERE

    $post = $request->user()->posts()->create([
        'slug'          => unique_slug('Post', $request->input('title')),
        'title'         => $request->input('title'),
        'description'   => $request->input('description'),
        'summary'       => $request->input('summary'),
    ]);

    $post->tags()->attach($tags);

    return redirect(route('theme.post.show', [$theme->slug, $post->slug]))->with(['success', trans('messages.post.store')]);
}

在多个控制器中使用时,代码有点草率和冗余。

为了解决这个问题,我创建了一个ValidationServiceProvider来扩展核心验证器规则。像这样:

$this->app['validator']->extend('tags', function ($attribute, $value, $parameters)
{
    $tags = explode(',', $value);
    $tags = Tag::whereIn('title', $tags)->lists('id');

    if(count($tags) < 1 || count($tags) > 5))
    {
        return false;
    }
});

非常整洁。问题是我仍然需要能够访问控制器中的$tags变量(因为->attach($tags))。

有没有更好的方法来解决这个问题?或者我应该停止思考并使用(并重复)我的代码?

提前致谢,希望它有所作为。

2 个答案:

答案 0 :(得分:0)

我假设你理解了这个类的用法,因为我已经看到你定义了StorePostRequest类。因此,只是为了澄清,rules方法可能如下所示:

public function rules()
{
    return [
        'tags' => ['required', 'tags'] //kb
    ];
}

最后,如果所有工具都在正确的位置,您只需像这样操作控制器中的数据:

public function store(StorePostRequest $request)
{
    // at this point, the tags are already validated, so we, proceed get them:
    $tags = explode(',', $$request->get('tags'));

    $post = $request->user()->posts()->create([
        'slug'          => unique_slug('Post', $request->input('title')),
        'title'         => $request->input('title'),
        'description'   => $request->input('description'),
        'summary'       => $request->input('summary'),
    ]);

    $post->tags()->attach($tags);

    return redirect(route('theme.post.show', [$theme->slug, $post->slug]))->with(['success', trans('messages.post.store')]);
}
  

请记住,在控制器的功能中调试StorePostRequeststore它已经在验证并运行规则。

如果您确实正确定义了StorePostRequest规则,那就足够了。

答案 1 :(得分:0)

foreach($request->tags as $k=>$tags){
            $this->validate($request, [
                'tags.'.$k => 'required|string|max:20'
            ]);
        }