大话西游体之解决无耻之首“冗余代码”版权声明

原创

小哥 3年前 (2022-12-26) 阅读数 29 #大杂烩

排在最前面的是重复的代码。为什么它如此臭名昭著？观众可以阅读“消除不良品味”系列的开篇文章“重构，企业应用程序的圣经”，这篇文章没有重复，只是引用了以下几点：

提示：代码中的所有内容都应该只出现一次，而且只能出现一次。

下面我们将讨论如何消除几个典型的重复代码，以及如何养成消除重复代码的习惯。

在同一类中重复

第一个是同一类中存在重复代码，这是最容易识别和最容易解决的。查看以下示例：

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)
    }
}

在上面的示例中，有两个类在业务中没有连接。

有可能存在业务联系，或者直接类似，这次应该是 提取公共基类 ，然后根据第二次重复进行重构。

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