代码坏味道与重构之中间人和内幕交易

1. 中间人

定义：一种多度使用委托(某类一半以上方法都委托给其他类)的代码。

影响：当需求发生某些变化的时候，作为中间人的代码总会被牵连一并修改，代码越发臃肿。

重构目标：减少委托。

重构方法：

• 移除中间人
• 内联
• 以继承取代委托
• 以委托取代超类/子类

常见特点：

• 代码委托过多，代码臃肿，增加后期修改成本
• 代码被委托类中含有“不干实事”的方法

重构中可能用到的快捷键操作：

操作	快捷键
移除中间人	Ctrl + Alt + Shift + T -> Remove Middleman 鼠标右键 -> refactor -> Remove Middleman
用内联移除	Ctrl + Alt +N

2. 内幕交易

定义：模块之间互相引用，私下直接进行大量的数据访问和交换。

影响：增大模块间的耦合，容易导致循环依赖，加快架构腐化，甚至会朝着大泥球式的架构发展，严重影响可维护性。

重构目标：消除模块间的不合理依赖关系(特别是循环依赖)，将私下的数据访问和交换放到明面上，使模块间解耦，提高可维护性。

重构方法：

• 搬移函数
• 搬移字段
• 隐藏委托关系
• 以委托取代子类/超类

常见特点：

• 类间存在不合理的继承关系
• 模块间存在循环依赖
• 模块间业务边界不清晰，随时有引入循环依赖的风险

重构中可能用到的快捷键操作：

操作	快捷键
封装字段	Ctrl + Alt + Shift + T -> Excapsulate Fields 鼠标右键 -> refactor -> Excapsulate Fields
委托取代继承	Replace Inheritance With Delegation
移除中间人	Ctrl + Alt + Shift + T -> Remove Middleman 鼠标右键 -> refactor -> Remove Middleman
抽取方法	Ctrl + Alt + M
实例/静态方法搬移	F6
内联	Ctrl + Alt + N
实例方法变为静态方法	Ctrl + Alt + Shift + T -> Make Static 鼠标右键 -> refactor -> Make Static
静态方法变为实例方法	Ctrl + Alt + Shift + T -> Convert To Instance Method 鼠标右键 -> refactor -> Convert To Instance Method

声明：本文仅用于学习交流。