function ($attribute, $value, $fail) $attribute 貌似没有用到?

public function rules()
{
return [
'sku_id' => [
'required',
function ($attribute, $value, $fail) {
if (!$sku = ProductSku::find($value)) {
$fail('该商品不存在');
return;
}
if (!$sku->product->on_sale) {
$fail('该商品未上架');
return;
}
if ($sku->stock === 0) {
$fail('该商品已售完');
return;
}
if ($this->input('amount') > 0 && $sku->stock < $this->input('amount')) {
$fail('该商品库存不足');
return;
}
},
],
'amount' => ['required', 'integer', 'min:1'],
];
}

《L01 基础入门》
我们将带你从零开发一个项目并部署到线上,本课程教授 Web 开发中专业、实用的技能,如 Git 工作流、Laravel Mix 前端工作流等。
《L04 微信小程序从零到发布》
从小程序个人账户申请开始,带你一步步进行开发一个微信小程序,直到提交微信控制台上线发布。
leo
最佳答案

是的用不到,但这个匿名函数是 Laravel 调用而不是我们自己调用,所以必须保留这个参数

5年前 评论
讨论数量: 2
leo

是的用不到,但这个匿名函数是 Laravel 调用而不是我们自己调用,所以必须保留这个参数

5年前 评论

这种写法很好 代码很简洁 但是不易懂

                    if (!$sku = ProductSku::find($value)) {
                        return $fail('该商品不存在');
                    }
                    if (!$sku->product->on_sale) {
                        return $fail('该商品未上架');
                    }
                    if ($sku->stock === 0) {
                        return $fail('该商品已售完');
                    }

这样写 更容易明白些

                    $sku = ProductSku::find($value);
                    if (!$sku) {
                        return $fail('该商品不存在');
                    }
                    if (!$sku->product->on_sale) {
                        return $fail('该商品未上架');
                    }
                    if ($sku->stock === 0) {
                        return $fail('该商品已售完');
                    }
3年前 评论

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