我正在寻找一种更干净的方法来验证存储Post时的标记。
所有的输入验证都发生在我的自定义请求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)）。
有没有更好的方法来解决这个问题？或者我应该停止思考，只使用（并重复）我拥有的代码？
提前谢谢，希望能有所收获。
我假设您理解这个类的用法，因为我已经看到您定义了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的规则，那么这就足够了。
foreach($request->tags as $k=>$tags){
$this->validate($request, [
'tags.'.$k => 'required|string|max:20'
]);
}