我总结了代码评审中的常见问题如下:
- 代码可读性差,导致评审效率低下。
- 找到的缺陷大都是轻微缺陷。
- 快速评审很多代码,没有发现很多问题。
- 专家没有时间做代码评审。
- 专家发现的问题作者不认可。
为了应对上述问题,可以借鉴如下的最佳实践:
1、先做个人评审,再进行专家评审。
2、坚持使用代码评审检查单,不断细化补充检查单。
3、即使不能对所有代码做检查,也要对部分代码检查。
4、轻量级代码走查更高效,频繁的日常走查,选择核心代码审查。
5、每次检查少于 200~400 行代码,每次代码检查不超过 90 分钟。
6、建立量化目标,并获得相关指标数据,从而不断改进流程。
7、优先使用静态检查工具。
8、建立代码评审的文化,例如,结对走查机制、固定时间段评审的机制。