重构·改善既有代码的设计.02之代码的“坏味道”

简介: 之前在《重构·改善既有代码的设计.01》中初步了解了重构的基本前提,基础原则等入门知识。今天我们继续第二更......

前言

之前在《重构·改善既有代码的设计.01》中初步了解了重构的基本前提,基础原则等入门知识。今天我们继续第二更......

微信图片_20230305095040.jpg

识别代码的坏味道

Duplicated Code

重复代码。最单纯的Duplicated Code就是“同一个类中含有相同的表达式”或“两个互为兄弟的子类内含有相同表达式”。

如果你在一个以上的地点看到相同的程序结构,那么可以肯定:设法将他们合二为一,程序会变得更好。

Long Method

过长函数。当感觉需要用注释来说明点什么的时候,我们就把需要说明的东西写进一个独立函数中,并以其用途(而非实现手法)命名。

如何提炼一段代码,一个很好的技巧是:寻找注释。

Large Class

过大的类。通常如果类内的数个变量有着相同的前缀或字尾,这就意味有机会把它们提炼到某个组件内。

Long Parameter List

过长参数列表。对于OO(面向对象)语言来说,只需传给它足够的、让函数能从中获得自己需要的东西就行了。函数需要的东西多半可以在函数的宿主类中找到。

Divergent Change

发散式变化。某个类经常因为不同的原因在不同的方向上发生变化。指一个类受多种变化影响。

Shotgun Surgery

散弹式修改。如果每遇到某种变化,你都必须在许多不同的类内做出许多修改。顾名思义,霰弹枪发散,修改一个东西,发现修改的代码散布在四处。可以考虑把所有需要修改的代码放进同一个类中。指一个变化引发多个类相应修改。

Feature Envy

依恋情结。其实就是代码职责。函数对某个类的兴趣程序高于对自己所处类的兴趣。如果某个函数为了计算某个值,需要从另一个对象中调用几乎一般的取值函数,那么就该迁移到它该去的地方。

Data Clumps

数据泥团。评判方法是:删除众多数据项中的一项,其他数据项会不会因此失去意义?如果是,那么就是个明显的信号。你需要为他们产生一个新对象。

Primitive Obsession

基本类型偏执。

Switch Statements

switch惊悚现身。switch语句的问题在于重复。一般switch语句可以考虑用多态替换。

Parallel Inheritance Hierarchies

平行继承体系。单你为某个类增加一个子类,必须也为另一个类相应增加一个子类。如果你发现某个继承体系的类名称前缀和另一个继承体系的类名称前缀完全相同,那么便是这种坏味道。

Lazy Class

冗赘类。一个类的所得不值其身价,就让他消失。

Speculative Generality

夸夸其谈未来性。“我想我们总有一天需要做这事”,并因而企图以各式各样的钩子和特殊情况来处理一些非必要的事情。

Temporary Field

令人迷惑的暂时字段。比如某对象某个实例变量仅为某种特定情况而设。我们通常认为对象在所有时候都需要他的所有变量,在变量未被使用的情况下猜测当初其设置的目的,会很抓狂。

Message Chains

过度耦合的消息链。如一个对象请求另一个对象,然后再向后请求另一个对象,然后再向后请求另一个对象......

Refused Bequest

被拒绝的遗赠。继承体系中,子类应该继承超类的函数和数据。但如果子类复用了超累的行为(实现),却又不愿意支持超类的接口。就像他们得到所有礼物,却只从中挑选几样来玩。按传统的说法,这就意味着继承体系设计错误。

Comments

过多的注释。当你感觉需要攥写注释时,请先尝试重构,试着让所有注释都变得多余。试想一种情况:当你看到一段代码,有着很长的注释,然后你发现,这些注释之所以存在是因为代码写的很糟糕。视图用注释来说明函数的用意以及实现。这时候应该使用重构手法把这类坏味道去除,重构之后你会发现注释变得多余,因为代码已经说明了一切。

构筑测试体系

自测代码的价值:修复错误通常比较快,但找出错误却是噩梦一场。 “花合理实践抓出大多数bug”好过“穷尽一生抓出所有bug”
  1. 重构的第一步:构建可靠的测试环境
  2. 类应该包含他们自己的测试代码
  3. 确保所有测试都完全自动化,让它们检查自己的测试结果
  4. 一套测试就是一个强大的bug侦测器,能够大大缩减查找bug所需要的时间
  5. 攥写测试diamagnetic的最有用时机,是在开始编程之前
  6. 在重构之前先确保代码能够进行自我测试
  7. 频繁的进行测试。每次编译请把测试页考虑进去,每天至少执行每个测试一次
  8. 编写测试代码时, 一开始先让它们失败,测试一下测试代码的机制可以运行
  9. 每当收到一个bug报告,请先写一个单元测试来暴露这个bug
  10. 观察类该做的所有事情,然后针对任何功能的任何一种可能失败情况,进行测试
  11. 测试的要诀:测试你最担心出错的部分
  12. 编写未臻完善的测试并实际运行,好过对完美测试的无尽等待

作者提到,当测试数量达到一定程度之后,继续增加测试带来的效益会呈现递减态势,而非持续递增;如果试图编写太多测试,你也可能因为工作量太大而气馁,最后什么都写不成。你应该把测试集中在可能出错的地方。观察代码,看哪儿变得复杂;观察函数,思考哪些地方可能出错。

不要因为测试无法捕捉所有bug就不写测试,因为测试的确可以捕捉到大多数bug。

小结

到此,刚看完整本书的5个章节。代码的坏味道和构筑测试体系加深了对于重构的认知。重构和设计模式等诸多思想一样,是需要反复学习,反复实践的。重构的技术就是以微小的步伐修改程序。希望真正吸收这些之后,能够看到一个不一样的代码世界。借此,共勉~

持续学习更新中......

相关文章
|
设计模式
重构·改善既有代码的设计.03之重构手法(上)
之前的重构系列中,介绍了书中提到的重构基础,以及识别代码的坏味道。今天继续第三更,讲述那些重构手法(上)。看看哪些手法对你的项目能有所帮助......
19256 1
重构·改善既有代码的设计.03之重构手法(上)
|
设计模式
重构·改善既有代码的设计.04之重构手法(下)完结
重构改善既有代码的设计完结篇,汇总了全部的重构手法。看看哪些手法对你的项目能有所帮助…
7407 2
重构·改善既有代码的设计.04之重构手法(下)完结
|
设计模式 程序员 开发者
重构·改善既有代码的设计.01之入门基础
近期在看Martin Fowler著作的《重构.改善既有代码的设计》这本书,这是一本经典著作。书本封面誉为软件开发的不朽经典。书中从一个简单的案例揭示了重构的过程以及最佳实践。同时给出了重构原则,何时重构,以及重构的手法。用来改善既有代码的设计,提升软件的可维护性。
630 1
重构·改善既有代码的设计.01之入门基础
|
Java C语言
看似无害的代码如何搞垮系统
编程就像魔法。最近遇到一个诡异的问题:添加一段看似无害的简单代码后,系统原有功能不可用了。 ## 复现演示 jdk 8 可使用如下演示代码复现这个问题。 `TaskCenter` 是一个任务框架,可添加多个任务,随后框架将执行这些任务。 `First` 任务是新增代码,看起来简单无害,且看不出对原有任务 `Count` 有何影响,但添加 `First` 任务后,其自身执行正常,原本正常的 `C
127 0
|
Java 数据库 容器
项目重构,我是如何优化大量屎一样的 if else 代码的?
  目录:   if else策略模式1、首先抽象业务处理器2、将业务处理器和其支持处理的类型放到一个容器中,java里Map就是最常用的容器之一3、定义不同的处理器4、测试类   前段时间,我将公司系统中的批量审单的功能进行了重构,用到了java的并发编程进行异步化处理,数据库的乐观锁机制处理多线程并发更新数据。   其中批量审单的业务处理涉及到多种任务类型,对应不同的业务方法进行处理,比如转仓,转快递,添加赠品,删除赠品,拆分订单,批量驳回,批量作废等等,其中就用到了策略模式。
260 0
|
UED
为什么有些设计初衷很好,结果却很糟糕
本文讲的是为什么有些设计初衷很好,结果却很糟糕,刚刚,在她意识到自己已经买了机票之后,她又打开另一个标签页,预定这次旅行的酒店房间,又租了一辆车。随后返回到美国航空的标签页获取她的确认编号,同时记录在她的日历上。
1398 0

相关实验场景

更多