搜索
关闭此搜索框。

如何做代码评审?

谈及TDD的好处时,其中之一就是随时随地的Code Review,所以,貌似TDD是不需要Code Review的。但实际上,TDD和Code Review是两个正交的维度,做TDD并不妨碍Code Review。

这里就来聊聊我所在的项目是如何做Code Review的。我们有两种Code Review:Daily Code Review和Weekly Code Review。之所以有两种Code Review,因为每种Code Review的目的是不同的。

Daily Code Review

顾名思义,这是一种每天都做的Code Review。它的目的是让项目组全体成员了解每天的项目进展。

每天早上,运用git(这里可以替换成你所用的版本控制软件)的diff功能,找出前一天修改。全项目组的人在一起,逐行浏览这些修改。每当过到一处修改,这段代码的修改者就会针对修改,介绍一下这是哪个功能,为什么要做这些修改。项目组的其他人如果对这段代码有异议,都可以提出自己的问题。

这种Code Review使用的diff功能,了解的基本上只是一个片段,所以,关注点基本上是Clean Code。最经常出现的问题是,这段代码为什么写成这样。对于项目组的新人而言,这种Code Review是一个非常好的学习Clean Code以及了解项目隐含编码约定的机会。

Daily Code Review结束之后,每个人会针对其他人提出的修改意见对代码进行调整。在开始新一天的工作之前,最大程度地保证了代码库的整洁。

Weekly Code Review

诚如之前所述,Daily Code Review只能了解一个片段,人们缺乏对一个功能或一个故事的整体认识。Weekly Code Review弥补了这种缺陷。

Weekly Code Review更类似于许多企业的传统Code Review方式。由专人(一个人或一对pair)选取一个近期开发比较大的功能(或故事),组织全项目组的人进行Review。

在这个过程中,组织者会带领所有人浏览针对这个功能的相关代码。我们的主要关注点是让项目组的所有人对这个功能有所了解。这时项目组成员,也会从业务逻辑,以及局部设计的角度对代码进行分析,提出一些改进建议。如果过程中出现了一些难于理解的部分,组织者要负责为大家答疑解惑。

同样,在Weekly Code Review中发现的问题,也会被修正到代码之中。

无论是Daily Code Review还是Weekly Code Review,最容易出现的问题都是时间控制。如前所述,不同类型的Code Review有其侧重点,所以,讨论的组织者要负责把握讨论的方向,防止发散,在Code Review的环节中,我们常说的一句话是:“线下讨论”,以此扔掉跑偏的话题。

转载自:http://dreamhead.blogbus.com/logs/216262052.html

火爆 售票中
Scrum.Org 主办
搜索
近期公开班
大规模敏捷顾问SAFe SPC认证课徽章
6月15-18日​
SAFe认证-SPC SAFe认证培训师导师班
Marsha Xue , Alex Guan 授课
领导大规模敏捷Leading SAFe认证徽章
7月13-14日
Leading SAFe领导大规模敏捷认证课
Scott Wang 王庆付 授课
专业Scrum Master (PSM I) 认证徽章
7月13-14日
专业Scrum Master (PSM I) 认证公开课
Derek Ding 丁志润 授课
Scrum.org专业Scrum产品负责人(PSPO)认证徽章
8月03-04日
专业Scrum产品负责人(PSPO)认证公开课
Derek Ding 丁志润 授课
scrum alliance csm认证徽章
8月03-04日
Scrum Master (CSM) 认证课
Lance Zhang 授课
safe scrum master
8月10-11日
SAFe ScrumMaster 官方认证公开班
Eric Liao 廖靖斌 授课
Scrum联盟acsm认证徽章
8月24-25日
高级Scrum Master(A-CSM)认证公开课
Jim Wang 王军 授课
0
0
小时
0
分钟
0
由Scrum.org主办的 2024中国Scrum大会 8月17日将在上海开幕
0
0
小时
0
分钟
0
预约回电
留下您的手机号,我们会在第1时间联系您。
热线电话:400-696-6280
Get in Touch
预约回电
预约成功,我们会尽快联系您。
Get in Touch