重构——抽取方法

Posted 2021-03-17 chengweijun

tags:

篇首语：本文由小常识网(cha138.com)小编为大家整理，主要介绍了重构——抽取方法相关的知识，希望对你有一定的参考价值。

技术图片

到处都是重复的代码？方法体又臭又长看不懂？快来试试抽取方法（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的嵌套循环，让人不明所以，连走路迈腿都写在一块，如果没有注释的话，绝对让人无法理解

接下来见证重构的奇迹

简化逻辑：抓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));
}

分而治之：试图抓住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进行重构，方便又快捷

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

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

可以选择替换掉所有的重复代码（竟然有18处）
重构——更改方法签名

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

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

比如：Tom.扫地() √

? Tom.扫地With扫把() ×

? Tom.用扫把扫地() ×