重构第10天：提取方法(Extract Method)

理解：经常写的代码中，有一些计算逻辑比较复杂的方法，写下来一个很长很长的方法，我们可以把这个方法，根据功能，分解成单独的几个小方法。这样做不仅能够增加代码的可维护性，而且增加了易读性。

详解：

重构前代码：

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方法，里面用到了2个foreach循环，看一下，还真不宜理解，不晓得每个foreach主要做什么。

看看重构后的代码：

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;
}
}

重构后，我们把CalculateGrandTotal方法分解成3个小方法，每个方法名称根据其功能命名。CalculateTax顾名思义，就是计算税收，CalculateDiscounts就是计算折扣金额。

这样就把逻辑剥离的很清楚了。