learnScala编程与实践代码改变世界

大话重构 之 解决万恶之首“重复代码”

2015-07-23  本文已影响280人  优雅程序员

排在坏味道之首的是重复代码。为什么它如此臭名昭著?不需要一码再解释了吧。

提示:对于任何事物,代码中只应该出现一次,而且是唯一的一次。

下面我们将介绍如何消除几种典型的重复代码,以及如何培养消除重复代码的习惯。

在同一个类中重复

第一种是在同一个类中存在重复代码,它最容易识别,也最容易解决。看下面的例子:

try {
    executorService.runTasks(...)
    timer.scheduleAtFixedRate(
        () => saveProgress(getProgress()),
        period, period
    )
} finally {
    executorService.shutdown()
    timer.cancel()
    saveProgress(getProgress())
}

saveProgress(getProgress())出现在两个地方,可以通过 提取方法 来解决。

try {
    executorService.runTasks(...)
    timer.scheduleAtFixedRate(
        () => persistProgress(),
        period, period
    )
} finally {
    executorService.shutdown()
    timer.cancel()
    persistProgress()
}

def persistProgress() {
    saveProgress(getProgress())
}

如果一个类超级长,上千行,那即使在一个类中也很难识别重复。这涉及到另一个坏味道“超长类”,《消灭坏味道》后续会有专门的文章来解决它。

在同一个类树下重复

第二种是在同一个类树下的不同子类中重复,比第一种识别起来困难些。

class Child1 extends Parent {
    def run() {
        init()
        process()
    }
    ...
}

class Child2 extends Parent {
    def run() {
        process()
    }
    ...
}

可以通过 上移方法模板方法 将公共部分上移到共同的父类。

class Parent {
    def run() { // 模板方法
        init()
        process()
    }
    
    def init() {}
    def process() //无函数体,等同于虚函数
}

class Child1 extends Parent {
    override def init() { ... }
    override def process() { ... }
}

class Child2 extends Parent {
    override def process() { ... }
}

在不相干的类中重复

第三种是在两个完全不相干的类中,如果不是专门地寻找很难发现。

class App1 {
    val last3 = scores.sort().take(3)
}

class App2 {
    val last10 = scores.sort().take(10)
}

可以先 提取方法 ,然后 移动方法 到新建的类,来消除重复。

class App1 {
    val last3 = Seqs.lastN(scores, 3)
}

class App2 {
    val last10 = Seqs.lastN(scores, 10)
}

object Seqs { // 新建的工具类
    def lastN[E](seq: Seq[E], n: Int)
        : Seq[E] = { // 返回值类型
        seq.sort().take(n)
    }
}

上面的例子,是在业务上没有联系的两个类。

有可能是业务上有联系,或者直接就是类似的,这个时候应该 抽取公共基类 ,然后按照第二种重复去重构。

细心的看官会发现,三种类型的重复有递进关系,离的越来越远,识别起来越来越难,解决起来也是。“在不相干的类中重复”可以通过重构转换成前两种,然后继续重构得以解决。

要培养好的编码习惯

good habitgood habit

解决重复并不困难,困难的是发现重复。发现重复并不困难,困难是培养发现重复的习惯。一码在写代码完成功能前,会习惯性地左右前后看看,别人是否已经做过相关的功能,和我要做的是否可以共用代码,这就很容易发现重复。

提示:如果你要动的是一个点,那么先了解她周围的一圈。

不仅解决了重复,而且经过一段时间,我就发现对整个系统的理解程度,在不知不觉中提高了不少。

其实和做人是一样的,著名的理财教练博多舍费尔在《小狗钱钱的爸爸 - 教你实现财务自由》一书中也强调:

不断将私人领域变成主导领域,这是你对未来的责任。

私人领域是生活中与我相关的,主导领域是由我直接或间接影响的。碰到事情不能光是自扫门前雪,需要积极地参与,变被动为主动,这样才会有更多的机会,让我们越来越好。

总结

希望各位看官不只是学会了消除重复的技能,而且理解了文中的两个重要提示。

  1. 对于任何事物,代码中只应该出现一次,而且是唯一的一次。
  2. 如果你要动的是一个点,那么先了解她周围的一圈。

关于示例代码的说明

一码觉得要讲如何写好代码,就得拿代码来说事。不过手机屏幕上代码的展示是个大问题。为了保证示例代码能让大家读得明白,一码做了些取舍:

  1. 用片段,而不是完整可运行的代码,突出重点
  2. 语言用Scala,因为她简洁易读

千万不要觉得不懂Scala,有压力,一码会用她里面最平易近人的部分,放心吧。

提示:Scala是JVM平台上的一种语言,拥有简洁和可扩展的特性,现在非常流行的大数据处理框架Spark就是用Scala开发的,学点Scala不会亏本的。

分类 大话重构

优雅程序员 原创 转载请注明出处

上一篇下一篇

猜你喜欢

热点阅读