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

我有一个类型表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);
                   })
           ]
]

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

《L05 电商实战》
从零开发一个电商项目,功能包括电商后台、商品 & SKU 管理、购物车、订单管理、支付宝支付、微信支付、订单退款流程、优惠券等
《L04 微信小程序从零到发布》
从小程序个人账户申请开始,带你一步步进行开发一个微信小程序,直到提交微信控制台上线发布。
讨论数量: 2

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

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

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

2年前 评论

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