c# - 来自基类构造函数的虚函数

Question

我在下面的代码片段中总结了我的问题

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

namespace St
{
    public  class Animal
    {
        public Animal()
        {
            Speak();
        }
      public virtual void Speak()
      {
          Console.WriteLine("Animal speak");
      }
    }
    public  class Dog:Animal
    {
        private StringBuilder sb = null;

        public Dog()
        {
                sb=new StringBuilder();
        }
        public override void Speak()
        {
            Console.WriteLine("bow...{0}",sb.Append("bow"));
        }
    }
    class Program
    {
        static void Main(string[] args)
        {
            Dog d=new Dog();
        }
    }
}

当我编译它时没有错误，但是当我运行它时却出现对象引用错误。

Answer 1

问题是您在 Animal 的构造函数中调用了虚方法。

这是一种危险的做法 - 正是出于这个原因。问题是，当您构造 Dog 时，“Animal”(基类)构造函数首先运行。此时，它调用 Speak()。但是，Dog 的 Speak 方法依赖于 Dog 的构造函数已经运行，因此 sb 尚未初始化，仍然是 null。

一般来说，在构造函数中调用虚方法是一个非常糟糕的主意 - 并且是设计缺陷的标志。我在这里推荐一种不同的方法。

---

我对修改的建议是简单地从构造函数中删除 Speak()。我会把这段代码写成:

public class Animal
{
    public Animal()
    {
        // Don't do this in the constructor
        // Speak();
    }
    public virtual void Speak()
    {
        Console.WriteLine("Animal speak");
    }
}
public class Dog : Animal
{
    private StringBuilder sb = null;

    public Dog()
    {
        sb = new StringBuilder();
    }
    public override void Speak()
    {
        Console.WriteLine("bow...{0}", sb.Append("bow"));
    }
}
class Program
{
    static void Main(string[] args)
    {
        Dog d = new Dog();
        d.Speak();
    }
}

从逻辑的角度来看，这对我来说更有意义。这一行:

Dog d = new Dog();

负责一个 Action ——构造一个新的Dog。作为类的使用者，我不希望它执行复杂的操作(即:说话)- 只是为了正确地创建和设置 Dog 及其内部状态。

当我想让它说话时，我专门调用:

d.Speak();