重构第1天：封装集合（Encapsulate Collection）

理解：封装集合就是把集合进行封装，只提供调用者所需要的功能行借口，保证集合的安全性。

详解：在大多的时候，我们没有必要把所有的操作暴露给调用者，只需要把调用者需要的相关操作暴露给他，这种情况中下我们就要对其进行封装，只把调用者需要的部分暴露出来。

我们先看一个例子：

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace _31DaysOfReflactor
{
    public class Order
    {
        private List<OrderLine> _orderLines;
        private double _orderTotal;

        public IList<OrderLine> OrderLines
        {
            get { return _orderLines; }
        }

        public void AddOrderLine(OrderLine orderLine)
        {
            _orderTotal += orderLine.Total;
            _orderLines.Add(orderLine);
        }

        public void RemoveOrderLine(OrderLine orderLine)
        {
            orderLine = _orderLines.Find(o => o == orderLine);

            if (orderLine == null)
                return;

            _orderTotal -= orderLine.Total;
            _orderLines.Remove(orderLine);
        }
    }

    public class OrderLine
    {
        public double Total { get; private set; }
    }
}

 如果我们对集合只允许调用端读取，遍历取值，而不允许其修改集合的值，那么我们把IList换成了IEnumerable。IEnumerable只包括一个返回值为IEnumerator的GetEnumerator()方法，用来遍历取值，就达到我们想要的目的。

重构后的代码如下：

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace _31DaysOfReflactor
{
    public class Order
    {
        private List<OrderLine> _orderLines;
        private double _orderTotal;

        public IEnumerable<OrderLine> OrderLines
        {
            get { return _orderLines; }
        }

        public void AddOrderLine(OrderLine orderLine)
        {
            _orderTotal += orderLine.Total;
            _orderLines.Add(orderLine);
        }

        public void RemoveOrderLine(OrderLine orderLine)
        {
            orderLine = _orderLines.Find(o => o == orderLine);

            if (orderLine == null)
                return;

            _orderTotal -= orderLine.Total;
            _orderLines.Remove(orderLine);
        }
    }

    public class OrderLine
    {
        public double Total { get; private set; }
    }
}

如上所示，我们对IList集合进行封装，没有把该集合的add/remove方法暴露给调用端。当然在.Net framework中提供了很多封装好的集合类，不同的类都有不同的作用，比如ReadOnlyCollection 集合，但是他们每个集合都有自己的使用局限性。上面的重构是一个值得注意的非常简单的例子，像这样的封装能够确保客户端不会错误的调用集合以免引进bugs。