重构——代码的坏味道

如果是尿布抽了,就换掉它。 —— Beck奶奶

3.1 Duplicated Code

最单纯的Duplicated Code是同一个类的两个函数含有相同的表达式;另一种常见的情况就是两个互为兄弟的子类内含相同的表达式,构筑超类或者使用模板方法;如果两个毫不相关的类出现Dulicated Code,提取独立类。

3.2 Long Method

拥有短函数的对象会获得比较好、比较长。间接层所能带来的全部利益——解释能力、共享能力、选择能力——都是由小型函数支持的。

3.3 Large Class

如果单个类想做太多的事情,期内往往会有太多的实例变量。这往往意味着Duplicated Code。

3.4 Long Parameter List

太长的参数无法理解,太多的参数同时也会容易造成前后不一致。

3.5 Divergent Change(发散式修改)

我们希望软件能够更容易被修改。如果某个类经常因为不同的原因在不同的方向上发生变化。最好能将类切开,这也是遵循单一职责原则。

3.6 Shotgun Surgery(散弹式修改)

Shotgun Surgery和Divergent Change类似,但是恰恰相反。如果每遇到某种变化,你都必须在许多不同类内做出许多小修改。
  总之,Divergent Change是指一个类受多种变化影响。Shortgun Surgery则是指一个变化引起多个类相应修改。

3.7 Feature Envy(依恋情节)

对象技术的全部要点在于:这是一种“将数据和对数据的操作行为包装在一起”的技术。有一种经典的气味:函数对某个类的兴趣搞过对自己所处类的兴趣。例如:某个类为了计算某个值,从某个类中获取几乎半打的取值函数。这种函数,应该将这个函数移动到它应该在的地方。
判断类的地址的原则:判断哪个类拥有最多被此函数使用的数据,然后该函数就乖乖去了那里。Strategy和Visitor就是对抗Divergent Change。最根本的原则:将总是一起变化的东西放在一起,保持变化永远只在一处发生。当然,你不得不多加一层抽象层次。

3.8 Data Clumps

数据项就像小孩子,喜欢成群结队地待在一块儿。常常可以在很多地方看到相同的三四项数据:两个类中的相同字段、许多函数签名中的参数。这些总是绑定在一起出现的数据真应该拥有属于它们自己的对象。
  一个好的评判方法:删除众多数据中的一项。这么做,其他数据没有失去意义?如果不再有意义,这就是明确的意义,告诉你应该为他们产生一个新对象。

3.9 Primitive Obsession(基本类型偏执)

大多数编程环境都有两种数据:结构类型允许你将数据组织成有意义的形式;基本类型则是构筑结构类型的积木块。结构总是会带来一定的额外开销。他们可能代表着数据中的表,如果仅仅为了做一两件事而穿件结构类型也可能显得太麻烦了。

3.10 Switch Statement(惊悚的switch)

面向对象程序的一个最棉线特征就是:少用switch(或case)语句。
  大多数的时候,一看到switch语句,你就应该考虑以多态来替代它;但是在单一函数中有些选择事例,且并不想改动它们,那么多态就有点杀鸡用牛刀了。

3.11 Parallel Inheritance Hierarchies(平行继承体系)

其实是Shotgun Surgery的特殊情况。即一个类增加一个子类,另一个类也必须增加一个子类。
  一般则略:让一个继承体系的实例引用另一个体系的实例。

3.12 Lazy Class(冗余类)

如果一个类不值其身价,它就应该消失。

3.13 Speculative Generality(夸夸其谈未来性)

因为主观臆测的未来性(未来某天我们需要做这件事情),并其余以各种各样的钩子和特殊情况来处理一些非必要的事情。
  注意:当函数或者类的唯一用户是测试用例,坏味道就飘出来了。

3.14 Temporary Field(令人迷惑的临时字段)

对象内某个实例变量仅为某特定情况而设。这样的代码会让让人无法理解,通常认为对象在所有时候都需要它所有变量。猜测变量的用途会让人crazy。

3.15 Message Chain(过度偶尔得消息链)

如果你可以看到用户向一个对象请求另一个对象,然后再向后者请求另一个对象,然后再请求另一个对象……这就是消息链。采取这种方式,意味着用户代码将与查找过程中的导航结构紧密耦合。一旦对象间的关系发生任何变化,客户端也的变化。
  可以使用消息队列模式来进行重构。

3.16 Middle Man

对象的基本特征之一就是封装——对外部空间隐藏起内部实现细节。封装也代表着委托。但是也有时候会成为过度的委托。我们会看到某个类接口将其一半的实现委托给其他类。

3.17 Inappropriate Intimacy(狎昵关系)

有时你会看到两个类过于亲密,花费太多时间探究彼此的private成分。过分狎昵必须分开。集成往往造成过度亲密,因为子类对超类的了解总是超过后者的主观愿望。需要的时候,让类离开类继承关系链。

3.18 Alternative Classes with Diffierent Interaces(异曲同工的类)

如果两个函数做同一件事情,却有着不同的签名,请根据用于合并和修改。

3.19 Incomplete Library Class(不完美的库类)

复用被视为对象的终极目的。不过,往往复用的意义经常被高估——大多数对象只要够用就好。有时候需要根据需要修改类库。

3.20 Data Class(纯稚的数据类)

Data Class是指它们拥有一些字段,以及用于访问(读写)这些字段的函数,除此之外一无长处。这种类往往被其他类过分操纵。这些类可能拥有public字段,果真如此,需要在别人注意到之前封装起来;如果这些类内含容器类的字段,你应该前叉它们是否得到了恰当的封装。如果没有则正确封装它们。然后找出取值/设值函数被其他类运用的地点。尝试以Move Method将调用行为移动到Data Class中。

3.21 Refused Bequest(被拒绝的遗赠)

子类应该继承超类的函数和数据。但是如果不想继承所有,怎么选择其中一部分。这就意味着继承体系设计错误。给子类设计兄弟类,下移动一部分字段到兄弟类中。从而正确构建体系。

3.22 Comments(过多的注释)

Comments本身是带着香味,然而不恰当的注释是相当臭气味。只有当不知道如何重构的时候,使用注释才是优质的方式。

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

推荐阅读更多精彩内容