代码审查专家

问题背景

代码审查常面临三类痛点：评审标准碎片化（不同开发者关注点不一）、缺陷模式识别不全（如空指针、竞态条件易被忽略）、评论表述模糊（导致执行效率低）。尤其在团队协作中，若CR流程缺乏结构化方法，关键风险点易被遗漏，甚至引发线上故障。

核心能力与工作流程

代码审查专家通过系统化清单解决上述问题：

• 多维度清单：覆盖正确性、安全性、性能、可维护性、测试覆盖五大维度，确保无死角评审。
• 缺陷模式库：内置空指针、并发竞态、注入漏洞、资源泄漏等高频问题模式，提供检查项模板。
• 评审礼仪规范：定义清晰可执行的评论写法，例如“建议将魔法值0xFF替换为常量MAX_VALUE”，避免模糊表述。
• 分级机制：将问题分为阻塞级（必须修改）与建议级（可后续优化），明确优先级。

工作流程分四步：

1. 明确CR目标与场景边界（如新功能开发 vs. Bug修复）。
2. 套用清单逐项执行或自查，标注可疑代码段。
3. 输出结构化结果：包括问题列表、分级建议及改进方向。
4. 标注需人工复核的环节，如涉及业务逻辑的复杂决策。

适用边界与注意事项

• ✅ 适用场景：常规CR流程、评审清单设计、缺陷模式自查、代码质量提升等需要方法论指导的场景。
• ❌ 不适用场景：需实时数据或外部系统直连的自动化任务（如CI/CD实时拦截）、需持牌专业人士出具结论的领域（如法律合规、医疗代码）。
• ⚠️ 注意：审查辅助不替代人工决策，关键系统须结合双人评审与自动化门禁（如SonarQube规则）。