昨天公司进行了团队评比,其中有一项就是代码走查。目前有两种方式,一种是通过会议进行抽查,一次性多人参加;另一种是通过gerrit 自带的review 流程。当然我们团队采用的是第二种方式。这里只是想描述一下第一种方式的弊端。
一、组织会议有时间成本。如果通过会议室开会形式,难免会有会议室预定、准备的时间,有时还有可能因为某些同事拖沓一下而耽误时间。如果以大家都集中在某个位置进行走查,此种形式难免有的同事看着一堆不熟悉的逻辑不会开小差,但如果让这些同事去开发他们自己的需求,我想他们肯定会专注的多。
二、组织会议会打断开发人员的节奏。会议形式必然会要求参会人员放下手上的工作来参加会议,这样会打断他们既有的节奏。
三、会议形式走查覆盖面有限。因为会议形式走查时间成本太高,因此难免会有团队采用敷衍的态度应对项目的检查,这样走查的覆盖率是很难保证的。
而gerrit review 无需组织会议,committer 在座位上就可以完成,这种方式开发人员可以更专注的走查代码,做走查的人也不一定非要集中在某个时间review ,他们可以先完成手上的工作后再走查,避免了打乱自己的工作节奏。