敏捷软件开发实践-Code Review Process

简介:

介绍:


在敏捷软件开发中,从代码的产生速度上来看,要比传统Waterfall产生速度高很多。因为我们把时间安排的更加紧凑了。那么这么多的代码,如何能保证这些代码质量呢?很多人可能直接想到静态代码检测工具。没错,那些是可以定义一个代码检查规则来确保代码的质量,但是那个仅仅是从语言角度,那么逻辑是否已经最优化了?可重用性是否已经优化到极致了?这些是静态代码工具不能完成的,所以我们需要Code Review


实现方式:


对于已经在项目组很久的人来说:

虽然传统的code review就是把代码从仓库checkout出来,然后看下,但是对于大项目来说,那样的代码审查没有任何的效果,因为你除了看到代码还是代码,就像你在大海中看到的除了水就是天,很快就会迷失方向的。我们团队的经验,一般是采用crucible工具来进行代码审查,这个工具我以前博客已经有过介绍了:http://supercharles888.blog.51cto.com/609344/1229660

因为我们代码提交每次都有产生一个uuid,而我们提交更多是以子功能为单位,所以我们在crucible中也以提交为单位创建事件,可以很明确的知道对于具体某个功能,其实现的效果如何。

具体流程如下:


代码我们也采用了传统的peer review,因为自己看自己代码很难看出问题,但是用批判的眼光看别人的代码就很容易看出问题,所以我们结对的进行code review, 前端的人相互review,后端的人相互review.


对于刚来项目组的人:

刚来项目组的人,因为对业务逻辑不熟悉,直接让他去以提交为单位进行审查代码是没有任何意义的,他们最重要的是熟悉代码从而可以很快的上手项目,这时候,我就不主张他们用code review工具了,而是直接把代码全部签下来整体看,我的经验是:用调试模式启动服务器,然后在关键的行打上断点(后端代码断点),然后在前端你用Firefox 打开,开启FireBug,在关键js文件相应行也打上断点(前端代码断点),然后完全用单步走的方式,一步步走过来,同时watch关键的变量的值的变化,这样走一遍虽然很慢,但是你会对代码逻辑流程非常熟悉而且印象深刻。而且一个项目来说,虽然代码很多,但是关键流程并不多(判断依据就是这些流程是否最后要做Regression,如果要做Regression,那么就算关键流程),如果把握了关键流程,就是等于抓住了主要矛盾。这是最好的上手项目的习惯。按照我们团队的经验,一般一个senior engineer水平,2-3天就能上手项目并且开始接任务做了。


总结:

(1)对于项目组的老人来说,用代码审查工具来code review,从而可以从功能模块角度审查代码的实现

(2)对于新来项目组的成员来说,用调试模式单步走的策略,只抓住核心流程,从而以最快的速度把握项目核心流程。





本文转自 charles_wang888 51CTO博客,原文链接:http://blog.51cto.com/supercharles888/1262016,如需转载请自行联系原作者
目录
相关文章
|
5月前
|
人工智能 监控 测试技术
|
7月前
|
算法 Java 测试技术
卓越工程之如何做好Code Review
本文主要从我们为什么需要CR?CR面临哪些挑战?CR的最佳实践几个方面分析,希望可以给读者一些参考。
|
7月前
|
Java 测试技术 p3c
我们如何做Code Review
我们如何做Code Review
130 0
|
SQL 缓存 架构师
一文梳理 Code Review 方法论与实践总结
作为卓越工程文化的一部分,Code Review 其实一直在进行中,只是各团队根据自身情况张驰有度,松紧可能也不一,这里简单梳理一下 CR 的方法和团队实践。
666 9
一文梳理 Code Review 方法论与实践总结
|
SQL 缓存 架构师
一文梳理Code Review方法论与实践总结
相信Code Review每天都在进行,但代码评审应该怎么做,评审范围有哪些,评审标准在哪里,怎样可以避免流于形式,这里结合自己的实践来聊一聊。
29448 10
一文梳理Code Review方法论与实践总结
|
缓存 人工智能 监控
【如何有效做Code Review】8行代码提出的21个问题
- 很多同学都有这个疑问,如何结构化体系化的做CR?如何综合应用各种手段尽快及早的发现代码问题和缺陷? - 下面围绕这个实例,抛砖引玉,大家可以一起探讨;  - 实例如下 ,短短8行代码,通过CR可以发现多少问题呢?21处;这段代码谁写的不重要,探讨的重点是如何全面发现其中的问题和隐患;  
5902 0
【如何有效做Code Review】8行代码提出的21个问题
|
设计模式 测试技术 程序员
7 个建议让 Code Review 高效又高质
Code Review(CR) 的本质是什么?是为了查错?还是为了 KPI?本文分享阿里资深技术专家的看法:CR 是一种关于社会学的长期行为和组织文化,通过 CR,形成一种良性互动的技术氛围,传播和分享知识,提升代码质量,并给出了 7 个提高 CR 效率和质量的实践建议。
4395 0
7 个建议让 Code Review 高效又高质
聊Code review(下)
上篇描述了什么是code review, 为什么要去做,主要包含哪些内容。本文主要讲如何去做Code review?会带来什么惊喜? 本文的分享者易荣平认为思路比方法重要,Code review 是为了推广代码的最佳实践,而不是挑出代码存在的问题。猿猿们精彩不容错过。
4531 0
|
程序员 开发者