什么时候需要重构,当你在项目代码里面嗅到这个味道的时候,就要进行重构。

 

首个介绍的味道是重复代码的味道。

 

它表现出来的特征是这些:

 

1.一个类里面,两个函数中,含有相同的代码,类似的代码;或者三个函数中,含有相同的代码,类似的代码。这是重复代码的味道。

 

2.两个互为兄弟的子类内含有相同的表达式。这也是重复代码的味道。

 

3.两个不相关的类,出现了重复代码。这也是重复代码的味道。

 
 

如何处理,如何重构

 

1.对于两个函数,出现重复代码,使用Extract Method方法。把某段代码组织到一个新的函数中,然后,将原来函数的某段代码替换为对新函数的调用。

 
 

Extract Method手法介绍
使用Extract Method手法处理原函数某段重复代码时,会面遇到如下几种情况:

 

a.该段代码没有局部变量。
对于a,直接将代码复制到一个新建的函数体中。

 
 

b.该段代码有局部变量。
如果,在段代码中,只是读取局部变量,并没有修改局部变量;或者,局部变量是个对象,这段代码只是调用该对象的某个方法。那么,只需要把这个局部变量当作新建函数的参数便可。

 
 

c.该段代码对局部变量再赋值。
被提炼代码段中,对源函数的某个局部变量进行了修改。
c1.并且,在被提炼代码段的后面,使用该修改后的局部变量。
这个处理手法是这样:
1.局部变量被修改后的结果作为新建函数的返回值。
2.源函数接收新建函数的返回值。

 

遵守的准则:一个函数,只有一个返回值。

 
 

------------------------------------------------------------------------------------------

 

2.两个互为兄弟的子类内含有相同的表达式。这也是重复代码的味道。

 

处理方式,首先使用Extract Method手法;然后,再对Extract Method手法处理后的结果使用Pull up Method,把提炼出来的代码推入超类。

 

Pull up Method(函数上移)手法介绍

 
针对不同情况的做法:

 

1.如果,兄弟类中,出现了相同的函数体。那么,在超类中新建一个函数,存放函数体,在各个子类冲调用该新建函数。

 

2.如果,兄弟类中,出现了相同的函数体,但是,该函数体还使用了所在类的一个字段,可以把该字段也提升到超类。

 

3.如果,兄弟类中,出现了相同的函数体,并且该函数体,还调用了所在类的一个函数。那么,可以考虑,在超类中声明一个抽象函数,由子类实现。然后,在新建的函数体中,调用超类声明的抽象函数。

 

4.兄弟类,有时候,不是出现完全相同的函数体,但是,是相似,这些函数体可以通过某种形式的参数调整,最后将相似的多个函数体调整为一个可以使用某种参数表示新函数。

 
 

-------------------------------------------------------------------------------------------------------

3.Form Template Method

 

两段函数体,它们以相同的顺序执行类似的操作,区别只是每个具体操作是不同的。那么,这是执行顺序的相同,也可以列为重复代码。

 
如下,兄弟类,各自某段函数执行操作顺序相同,每个操作的含义相同,区别只是操作的具体实现不同。
那么,可以这样做,把每段操作,抽象为一个父类函数。
然后,父类新建一个函数,该函数包含的是兄弟类中执行顺序和操作含义相同的代码。
 
比如:
类A
double base = _units*_rate*0.5; ----------------getBaseAmount();
double tax = base*Site.TAX_RATE*0.2;---------getTaxAmount();
return base+tax;-----------------return getBaseAmount()+getTaxAmount();
 
类B
步骤一样,操作步骤含义一样。
 
将getBaseAmount(),getTaxAmount()提到超类中,作为抽象函数,由具体的子类实现。
在超类中,实现 getBillableAmount()函数,它的返回值是getBaseAmount()+getTaxAmount()。
这样,就可以将类B,类A中类似的代码,提取为超类的 getBillableAmount()函数。
 

---------------------------------------------------------------------------

4.Extract Class 提炼类
 
两个不相关的类,出现了重复代码。这也是重复代码的味道。针对这种重复代码的重构手段,是使用Extract Class方法。
 
 
Extract Class手法的介绍
 
一个类,随着开发的进行,你不断往类里面添加方法和字段,不断给类添加各种职责。有一天,你回头,再来看这个类的时候,你发现,这个类过于庞大。
于是,你需要对这个类进行分解,提炼。
提炼的步骤简单的概括为:
1.关键是识别出类里面的各个职责。比如,在一个Person类中,识别出了专门用来打电话的代码,将电话相关的代码提炼到一个TelephoneNumber类中。这样就分离出了Person类中,打电话的职责。
 
 2.然后,在Person类中,使用一个提炼出来的类,作为成员。Person类中,原来需要的打电话功能,可以委托这个TelephoneNumber成员来执行。只保留,从Person到TelephoneNumber的单向连接。
 
3.决定,提炼出来的类,是一个引用类,还是一个值不可变的类。引用类,就是,值会被修改;值不可变类,则是值不会被修改,在首次初始化之后。
 
4.决定,提炼出来的类,是只放在原来的类中,作为原来类中的成员;还是公开,public,被其它类使用。这要考虑。
 
Extract Class手法处理两个不相关的类中的重复代码
 
1.识别出,两个类中,相同的重复代码。
2.将重复代码,提炼到一个新建的类中。
3.新建的类,包含了原来两个类的重复代码。新建的类,作为原来两个类的成员。
4.原来的两个类,委托新建的成员类进行访问,获得原有重复代码的功能。

重复代码Duplicated Code---要重构的信号的更多相关文章

  1. 代码的坏味道(14)——重复代码(Duplicate Code)

    坏味道--重复代码(Duplicate Code) 重复代码堪称为代码坏味道之首.消除重复代码总是有利无害的. 特征 两个代码片段看上去几乎一样. 问题原因 重复代码通常发生在多个程序员同时在同一程序 ...

  2. 【重构】 代码的坏味道总结 Bad Smell (一) (重复代码 | 过长函数 | 过大的类 | 过长参数列 | 发散式变化 | 霰弹式修改)

    膜拜下 Martin Fowler 大神 , 开始学习 圣经 重构-改善既有代码设计 . 代码的坏味道就意味着需要重构, 对代码的坏味道了然于心是重构的比要前提; . 作者 : 万境绝尘 转载请注明出 ...

  3. 重构 之 总结代码的坏味道 Bad Smell (一) 重复代码 过长函数 过大的类 过长参数列 发散式变化 霰弹式修改

    膜拜下 Martin Fowler 大神 , 开始学习 圣经 重构-改善既有代码设计 . 代码的坏味道就意味着需要重构, 对代码的坏味道了然于心是重构的比要前提; . 作者 : 万境绝尘 转载请注明出 ...

  4. For循环重复代码的重构

    DRY(don't repeat yourself),重复往往是代码腐烂的开始,我们一般的处理手法是将重复的代码提取成一个方法,然后用新方法替换掉原来的代码. 但是对于for循环里面的重复代码要如何处 ...

  5. IntelliJ IDEA “Finds duplicated code”提示如何关闭

    发现重复的代码这个提示真的很烦啊,我们怎么关闭他呢. 设置在这里: Settings -> Editor -> Inspections -> General -> Duplic ...

  6. IntelliJ IDEA 自动导入包 关闭重复代码提示

    idea可以自动优化导入包,但是有多个同名的类调用不同的包,必须自己手动Alt+Enter设置 设置idea导入包 勾选标注 1 选项,IntelliJ IDEA 将在我们书写代码的时候自动帮我们优化 ...

  7. IntelliJ IDEA 自动导入包 快捷方式 关闭重复代码提示

    idea可以自动优化导入包,但是有多个同名的类调用不同的包,必须自己手动Alt+Enter设置 设置idea导入包 勾选标注 1 选项,IntelliJ IDEA 将在我们书写代码的时候自动帮我们优化 ...

  8. 关闭Found duplicated code

    IDEA中的这个“发现重复代码 - Found duplicated code“的这个提示甚是烦躁. Settings —> Editor —> Inspections —> Gen ...

  9. Large Class--过大的类--要重构的信号

    如果想利用单个类做太多事情,其内往往就会出现太多实例变量.一旦如此,Duplicated Code也就接踵而至.     解决方法:     1.将类内彼此相关的变量,将它们放在一起.使用Extrac ...

随机推荐

  1. Thunder团队第六周 - Scrum会7

    Scrum会议7 小组名称:Thunder 项目名称:i阅app Scrum Master:杨梓瑞 工作照片: 参会成员: 王航:http://www.cnblogs.com/wangh013/ 李传 ...

  2. 3dContactPointAnnotationTool开发日志(十四)

      貌似每次让用户手动输入文件路径太不人道了,于是参考Unity 实用教程 之 调用系统窗口选择文件或路径增加了让用户浏览文件的功能,点击输入框旁边的+就可以找到文件并加载进来:   貌似调整位置再计 ...

  3. 网卡多ip 再看arp; arp队列也会缓存skb

    [结论] 当协议失效的时候,skb会挂载arp的neigt的一个链表上,然后直接返回了,相当于数据包发下了,当arp收到数据包去修复neigh的目的地址的时候,会把之前所有的neihe中等待的skb全 ...

  4. 第52天:offset家族、scroll家族和client家族的区别

    一.offset家族 1.offsetWidth offsetHeight offsetLeft offsetTop offsetParent共同组成了offset家族,用来获取元素尺寸. offse ...

  5. java基础简介

    一.软件开发 软件:是由数据和指令组成的(例:计算器) 如何实现软件开发呢?    就是使用开发工具和计算机语言做出东西来 二.常用dos命令 d: 回车 盘符切换 dir(directory):列出 ...

  6. WPF 进度条ProgressBar

    今天研究了一下wpf的进度条ProgressBar 1.传统ProgressBar WPF进度条ProgressBar 这个控件,如果直接写到循环里,会死掉,界面会卡死,不会有进度.需要把进度条放到单 ...

  7. 【题解】SHOI2014概率充电器

    首先发现答案就是每个节点有电的概率之和.有电的概率牵扯太广不好求,所以转化为求没有电的概率.这题最难的部分在于:一个节点如果有电,可以来自儿子,也可以来自父亲.我们考虑将这两个部分分离开来:建立状态 ...

  8. 【题解】HNOI2004敲砖块

    题目传送门:洛谷1437 决定要养成随手记录做过的题目的好习惯呀- 这道题目乍看起来和数字三角形有一点像,但是仔细分析就会发现,因为选定一个数所需要的条件和另一个数所需要的条件会有重复的部分,所以状态 ...

  9. CF724E Goods transportation 最小割 DP

    照惯例CF的题不放原题链接... 题意:一个序列上有n个点,每个点有权值pi和si.表示这个点一开始有pi个物品,最多可以卖出si个物品,每个点都可以把物品向编号更大的点运输,但是对于i < j ...

  10. [AHOI2009]最小割 最小割可行边&必须边

    ~~~题面~~~ 题解: 做这题的时候才知道有最小割可行边和必须边这种东西..... 1,最小割可行边, 意思就是最小割中可能出现的边. 充要条件: 1,满流 2,在残余网络中找不到x ---> ...