【转】编写高质量代码改善C#程序的157个建议——建议92：谨慎将数组或集合作为属性

建议92：谨慎将数组或集合作为属性

数组或集合作为属性会引起这样一个问题：如果属性是只读的，我们通常会认为它是不可用改变的，但如果将只读属性应用于数座或集合，而元素的内容和数量却仍旧可以随意改变。如下所示：

        static void Main(string[] args)
        {
            Company microsoft = new Company();
            microsoft.Employees[0].Name = "LiMing";
            foreach (var item in microsoft.Employees)
            {
                Console.WriteLine(item.Name);
            }
            Console.ReadKey();
        }

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

        class Company
        {
            public Company()
            {
                Employees = new List<Employee>()
                {
                    new Employee(){ Name = "Bill Gates"}
                };
            }
            public IList<Employee> Employees { get; private set; }
        }

在上面的代码中，我们可以随意对Employees进行集合操作，它不改变的只是自身的引用而已。

如果某个类型含有集合概念的属性，那么他的可见性应该是private或protected，并且，它更应该是一个字段。类型对外只公开必要的方法来操作这个集合。

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