【转】编写高质量代码改善C#程序的157个建议——建议23：避免将List<T>作为自定义集合类的基类

 

建议23：避免将List<T>作为自定义集合类的基类

如果要实现一个自定义的集合类，不应该以一个FCL集合类为基类，反而应扩展相应的泛型接口。FCL结合类应该以组合的形式包含至自定义的集合类，需要扩展的泛型接口通常是IEnumerable<T>和ICollection<T>（或ICollection<T>的子接口，如IList<T>），前者规范了集合类的迭代功能，后者规范了一个集合通常会有的操作。

一般的情况下，下面两个实现的集合类都能完成默认的需求：

class Employees1 : List<Employee>
class Employees2 : IEnumerable<Employee>, ICollection<Employee>

不过，List<T>基本上没有提供可供子类使用的protected成员（从object中继承的Finalize和MemberwiseClone方法除外），所以继承List<T>并没有带来任何继承上的优势，反而丧失了面向接口编程的灵活性。稍加不注意，隐含的Bug就会接踵而至。

以Employees1为例，如果要在Add方法中加入某些需求方面的变化，比如，为名字添加一个后缀“Changed!"，但是客户端的开发人员也许已经习惯了面向接口编程的方式，他在为集合添加一个元素是使用了如下的语法：

        static void Main(string[] args)
        {
            Employees1 employees1 = new Employees1() 
            {
                new Employee(){ Name = "Mike" },
                new Employee(){ Name = "Rose" }
            };
            IList<Employee> employees = employees1;
            employees.Add(new Employee() { Name = "Steve" });
            foreach (var item in employees1)
            {
                Console.WriteLine(item.Name);
            }
        }

        class Employee
        {
            public string Name { get; set; }
        }

        class Employees1 : List<Employee>
        {
            public new void Add(Employee item)
            {
                item.Name += " Changed!";
                base.Add(item);
            }
        }

于是，代码的实际输出会偏离集合类设计者的设想。代码输出为：

Mike Changed!
Rose Changed!
Steve

要纠正这类行为，应该采用Employees2的方式：

        static void Main(string[] args)
        {
            Employees2 employees2 = new Employees2() 
            {
                new Employee(){ Name = "Mike" },
                new Employee(){ Name = "Rose" }
            };
            ICollection<Employee> employees = employees2;
            employees.Add(new Employee() { Name = "Steve" });
            foreach (var item in employees2)
            {
                Console.WriteLine(item.Name);
            }
        }

        class Employees2 : IEnumerable<Employee>, ICollection<Employee>
        {
            List<Employee> items = new List<Employee>();

            #region IEnumerable<Employee> 成员

            public IEnumerator<Employee> GetEnumerator()
            {
                return items.GetEnumerator();
            }

            #endregion

            #region ICollection<Employee> 成员

            public void Add(Employee item)
            {
                item.Name += " Changed!";
                items.Add(item);
            }

            //省略

            #endregion
        }

输出结果为：

Mike Changed!
Rose Changed!
Steve Changed!

 

 

转自：《编写高质量代码改善C#程序的157个建议》陆敏技