求求你别再写上千行的类了，试试这些牛逼的重构技巧吧，zookeeper和dubbo面试题

为了防止这样的惨剧再次发生在我的身上，我觉得有必要写一篇博客来让广大程序猿同胞知道代码重构的重要性

如果你身边有一个类写上千行的猿，一定要把此文转给ta

为什么类不能过长？

---

《一线大厂Java面试题解析+后端开发学习笔记+最新架构讲解视频+实战项目源码讲义》

【docs.qq.com/doc/DSmxTbFJ1cmN1R2dB】 完整内容开源分享

类过长——读不通，扩展不通

推荐一些很不错的计算机学习教程，包括：数据结构、算法、计算机网络、操作系统、Java（spring、springmvc、springboot、springcloud等）等等 ，全部收集于网络，如果有侵权，请联系删除！

下面是部分截图：

获取方式

点击下方公众号，回复：好好学Java，即可获取。

• 读不通——直接用滚轮滚都得好几秒，就算是原作者，时间长了恐怕也难以理清整个类，更不用说是其他的读者

• 扩展不通——一个类有过多的接口，会让扩展这个类变得异常困难，一动千行

类过长——可能有冗余代码

此时CV工程师打了个冷战

冗余代码，就是重复代码，通常出自使用Ctrl+C，Ctrl+V来生产代码的CV工程师之手，冗余代码的危害非常大：

• 冗余代码使方法、类过长，不简洁

• 冗余代码会造成发散式修改（冗余代码需要变动时，每一处Ctrl+V都需要修改）

类过长——多半是职责过多

一个类开放几十个接口，绝对存在职责过多的问题，就像图中的Tom猫一样手忙脚乱，一个类的职责过多也有巨大问题：

1、违反设计原则——单一职责原则（单一职责原则要求一个类只实现一个职责，比如一只Tom只做扫地、擦桌、拖地中的一件事，而其他事的实现可以转移给史派克狗或肥胖女佣），违反了这个原则会导致发散式变化、发散式修改、类过长等代码问题，还会让你的类难以扩展，甚至会让其他程序猿认为你不专业

2、发散式变化（指引发此类修改的地方很多），如果一个类的职责很多，那它的扇入（调用者）一定很多，每个调用者的修改都有可能让你这个类不得不随之修改，也就是发散式变化

就是说不管哪儿出了问题，你这个类都得遭殃

3、发散式修改（指此类修改引发修改的地方很多），相同的，如果一个类职责很多，那支撑它实现的下级，即扇出（被调用方）一定很多，如果此类逻辑发生变动，所有下级被调用者可能都得随之修改，也就是发散式修改

就是说你这个类出了问题，不管哪儿都会遭殃

4、难以扩展，如果你的一个类接口非常多，那它的子类怎么办？它的包装类怎么办？难道全部都要实现这么多接口，全部都要承担同样多的职责吗？扩展起来真的非常麻烦

5、触发机关【测试之怒】【运维之怒】

我已经写了几千行了，怎么办？

---

重构——抽取冗余代码

抽取冗余代码就是将重复代码抽取成一个独立的方法，之后再使用这段代码时就不再需要Ctrl + C，Ctrl + V，而是直接调用对应的方法即可

这样做也可以缩短原方法，使原方法更加简洁易懂

更值得一提的是如果这段代码需要修改，也只需修改一处，而不是发散式地到处修改

真是一箭三雕

使用IDEA进行冗余代码的抽取

1.找到重复代码

2.进行方法抽取 右键->选择重构->抽取->方法 （或者直接使用快捷键Ctrl + Alt + m）

自动检测出个别重复代码的细微差别，有些代码可能只改动一两个变量，IDEA会自动检测出来，并在抽取方法时提醒我们，选择左侧Accept Signature Change（接受签名变动）可以使抽取的方法自动替换更多的重复点

可以选择替换掉所有的重复代码（竟然有18处）

3.重构——更改方法签名

如果你对抽取出的方法的名字、参数、返回值或是修饰符不满意，不要使用Ctrl + R 修改，IDEA提供了重构方法——更改签名（快捷键Ctrl + F6）

注意：方法的名字指的是方法做了什么，而非怎么去做，最好是 动词+名词 格式

比如：

• Tom.扫地() √

• Tom.扫地With扫把() ×

• Tom.用扫把扫地() ×

重构——转移成员变量+函数（转移职责）

将不应该由自己管理的成员变量和函数转移出去

那就要考虑两个问题：该转移谁？转移给谁？

来看一个图