CodeReview

2020/06/16

Code review必要性Code
Review不只是一种管理方法,也是开发者特有的沟通方式,更是一种团队文化。主要目的是保障我们的代码质量和软件产品质量的同时提高团队整体技术实力,共同成长。Code Review机制是否健全是评价一个研发团队技术氛围好坏的重要参考。

Code Review文化
通过多层次的Code Review体系,在团队中建立起集体认知:代码是团队共有的知识财富,而不是个人的私有地所有人对所有代码的最终质量负责,而非某个人对某些代码负责完成功能只是开始,代码需要持续改进以符合团队标准Code Review对于团队中的新人来说是很好的锻炼机会,我们团队的新人首次提交代码,被打回更新四五次是很正常的事情。集体对代码质量的追求,也能提升新人的成就感、荣誉感。

Code review注意事项(规范 心态)
不要刻意的去寻找代码bug不要按照自己的编程风格去评论别人的代码(按照约定好的规范)不要带着抨击和质疑别人能力的心态去进行代码评审不要在不确定的问题上争来争去不要听不进别人的意见参与者最好不要自己都没想明白就提意见

团队如何进行代码review
前期建议指定项目负责人进行代码提交的审核 落实团队代码规范(配合gitlab的PR)规范熟悉之后建议轮流互相review 但是不可流于形式(减轻负责人压力)形成习惯之后 争取从管理制度变成团队文化 每个月进行review记录分享

一些共性问题:
有时候参与Review的人太多了,意见太分散,Review时间拉的很长,发现问题效率低;
有时候会发现一个CodeReview时间很长,参与者会觉得煎熬和浪费时间;
有时候不太了解对方评审的东西,没法跟上大家的思路,影响效率;
有时候走查的代码量太大了,无法做到详细走查;
有时候会看到有些人无所事事、精神不集中、不发言,影响效果。

Search

    Table of Contents