代码Review系统

本文章转载于搜狗测试

今天,我们来聊一聊软件质量控制其中一个非常重要的环节:代码Review。

代码Review作用

1.提高软件的质量。软件的质量控制中,代码Review其实是非常重要的一个环节,它可以发现大量的问题并且很有效。微软曾经做过类似的统计,结果显示:用代码Review这种一步到位的方法找出并修改一个错误要花费3个工作时,而通过测试这种两部完成的方法则要花费12个工作时。在一个由400名程序猿创建的有70万行代码的程序中,代码Review的成本收益要比测试高出好几倍。因此,代码Review对于质量控制具有很大的作用

2.提高开发人员的代码能力。通过代码Review可以发现开发人员编写代码中的各类问题,比如:变量命名、代码结构设计、算法实现、注释说明等问题,通过问题的交流,也会提升开发人员的代码能力。

代码Review的角色

开发Leader或有经验的开发人员对他人进行Review(一般多是这种情况)

开发人员之间相互Review

测试人员对开发代码进行抽查。小编曾经在测试项目过程时会Review代码,经常会发现开发复制粘贴时没有修改变量名的问题,屡试不爽。

代码Review形式

为了保证代码Review这一环节能够贯彻执行下去,需要一定的手段来保证流程运转。有些团队采用的是"自律性"方法,即在提交代码之前进行Review工作,然后将Review情况在changelog中体现,例如:

这种方法的弊端是完全依赖于开发人员的规范执行情况,如果出现新人或者自律性不足,那么Review工作有可能就不会执行下去。

代码Review系统

代码Review系统是小编所在团队正在使用的方法,它可以有效解决以上规范执行的问题,它的工作流程大概如下:

1.开发人员提交代码。代码不会马上进入SVN库,而是会先提交到Review board系统。

2.代码的Diff查看。其他具有Review权限的人员对提交记录进行Diff内容查看。

点击右上角的View Diff,查看变更内容。

3.Review通过。如果代码没有问题,点击提交记录中右上角的Ship it!,代码通过Review自动进入SVN库中。

备注:有些项目会设置必须超过n个人(n可以配置)Review通过,代码才会进入SVN库中。

4.Review不通过提出问题。如果代码存在疑问,在Diff代码界面中,可以对有问题的代码行进行备注或者提交问题。

以上为代码Review的大致使用方法。

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 212,542评论 6 493
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 90,596评论 3 385
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 158,021评论 0 348
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 56,682评论 1 284
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 65,792评论 6 386
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 49,985评论 1 291
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 39,107评论 3 410
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 37,845评论 0 268
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 44,299评论 1 303
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 36,612评论 2 327
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 38,747评论 1 341
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 34,441评论 4 333
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 40,072评论 3 317
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 30,828评论 0 21
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 32,069评论 1 267
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 46,545评论 2 362
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 43,658评论 2 350

推荐阅读更多精彩内容

  • Android 自定义View的各种姿势1 Activity的显示之ViewRootImpl详解 Activity...
    passiontim阅读 171,846评论 25 707
  • Spring Cloud为开发人员提供了快速构建分布式系统中一些常见模式的工具(例如配置管理,服务发现,断路器,智...
    卡卡罗2017阅读 134,637评论 18 139
  • 吹了一天的风,被一场细雨平息。 滴滴答答的小雨,被一场夜幕霸占。 浩瀚的星空,被漆黑笼罩。 鲜明的街灯,亮等来日的天明。
    鐘玥阅读 251评论 0 0
  • 过去一年有余,我经历了一次彻底的创业,做了一款手机网游,进行了三次角色转变,其心路历程复杂而坎坷。信心多次崩坏而重...
    闲好手游阅读 489评论 0 3
  • 人为什么总是喜欢怀念小时候?因为那时我们一无所有,整日无忧。 长大后,我们拥有的越来越多,想拥有的也越来越多。 每...
    Lancy日记阅读 2,719评论 0 15