为什么QA不喜欢重构?

经常听到开发人员抱怨 “这么烂的代码,我来重构一下吧!”,“这代码怎么能这么写呢?谁来重构一下?”,“这儿有个坏味道,重构吧!”,作为一名QA,每次听到“重构”两个字,既想给追求卓越代码的开发人员点个赞,同时又会感觉非常紧张,为什么又要重构?马上就要上线了,怎么还要改?是不是应该阻止开发人员做重构?

重构几乎是开发人员最喜欢的一项实践了,可QA们却充满了顾虑,那么为什么QA不喜欢重构呢?

老功能被破坏

不止一次遇到这样的场景,突然某一天一个老功能被破坏了,QA们感到奇怪,产品这块的功能已经很稳定了,也没有在这部分开发什么新功能,为什么突然出问题了呢?追查下去发现是近期做了重构,再追问的话,对于老代码,测试已经几乎看不太懂了,可是看到坏味道又实在心痒痒,就想重构,于是功能破坏了。

在快速迭代的开发模式下,QA们主要关注用户故事的生命周期,重点测试新的特性功能,所以对于老功能回归测试的投入是非常有限的,如果开发人员突然对老功能进行了重构又没有告知团队,很可能这样的问题就会进入生产环境,这样是非常有风险的,所以很多开发人员重构老功能时会告知QA,虽然这样可以提前发现和修复导致的问题,但无疑会大大增加QA做回归测试的负担。

QA发现功能破坏后找出重构导致

新功能推迟/重复测试

按照用户故事的开发流程,开发人员完成功能后多方角色会在开发人员机器上进行用户故事的快速验收以及探索性测试,然后我们期望的是开发人员提交代码,拿包测试,但有的时候我们在开发机器上验收之后,开发人员才会进行重构,曾经经历过故事验收的时候功能都是正常的,拿到包之后好多功能不工作了,跟开发人员确认,又是重构导致的。

所以有时候开发人员会在验收之后告知QA还会继续重构,QA可以等到重构完成后再做测试,或者开发人员会先重构再做验收,前者会导致用户故事的测试时间推迟,后者则会导致重复的验收测试。

在开发机器上重复验收

无计划不可见

开发人员的重构时机对我们来说是无规律的,有的时候一边开发用户故事一边重构,有的时候会在故事完成后QA在开发机器上验证结束后才做重构,有的是代码评审后大家指出问题后做重构,有的甚至依赖于项目组来了有经验的开发人员才开始重构。对QA来说,重构的时机是无计划的。

另外重构也是不可见的,我们总谈可视化,日常的开发工作由用户故事和缺陷来可视化,而重构经常是幕后进行的,不被跟踪不被记录不可见,有经验的开发人员会在进行“大”的重构之前告知团队,如果没有告知,就在不知不觉中发生了,只有功能被破坏了才能意识到。

重构总是不知不觉发生

总结

以上列出了QA不喜欢重构的三个理由,归根结底其实都是重构不当导致。代码重构(Code refactoring)指对软件代码做任何更动以增加可读性或者简化结构而不影响输出结果。而不当的重构往往只关注了前半部分却忽略了对输出结果的影响。

重构的目标是为了改善代码质量,长远来说应该是可以减少软件缺陷的,从这个角度来说QA和开发人员的目标是一致的,我们相信,如果重构恰当,必定对项目是有利无害的。

如果能够符合以下几个规则,也许可以很大程度减少QA对重构的顾虑:

1. 充足的自动化测试是保障输出结果的一个有效途径。不管对于历史代码还是新代码,进行重构的前提应该是确保这部分功能已经有了足够的自动化测试覆盖,否则就不要进行重构。

2. 对于新功能的重构,应该融入到正常开发过程中,在故事验收之前已经完成了相应的重构,因为自动化测试不是万能的,也不可能达到100%的覆盖率,所以还是需要QA的验证的。

3. 小步前进,尽量避免或者减少大的重构,这样可以减少突然增多的回归测试工作量,也会减少功能被破坏的风险。

4. 对于一些核心功能或者组件,进行重构之前尽早告知团队QA,以便QA可以合理安排测试计划。

5. 尽量避免在产品上线前进行重构,不仅可以减轻QA回归测试的负担,也会减少功能破坏后来不及修复的风险。

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

推荐阅读更多精彩内容

  • 文/刘建华 经常听到开发人员抱怨 ,“这么烂的代码,我来重构一下!”,“这代码怎么能这么写呢?谁来重构一下?”,“...
    ThoughtWorks阅读 570评论 3 8
  • Android 自定义View的各种姿势1 Activity的显示之ViewRootImpl详解 Activity...
    passiontim阅读 171,431评论 25 707
  • 先说项目开发过程中团队人员的分工协作。 一 人员安排 毕业至今的大部分项目都是独立完成,虽然也有和其他同事协作的时...
    SnowflakeCloud阅读 10,736评论 3 59
  • 你是风中的一粒尘土,飘落我的心间,挥之不去;为此我紧闭双目,怕你在我眼中修篱种树。最怕风停了,雨不住;雪不融,太炫...
    孤灯夏阅读 440评论 2 19
  • 每个人的一生中都会有刻骨铭心的经历,遇到难以忘怀的人。我们会从经历的大大小小的事情中学会某些东西,比如做人、做事。...
    兰因素果阅读 460评论 0 0