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

发布于:2022-11-28 ⋅ 阅读:(250) ⋅ 点赞:(0)

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

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

本文含有隐藏内容,请 开通VIP 后查看

网站公告

今日签到

点亮在社区的每一天
去签到