Laravel使用laravel-couponables包时优惠券无效仍注册用户问题

当前代码执行顺序存在逻辑漏洞：先调用User::create()将用户数据写入数据库生成账号，之后才执行$user->redeemCoupon($request->code)处理优惠券核销。当优惠券无效、已被使用时，核销逻辑抛出异常也不会回滚已经落库的用户记录，自然出现无效优惠券也能成功注册的问题。

核心是通过数据库事务保证「创建用户」和「核销优惠券」两个操作的原子性：任意环节失败就回滚所有数据库操作，不会残留用户数据；同时提前校验优惠券码参数，捕获优惠券相关异常返回友好提示。

具体调整点 #

• 在请求验证规则中新增code字段校验，要求必填
• 用DB::transaction()包裹用户创建、优惠券核销逻辑，触发异常时自动回滚
• 捕获优惠券组件抛出的专用异常，返回对应错误信息，中断后续注册、登录流程
• （可选）事务前提前校验优惠券存在性，减少不必要的事务开启开销

修正后的代码 #

use Illuminate\Support\Facades\DB;
use MichaelRubel\Couponables\Exceptions\CouponException;

public function store(Request $request)
{
    $request->validate([
        'name' => ['required', 'string', 'max:255'],
        'username' => ['required', 'string', 'max:255', 'unique:users'],
        'email' => ['required', 'string', 'email', 'max:255', 'unique:users'],
        'password' => ['required', 'confirmed', Rules\Password::defaults()],
        'code' => ['required', 'string'],
    ]);

    try {
        $user = DB::transaction(function () use ($request) {
            $user = User::create([
                'username' => $request->username,
                'name' => $request->name,
                'email' => $request->email,
                'password' => Hash::make($request->password),
            ]);

            // 优惠券无效时此处会抛出CouponException，触发事务回滚，已创建的用户会被自动删除
            $user->redeemCoupon($request->code);

            return $user;
        });
    } catch (CouponException $e) {
        return back()->withErrors(['code' => $e->getMessage()])->withInput();
    }

    event(new Registered($user));
    Auth::login($user);

    return redirect(RouteServiceProvider::HOME);
}

注意：不建议跳过事务直接把优惠券校验放到用户创建之前，校验和核销两个操作之间存在并发窗口，可能出现校验通过但实际核销时优惠券已被使用的情况，事务+核销时的原子校验才是可靠的实现方式。

内容的提问来源于stack exchange，提问作者Hyanu Oluwa