影片出租店重构方法（6）

最新推荐文章于 2020-02-27 23:13:22 发布

giantmfc123

最新推荐文章于 2020-02-27 23:13:22 发布

阅读量848

点赞数

CC 4.0 BY-SA版权

分类专栏：重构

本文链接：https://blog.youkuaiyun.com/mafucun1988/article/details/89254664

重构专栏收录该内容

84 篇文章

订阅专栏

本文通过重构《重构：改善既有代码的设计》中的影片出租店代码示例，详细展示了如何优化GetCharge和GetFrequentRenterPoints函数，将其移动至Movie类中以减少未来维护成本。通过对代码的修改，不仅提高了模块的独立性和可读性，还降低了耦合度。

摘要生成于 C知道，由 DeepSeek-R1 满血版支持，前往体验 >

影片出租店是《重构：改善既有代码的设计》里面的第一个例子。使用了多种重构方法。
影片出租店：计算每一位顾客的消费金额并打印详单。
操作者告诉程序：顾客租了哪些影片,租期多长，程序便根据租赁时间和影片类型算出费用。影片分为三类：普通片，儿童片和新片。除了计算费用，还要为常客计算积分，积分会根据租片种类是否为新片而有不同。

第5次重构：

Move Method: 由于GetCharge/GetFrequentRenterPoints主要用到Movie类中的函数，将两者的变化可以控制在movie类中，将来变化时改动要小一些。
- 最好不要在另一个对象的属性基础上运用switch语句。如果不得不使用，也应该在对象自己的数据上使用。
- 将GetCharge移动到Movie类中。
- 将GetFrequentRenterPoints移动到Movie类中。

修改后的代码：

int Rental::GetFrequentRenterPoints()
{
    return GetMovie()->GetFrequentRenterPoints(GetDaysRented());
}
double Rental::GetCharge()
{
    return GetMovie()->GetCharge(GetDaysRented());
}

int Movie::GetFrequentRenterPoints(int daysRented)
{
    if (NEW_RELEASE == (GetPriceCode()) &&
            daysRented > 1 )
    {
        return 2;
    }
    return 1;
}

double Movie::GetCharge(int daysRented)
{
    double Result = 0;
    switch (GetPriceCode())
    {
    case Movie::REGULAR:
        Result += 2;
        if (daysRented > 2)
        {
            Result += (daysRented - 2)*1.5;
        }
        break;
    case Movie::NEW_RELEASE:
        Result += daysRented * 3;
        break;
    case Movie::CHILDREN:
        Result += 1.5;
        if (daysRented > 3)
        {
            Result += (daysRented - 3)*1.5;
        }
        break;
    }
    return Result;
}

测试结果：