老师在微信群中提了一个思考题：学习重构这部分需要注意的地方 - 第一个，你们可以反思一下，bad smell 是什么的，是怎么发现的。这个最重要。 - 第二个，对比重构前和重构后的代码结构。到底做了什么改变。这个慢慢你们就会养成习惯，有重构的大局观。 - 第三个，列一下用的重构手法，看如何改变的。这个反而没那么重要，每个人做法也不一样。 我的理解 - 一、bad smell 是什么的，是怎么发现的 - 违反**DRY原则**（Don’t repeat yourself）的最多 - 实现逻辑：代码重复基本可以直接看出来（收留心观察就行） - 测试类：需要在重构后留意是否有重复的逻辑 - 违反**YAGNI原则**（You ain’t gonna need it） - 实现逻辑：一波操作后，Idea 会提示没有被用到的变量、方法 - 测试类：需要在重构后留意是否有没有被用到的代码 - 违反 **KISS原则**（Keep it simple, stupid） - 实现逻辑：尽量让代码朝着更简单的方向发展 - 违反**单一职责原则** - 一个类或方法承担了多个职责 - 总结：设计原则的时候，就是 bad smell，需要及时重构 - 二、对比重构前和重构后的代码结构，到底做了什么改变 - 重构前： - 简单粗暴实现功能，不要给我说什么设计原则、设计模式、数据结构、拿起键盘就是干！一把梭！ - 对测试友好（因为是 TDD，代码再烂也不能直接翻车） - 对后期维护不友好，代码有点乱 - 重构后： - 大部分代码符合设计原则、合理套用设计模式、正确使用数据结构 - 对测试友好 - 对后期维护友好，代码整洁、逻辑清晰 - 三、列一下用的重构手法，看如何改变的 - 使用 Idea 快捷键进行重构（Extract 变量、方法参数、方法、类；inLine 方法；构造工厂方法；尽量使用接口等） - 通过**绞杀植物模式**替换旧的实现 - 姚琪琳老师的 [遗留系统现代化实战 | 06 | 以增量演进为手段：为什么历时一年的改造到头来是一场空？](http://gk.link/a/11lAM)中有详细介绍 附录 设计原则：https://wyyl1.com/post/18/02/#51-%E4%B8%BA%E4%BD%95%E8%A6%81%E5%85%B3%E5%BF%83%E8%AE%BE%E8%AE%A1 格式化后的思考题：https://wyyl1.com/post/19/wq/#%E8%AF%B4%E4%B8%80%E4%B8%8B%E5%AD%A6%E4%B9%A0%E9%87%8D%E6%9E%84%E8%BF%99%E9%83%A8%E5%88%86-