资深开发者视角审查代码变更，提供运维建议，帮助初级开发者提升…

# 购物车/库存扣减并发问题代码审查指南

由于我无法直接访问你的GitHub仓库，以下提供一套针对“Java库存扣减并发处理”的标准审查清单，你可以对照自查；也欢迎把代码片段贴出来，我帮你逐行分析。

一、库存扣减最常见的高危问题

1. 超卖（先查后改的竞态条件）

• 危险写法：SELECT stock → Java判断 stock > 0 → UPDATE stock = stock - 1。并发下多个请求都读到“还有库存”，导致超卖。
• 正确做法：用条件更新原子操作：

UPDATE stock SET num = num - 1 WHERE sku_id = ? AND num >= 1，根据影响行数判断成败。

2. 锁的使用是否正确

• synchronized：单机JVM内有效，多实例部署失效；且锁粒度过大（锁整个方法）会严重压低吞吐。
• Redis分布式锁：检查是否设置过期时间、是否使用Lua脚本保证“判断+扣减”原子性、释放锁时是否校验持有者身份。
• 数据库悲观锁（SELECT ... FOR UPDATE）：确认事务范围是否过大，避免长事务阻塞。

二、需重点检查的代码点

• 事务边界：扣库存与生成订单是否在同一事务？回滚时库存是否正确回补？
• 幂等性：重复提交/消息重试是否会导致重复扣减？应通过唯一请求ID或订单号做幂等控制。
• Redis缓存库存一致性：若用Redis预扣库存，检查是否有Lua原子脚本、Redis与DB最终一致性方案（如延迟对账、消息补偿）。
• 异常回补：扣减成功后下游失败（如优惠券核销失败），是否有库存归还逻辑且不会重复归还。

三、购物车模块特有的关联问题

• 加购与扣库存是否耦合？购物车改数量时是否会误触库存扣减。
• 未支付订单超时关单后，库存释放是否可靠（延迟队列/定时任务兜底）。

四、建议的验证方式

• 用 JMeter/wrk 压测库存接口，设置库存为1并发下单，观察是否超卖。
• 写并发单元测试：CountDownLatch 模拟N线程同时扣减。
• 检查数据库唯一索引兜底（订单号唯一）。

建议下一步

将涉及扣库存的核心方法（约几十行即可）贴出，我可以做针对性逐行审查，指出具体的修改建议。