Eclipse重构——Move Method(I)

Motivation

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

Mechanics

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

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

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

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

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

public class Account {
	private AccountType _type;
	double bankCharge() {
		double result = 4.5;
		if (_daysOverdrawn > 0)
			result += overdraftCharge(_daysOverdrawn);
		return result;
	}    

// AccountType 目标类   
public class AccountType {        }

应用[Move]方法， 目标类中bankCharge方法自动多一个Account account参数

public class Account {
	private AccountType _type;
        double bankCharge() {
		return _type.bankCharge(this);
	}

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

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

因为内部类引用外部类的成员和方法时，不需要使用外部类的实例，因此[Move] method 时，实际不需要传入源类对象作为函数参数。这种情况下无需使用[Move]选项，直接copy函数到内部类，然后将外部类委托给内部类方法即可。 如果还有需要，可以使用[Inline],将所有对外部类方法的引用改为对内部类方法的引用（从封装性角度看，一般不会这么做）。

public class Account {
	private AccountType _type;
	double bankCharge() {
		double result = 4.5;
		if (_daysOverdrawn > 0)
			result += overdraftCharge(_daysOverdrawn);
		return result;
	}    
        // AccountType 是Account 的内部类   
        class AccountType {        }

直接copy，完成重构

public class Account {
	private AccountType _type;
	double bankCharge() {
		return _type.bankCharge();
	}    
        // AccountType 是Account 的内部类   
        class AccountType {        
            double bankCharge() {
		double result = 4.5;
		if (_daysOverdrawn > 0)
			result += overdraftCharge(_daysOverdrawn);
		return result;
	    }    
        }

over

上面完整的源码实例可以 here 下载