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