代码审查：不只是找Bug，更是团队合作的桥梁

意义

• 团队知识共享

  代码审查，就是一个很好的知识共享的方式。通过代码审查，高手可以直接指出新手代码中的问题，新手可以马上从高手的反馈中学习到好的实践，得到更快的成长

• 代码质量

  对于代码质量来说，很多问题通过测试是测试不出来的，只能通过代码审查。比如说代码的可读性可维护性，比如代码的结构，比如一些特定条件才触发的死循环、逻辑算法错误，还有一些安全上的漏洞也更容易通过代码审查发现和预防。

• 团队规范

​ 每个团队都有自己的代码规范，有自己的基于架构设计的开发规范，然而时间一长，就会发现代码中出现很多不遵 守代码规范的情况，有很多绕过架构设计的代码。比如难以理解和不规范的命名，比如三层架构里面UI层绕过业务逻 辑层直接调用数据访问层代码。

方式

工具扫描　＋　人工审查

工具扫描适合的方面

1. 代码风格与格式: 自动化工具非常适合检查代码风格一致性，包括缩进、空格、命名规范等，确保代码遵循固定的风格指南。
2. 错误模式检测: 静态分析工具如 SonarQube, FindBugs, 或 PMD 可以识别常见的编程错误，如空指针访问、未关闭的资源、潜在的线程安全问题等。
3. 复杂度计算: 工具可以计算函数或类的复杂度，帮助识别过于复杂难以维护的代码区域。
4. 安全漏洞扫描: 专门的安全扫描工具能够检测到潜