关于代码走查的思考

昨天公司进行了团队评比,其中有一项就是代码走查。目前有两种方式,一种是通过会议进行抽查,一次性多人参加;另一种是通过gerrit 自带的review 流程。当然我们团队采用的是第二种方式。这里只是想描述一下第一种方式的弊端。

一、组织会议有时间成本。如果通过会议室开会形式,难免会有会议室预定、准备的时间,有时还有可能因为某些同事拖沓一下而耽误时间。如果以大家都集中在某个位置进行走查,此种形式难免有的同事看着一堆不熟悉的逻辑不会开小差,但如果让这些同事去开发他们自己的需求,我想他们肯定会专注的多。

二、组织会议会打断开发人员的节奏。会议形式必然会要求参会人员放下手上的工作来参加会议,这样会打断他们既有的节奏。

三、会议形式走查覆盖面有限。因为会议形式走查时间成本太高,因此难免会有团队采用敷衍的态度应对项目的检查,这样走查的覆盖率是很难保证的。

而gerrit review 无需组织会议,committer 在座位上就可以完成,这种方式开发人员可以更专注的走查代码,做走查的人也不一定非要集中在某个时间review ,他们可以先完成手上的工作后再走查,避免了打乱自己的工作节奏。

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
平台声明:文章内容(如有图片或视频亦包括在内)由作者上传并发布,文章内容仅代表作者本人观点,简书系信息发布平台,仅提供信息存储服务。

推荐阅读更多精彩内容

  • 近年来,由于开源项目、社区的活跃热度大增,进而引来持续集成(CI)系统的诞生,也越发的听到更多的人在说协同开发、敏...
    a6fc544968bb阅读 13,129评论 0 30
  • Android 自定义View的各种姿势1 Activity的显示之ViewRootImpl详解 Activity...
    passiontim阅读 175,707评论 25 709
  • 医的对象是活体,是整体,是不断变化的辩证施治,西医是解刨学基础上的静态的、部分对症施治,中国哲学研究的是动态的、变...
    禹音阅读 3,339评论 0 2
  • 曾经多么爱她 现在既不爱又不狠,也许就算过去了吧 但还是许下老死不相往来的誓言 也许还是内心深处无法忘怀吧 都说时...
    Z栋阅读 2,224评论 0 1
  • 阅读是学习之匙 读第60条《必须教会少年阅读》 莱州市文峰中学 程挺模 苏霍的教育建议从儿童说到少年了,或者说从小...
    赶潮儿阅读 2,738评论 0 0