关于代码走查和代码审查

代码走查和代码审查是白盒测试方法中的2种。

代码走查:

从参加人员来说,应该是项目的整体参与者,如果项目太大,整体参加的成本很高,那么可以以模块为组进行走查。因为他们之间负责的业务是紧密相关的,使用的技术是接近程度比较大的,因而开发的规范应该是统一的。

从走查内容来说,应该是代码的命名规范,以及组织结构。每个项目都有自己的规范,但是如果项目内部使用不同的规范必然会增加发现问题、解决问题的难度,同时增加后期的维护成本。

从走查时间来说,应该在每个模块开发完成之后进行,便于开发人员之间交流问题以及体会,并且每个人的讲解时间不要超过30分钟,因为模块的业务复杂度不会那么复杂,30分钟都讲不清的业务逻辑如何保证代码是清晰的。

从走查的结果来说,经过走查的代码应该是参加成员大部分能认同的,并且参加者每个人都能读懂的逻辑清晰的代码,并且通过交流提高项目成员的凝聚力,提高其业务认知度,最好能形成项目之间可以共同使用的产品。

序号检查项

1代码的注释与代码是否一致?注释是否是多余的?

2是否存在超过3层嵌套的循环与/或判断?

3变量的命名是否代表了其作用?

4所有的循环边界是否正确?

5所有的判断条件边界是否正确?

6输入参数的异常是否处理了?

7程序中所有的异常是否处理了?

8是否存在重复的代码?

9是否存在超过20行的方法?

10是否存在超过7个方法的类?

11方法的参数是否超过3个?

12是否有多种原因导致修改某个类?

13当发生某个功能变化时,是否需要修改多个类?

14代码中的常量是否合适?

15一个方法是否访问了其他类的多个属性?

16某几项数据是否总是同时出现,而又不是一个类的属性?

17switch语句是否可以用类来替代?

18是否有一类的职责很少?

19是否有一个类的某些属性或者方法没有被其他类所使用?

20在类的方法中是否存在如下的调用形式:a.b().c()?

21是否某个类的方法总是调用另外一个类的同名方法?

22是否某个类总是访问另外一个类的属性与方法?

23是否两个类完成了类似的工作,使用了不同的方法名,却没有拥有同一个父类?

24是否某个类仅有字段和简单的赋值方法与取值方法构成?

25是否某个子类仅使用了父类的部分属性或方法?

代码审查:

代码走查(code walkthrough)和代码审查(code inspection)是两种不同的代码评审方法,

代码审查是一种正式的评审活动,而代码走查的讨论过程是非正式的。

最近对项目组进行代码评审,发觉需要对代码评审中找到的问题进行一下分类,大概可以分成以下几类问题:

1. Comment

注释没写,或者格式不对,或者毫无意义

2. Coding Standard

没遵守代码规范

3. Existing Wheel

重复现成的代码,或者是开源项目,或者公司已有代码

4. Better practice

Java或者开源项目,有更好的写法

5. Performance bottle and Improvement

性能瓶颈和提高

6. Code Logic Error

代码逻辑错误

7. Business Logic Error

业务逻辑错误

代码审查列出问题的类型,并有解决情况报告

一般情况代码走查有专门的工具比如:emma,该工具使用与大型项目,为开源的,代码走查完毕之后需要对结论给出一个代码走查报告。

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

推荐阅读更多精彩内容

  • Android 自定义View的各种姿势1 Activity的显示之ViewRootImpl详解 Activity...
    passiontim阅读 172,251评论 25 707
  • 日总结8.30 修身: 1.听书,有些漫无目的,九月待规整 2.阅读,读书笔记得留着,至少章节的收获总结要有,一边...
    隔世的风阅读 145评论 0 0
  • 今天在windows安装完redis启动时遇到一个错误: 大概意思是:没有足够的可用空间,可以增加系统的大小分页文...
    郭之源阅读 6,498评论 0 0
  • 以前听说过这样一句话:“女人是一种神奇的生物,每个月流血7天还不死的生物”。的确,女人确实是一种神奇的生物,可柔可...
    HX初三阅读 29,782评论 0 1
  • 高中时期会坚定不移的相信生活不止眼前的苟且,还有诗和远方的田野。于是乎报志愿时为了逃离父母的手掌,追求所谓的远方,...
    布丁怪兽o阅读 280评论 0 0