zoehis-code-review

ZOEHIS AI 局部审查（Step 8）

改动完成后、人工审查前，对 本次 diff 做局部审查并输出结论。详见 docs/workflow.md Step 8。

本规则将「Java/Spring Boot 企业级代码生成约束」转换为 审查视角 的检查清单：生成阶段该遵守的，审查阶段逐条核对是否已遵守；命中即标记 违规，给出改法，结论置为 需修改，不得放行进入 Step 9。

1. 规范与数据流

• 命名、代码风格（dev/rules/zoehis-*.mdc）
• 池表 / 主细表 / 预交金 / 流水完整性（同 Step 7，针对改动点）

2. Java/Spring Boot 企业级规范（必查）

对新增/修改的 Java/Spring 代码逐项核查；命中任一项即标记 违规 并给出改法，不得进入 Step 9。

2.1 异常处理（强制）

• 禁止空 catch：catch (Exception e) {} → 必须记录日志并向上抛出
• 禁止吞 cause：禁止 throw new RuntimeException("失败") 丢原异常 → 必须 throw new RuntimeException("失败", e)
• 必须保留原始 cause、记录完整异常日志（含堆栈 log.error("执行失败", e)）
• 禁止静默吞异常（catch 后不处理也不抛出，return 掩盖错误）

2.2 事务规范（强制）

• @Transactional 仅标注 public 方法；禁止 private/protected 方法加事务（不生效）
• 必须声明 rollbackFor = Exception.class，禁止依赖默认回滚（默认仅回滚 RuntimeException）
• 禁止同类内部 self-invoke 调用本类事务方法（事务失效）：改用 AOP 代理调用 / 抽取独立 Service / 注入自身代理
• 多表写、池表删增未同事务 → 标记风险并提示

2.3 afterCommit（最高优先级）

• 若使用 TransactionSynchronizationManager / afterCommit() / TransactionSynchronizationAdapter
• 其中任意 DB 写（insert/update/delete/save/saveBatch/Mapper/Repository）必须 新开物理事务，满足其一：
  • ① @Transactional(propagation = Propagation.REQUIRES_NEW)
  • ② TransactionTemplate
  • ③ AOP 代理调用且代理方法声明 REQUIRES_NEW
• 禁止 afterCommit 中直接写库而未开启新事务（主事务已提交，内部写不受控 / 事务失效）

2.4 金额计算（强制）

• 金额字段、金额运算必须使用 BigDecimal，禁止 double/Double/float/Float
• 禁止金额 == 比较，必须用 compareTo() / equals()
• 禁止浮点字面量参与金额运算（如 1.1 * qty → new BigDecimal("1.1").multiply(...)）

2.5 线程安全

• 禁止共享 SimpleDateFormat（静态共享变量）→ 改用 DateTimeFormatter 或局部变量/ThreadLocal
• 禁止共享可变 HashMap/ArrayList 等非线程安全容器（线程间复用导致数据错乱）
• 共享对象必须保证线程安全（或改为方法局部变量）

2.6 异步任务

• 出现 @Async 的类中，项目须存在 @EnableAsync 与 ThreadPoolTaskExecutor 配置；缺一则 @Async 不生效 → 补全配置或移除注解
• 异步方法内异常需有明确兜底，避免静默丢失

2.7 依赖注入

• 统一 @RequiredArgsConstructor + 构造器注入；禁止 @Autowired 字段注入
• 禁止在字段上直接 @Autowired（破坏不可变性与可测试性）