1. 冗赘的元素
定义
:由于过度设计或在代码演进过程中,产生的冗余、废弃或不足以独立承担其职责的类、方法、变量等。
影响
:代码可读性差,可维护性差。
重构目标
:消除冗赘元素,提升代码可读性和可维护性。
重构方法
:
- 内联函数或内联类封装变量
- 如果这个类在一个继承体系中,可以使用折叠继承体系
- 安全删除冗余元素
常见特点
:
- 注释掉的代码、未被使用的参数、变量、函数、类
- 函数或方法过于简单,不足以独立承担其职责
重构中可能用到的快捷键操作
:
操作 | 快捷键 |
---|---|
封装变量 | Ctrl + Alt + Shift + T -> Encapsulate Fields 鼠标右键 -> refactor -> Encapsulate Fields |
提取字段/参数/变量 | Ctrl + Alt + F/P/V |
用内联移除 | Ctrl + Alt +N |
安全删除 | Alt + Del |
2. 夸夸其谈的通用性
定义
:多度的考虑程序通用性。
影响
:代码可读性差、不易维护。
重构目标
:删除多度设计的代码。
重构方法
:
- 折叠继承体系
- 内联类
- 内联函数
- 改变函数声明
- 移除无用代码
常见特点
:
- 抽象类和接口类没有太大作用——折叠继承体系
- 不必要的委托——内联类、内联函数
- 函数的参数未被使用——改变函数声明
- 函数只被测试用例调用或者未被调用——移无用代码
重构中可能用到的快捷键操作
:
操作 | 快捷键 |
---|---|
移除超类 | Ctrl + Alt + N |
删除参数 | Alt + Del |
声明:本文仅用于学习交流。