代码评审(Code Review)

6,605 阅读2分钟

代码评审的目的

不是为了去刻意批斗某个Coder,而是为了团队成员之间相互了解学习,加深成员对系统的理解,使团队成员的代码更加健壮,提早发现代码缺陷。

那么应该如何做代码评审呢?

流程说明:

1.代码送审者每次提交最好是一个完整的功能,而不是一个小功能分很多次提交。
2.代码送审时候需要填写代码说明/审核人/功能链接/bug链接
3.审查人员收到邮箱通知后,查看审查任务,进行代码评审。(需要定义一些审核规范,一些基本的规范可以通过工具在在控制,自动审核)
4.审查人员根据团队之前达成的共识(代码规范)去评审一些代码,然后给出通过或者不通过的决定
5.送审人员根据驳回的意见进行修改后,然后在次送审。
6.如果代码通过,则合并到分支库里面去。

代码评审的好处

  • 提升系统的可维护性

  • 及早发现潜在缺陷与BUG,降低事故成本。

  • 促进团队内部知识共享,提高团队整体水平。

  • 评审过程对于评审人员来说,也是一种思路重构的过程,可以帮助更多的人理解系统。

  • 交叉审查代码,类似于结对编程,彼此都能熟悉对方模块业务,降低因人员流失的运营成本及风险。

后记:

1.代码审查建议每半月一次或一月一次,审查追求的是质量而不是数量。不要过分要求程序员做代码审查。如果你强迫他们每天做一小时的代码审查,他们很快就会痛恨它,把它当成一种无趣的任务。

2.代码审查是针对代码,不是针对人。代码审查是一种学习,是表扬,是获得反馈,是一种十分社交性的活动。代码审查应该是有趣的,不要让它变的无聊。