如何进行代码review

Posted 2021-11-06

代码review是质量保证（QA）的手段之一，但不是用来替代测试的，特别是自测。

一个软件项目的质量定义并不是代码review的职责，换句话说，良好的质量定义是代码review发挥效果的必要前提。

代码review到底要review哪些东西？

• 代码风格
• 代码结构（架构与设计）
• 核心逻辑

想要通过代码review来检测每一行代码，并确保检查出所有问题是不可能的，它更侧重于处理核心且明显的问题。

谁来review？

这个要看开发组采取的review形式，一般分为独立review和集中review，前者指定要review的人，后者开发组都在场集中review。

独立review适合项目进度没那么紧张，且人员配置充足的情况，可以对每一笔提交进行review。

集中review则相反，比较适合人员和时间有限的情况，一般每周一次，需要投屏。

如何review？

明确需求（case） -> 代码结构 -> 核心逻辑 -> 代码风格。

review过程产生的问题，要记录下来，一般review工具（比如gerrit）提供的有记录方式，review结束后要推动问题的修改落实。