《重构：改善既有代码的设计》在对象之间搬移特性之 1

《重构：改善既有代码的设计》中提到过很多重构方法，关于在对象之间搬移特性的方法有9种。本文介绍：
搬移函数 move method

名称：搬移函数 move method
概要：在该函数最常引用的类中建立一个有着类似行为的新函数。将旧函数编程一个单纯的委托函数，或是将旧函数完全移除。
动机：在程序中，有个函数与其所驻类之外的另一个类进行更多交流，调用后者，或者被后者调用。

做法：
- 检查源类中被源函数所使用的一切特性（包括字段和函数），考虑它们是否应该被搬移
- 检查源类的子类和超类，看看是否有该函数的其他声明。如果出现其他声明，可能无法进行搬移，除非目标类中也同样表现出多态性。
- 在目标类中声明这个函数
- 将源函数的代码复制到目标函数中。调整目标函数，使之能够正常运行。如果目标函数中使用了源类中的特性，你得决定如何从目标函数引用源对象。如果目标类中没有相应的引用机制，就把源对象的引用当作参数，传给新建立的目标函数。如果源函数中包含异常处理，你得判断逻辑上应该由哪个类来处理这个异常。如果由源类来负责，就把异常处理留在原地。
- 编译目标类
- 决定如何从源函数正确引用目标对象。如果有现成的字段或者函数能够取得目标对象，如果没有，就建立一个。如果还是不行，就在源类中新建一个字段来保存目标对象。
- 修改源函数，使之成为一个纯委托函数
- 编译，测试
- 决定是否删除源函数，或者将它作为一个委托函数保留下来。如果经常在源对象中引用目标函数，就将源函数作为委托函数保留下来会比较简单。
- 如果要移除源函数，请将源类中对源函数的所有调用，替换为对目标函数的调用。
- 编译，测试
代码演示

修改之前的代码：

///////////////////////////.h
#ifndef REFACTORMOVE_H
#define REFACTORMOVE_H

class AccountType
{
public:
    bool isPremium();
private:
    int m_type;
};
class Account
{
public:
   double overdraftCharge();
   double bankCharge();
private:
   AccountType m_type;
   int m_daysOverdrawn;
};


#endif // REFACTORMOVE_H



///////////////////////////.cpp
#include "RefactorMove.h"


double Account::overdraftCharge()
{
        if (isPremium())
    {
        double result = 10;
        if (m_daysOverdrawn> 7)
        {
            result += (m_daysOverdrawn- 7) * 0.85;
        }
        return result;
    }
    return daysOverdrawn* 1.75;    
}

double Account::bankCharge()
{
    double result = 4.5;
    if (m_daysOverdrawn > 0)
    {
        result += m_type.overdraftCharge(m_daysOverdrawn);
    }
    return result;
}

bool AccountType::isPremium()
{
    return m_type > 0;
}

由于每种账户都有自己的“透支金额计费规则”，所以要把overdraftCharge()搬移到AccountType类中。对于overdraftCharge()的每一个特性，都要考虑是否搬走。isPremium是AccountType类， result是临时变量，所以要搬移。由于m_daysOverdrawn不会随账户类型而变化，所以留下来。当使用源类的特性时，有四种选择，本例中选第4种。
1）将这个特性也移到目标类中
2）建立或使用一个从目标类到源类的引用关系
3）将源对象当作参数传给目标函数
4）如果所需特性是个变量，将它当作参数传给目标函数
修改之后的代码：

///////////////////////////.h
#ifndef REFACTORMOVE_H
#define REFACTORMOVE_H

class AccountType
{
public:
    bool isPremium();
    double overdraftCharge(int daysOverdrawn);
private:
    int m_type;
};
class Account
{
public:
//   double overdraftCharge();
   double bankCharge();
private:
   AccountType m_type;
   int m_daysOverdrawn;
};


#endif // REFACTORMOVE_H
///////////////////////////.cpp
#include "RefactorMove.h"


//double Account::overdraftCharge()
//{
//    return m_type.overdraftCharge(m_daysOverdrawn);
//}

double Account::bankCharge()
{
    double result = 4.5;
    if (m_daysOverdrawn > 0)
    {
        result += m_type.overdraftCharge(m_daysOverdrawn);
    }
    return result;
}

bool AccountType::isPremium()
{
    return m_type > 0;
}

double AccountType::overdraftCharge(int daysOverdrawn)
{
    if (isPremium())
    {
        double result = 10;
        if (daysOverdrawn > 7)
        {
            result += (daysOverdrawn - 7) * 0.85;
        }
        return result;
    }
    return daysOverdrawn* 1.75;
}

如果需要源类的多个特性，就要将源对象传给目标函数。如果目标函数需要太多的源类特性，就要进一步重构。

《重构：改善既有代码的设计》 在对象之间搬移特性 之 1

猜你喜欢

《重构：改善既有代码的设计》在对象之间搬移特性之 1