第10天：提取方法

返回顶部
查看留言
转到底部

现在的位置: 首页 > 综合 > 正文

第10天：提取方法

2012年06月10日 ⁄ 综合 ⁄ 共 1183字 ⁄ 字号小中大 ⁄ 评论关闭

这一条重构的建议是尽量提取方法，使得代码更加清晰明了

旧的代码：

Code
public class Receipt
{
private IList<decimal> Discounts { get; set; }
private IList<decimal> ItemTotals { get; set; }
public decimal CalculateGrandTotal()
{
decimal subTotal = 0m;
foreach (decimal itemTotal in ItemTotals)
subTotal += itemTotal;
if (Discounts.Count > 0)
{
foreach (decimal discount in Discounts)
subTotal -= discount;
}
decimal tax = subTotal * 0.065m;
subTotal += tax;
return subTotal;
}
}

为了弄明白CalculateGrandTotal() 这个方法的意思你会把代码整个看一遍，从而知道是做了3件事情

1，计算总额

2，计算优惠折扣

3，添加消费税

相比较而言我们可以作如下重构

Code
public class Receipt
{
private IList<decimal> Discounts { get; set; }
private IList<decimal> ItemTotals { get; set; }
public decimal CalculateGrandTotal()
{
decimal subTotal = CalculateSubTotal();
subTotal = CalculateDiscounts(subTotal);
subTotal = CalculateTax(subTotal);
return subTotal;
}
private decimal CalculateTax(decimal subTotal)
{
decimal tax = subTotal * 0.065m;
subTotal += tax;
return subTotal;
}
private decimal CalculateDiscounts(decimal subTotal)
{
if (Discounts.Count > 0)
{
foreach (decimal discount in Discounts)
subTotal -= discount;
}
return subTotal;
}
private decimal CalculateSubTotal()
{
decimal subTotal = 0m;
foreach (decimal itemTotal in ItemTotals)
subTotal += itemTotal;
return subTotal;
}
}