综合练习

把本 track 串起来：审查一段 Spring Boot 代码，按坏味道与审查清单找问题，在测试保护下 小步重构，最后以规范 PR 提交改进。目标不是重写项目，而是形成 可复用的质量工作流。

学习目标

• 独立完成 Review → 重构 → 测试 → PR 全流程
• 对照 cq-intro 至 cq-team 的检查项
• 与 spring-boot-essentials、git-essentials 实战衔接

任务说明

任选 其一：

A. 审查 CodeRoadMap backend（推荐）

打开 backend 模块，选一个 Service 或 Controller（如 tracks 相关），完成下方清单。脱敏前提下可 fork 到自己仓库练习 PR。

B. 审查你工作中的 Spring 模块

去掉真实密钥与客户名，对一个小模块做同样流程。

审查清单

1. 可读与 smell

• 是否存在长方法、重复校验（cq-smells）
• 命名是否表达业务意图
• 分层是否清晰（spring-boot-essentials）

2. 测试

• 核心逻辑是否有单元测试（testing-essentials）
• 改完后 mvn test 是否全绿

3. 规范与工具

• 格式/lint 是否通过（cq-style）
• 是否引入新的静态分析问题（cq-tools）

4. 文档与 PR

• README 或 JavaDoc 是否需补一句（cq-docs）
• PR 描述含 Test plan（cq-pr）

交付格式

## 摘要
- 模块：xxxService
- 主要问题：重复 slug 校验、方法过长

## 改进（≤3 项可 merge）
1. Extract validateSlug()
2. 补 shouldRejectInvalidSlug 测试

## PR 链接
（或本地分支名 + commit 列表）

重构遵守 cq-refactoring：一步一测一 commit。若用 AI 辅助，披露并人工 diff，见 ai-coding-essentials。

反思题

1. 哪类问题出现最多：smell、缺测还是文档？
2. 若只能修一项，你选什么？依据 cq-debt 的「利息」？
3. 你的团队 DoD 还缺哪一条（cq-team）？

练习

• 完成清单 至少 12 条 评估
• 提交 ≥1 个 含测试的重构 commit（或 PR）
• 写 3 条 constructive Review 评论（自审或同伴互审）

小结

质量实践 = 清单审查 + 测试守护的小步重构 + 规范 PR。 本 track 完结；持续质量靠每次 merge 的纪律与团队 DoD。