阿里巴巴 Java 代码审查

要解决什么问题

团队做 Java Code Review 时，规范往往散落在《阿里巴巴 Java 开发手册》PDF 或 wiki 里，审查者凭记忆容易漏项，反馈也不统一：有人只盯命名，有人忽略并发与 SQL。本技能把手册规约落成可执行的审查流程，要求对照规则编号给出违规等级、位置和修复建议，而不是泛泛说「这里不太好」。

技能如何工作

审查从读代码上下文开始：Java 版本、框架（如 Spring Boot）、文件角色（Controller、Service、DAO、测试等）。随后按代码内容按需加载 L1 规则文件，而不是一次性塞进全部条文：

• reference/rules-programming.md：命名、常量、格式、OOP、集合、并发、控制语句、注释（N/C/F/O/COL/CON/CTRL/COM）
• reference/rules-exception-log.md：异常与日志（E/L）
• reference/rules-test-security.md：单测与安全（UT/S）
• reference/rules-mysql-project.md：MySQL、工程结构、服务端（DB/PRJ/SRV）

逐模块对照已加载规则，每条违规记录文件路径 + 行号、等级（BLOCKER / CRITICAL / MAJOR）、规则编号与描述。输出阶段可加载 reference/report-template.md 与 reference/examples.md，在报告里附上示例修复代码。用户说「快速 review」时可先读 reference/checklist.md 做 BLOCKER 级速查。

适用边界与注意点

• 规约来源是手册嵩山版的整理条目，不能替代你们团队自定义规范或安全合规审计。
• 渐进式加载依赖正确识别代码模块；多文件全栈改动时通常需要加载全部 L1 文件，否则可能漏查 DB/安全类问题。
• 本技能侧重审查与报告；是否合入分支、例外审批仍由人和团队流程决定。