关联表的唯一验证这样写对吗?

我有一个类型表categroy,一个商品表goods。goods里有个字段c_id 关联categroy里的id,还有一个字段商品标题title。现在我要对商品编辑接口请求参数title进行同商品类型下的唯一性检查,表单验证代码如下:
app/Http/Requests/GoodsRequest.php

$id = $this->route('goods');
$cid = $this->input('cid');
if(!$cid){
    $goods = Field::find($id);
    if(!$goods){
        throw ValidationException::withMessages(['商品不存在']);
    }
    $cid = $goods->c_id;
}
$rules = [
'title' => ['bail', 
            'max:60',
             Rule::unique('goods','title')
                   ->ignore($id)
                   ->where(function($query) use ($cid){
                         $query->where('c_id',$cid);
                   })
           ]
]

这样写合理吗?或者放到后面控制器或者业务逻辑代码中去验证更合理?

《L02 从零构建论坛系统》
以构建论坛项目 LaraBBS 为线索,展开对 Laravel 框架的全面学习。应用程序架构思路贴近 Laravel 框架的设计哲学。
《G01 Go 实战入门》
从零开始带你一步步开发一个 Go 博客项目,让你在最短的时间内学会使用 Go 进行编码。项目结构很大程度上参考了 Laravel。
讨论数量: 2

遇到过这个问题,之前为了代码的优雅性将这部分封装成了一个新的验证器,直接用自带的表单验证即可。

Validator::extend('foo', function ($attribute, $value, $parameters, $validator) {
        // 逻辑代码
});
2年前 评论

反正都是要查询数据库的 先实现了再优雅

2年前 评论

讨论应以学习和精进为目的。请勿发布不友善或者负能量的内容,与人为善,比聪明更重要!