【转】编写高质量代码改善C#程序的157个建议——建议95:避免在构造方法中调用虚成员

建议95:避免在构造方法中调用虚成员

在构造方法中调用虚方法会带来一些意想不到的错误,虽然这种方法不常见,但还是需要注意这类陷阱。

        static void Main()
        {
            American american = new American();
            Console.ReadKey();
        }

        class Person
        {
            public Person()
            {
                InitSkin();
            }

            protected virtual void InitSkin()
            {
                //省略
            }
        }

        class American : Person
        {
            Race Race;

            public American()
                : base()
            {
                Race = new Race() { Name = "White" };
            }

            protected override void InitSkin()
            {
                Console.WriteLine(Race.Name);
            }
        }

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

运行会出现异常NullReferenceException:未将对象引用设置到对象的实例。

在调用者代码中,我们需要创建一个American的实例对象american。由于发现实例还存在一个积累Person,所以运行时会首先调用基类的构造方法。在构造方法中Person调用了虚方法InitSkin。由于是虚方法,所以会在运行时调用子类的InitSkin方法。子类的InitSkin方法中,需要打印出名字。而这个时候,方法的调用堆栈还一直在基类的构造方法内,也就是在子类的构造方法中的代码还完全没有执行:

Race = new Race() { Name = "White" };

所以会抛出异常。

基于以上原因,建议不要在构造方法中调用虚成员。

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