影片出租店 重构方法(4)

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

第3次重构:

  • Replace Temp with Query: 去除临时变量
    • ThisAmount是个临时变量,可以直接去除。使用 GetCharge()来代替。
    • 但如果GetCharge()计算很复杂,很耗时间,可以保留ThisAmount这个临时变量
  • Extract Method: 将积分计算抽出来。
    • 积分计算根据影片种类而有不同,可以将其单独成立一个函数
    • 将积分计算函数移动到rental类中

修改后的代码:

QString Customer::Statement()
{
    double TotalAmout = 0;
    int FrequentRenterPoints = 0;
    QString result = "Rental Record for " + GetName() + "\n";
    for (Rental *rental : GetRentals())
    {
        double ThisAmount = 0;
        ThisAmount = AmountFor(rental);
        FrequentRenterPoints += rental->GetFrequentRenterPoints();
        result += "\t" + rental->GetMovie()->GetTitle() + "\t" + QString("%1").arg(ThisAmount) + "\n";
        TotalAmout += ThisAmount;
    }

    result += "Amount owed is " + QString("%1").arg(TotalAmout) + "\n";
    result += "You earned " + QString("%1").arg(FrequentRenterPoints) + " frequent renter points";
    qDebug() << qPrintable(result) ;
    return  result;
}

int Rental::GetFrequentRenterPoints()
{
    if (Movie::NEW_RELEASE == (GetMovie()->GetPriceCode()) &&
            GetDaysRented() > 1 )
    {
        return 2;
    }
    return 1;
}

测试结果:影片出租店 重构方法(4)_第2张图片

你可能感兴趣的:(重构)