书店会员销售系统（二）

书店会员销售系统（二）
                     ――OO、Refactoring and Design Pattern
本节目的：
1.         学习使用策略模式。
2.         使用重构手法。

客户：  “我觉得你们的打折算法有点问题。”
项目经理：“有什么问题？
客户：   “在这种算法中，顾客要消费至少1000元才能享受折扣优惠，我想根据书店的规模，采取不同的折扣   算法,这就需要让我能设置折扣算法。”
项目经理：“你说得比较有道理，我们会修改程序，满足你的要求。”
客户：    “那下个版本给我实现这个功能吧？”
项目经理：“没有问题。”

     听到这段对话，我们可以知道下一步我们需添加的新功能是配置折扣算法，但在实现之前，让我们先戴上“重构”的帽子。
    在CMember会员类中，根据累计点数计算折扣这个功能有很不稳定的因素，我们应该把它提炼到一个类里去。
class CRebateRule 
{
public:
    CRebateRule();
    ~CRebateRule();
    float CalcRebate(int nPoint);

}; 

float CRebateRule::CalcRebate(int nPoint)
{
    if(nPoint<=100)
        return 10;
    else if(nPoint>100 && nPoint <200)
        return 9.5;
    else if(nPoint>=200 && nPoint <400)
        return 9;
    else if(nPoint>=400 && nPoint <600)
        return 8.5;
    else
        return 8.0;
}
    然后要对CMember::GetRebate()进行修改。
float CMember::GetRebate()
{
    CRebateRule RebateRule;
    return RebateRule.CalcRebate(m_nPoint);
}
    编译程序，通过测试。
    客户不是要配置自己的打折算法吗？我们可以把打折算法写在配置文件中，然后读取到数据结构中，这样就比较灵活了。现在是脱下“重构”的帽子，戴上“添加新功能”的帽子了。
    为CRebateRule类添加一函数。
bool  CRebateRule::Init(char *pszConfigFilename)
{
    //从配置文件中读取打折算法，在这里我就不多写了。
    return true;
} 

float CRebateRule::CalcRebate(int nPoint)
{
    //为了方便，就假设这是从配置文件中读取的。
    if(nPoint<=100)
        return 10;
    else if(nPoint>100 && nPoint <200)
        return 9.5;
    else if(nPoint>=200 && nPoint <400)
        return 9;
    else if(nPoint>=400 && nPoint <600)
        return 8.5;
    else
        return 8.0;
}

    修改CMember::GetRebate()函数：
float CMember::GetRebate()
{
    CRebateRule RebateRule;
    RebateRule.Init("Configfile.txt");
    return RebateRule.CalcRebate(m_nPoint);
}
    编译程序，通过测试。
    然而过了一段时间，客户又开始抱怨了：“我想在一些节假日实现折上加折的功能，现在的程序还不能实现。”
    到现在，策略模式就该上场了，然而在加新功能之前，我们还是得考虑一下，问自己一个问题：“现在我加上这个功能，需要改动原有类的一些代码吗？”回答是肯定的。那不妨我们再戴上“重构”的帽子。
    仔细观察CCalculate类，其中的两个函数，一个是累计会员的点数，一个是根据会员享受的折扣来计算消费的金额，我想我们可以把他们移到CMember类中去会更好些。
    先修改main函数：
int main(int argc, char* argv[])
{
    char    szMemberID[MAX_PATH];
    strcpy(szMemberID,"00000001");
    float fConsumeSum = 120.0;
    int   nPoint    = 0; 

    CMember     *pMember = new CMember(szMemberID,1000.0,75);
    nPoint = pMember->GetPoint();
    nPoint += pMember->CalculatePoint(fConsumeSum);
    fConsumeSum = pMember->CalcMoney(fConsumeSum);
    pMember->SetPoint(nPoint); 

    assert(nPoint == 87);
    assert(fConsumeSum == 120.0); 

    printf("Point = %d/n",nPoint);
    printf("ConsumeSum = %f/n",fConsumeSum); 

    delete pMember;
    return 0;
}

而CMember中加了两函数：
int CMember::CalculatePoint(float fSum)
{
    int nPoint = (int)(fSum/10);
    return nPoint;
}

 float CMember::CalcMoney(float fSum)
{
    float fRebateSum = 0.0;
    fRebateSum = fSum * GetRebate()/10;
    return fRebateSum;
}
    编译程序，通过测试。
    让我们再来分析一下，CalculatePoint函数是否要向外公开了，我们完全可以在计算折扣金额的时候一起累计点数。
    所以main函数我们又可以修改了。
int main(int argc, char* argv[])
{
    char    szMemberID[MAX_PATH];
    strcpy(szMemberID,"00000001");
    float fConsumeSum = 120.0; 

    CMember     *pMember = new CMember(szMemberID,1000.0,75);
    fConsumeSum = pMember->CalcMoney(fConsumeSum);
    assert(pMember->GetPoint() == 87);
    assert(fConsumeSum == 120.0); 

    printf("Point = %d/n",nPoint);
    printf("ConsumeSum = %f/n",fConsumeSum); 

    delete pMember;
    return 0;
}
修改CMember::CalculatePoint为私有函数。
修改CMember::CalcMoney函数。
float CMember::CalcMoney(float fSum)
{
    float fRebateSum = 0.0;
    fRebateSum = fSum * GetRebate()/10;
    CalculatePoint(fSum);
    return fRebateSum;
}
    编译程序通过，可调试不通过。简单的查了下原因，原来是CMember::CalculatePoint还没有修改正确。
    修改后：
int CMember::CalculatePoint(float fSum)
{
    m_nPoint = m_nPoint + (int)(fSum/10);
    return m_nPoint;
}
    编译程序，通过测试。
    CMember::CalculatePoint还需要返回参数吗？改成void的吧。别忘了编译测试，保证每次的一小步，我们也能走到“黄河”。
    再思考一下，CalcMoney这个名字适合吗？RebateSumAndCumulatePoint可能会更好点。
    以上的重构都不是为了添加新功能，只是为了程序更简洁、更具可读性。下面的重构才开始为了添加新功能而做的。UML图如下：

    把现有的CRebateRule改名为CNormalRebateRule，然后修改程序让其通过编译测试。再创建一个新的抽象类CRebateRule。
class CRebateRule 
{
public:
    virtual ~CRebateRule();
    virtual bool  Init(char *pszConfigFilename) = 0;
    virtual float CalcRebate(int nPoint) = 0;
};
    根据设计，我们继续修改程序，使它通过编译测试。到现在为止，我们可以加新功能了。这样我们添加CSpecialRebateRule类，现有的类都不需要修改了，戴上“添加新功能”的帽子吧。
    我们先规定折上在打9折。先写main函数：
int main(int argc, char* argv[])
{
    char    szMemberID[MAX_PATH];
    strcpy(szMemberID,"00000001");
    float fConsumeSum = 120.0; 

    CMember     *pMember = new CMember(szMemberID,1000.0,75);
    CRebateRule *pRebateRule = new CNormalRebateRule;
    pRebateRule->Init("Configfile.txt"); 
    fConsumeSum = pMember->RebateSumAndCumulatePoint(fConsumeSum,pRebateRule);
    assert(pMember->GetPoint() == 87);
    assert(fConsumeSum == 120.0);
    printf("Point = %d/n",pMember->GetPoint());
    printf("ConsumeSum = %f/n",fConsumeSum); 

    delete pRebateRule;
    pRebateRule = new CSpecialRebateRule;
    pRebateRule->Init("Configfile.txt");    
    fConsumeSum = pMember->RebateSumAndCumulatePoint(300.0,pRebateRule);
    assert(pMember->GetPoint() == 117);
    assert(fConsumeSum == 270.0);
    printf("Point = %d/n",pMember->GetPoint());
    printf("ConsumeSum = %f/n",fConsumeSum);   

    delete pMember;
    delete pRebateRule;
    return 0;
}
    增加CSpecialRebateRule类及函数，这里类的代码我就不写出来了，可以下载代码。
float CSpecialRebateRule::CalcRebate(int nPoint)
{
    //假设现在用户已经享受9.5折的优惠。
    return (9.5*9.0/10);
}
    编译程序，通过测试。
    也许有人会问，这里为何不用Factory Method设计模式呢？在这里折扣算法类只有CMember类使用，而不是像Log类样到处使用的，毕竟使用Factory Method还会引入Factory类，会增加程序的复杂度，在不必要用的时候，我们还是避免使用，简单的可以使用Simple Factory来替代。
    记住一点：不要一开始就添加新功能，而是通过重构的手法，不改变软件的原来功能，逐步地修改程序，使其适合新功能的加入。
    好了，说的也比较罗嗦了，但这是一个编程的过程。
    使用C++语言编写，在VC++ 6.0环境调试通过。
    下载代码 

参考资料：
 《Refactoring: Improving the Design of Existing Code》 ――Martin Fowler
 《Design Patterns - Elements of Reusable Object-Oriented Software》 ――GoF

http://goodcandle.cnblogs.com/archive/2006/03/10/booksell2.html