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

版权声明:请注明转发出处 https://blog.csdn.net/mafucun1988/article/details/89391378

《重构:改善既有代码的设计》中提到过很多重构方法,关于在对象之间搬移特性的方法有8种。本文介绍:
将类内联化 inline class

  • 名称: 将类内联化 inline class
  • 概要:某个类没有做太多的事情,将这个类的所有特性搬移到另一个类中,然后移除原类。
  • 动机: inline class 正好与extract class相反。如果一个类不再承担足够责任,不再由单独存在的理由。挑选这个“萎缩类”的最频繁用户,将其塞入另一个类中。

  • 做法:
    • 在目标类身上声明源类的public 协议,并将其中所有函数委托至源类。如果“以一个独立接口表示源类函数”更合适,就应该在内联之前先使用extract interface
    • 修改所有源类引用点,改为引用目标类。将源类声明为private,以斩断包之外的所有引用可能。同时修改源类的名称,编译器可以捕捉一些隐藏的引用点。
    • 编译,测试
    • 运用move method 和move field将源类的特性全部搬移到目标类
  • 代码演示

修改之前的代码:

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

class TelephoneNumber
{
public:
    void SetAreaCode(QString areaCode);
    QString GetAreaCode();
    QString GetTelephoneNumber();
    QString GetNumber();
    void SetNumber(QString officeNumber);
private:
    QString m_AreaCode;
    QString m_Number;
};

class Person
{
public:
    QString GetName();

    QString GetOfficeAreaCode();
    void SetOfficeAreaCode(QString officeAreaCode);

private:
    QString m_Name;
    QString m_OfficeAreaCode;
    TelephoneNumber m_TelephoneNumber;
};


#endif // REFACTORMOVE_H

///////////////////////////.cpp
QString Person::GetName()
{
    return m_Name;
}

QString TelephoneNumber::GetTelephoneNumber()
{
    return "(" + m_AreaCode + ") " +m_Number;
}

QString Person::GetOfficeAreaCode()
{
    return m_TelephoneNumber.GetAreaCode();
}

void Person::SetOfficeAreaCode(QString officeAreaCode)
{
    m_TelephoneNumber.SetAreaCode(officeAreaCode);
}

QString TelephoneNumber::GetNumber()
{
    return m_Number;
}

void TelephoneNumber::SetNumber(QString officeNumber)
{
    m_Number = officeNumber;
}

修改之后的代码:
1)在person中声明TelephoneNumber的所有“可见”(public)函数。
2) 找到TelephoneNumber的所有用户,转而使用person的接口
3)反复使用move method 和move field,直到TelephoneNumber不存在
 

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

class Person
{
public:
    QString GetName();
    QString GetTelephoneNumber();
    QString GetOfficeAreaCode();
    void SetOfficeAreaCode(QString officeAreaCode);
    QString GetOfficeNumber();
    void SetOfficeNumber(QString officeNumber);
private:
    QString m_Name;
    QString m_OfficeNumber;
    QString m_OfficeAreaCode;
};

#endif // REFACTORMOVE_H



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

QString Person::GetName()
{
    return m_Name;
}

QString Person::GetTelephoneNumber()
{
    return "(" + m_OfficeAreaCode + ") " +m_OfficeNumber;
}

QString Person::GetOfficeAreaCode()
{
    return m_OfficeAreaCode;
}

void Person::SetOfficeAreaCode(QString officeAreaCode)
{
    m_OfficeAreaCode = officeAreaCode;
}

QString Person::GetOfficeNumber()
{
    return m_OfficeNumber;
}

void Person::SetOfficeNumber(QString officeNumber)
{
    m_OfficeNumber = officeNumber;
}


 

猜你喜欢

转载自blog.csdn.net/mafucun1988/article/details/89391378