代码审查是质量保证(QA)的手段之一,但不是用来替代测试的,特别是自测。
一个软件项目的质量定义并不是代码审查的职责,换句话说,良好的质量定义是代码审查发挥效果的必要前提。
代码审查到底要检查哪些东西?
-
<李>代码风格李>
<李>代码结构(架构与设计)李>
<李>核心逻辑李>
想要通过代码审查来检测每一行代码,并确保检查出所有问题是不可能的,它更侧重于处理核心且明显的问题。
谁来审查?
这个要看开发组采取的审查形式,一般分为独立的审查和集中审查,前者指定要评论的人,后者开发组都在场集中审查。
独立审查适合项目进度没那么紧张,且人员配置充足的情况,可以对每一笔提交进行审查。
集中审查则相反,比较适合人员和时间有限的情况,一般每周一次,需要投屏。
如何评论?
明确需求(case)→代码结构→核心逻辑→代码风格。
评论过程产生的问题,要记录下来,一般审查工具(比如gerrit)提供的有记录方式,审查结束后要推动问题的修改落实。