Eclipse重构——Move Method(III)

《重构——改善既有代码》

Motivation

出现依恋情节或者职责不清，应将行为移动到正确的类中。

Mechanics

1  检查源函数中所使用一切特性（包括field和函数）

2  检查子类和父类是否也定义了该函数

3  先委托，然后根据实际情况，决定是否将对源函数的调用替换为对目标函数的调用。

Eclipse refactor菜单下直接有Move 选项，可以直接使用该选项完成Move Method重构。但实际应用中，还是有一些细致的不同，下面详细说明：

默认情形：[Move] method,  必须要求源类中必须以目标类对象为成员变量，否则找不到目标对象。另外，移动后，默认总是以源类对象作为函数参数。

特殊情形1：目标类是源类的内部类，实际使用copy代码的方法。

特殊情形2：目标类中成员中已经有源类的对象，实际使用[Move] + [Rename] + [Change Method Signature]的方法

特殊情形3：有时候，只需要将源类的一个成员变量作为函数参数传入，而没有必要传入整个源类对象。实际使用[ExtractLocal Variable] + [Introduce Parameter] / [Extract Method]   + [Move]，

也可以使用[ExtractLocal Variable] + [Extract Method]  + [Inline] + [Move]

double bankCharge(Account account) {
		double result = 4.5;
		if (account._daysOverdrawn > 0)
			result += account.overdraftCharge();
		return result;
	}

// 1.  对需要move的函数中的成员变量，应用[Extract Local Variable]替换为局部变量，同时必然会增加“localVariable = _field”这样一句赋值语句。

publicclass Account {

    double overdraftCharge() {

        finalintdaysOverdrawn =_daysOverdrawn;

        if (_type.isPremium()) {

            double result = 10;

            if (daysOverdrawn> 7)

                result += (daysOverdrawn - 7) * 0.85;

            return result;

        } else

            returndaysOverdrawn* 1.75;

}

// 2. 对上面新增的赋值语句"final int daysOverdrawn = _daysOverdrawn;"，选中field " _daysOverdrawn",应用Introduce Parmeter，将该field作为函数参数传入。

public class Account {

    double overdraftCharge(int dayOverdrawn) {

        finalint daysOverdrawn =_daysOverdrawn;

        final int daysOverdrawn = dayOverdrawn;

        if ( _type.isPremium()){

// 3.   删除该赋值语句”localVariable = _field”。

public class Account {

    double overdraftCharge(int dayOverdrawn) {

        finalint daysOverdrawn =dayOverdrawn;

        if ( _type.isPremium()){

=========================================================================================

还有一种更通用的调整方法，其核心思想是将旧方法接口委托给新方法，然后应用内联将所有旧方法替换为新方法，最后move。

第一步同上，第二步改为使用[Extract Method]，将 赋值语句"final int daysOverdrawn = _daysOverdrawn;"以下的部分抽取为新的函数，函数名与目标函数相同，都是overdraftCharge.

public classAccount {

    double overdraftCharge() {

        final int daysOverdrawn = _daysOverdrawn;

        return overdraftCharge(daysOverdrawn);

    }

    private double overdraftCharge(final int daysOverdrawn) {

        if (_type.isPremium()){

            double result = 10;

            if (daysOverdrawn > 7)

                result +=(daysOverdrawn - 7) * 0.85;

            return result;

        } else

            return daysOverdrawn * 1.75;

    }

//3. 使用[Inline]替换掉目标函数

先对daysOverdrawn应用[Inline]

double overdraftCharge() {

        finalint daysOverdrawn =_daysOverdrawn;

        return overdraftCharge(_daysOverdrawn);

    }

再对overdraftCharge() 应用[Inline]

doubleoverdraftCharge() {

        return overdraftCharge(_daysOverdrawn);

    }

原来调用overdraftCharge()的地方自动调整为 overdraftCharge(_daysOverdrawn)

result += overdraftCharge();  -->  result += overdraftCharge(_daysOverdrawn);

 

over

以上完整源码可以在here下载