Eclipse重构——Move Method(II)

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

Motivation

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

Mechanics

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

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

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

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

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

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

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

因为目标类成员中已经有源类对象，所以应该直接使用该成员去引用目标类其他方法，而无需将源类实例作为函数参数传入。这种情况下，先[Move], 然后通过[Rename]将函数参数的形参名改为与目标类中定义的源类对象变量名相同。 接着在手动修改形参名为其他任意名称，最后通过[Change Method Signature]删除需要的参数。

// 需要移动bankCharge到AccountType
public class Account {
	private AccountType _type;
	double bankCharge() {
		double result = 4.5;
		if (_daysOverdrawn > 0)
			result += overdraftCharge(_daysOverdrawn);
		return result;
	}

class AccountType {
		Account mAccount;

//1.     [ Move Method]  and   [ Rename]， 修改传入的源类实例的形参名称，使与目标类中定义的成员变量一致,都是mAccount。

class AccountType ...

        Account mAccount;

        double bankCharge(Account mAccount) {

            double result = 4.5;

            if (mAccount._daysOverdrawn > 0)

                result +=mAccount.overdraftCharge(mAccount._daysOverdrawn);

            return result;

     }

// 2. 直接手动修改形参名，mAccount为account。但不要使用[Rename]

class AccountType {

        Account mAccount;

        double bankCharge(Accountaccount) {

            double result = 4.5;

         if (mAccount._daysOverdrawn> 0)

// 3.  应用Change Method Signature，将源类实例形参删除。

class AccountType {

        Account mAccount;

        double bankCharge(Accountaccount) {

            double result = 4.5;

         if (mAccount._daysOverdrawn> 0)

over.

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