重复代码Duplicated Code---要重构的信号
什么时候需要重构,当你在项目代码里面嗅到这个味道的时候,就要进行重构。
首个介绍的味道是重复代码的味道。
它表现出来的特征是这些:
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
两段函数体,它们以相同的顺序执行类似的操作,区别只是每个具体操作是不同的。那么,这是执行顺序的相同,也可以列为重复代码。
---------------------------------------------------------------------------
重复代码Duplicated Code---要重构的信号的更多相关文章
- 代码的坏味道(14)——重复代码(Duplicate Code)
坏味道--重复代码(Duplicate Code) 重复代码堪称为代码坏味道之首.消除重复代码总是有利无害的. 特征 两个代码片段看上去几乎一样. 问题原因 重复代码通常发生在多个程序员同时在同一程序 ...
- 【重构】 代码的坏味道总结 Bad Smell (一) (重复代码 | 过长函数 | 过大的类 | 过长参数列 | 发散式变化 | 霰弹式修改)
膜拜下 Martin Fowler 大神 , 开始学习 圣经 重构-改善既有代码设计 . 代码的坏味道就意味着需要重构, 对代码的坏味道了然于心是重构的比要前提; . 作者 : 万境绝尘 转载请注明出 ...
- 重构 之 总结代码的坏味道 Bad Smell (一) 重复代码 过长函数 过大的类 过长参数列 发散式变化 霰弹式修改
膜拜下 Martin Fowler 大神 , 开始学习 圣经 重构-改善既有代码设计 . 代码的坏味道就意味着需要重构, 对代码的坏味道了然于心是重构的比要前提; . 作者 : 万境绝尘 转载请注明出 ...
- For循环重复代码的重构
DRY(don't repeat yourself),重复往往是代码腐烂的开始,我们一般的处理手法是将重复的代码提取成一个方法,然后用新方法替换掉原来的代码. 但是对于for循环里面的重复代码要如何处 ...
- IntelliJ IDEA “Finds duplicated code”提示如何关闭
发现重复的代码这个提示真的很烦啊,我们怎么关闭他呢. 设置在这里: Settings -> Editor -> Inspections -> General -> Duplic ...
- IntelliJ IDEA 自动导入包 关闭重复代码提示
idea可以自动优化导入包,但是有多个同名的类调用不同的包,必须自己手动Alt+Enter设置 设置idea导入包 勾选标注 1 选项,IntelliJ IDEA 将在我们书写代码的时候自动帮我们优化 ...
- IntelliJ IDEA 自动导入包 快捷方式 关闭重复代码提示
idea可以自动优化导入包,但是有多个同名的类调用不同的包,必须自己手动Alt+Enter设置 设置idea导入包 勾选标注 1 选项,IntelliJ IDEA 将在我们书写代码的时候自动帮我们优化 ...
- 关闭Found duplicated code
IDEA中的这个“发现重复代码 - Found duplicated code“的这个提示甚是烦躁. Settings —> Editor —> Inspections —> Gen ...
- Large Class--过大的类--要重构的信号
如果想利用单个类做太多事情,其内往往就会出现太多实例变量.一旦如此,Duplicated Code也就接踵而至. 解决方法: 1.将类内彼此相关的变量,将它们放在一起.使用Extrac ...
随机推荐
- Thunder团队第二周 - Scrum会议2
Scrum会议2 小组名称:Thunder 项目名称:爱阅app Scrum Master:胡佑蓉 工作照片: 参会成员: 王航:http://www.cnblogs.com/wangh013/ 李传 ...
- 【转载】Windows下Mysql5.7开启binlog步骤及注意事项
转自:https://www.cnblogs.com/wangwust/p/6433453.html 1.查看是否开启了binlog:show binary logs; 默认情况下是不开启的. 2.开 ...
- js 复制到剪贴板 兼容还得自己想办法
clipboard.js https://github.com/zenorocha/clipboard.js/ 主要问题还是ie8, 可以使用ie 特有的方法 if (window.clipboard ...
- 使用bat执行java项目
前提:java项目要有main方法 类似写法如下: set JAVA_HOME=C:\jdk1.6 set LIB_HOME=. set JAVA_JAR=. set JAVA_JAR=%JAVA_J ...
- Foundation框架—时间处理对象NSDate
NSDate类用于保存时间值,同时提供了一些方法来处理一些基于秒级别时差(Time Interval)运算和日期之间的早晚比较等. 1. NSDate的构造方法和构造获取实例的属性 用于创建NSDat ...
- BZOJ 1834 网络扩容(最大流+费用流)
对于第一问,直接求最大流. 对于第二问,建源点s和汇点t,s连1容量为INF,费用为0的边,n连t容量为最大流+k,费用为0的边.这样就把最大流限制为最多增加k了. 限制需要求扩充的最小费用,原图的边 ...
- HDU 4869 Turn the pokers(思维+逆元)
考试的时候没有做出来... 想到了答案一定是一段连续的区间,一直在纠结BFS判断最后的可行1数. 原来直接模拟一遍就可以算出来最后的端点... 剩下的就是组合数取模了,用逆元就行了... # incl ...
- Django 2.0 学习(11):Django setuptools
应用打包 当前状态的Python包与各种工具有点儿混乱,本结我们将学习使用setuptools来构建应用包.该工具是强烈推荐使用的打包工具,之后我们也会使用pip去安装和卸载它. Python打包指的 ...
- 计蒜客 17417 Highest Tower(思维+图论)
题解: 实际上一个可行解即选取长和宽的一个,使得最后每一组选第一维的数值都不同 在此基础上,使得另一维的和最大. 然后建立图论模型 对于每一个方块,在a和b之间连边. 对于选择的方案,如果选择a-&g ...
- html的body内标签之input系列1
1. Form的作用:提交当前的表单. 类似于去了银行提交的纸质单子,递到后台去办理相关业务. text,password只有输入的功能:button,submit只有点击的功能.想要把这些信息提交, ...