代码审查

代码审查也叫代码复查,即通过阅读代码的方式来检查代码是否符合要求

谷歌代码审查指南:https://jimmysong.io/eng-practices/docs/review/

为何要代码审查

代码审查是提高代码质量,提前发现 BUG ,统一团队代码规范,促进团员成长的一个重要途径

针对目标使用数据驱动,监测相关目标的推动因素,达到一个正反馈,从而促进质量的提升

谁来审查

审查什么

审查策略

必备检查项

业务高压线

历史故障点

变更分级

内容分级

审查标准

审查原则

审查工具

审查形式

原则上,每次被审查的代码都应保持较少的水平,审查这些代码的人员一般都要求是经验较为丰富的高级开发人员,不仅可以通过审查发现代码的缺陷、不足,同时对于研发来说审查也是高级开发人员向团队新手分享知识的一种途径,对于测试来说也可以明确变更的影响范围及提升系统质量

审查的过程应该是纯粹的,禁止有情感、绩效因素。

同时审查也不应该是保证代码规范的手段,代码规范应该有工具化、自动化的手段

CR讲解

开发者讲解代码,评审者听取并发现问题、提问讨论

针对分歧,当场解决

CR Review

开发者异步提交给评审者

审查内容

代码飞检

万物评审

通过多人参与集思广益,多样的视角,不同的观点发现其中存在的问题

为什么要做,怎么做,哪里做,何时做,谁来做?

成本与收益比? 目标是什么,怎么执行,得到什么结果

评审形式

参与人