构造函数中的虚拟成员调用

我从ReSharper收到一条关于从对象构造函数调用虚拟成员的警告。

为什么这是不该做的？

当前回答

小心盲目地听从Resharper的建议，让课堂变得封闭！如果它是EF Code First中的模型，它将删除虚拟关键字，这将禁用其关系的延迟加载。

    public **virtual** User User{ get; set; }

其他回答

在这个特定的例子中，C++和C#之间有区别。在C++中，对象未初始化，因此在构造函数中调用病毒函数是不安全的。在C#中，当创建类对象时，其所有成员都是零初始化的。可以在构造函数中调用虚拟函数，但如果您将访问仍然为零的成员。如果您不需要访问成员，那么在C#中调用虚拟函数是非常安全的。

因为在构造函数完成执行之前，对象不会完全实例化。虚拟函数引用的任何成员都不能初始化。在C++中，当您处于构造函数中时，这仅指所处构造函数的静态类型，而不是所创建对象的实际动态类型。这意味着虚拟函数调用甚至可能不会到达您期望的位置。

警告的原因已经描述过了，但您将如何修复警告？您必须密封类或虚拟成员。

  class B
  {
    protected virtual void Foo() { }
  }

  class A : B
  {
    public A()
    {
      Foo(); // warning here
    }
  }

您可以密封A类：

  sealed class A : B
  {
    public A()
    {
      Foo(); // no warning
    }
  }

或者您可以密封方法Foo：

  class A : B
  {
    public A()
    {
      Foo(); // no warning
    }

    protected sealed override void Foo()
    {
      base.Foo();
    }
  }

小心盲目地听从Resharper的建议，让课堂变得封闭！如果它是EF Code First中的模型，它将删除虚拟关键字，这将禁用其关系的延迟加载。

    public **virtual** User User{ get; set; }

是的，在构造函数中调用虚拟方法通常是不好的。

此时，对象可能还没有完全构造，方法所期望的不变量可能还不成立。

推荐文章