编写高质量代码改善C#程序的157个建议——建议25：谨慎集合属性的可写操作

建议25：谨慎集合属性的可写操作

如果类型的属性中有集合属性，那么应该保证属性对象是由类型本身产生的。如果将属性设置为可写，则会增加抛出异常的几率。一般情况下，如果集合属性没有值，则它返回的Count等于0，而不是集合属性的值为null。下面的代码将产生一个NullReferenceException异常：

    class Program

    {

        static List<Student> listStudent = new List<Student>()

            {

                new Student(){ Name = "Mike", Age = },

                new Student(){ Name = "Rose", Age = }

            };

        static void Main(string[] args)

        {

            StudentTeamA teamA = new StudentTeamA();

            Thread t1 = new Thread(() =>

            {

                teamA.Students = listStudent;

                Thread.Sleep();

                Console.WriteLine(listStudent.Count); //模拟对

                //集合属性进行一些运算

            });

            t1.Start();

            Thread t2 = new Thread(() =>

            {

                listStudent = null;   //模拟在别的地方对list1而

                //不是属性本身赋值为null

            });

            t2.Start();

        }

    }

    class Student

    {

        public string Name { get; set; }

        public int Age { get; set; }

    }

    class StudentTeamA

    {

        public List<Student> Students { get; set; }

    }

上面代码的问题是：线程t1模拟将对类型StudentTeamA的Students属性进行赋值，它是一个可读/可写的属性。由于集合属性是一个引用类型，而当前针对该属性对象的引用却又两个，即集合本身和调用者的类型变量listStudent，线程t2也许是另一个程序员写的，但他看到的只有listStudent，结果，针对listStudent的修改hi直接影响到另一个工作线程的对象。在例子中，我们将listStudent赋值为null，模拟在StudentTeamA（或者说工作线程t1）不知情的情况下使得集合属性变为null。接着，线程t1模拟针对Students属性进行若干操作，导致抛出异常。

下面的StudentTeamA版本是一个改进过的版本。首先，将类型的集合属性设置为只读；其次，集合对象由类型自身创建，这保证了集合属性永远只有一个引用：

    class Program

    {

        static List<Student> listStudent = new List<Student>()

            {

                new Student(){ Name = "Mike", Age = },

                new Student(){ Name = "Rose", Age = }

            };

        static void Main(string[] args)

        {

            StudentTeamA teamA2 = new StudentTeamA();

            teamA2.Students.Add(new Student() { Name = "Steve", Age =  });

            teamA2.Students.AddRange(listStudent);

            Console.WriteLine(teamA2.Students.Count);

            //也可以像下面这样实现

            StudentTeamA teamA3 = new StudentTeamA(listStudent);

            Console.WriteLine(teamA3.Students.Count);

        }

    class Student

    {

        public string Name { get; set; }

        public int Age { get; set; }

    }

    class StudentTeamA

    {

        public List<Student> Students { get; private set; }

        public StudentTeamA()

        {

            Students = new List<Student>();

        }

        public StudentTeamA(IEnumerable<Student> studentList)

            : this()

        {

            Students.AddRange(studentList);

        }

    }

在改进版本的StudentTeamA中尝试对属性Students进行赋值：

 teamA.Students = listStudent;

将导致编译通不过。

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

巴特西

编写高质量代码改善C#程序的157个建议——建议25：谨慎集合属性的可写操作

最新文章

热门文章