code review 的几条心得

149 阅读2分钟

什么是 code review

code review 即代码评审,开发人员之间,互相检查彼此代码,验证需求、发现错误、提高代码质量的过程。

为什么要 code review

1) 旁观者清

首先,通常自己写的代码都难以发现问题,需要以第二双眼睛再次检查代码;

其次,很多开发人员,往往在项目中,对具体实现细节钻研过深,但是却忽视了从整体考虑问题,从旁人角度出发,更容易找到关键点,避免片面的头疼医头、脚疼医脚现象。

2) 更好掌握整体业务

一个项目是由多人负责的,每人只负责一个部分,在 CR 的过程中,互相了解彼此业务,更好配合,甚至在缺人手时,其他人能够快速顶上。

3) 对开发者的好处

加深需求理解,弥补不足,提高表达能力、逻辑思维能力、心理承受能力。

4) 对评审者的好处

掌握整体业务,学习优秀的设计思路。

5) 对项目组的好处

提高凝聚力,统一代码风格,提高项目质量,培养优秀工程师。

code review 原则

1. CR不是挑刺,不是炫耀谁能力强

CR是提高团队整体能力,提高短板,并非针对个人设置的“检查关卡”。

2. CR是必要的,但是要结合团队现状

CR需要找一个大伙都空闲的时间,在工期紧张的情况下CR效果不会很好。

3. 不刻意追求完美,而是追求持续的提高

良好的代码需要时间和业务发展的双重打磨,完美代码不会一蹴而就。

4. 每次代码改动要获取所有人的认同

避免之后出现“我当初就说了这样不行”这种扯皮现象,难以达成共识要拉起更大的会来讨论。

5. 保持谦逊积极的态度

傲慢与偏见是团队合作的大敌,无论评审者还是开发者,都要悉心聆听他人观点,理解之后再提出自己的观点。