重构——抽取方法

到处都是重复的代码?方法体又臭又长看不懂?快来试试抽取方法(Extract Method)吧,保证药到病除!

何为抽取方法

将重复的代码或语义独立的代码抽取成一个方法

看好了:

Tom抓Jerry(又臭又长版)

class Tom {

/**
* Tom当前的位置
**/
private int curPos;

public void catchJerry() {

//只要没抓到Jerry就一直抓
while(true) {
//找Jerry
//左看看
int posOfJerry = lookLeft();
//右看看
if(posOfJerry == -1)
posOfJerry = lookRight();
//上看看,下看看
//.....省略

//前往Jerry的位置
while(curPos <= posOfJerry) {
//迈前腿
curPos++;
//迈后腿 保持平衡
//....
}

//抬起爪子
upHands();
//挥爪
Object objInHand = waveHands();
//Jerry是否在手中
if(Jerry.equals(objInHand))
break;
}

}
}

又臭又长,两个While的嵌套循环,让人不明所以,连走路迈腿都写在一块,如果没有注释的话,绝对让人无法理解

接下来见证重构的奇迹

  1. 简化逻辑:抓Jerry = 抓不到Jerry就一直抓

    public void catchJerry() {
    //只要没抓到Jerry就一直抓
    while(!tryCatchJerry());
    }

    catchJerry()只有一行了,此处应有掌声

    但是我们又创建了又臭又长的方法——试图抓住Jerry

    private boolean tryCatchJerry() {
    //找Jerry
    //左看看
    int posOfJerry = lookLeft();
    //右看看
    if(posOfJerry == -1)
    posOfJerry = lookRight();
    //上看看,下看看
    //.....省略

    //前往Jerry的位置
    while(curPos <= posOfJerry) {
    //迈前腿
    curPos++;
    //迈后腿 保持平衡
    //....
    }

    //抬起爪子
    upHands();
    //挥爪
    Object objInHand = waveHands();
    //Jerry是否在手中
    return Jerry.equals(objInHand));
    }
  2. 分而治之:试图抓住Jerry = 找Jerry + 前往Jerry位置 + 挥爪抓住Jerry

    private boolean tryCatchJerry() {
    //找Jerry并前往Jerry的位置
    go(findPosOfJerry());

    //挥爪抓Jerry
    return waveHandsCatchSomething(Jerry);
    }

    好的,tryCatchJerry简单又清晰了

    我对剩下的方法进行了微调,让它们更有逻辑感

    private int findPosOfJerry() {
    //左看看
    int posOfJerry = lookLeft();
    if(posOfJerry != -1)
    return posOfJerry;

    //右看看
    posOfJerry = lookRight();
    if(posOfJerry != -1)
    return posOfJerry;
    //上看看,下看看
    //.....省略
    }

    private void go(int pos) {
    while(curPos <= pos) {
    //迈前腿
    curPos++;
    //迈后腿 保持平衡
    //....
    }
    }

    private boolean waveHandsCatchSomething(Object something) {
    //抬起爪子
    upHands();
    //挥抓,然后看看指定东西是否在手中
    return something.equals(waveHands());
    }

新版Tom抓Jerry

铛铛!掌声在哪里?

class Tom {

/**
* Tom当前的位置
**/
private int curPos;

public void catchJerry() {
//只要没抓到Jerry就一直抓
while(!tryCatchJerry());
}

private boolean tryCatchJerry() {
//找Jerry并前往Jerry的位置
go(findPosOfJerry());

//挥爪抓Jerry
return waveHandsCatchSomething(Jerry);
}

private int findPosOfJerry() {
//左看看
int posOfJerry = lookLeft();
if(posOfJerry != -1)
return posOfJerry;

//右看看
posOfJerry = lookRight();
if(posOfJerry != -1)
return posOfJerry;
//上看看,下看看
//.....省略
}

private void go(int pos) {
while(curPos <= pos) {
//迈前腿
curPos++;
//迈后腿 保持平衡
//....
}
}

private boolean waveHandsCatchSomething(Object something) {
//抬起爪子
upHands();
//挥抓,然后看看指定东西是否在手中
return something.equals(waveHands());
}
}

何时应该抽取方法

有重复代码

方法体又臭又长看不懂

怎么抽取方法

IDEA提供了重构的快捷方式和快捷键,使用IDEA进行重构,方便又快捷

  1. 选中要抽取的代码

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

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

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

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

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

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

    比如:Tom.扫地() √

    ​ Tom.扫地With扫把() ×

    ​ Tom.用扫把扫地() ×

更多重构手段

等我把这些文章全写完发了,再给你们提供链接

  • 抽取方法

    到处都是重复的代码?方法体又臭又长看不懂?快来试试抽取方法(Extract Method)吧,保证药到病除!

  • 使用对象封装参数集

    答应我,别再给方法弄一长溜的参数了,好吗?

  • 使用查询方法取代临时变量

    你这方法里怎么这么多临时变量?我完全记不住每个变量是用来干什么的!

  • 使用多态取代switch分支

    “来看看外包老哥写的超长switch语句”这种文章又登上首页了

  • 及时打断if条件分支

    外包老哥总是被黑的幕后黑手找到了!

  • 转移成员变量和函数

    道理我都懂,可是你一直在用的成员变量为什么要让我负责管理?

  • 抽取类

    总觉得,是时候new一个替身出来帮我干活了

  • 将数据的处理函数转移给持有数据的类

    或许,这个锅应该由你来背

  • 向父类转移成员变量和函数

    所有子类都需要做的事,为什么不让父类直接做了?

  • 注意集合型成员变量的get/set方法!

    有人动了我的集合对象,可是我却不知道!



重构      重构

本博客所有文章除特别声明外,均采用 CC BY-SA 3.0协议 。转载请注明出处!