Evaluate and enhance solutions…

# 电商系统接口代码校验——Bug清单框架

由于您尚未提供实际代码，我先给出高频风险点排查清单。您可对照自查，也可将代码贴出后我逐条核实。

---

一、库存扣减并发问题（高危）

1. 典型Bug：先查后改（非原子操作）

• 现象：select stock → 判断 >0 → update stock - 1，并发下多个请求同时读到相同库存，导致超卖。
• 修复：改用数据库原子更新：

```sql

UPDATE stock SET num = num - #{qty} WHERE sku_id = ? AND num >= #{qty}

```

根据影响行数判断是否扣减成功。

2. 乐观锁缺失

• 现象：update时未校验版本号或库存余量条件。
• 修复：加 WHERE num >= qty 或引入 version 字段；重试需设上限（如3次）防活锁。

3. 分布式场景漏洞

• 多实例部署时，JVM级 synchronized 无效；需用数据库行锁、Redis分布式锁（需正确设置超时+续期）或Redis+Lua原子扣减。
• Redis扣减需注意与DB的最终一致性（如通过MQ补偿、对账）。

4. 事务边界问题

• 现象：扣库存与创建订单不在同一事务，或事务内调用外部RPC，导致部分成功。
• 修复：同库用 @Transactional；跨服务用TCC/Saga/本地消息表。

5. 其他

• 扣减未校验库存上限（负库存、回补超上限）。
• 取消订单回补库存未做幂等，可能重复回补。

---

二、订单金额计算精度异常（高危）

1. 禁用 double/float

• 现象：用 double 存金额，出现 0.1+0.2 ≠ 0.3 类精度丢失。
• 修复：金额一律用 BigDecimal，数据库用 DECIMAL(x,y)。

2. BigDecimal 错误用法

• 用 new BigDecimal(0.1)（传入double仍有精度问题）→ 应使用 new BigDecimal("0.1") 或 BigDecimal.valueOf()。
• 用 equals 比较金额（scale不同会判false）→ 用 compareTo。
• 除法未指定精度和舍入模式，遇除不尽抛异常 → divide(b, 2, RoundingMode.HALF_UP)。

3. 分摊与舍入不一致

• 优惠券/满减分摊到多商品时，“各明细四舍五入之和 ≠ 订单总额”，产生分/厘级误差。
• 修复：最后一项用“总额 - 已分摊之和”兜底。

4. 计算顺序错误

• 现象：先合计再打折 vs 先打折再合计，结果不一致；折扣连乘顺序影响精度。
• 修复：统一计算规则并文档化，核心链路增加金额校验断言。

---

三、建议补充检查项

• 30个接口是否统一返回结构、异常码。
• 幂等性：下单/支付回调是否防重复提交（唯一索引/幂等token）。
• 参数校验：负数购买数量、超量下单。

如需精确到行的bug清单，请贴出代码。