代码的坏味道
重复代码(Duplicated Code)
如果你在一个以上的地点看到相同的程序结构,那么可以肯定:设法将它们和而为一,程序会变得更好。
同一个类的2个函数含有相同的表达式,这时可以采用Extract Method (提炼函数)提炼出重复的代码,然后让这2个地点都调用被提炼出来的那段代码。
2个互为兄弟的子类内含相同表达式,只需对2个类都是用Extract Method (提炼函数),然后对被提炼出来的函数是用Pull Up Method (方法上移) ,将它推入超类。如果代码之间只是类似, 并非完全相同,那么就得运用Extract Method (提炼函数)将相似部分和差异部分隔开,构成单独一个的函数。然后你可能发现可以运用Form Template Method (塑造模板函数)获得一个 Template Method设计模式。如果有些函数以不同的算法做相同的事,你可以选择其中较清晰地一个,并是用 Substitute Algorithm (替换算法)将其他函数的算法替换掉。
如果2个毫不相关的类出现 重复代码,你应该考虑对其中一个运用 Extract Class (提炼类),将重复代码提炼到一个独立类中,然后在另一个类内使用这个新类。但是,重复代码所在的函数可能只应该属于某个类,另一个类只能调用它,抑或这个函数可能属于第三个类,而另2个类应该引用这第三个类。你必须决定这个函数放在哪儿最合适,并确保它被安置后就不会再在其他任何地方出现。
过长函数(Long Method)
90%的场合里,要把函数变小,只需使用Extract Method (提炼函数),找到函数中适合集中在一起的部分,将它们提炼出来形成新函数。
如果函数内有大量的参数和临时变量,它们会对你的函数提炼形成障碍。你可以经常运用Replace Temp with Query (以查询取代临时变量),来消除这些临时元素。Introduce Parameter Object (引入参数对象),Preserve Whole Object (保持对象完整)则可以将过长的参数列变得简洁一些。
如果已经这么做了,仍然有太多的临时变量和参数,就应该使用 Replace Method with Method Object (以函数对象取代函数)。如何确定提炼哪一段代码呢?一个很好的技巧是:寻找注释。它们通常指出代码用途和实现手法之间的语义距离。如果代码前方有一行注释,就是在提醒你:可以将这段代码替换成一个函数,而且可以在注释的基础上给这个函数命名。就算只有一行代码,如果它需要以注释来说明,那也值得将它提炼到独立函数中。
条件表达式和循环常常也是提炼的信号。可以使用 Decompose Conditional (分解条件表达式)处理条件表达式。至于循环,应该将循环和其内的代码提炼到独立函数中。
过大的类(Large Class)
如果想利用单个类做太多的事,其内往往就会出现太多实例变量。这样 重复代码(Duplicated Code)就接踵而至了。
运用Extract Class (提炼类)将几个变量一起提炼到新类里。提炼类时应该选择类内彼此相关的变量,将它们放在一起。通常类内的数个变量有相同的前缀或字尾,这就意味有机会把它们提炼到某个组件内。如果这个组件适合作为一个子类,就可以运用 Extract Subclass (提炼子类)。
有时类并非在所有时刻都使用所有实例变量。可多次使用Extract Class (提炼类)和Extract Subclass (提炼子类)
一个类如果拥有太多代码,往往也适合使用Extract Class (提炼类)和Extract Subclass (提炼子类)。技巧:先确定客户端如何使用它们,然后运用 Extract Interface (提炼接口) 为每个使用方式提炼出一个接口。可帮助你看清楚如何分解这个类。
如果是个GUI类,可能需要把数据和行为移到一个独立的领域对象去。可能需要2边各保留一些重复数据,并保持2边同步。Duplicate Observed Data (复制”被监视数据”)告诉你该这么做。
过长参数列(Long Parameter List)
如果向已有的对象发出1条请求就可以取代1个参数,那么就运用 Replace Parameter with Methods (以函数取代参数)。在这里,”已有的对象”可能是函数所属类里的1个字段,也可能是另一个参数。还可以运用 Preserve Whole Object (保持对象完整)将来自同一对象的一堆数据收集起来,并以该对象替换它们。如果某些数据缺乏合理的对象归属,可使用 Introduce Parameter Object (引入参数对象)为它们制造出一个参数对象。
重要的例外:如果你明显不想造成”被调用对象”与”较大对象”间的某种依赖关系。这时候将数据从对象中拆解出来单独作为参数,也很合情理。但是请注意其所引发的代价。如果参数列太长或变化太频繁,就需要重新考虑自己的依赖结构了。
发散式变化(Divergent Change)
如果某个类经常因为不同的原因在不同的方向上发生变化,发散式变化(Divergent Change)就出现了。那么此时也许将这个对象分成2个会更好,这么一来每个对象就可以只因1种变化而需要修改。针对某以外界变化的所有相应修改,都只应该发生在单一类中,而这个新类内的所有内容都应该反应此变化。为此应该找出某特定原因而造成的所有变化,然后运用Extract Class (提炼类)将它们提炼到另一个类中。
散弹式修改(Shotgun Surgery)
散弹式修改(Shotgun Surgery)类似 发散式变化(Divergent Change),但恰恰相反。如果每遇到某种变化,都必须在许多不同的类中做出许多小修改,你所面临的坏味道就是散弹式修改(Shotgun Surgery)。如果需要修改的代码散布在四处,你不但很难找到它们,也很容易忘记某个重要的修改。
可以使用 Move Method (搬移函数)和 Move Field (搬移字段)把所有需要修改的代码放进同1个类。如果暂时没有合适的类,就创建一个。通常可以运用 Inline Class (将类内联化) 把一系列相关行为放进同一个类。这可能会造成少量的发散式变化(Divergent Change),但可以轻易出来它。
发散式变化(Divergent Change)是指”一个类受多种变化的影响”,散弹式修改(Shotgun Surgery)则是指”一种变化引起多个类相应修改”。这2种情况都会希望整理代码,使”外界变化”与”需要修改的类”趋于一一对应。
依恋情结(Feature Envy)
函数对某个类的兴趣高过对自己所处的类,通常的焦点就是数据,某个函数为了计算某个值,从另一个对象那儿调用几乎半打的取值函数。这时一个运用 Move Method (搬移函数)把它移到自己该去的地方。有时候函数中只有一部分受这种依恋之苦,这时候使用Extract Method (提炼函数)把这部分提炼到独立函数中,再使用Move Method (搬移函数)带它去它的梦中家园。
一个函数往往会用到几个类的功能,那么它究竟该被置与何处呢?原则:判断哪个类拥有最多被此函数使用的数据,然后就把这个函数和那些数据摆在一起。如果先以Extract Method (提炼函数)将这个函数分解为数个较小函数并分别置于不同地点,上述步骤就比较容易完成了。
数据泥团(Data Clumps)
常常可以在很多地方看到相同的3、4项数据:2个类中相同的字段、许多函数签名中相同的参数。这些总是绑在一起出现的数据真应该拥有属于它们自己的对象。首先找出这些数据以字段形式出现的地方,运用Extract Class (提炼类)将它们提炼到一个独立对象中。然后将注意力转移到函数签名上,运用Introduce Parameter Object (引入参数对象)或Preserve Whole Object (保持对象完整)为它减肥。这么做的直接好处是可以将很多参数列缩短,简化函数调用。不必在意数据泥团(Data Clumps)只用上新对象的一部分字段,只要以新对象取代2(或更多)个字段,就值得了。
一个好的评判方法是:删除众多数据中的一项。这么做,其他数据有没有因而失去意义?如果它们不再有意义,这就是个明确信号:你应该为它们产生一个新对象。
基本类型偏执(Primitive Obsession)
对象技术的新手通常不愿意在小任务上运用小对象—像是结合数值和币种的money类,由一个起始值和结束值组成的range类等。你可以使用 Replace Data Value with Object (以对象取代数据值)将原本单独存在的数据值替换为对象。如果想要替换的数据值是类型码,而它并不影响行为,则可以运用 Replace Type Code with Class (以类取代类型码)将它替换掉。如果你有与类型码相关的条件表达式,可运用 Replace Type Code with Subclass (以子类取代类型码)或 Replace Type Code with State/Strategy (以状态/策略取代类型码)加以处理。
如果你有一组应该总是被放在一起的字段,可运用Extract Class (提炼类)。如果你在参数列中看到基本数据类型,不妨试试 Introduce Parameter Object (引入参数对象)。如果你发现自己正从数组中挑选数据,可运用 Replace Array with Object (以对象取代数组)
switch 惊悚现身(Switch Statements)
面向对象程序的一个最明显特征就是:少用switch或(case)语句。从本质上说,switch语句的问题在于重复。你常会发现switch语句散布于不同地点。如果要为它添加一个新的case子句,就必须找到所有switch语句并修改它们。面向对象中的多态概念可为此带来优雅的解决办法。
大多数时候,一看到switch语句,就应该考虑以多态来替换它。问题是多态该出现在哪?switch语句常常根据类型码进行选择,你要的是“与该类型码相关的函数或类”,所以应该使用 Extract Method (提炼函数)将switch语句提炼到一个独立函数中,再以 Move Method (搬移函数)将它搬移到需要多态性的那个类里。此时你必须决定是否使用 Replace Type Code with Subclass (以子类取代类型码)或 Replace Type Code with State/Strategy (以状态/策略取代类型码)。一旦完成这样继承结构后,就可以运用 Replace Conditional with Polymorphism (以多态取代条件表达式)了。
如果你只是在单一函数中有些选择事例,且并不想改动它们,那么多态就不必要了。这时可运用 Replace Parameter with Explicit Methods (以明确函数取代参数)。如果你的选择条件之一是null,可以试试 Introduce Null Object (引入Null 对象)
平行继承体系(Parallel Inheritance Hierarchies)
平行继承体系其实是散弹式修改(Shotgun Surgery)的特殊情况。在这种情况下,每当你为某个类增加1个子类,必须也为另一个类相应增加1个子类。如果你发现某个继承体系的类名前缀和另一个继承体系的类名前缀完全相同,便是闻到了这种坏味道。
消除这种重复性的一般策略是:让一个继承体系的实例引用另一个继承体系的实例。如果再接再厉运用 Move Method (搬移函数)和Move Field (搬移字段),,就可以将引用端的继承体系取消。
冗赘类(Lazy Class)
如果某些子类没有足够的工作,试试Collapse Hierarch (折叠继承体系)。对应几乎没用的组件,应该以Inline Class (将类内联化)对付它们。
夸夸其谈未来性(Speculative Generality)
如果你的某个抽象类其实没有太大作用,请运用 Collapse Hierarch (折叠继承体系)。不必要的委托可运用 Inline Class (将类内联化)除掉。如果函数的某些参数未被用上,可对它实施 Remove Parameter (移除参数)。如果函数名称带有多余的抽象意味,应该对它实施Rename Method (函数改名)
如果函数或类的唯一用户是测试用例,这就飘出了坏味道 夸夸其谈未来性(Speculative Generality)。 如果有这样的函数或类,请把它们连同其测试用例一并删除。但如果它们的用途是帮助测试用例检测正当功能,则不能删除。
令人迷惑的临时字段(Temporary Field)
有时你会看到这样的对象:其内某个实例变量仅为某种特定情况而设。这样的代码让人不易理解,因为你通常认为对象在所有时候都需要它的所有变量。在变量未被使用的情况下猜测当初设置目的,会让你发疯。
请使用Extract Class (提炼类)给这些变量创造一个家,然后把所有和这些变量相关的代码都放进这个新家,也许你还可以使用 Introduce Null Object (引入Null 对象)在变量不合法的情况下创建一个null对象,从而避免写出条件式代码。
如果类中有一个复杂算法,需要好几个变量,往往就可能导致坏味道令人迷惑的临时字段(Temporary Field)出现。由于实现者不希望传递一长串参数,所以他把这些参数都放进字段。但是这些字段只在使用该算法时才有效,其他情况下只会让人迷惑。这时可以利用 Extract Class (提炼类)把这些变量和其相关函数提炼到一个独立的类中。提炼后的新对象将是一个函数对象。
过度耦合的消息链(Message Chains)
如果你看到用户向一个对象请求另一个对象,然后再向后者请求另一个对象,然后再请求另一个对象……这就是消息链。实际代码中你看到的可能是一长串getThis()或一长串临时变量。采取这种方式,意味客户代码将与查找过程中的导航紧密耦合。一旦对象间关系发生任何变化,客户端就不得不做出相应的修改。
这时候应该使用 Hide Delegate (隐藏委托关系)。你可以在消息链的不同位置进行这种重构。理论上可以重构消息链上任何对象,但这么做往往会把一系列对象都变成Middle Man(中间人)。通常更好的选择是:先观察消息链最终得到的对象是用来干什么的,看看能否以 Extract Method (提炼函数)把使用该对象的代码提炼到一个独立函数中,再运用 Move Method (搬移函数) 把这个函数推入消息链。
中间人(Middle Man)
对象的基本特征之一就是封装:对外部世界隐藏其内部细节。封装往往伴随委托。比如说你问你主管是否有时间参加一个会议,他就把这个消息“委托”给他的记事簿,然后才能回答你。你没必要知道这位主管到底使用传统记事簿或电子记事簿或秘书来记录自己的约会。
但是人们可能过度运用委托。你也许会看到某个类有一半的函数都委托给其他类,这样就是过度运用。这时应该使用 Remove Middle Man (移除中间人),直接和真正负责的对象打交道。如果不干实事的函数只有少数几个,可以运用 Inline Method (内联函数)把它们放进调用端。如果这些中间人还有其他行为,可以运用 Replace Delegation with Inheritance (以继承取代委托)把它们变成实责对象的子类,这样你即可以扩展原对象的行为,又不必负担那么多的委托动作。
狎昵关系(Inappropriate Intimacy)
有时候你会看到2个类过于亲密,花费太多时间起探究彼此的private成分。你可以采用Move Method (搬移函数)和 Move Field (搬移字段)帮他们划清界限。你也可以看看是否可以运用 Change Bidirectional Association to Unidirectional (将双向关联改为单向关联)让其中一个类对另一个斩断情丝。如果2个类实在是情投意合,可以运用 Extract Class (提炼类)把2者共同点提炼到一个安全地点,让它们坦荡的使用这个新类。或者可以尝试运用 Hide Delegate (隐藏委托关系)让另一个类来为它们传递相思情。
继承往往造成过度亲密,因为子类对超类的了解总是超过后者的主观愿望,如果你觉得该让这个孩子独自生活了,请运用 Replace Inheritance with Delegation (以委托取代继承)让它离开继承关系。
异曲同工的类(Alternative Classes with Different Interfaces)
如果2个函数做同一件事,却有着不同的签名,请运用 Rename Method (函数改名)根据它们的用途重新命名。但这往往不够,请反复运用 Move Method (搬移函数)将某些行为移入类,直到2者的协议一致为止。如果你必须反复而赘余的移入代码才能完成这些,或许可运用Extract Superclass (提炼超类)。
不完美的库类(Incomplete Library Class)
复用常被视为对象的终极目的。不过复用的意义常被高估:大多数对象只要够用就好。但是无可否认,许多编程技术都建立在程序库的基础上。
库类的构筑者没用未卜先知的能力,不能因此责怪他们。麻烦的是库往往构造的不够好,而且往往不可能让我们修改其中的类使它们完成我们希望完成的工作。这是否意味着那些经过实践检验的技术,如今都派不上用场了?
幸好我们有2个专门应付这种情况的工具。如果你只想修改类库的一两个函数,可以运用Introduce Foreign Method (引入外加函数);如果想要添加一大堆额外行为,就得运用 Introduce Local Extension (引入本地扩展) 。
纯稚的数据类(Data Class)
所谓的Data Class是指:它们拥有一些字段,以及用于访问这些字段的函数,除此之外一无长物。这样的类只是不会说话的数据容器,它们几乎一定被其他类过分细琐的操控着。这些类早期可能拥有public字段,果真如此你应该在别人注意到它们之前,立刻运用 Encapsulated Field (封装字段)将它们封装起来。如果这些类含容器类的字段,你应该检查它们是不是得到了恰当的封装;如果没有,就运用 Encapsulated Collection (封装集合)把它们封装起来。对于那些不该被其他类修改的字段,请运用 Remove Setting Method (移除设置函数)。
然后,找出这些取值/设值函数被其他类运用的地点。尝试以 Move Method (搬移函数)把那些调用行为搬移到Data Class来。如果无法搬移这个函数,就运用 Extract Method (提炼函数)产生一个可搬移的函数。不久之后就可以运用 Hide Method (隐藏函数)把这些取值/设值函数隐藏起来了。
被拒绝的遗赠(Refused Bequest)
子类应该继承超类的函数和数据。但如果它们不想或不需要继承,又该这么办呢?它们得到所有礼物,却只从中挑选几样来玩。
按传统说法,这就意味着继承体系设计错误。你需要为这个子类新建一个兄弟类,再次运用 push down Method (函数下移)和 push down field (字段下移)把所有用不到的函数下推给那个兄弟。这样一来,超类就只持有所有子类共享的东西。你常常会听到这样的建议:所有超类都应该是抽象的。
既然使用“传统说法”这个略带贬义的词,你就可以猜到,我们不建议你这么做,起码不建议你每次都这么做。我们继承利用继承来复用一些行为,并发现可以很好的应用于日常工作。这也是一种坏味道,我们不否认,但气味通常并不强烈。所以我们说:如果Refused Bequest引起困惑和问题,请遵循传统忠告。但不必认为你每次都得那么做。十有八九这种坏味道很淡,不值得理睬。
如果子类复用了超类的行为,却有不愿支持超类的接口,Refused Bequest的坏味道就会变得浓烈。拒绝继承超类的实现,这点我们不介意;但如果拒绝继承超类的接口,我们不以为然。不过即使你不愿意继承接口,也不要胡乱修改继承体系,应用运用 Replace Inheritance with Delegation (以委托取代继承)来达到目的。
过多的注释(Comments)
我们之所以在这里提到注释,是因为人们常把它当做除臭剂来使用。常常会有这样的情况:你看到一段代码有着长长的注释,然后发现,这些注释之所以存在是因为代码很糟糕。
注释可以带我们找到代码中的坏味道。找到坏味道后,我们首先应该以各种重构手法把坏味道去除。完成之后我们常常会发现:注释已经变得多余了,因为代码已经清楚说明了一切。
如果你需要注释来解释一块代码做了说明,试试Extract Method (提炼函数);如果函数已经提炼出来,但还是需要注释来解释其行为,试试 Rename Method (函数改名);如果你需要注释说明某些系统的需求规格,试试 Introduce Assertion (引入断言)。
重构手法
重新组织函数
- Extract Method (提炼函数)
- Inline Method (内联函数)
- Inline Temp (内联临时变量)
- Replace Temp with Query (以查询取代临时变量)
- Introduce Explaining Variable (引入解释性变量)
- Split Temporary Variable (分解临时变量)
- Remove Assignments to Parameters (移除对参数的赋值)
- Replace Method with Method Object (以函数对象取代函数)
- Substitute Algorithm (替换算法)
在对象之间搬移特性
- Move Method (搬移函数)
- Move Field (搬移字段)
- Extract Class (提炼类)
- Inline Class (将类内联化)
- Hide Delegate (隐藏委托关系)
- Remove Middle Man (移除中间人)
- Introduce Foreign Method (引入外加函数)
- Introduce Local Extension (引入本地扩展)
重新组织数据
- Self Encapsulate Field (自封装字段)
- Replace Data Value with Object (以对象取代数据值)
- Change Value to Reference (将值对象改为引用对象)
- Change Reference to Value (将引用对象改为值对象)
- Replace Array with Object (以对象取代数组)
- Duplicate Observed Data (复制“被监视数据”)
- Change Unidirectional Association to Bidirectional (将单向关联改为双向关联)
- Change Bidirectional Association to Unidirectional (将双向关联改为单向关联)
- Replace Magic Number with SymBolic Constant (以字面常量取代魔法数)
- Encapsulated Field (封装字段)
- Encapsulated Collection (封装集合)
- Replace Record with Data Class (以数据类取代记录)
- Replace Type Code with Class (以类取代类型码)
- Replace Type Code with Subclass (以子类取代类型码)
- Replace Type Code with State/Strategy (以状态/策略取代类型码)
- Replace Subclass with Field (以字段取代子类)
简化条件表达式
- Decompose Conditional (分解条件表达式)
- Consolidate Conditional Expression (合并条件表达式)
- Consolidate Duplicate Conditional Fragments (合并重复的条件片段)
- Remove Control Flag (移除控制标记)
- Replace Nested Conditional with Guard Clauses (以卫语句取代嵌套条件表达式)
- Replace Conditional with Polymorphism (以多态取代条件表达式)
- Introduce Null Object (引入Null 对象)
- Introduce Assertion (引入断言)
简化函数调用
- Rename Method (函数改名)
- Add Parameter (添加参数)
- Remove Parameter (移除参数)
- Separate Query form Modifier (将查询函数和修改函数分离)
- Parameterize Method (令函数携带参数)
- Replace Parameter with Explicit Methods (以明确函数取代参数)
- Preserve Whole Object (保持对象完整)
- Replace Parameter with Methods (以函数取代参数)
- Introduce Parameter Object (引入参数对象)
- Remove Setting Method (移除设置函数)
- Hide Method (隐藏函数)
- Replace Constructor with Factory Method (以工厂函数取代构造函数)
- Encapsulate Downcast (封装向下转型)
- Replace Error Code with Exception (以异常取代错误码)
- Replace Exception with Test (以测试取代异常)
处理概括关系
- Pull Up Field (字段上移)
- Pull Up Method (方法上移)
- Pull Up Constructor Body (构造函数本体上移)
- push down Method (函数下移)
- push down field (字段下移)
- Extract Subclass (提炼子类)
- Extract Superclass (提炼超类)
- Extract Interface (提炼接口)
- Collapse Hierarch (折叠继承体系)
- Form Template Method (塑造模板函数)
- Replace Inheritance with Delegation (以委托取代继承)
- Replace Delegation with Inheritance (以继承取代委托)