库存管理中实现下单后扣减商品库存的技术求助

原代码存在的问题 #

你的代码有几个关键错误导致无法正常执行：

• 重复关联了products表，正确的关联应该是products与cart_product表通过product_id匹配
• WHERE条件的字段和值顺序颠倒，且orWhere逻辑错误，这里不需要用orWhere，只需要筛选当前购物车的商品
• update()方法未指定更新规则，缺少库存扣除的核心逻辑

正确实现方式 #

方法一：数据库层面批量更新（高效简洁）

直接通过关联查询批量更新库存，适合不需要额外校验的场景：

DB::transaction(function () {
    DB::table('products')
        ->join('cart_product', 'products.id', '=', 'cart_product.product_id')
        ->where('cart_product.main_cart_id', '=', session('active_cart_id'))
        ->update([
            'products.stock' => DB::raw('products.stock - cart_product.number')
        ]);
});

• 用DB::transaction包裹操作，确保库存更新失败时回滚，避免数据不一致
• 通过product_id正确关联商品表和购物车商品表
• 用DB::raw直接在数据库中计算库存扣除，比先查询再更新效率更高

方法二：带库存校验的逐行更新（避免超卖）

如果需要校验库存是否充足，防止超卖，可以用这种方式：

DB::transaction(function () {
    // 获取当前购物车的所有商品
    $cartProducts = DB::table('cart_product')
        ->where('main_cart_id', '=', session('active_cart_id'))
        ->select('product_id', 'number')
        ->get();

    foreach ($cartProducts as $cartProduct) {
        // 加行锁，防止并发下单时的库存冲突
        $product = DB::table('products')
            ->where('id', '=', $cartProduct->product_id)
            ->lockForUpdate()
            ->first();

        // 校验库存是否足够
        if ($product->stock < $cartProduct->number) {
            throw new Exception("商品ID {$cartProduct->product_id} 库存不足，无法完成订单");
        }

        // 更新库存
        DB::table('products')
            ->where('id', '=', $cartProduct->product_id)
            ->update([
                'stock' => $product->stock - $cartProduct->number
            ]);
    }
});

• 使用lockForUpdate对商品行加锁，避免多个请求同时扣减同一商品库存导致超卖
• 增加库存校验逻辑，确保订单提交时库存充足

注意事项 #

• 必须用事务包裹库存更新操作，防止中途出错导致数据异常
• 高并发场景下，优先使用方法二的行锁机制，避免库存超卖
• 更新完成后，建议清空当前购物车的商品记录，避免重复扣减

内容的提问来源于stack exchange，提问作者Muhammet aytaç