《重构》阅读笔记-代码的坏味道

简介:

决定何时重构、何时停止和知道如何重构一样重要!

开发者必须通过实践培养自己的经验和直觉,培养出自己的判断力:学会判断一个类内有多少个实例变量算是太大、学会判断一个函数内有多少行代码才算太长。

  1. 重复代码(Duplicated Code)
    如果你在一个以上的地方看到相同的程序结构,那么可以肯定:设法将它们合而为一,程序会变得更美好!你需要决定这个重复的代码放在哪里比较合适,并确保它被安置之后就不会在别的地方再次出现。

  2. 过长函数(Long Method)
    程序越长越难以理解。现代OO语言几乎完全免去了进程内的函数调用开销,因此,你应该积极地分解函数。我们应该遵循原则:每当需要以注释来说明点什么的时候,我们就需要把需要说明的东西写进一个独立函数中,并以其用途(而非实现手法)命名。如何确定需要提炼哪一段代码:寻找注释、条件表达式、循环。

  3. 过大的类(Large Class)
    如果你发现一个类试图做太多事情,其内部就会出现很多不相关的实例变量,此时这个类的职责就不明确了。

  4. 过长参数列(Long Parameter List)
    太长的参数队列难以理解,太多参数的接口对于使用者来说十分不友好,而且容易出错。如果可以使用一个对象代替参数列表,那么就应该这么做。

  5. 发散式变化(Divergent Change) VS 霰弹式修改(Shotgun Surgery)
    一旦需要修改,我们希望只在系统的一个地方进行修改,否则,就属于两种非常相似的坏味道的一种:如果某个类经常因为不同的原因在不同的地方发生变化,那么Divergent Change就出现了;如果系统每遇到一个小变化,就需要在多个不同的类内进行许多小修改,这属于Shotgun Surgery。Divergent Change是指“一个类受多种变化的影响”,Shotgun Surgery则指的是“一种变化引发多个类的修改”。

  6. 平行继承体系(Parallel Inheritance Hierarchies)
    这其实是Shotgun Surgery的特殊情况——每当你为某个类添加一个子类,你也必须为它的兄弟类加一个子类。如果你发现某个继承体系的类名称前缀和另一个继承体系的类名称前缀完全相同,就属于这种情况。

  7. 依恋情结(Feature Envy)
    面向对象的精髓在于:“将数据和对数据的操作行为包装在一起”。有一种气味是:函数对某个类的兴趣高过自己所处的类的兴趣。有很多时候,我们看到一个函数为了计算某个值,从另一个对象那儿调用了几乎半打的取值函数。最根本的原则是:将总是一起变化的东西放在一起。

  8. 数据泥团(Data Clumps)
    你常常可以在很多地方看到相同的三四项数据:两个类中相同的字段、许多函数签名中相同的参数。这些绑在一起出现的数据真应该拥有属于它们自己的对象。减少字段和参数的个数,当然可以去除一些坏味道,但更重要的是:一旦拥有新对象,你就有机会寻找Feature Envy,这可以帮你指出能够移至新类中的种种方法

  9. 基本类型偏执(Primitive Obsession)
    对象技术的新手通常不愿意在小任务上运用对象——例如结合数值和币种的Money类、由一个起始值和一个结束值构成的Range类、电话号码或邮政号码等的特殊字符串。

  10. switch语句(Switch Statements)
    从本质上讲,switch语句的问题在于重复,面向对象的多态特性可以优雅地解决这个问题。如果你只是在单一函数内有些选择事例,那么用多态就属于杀鸡用牛刀了,这种情况下Replace Parameter with Explicit Methods是个不错的选择,如果你的选择之一是null,记得使用Introduce Null Object。

  11. 多余的类(Lazy Class)
    你所创建的每一个类,都得有人去理解它、维护它,这些工作都是要花钱的。如果一个类的所得不值得其身价,就应该消除这个类。

  12. 过度设计(Speculative Generality)
    当有人说“噢,我想我们有一天需要做这个事情”,并因此而企图以各种各样的钩子和特殊情况来处理一些非必要的事情,这种坏味道就出现了。软件设计不可过度设计,否则会使得系统难以理解和维护。

  13. 令人迷惑的暂时字段(Temporary Field)
    有时你会看到这样的现象:类内的某个实例变量仅为某种特定情况而设。这样的代码不易理解,因为通常认为对象在所有时候都需要它的所有变量。

  14. 过度耦合的消息链(Message Chains)
    如果你看到用户向一个对象请求另一个对象,然后再向后者请求另一个对象,然后再请求另一个对象……这就是消息链。实际代码中你看到的可能是一长串getXXX()调用,意味着客户代码将与查找目标对象过程中的导航结构紧密耦合,一旦对象间的关系发生任何变化,客户端就会受到影响。

  15. 中间人(Middle Man)
    对象的基本特征之一是封装——对外部世界隐藏其内部细节。封装往往伴随着委托,比如你问主管是否有时间参加一个会议,他就把这个消息“委托”给他的记事簿,然后才能回答你——你没有必要这位主管到底是使用传统记事簿或电子记事簿或秘书来记录自己的约会。但是,不要过度使用委托——你也许会看到某个类有一半接口都委托给其他类。

  16. 狎昵关系(Inappropriate Intimacy)
    类与类之间过分紧密的关系必须拆散——可以引入第三方类或者利用委托。

  17. 异曲同工的类(Alternative Classes with Different Interfaces)
    如果两个函数做同一件事,却有着不同的签名,请运用Rename Method根据它们的用途重新命名。但这往往不够,请反复运用Move Method将某些行为移入类,知道这两个函数的协议一致为止。如果你必须移动大量代码才可以完成这个工作,那还不如直接构建一个父类。

  18. 不完美的库类(Incomplete Library Class)
    复用常常被认为是面向对象技术的终极目标。很多第三方库提供的接口经常不能恰如其分得满足我们的需求,这时候就需要对第三方接口做一层转换,或者给它添加一定的行为。

  19. 数据类(Data Class)
    所谓Data Class,指的是:这种类拥有一些字段,以及用于访问(读、写)的函数,除此之外啥都没有。这样的类只是一种不会说话的数据容器,它们一定被其他类过分细碎得控制着。
    Data Class就像小孩子,作为一个起点很好,但若要让它们像成熟的对象那样参与整个系统的工作,它们就必须承担一定责任。但是,在Spring框架开发中,我们经常需要定义很多domain对象。

  20. 被拒绝的遗嘱(Refused Request)
    子类应该继承超类的函数和数据,但如果它们不想或者不需要继承,又该怎么办呢?按照传统说法,这就意味着继承体系的设计错误。你需要为这个子类新建一个兄弟类,然后让父类只包括两个子类共享的部分。
    一般而言,这就足够了,但是如果子类不愿意支持超类提供的接口,则说明不能使用继承处理,应该使用委托。

  21. 过多的注释(Comments)
    常常会有这样的情况:你看到一段代码有着长长的注释,然后发现,这些注释之所以存在乃是因为代码很糟糕。当你需要些注释时,要先尝试重构下代码,争取让代码拥有自说明性。


相关文章
|
6月前
|
设计模式 算法 程序员
程序员为何需要反复修改Bug?探寻代码编写中的挑战与现实
作为开发者,我们在日常开发过程中,往往会遇到反复修改bug的情况,而且不能一次性把代码写的完美无瑕,其实开发项目是一项复杂而富有挑战性的任务,即使经验丰富的程序员也难以在一次性编写完美无瑕地完成代码,我个人觉得一次性写好代码是不可能完成的事情。虽然在设计之初已经尽力思考全面,并在实际操作中力求精确,但程序员仍然需要花费大量时间和精力来调试和修复Bug。那么本文就来分享程序员需要反复修改Bug的原因,以及在开发中所面临的复杂性与挑战。
160 1
程序员为何需要反复修改Bug?探寻代码编写中的挑战与现实
|
3月前
|
设计模式 程序员
故意把代码写得很烂,这样的 “防御性编程“ 可取吗?
故意把代码写得很烂,这样的 “防御性编程“ 可取吗?
|
设计模式 测试技术
重构·改善既有代码的设计.02之代码的“坏味道”
之前在《重构·改善既有代码的设计.01》中初步了解了重构的基本前提,基础原则等入门知识。今天我们继续第二更......
201 1
重构·改善既有代码的设计.02之代码的“坏味道”
|
数据库
代码坏味道
代码坏味道
118 0
消除代码坏味道
读完本章的收获:代码规范、代码可读性、鲁棒性的代码好坏案例,写出好代码的能力。
消除代码坏味道
|
XML IDE 应用服务中间件
重构:勿以善小而不为
重构:勿以善小而不为
|
设计模式 移动开发 安全
与其硬啃“屎山”代码,不如用这六步有条不紊实现代码重构 李慧文
对大规模系统进行重构,如果一个人对着又臭又长的代码硬刚,即使花了大量的时间进行手工验证,最后仍然会有很多问题,特别是一些深路径及特殊场景下的问题。其实,大规模的系统级别重构时是有方法的。我们采访了 Thoughtworks 数字化转型与运营 资深咨询师黄俊彬(QCon+案例研习社讲师),请他来分享 MV*模式重构演进的方法和经验。
568 0
与其硬啃“屎山”代码,不如用这六步有条不紊实现代码重构 李慧文
|
设计模式 Oracle IDE
讲码德!避免这些代码坏味道,努力做一名优秀的程序员
讲码德!避免这些代码坏味道,努力做一名优秀的程序员
|
数据库
IO流Bug(记忆深刻的一课)
IO流死循环创建的文件夹无法删除,甚至在删除过程中导致 explorer.exe等 进程直接结束 只能强制重启
IO流Bug(记忆深刻的一课)
团队里的妹子让阿粉跟她说说怎样写出好的代码
昨天,团队里的妹子很突然地就让阿粉跟她说说怎么才能写出一手好的代码 阿粉本着负责任的态度,专门写了一篇文章出来,妹子嘛,就是要宠的嘛

相关实验场景

更多