如何检测c#中的虚方法是否被重写

Question

是否可以确定虚拟方法是否已被重写：

class ABase {

  public void DoSomething(object p)
  {
    p.Process();
    if( /* DoSomethingExtra is implemented */ )
      DoSomethingExtra(p);
  }
  public virtual void DoSomethingExtra(object p) { }

}

class ADerived {
  public override void DoSomethingExtra(object p)
  {
    p.ProcessMore();
  }
}

我意识到这个示例看起来很愚蠢（例如，为什么不直接调用 DoSomethingExtra() 因为它不执行任何操作）。我向你保证我有一个合法的理由。有什么想法吗？

Answer 1

因此，在 C# 中，可以声明虚拟方法而不实现它。

这是不可能的。您可以将一个方法声明为抽象，但如果该方法是虚拟的，它将有一些实现（即使该实现实际上是一个空操作）。

上面的代码报告错误Error 1 'ABase.DoSomethingExtra(object)'必须声明一个主体，因为它没有标记为抽象、外部或部分。

处理此问题的典型方法是仅声明具有空操作实现的方法，并调用它：

class ABase {
  public void DoSomething(object p)
  {
    p.Process();
    DoSomethingExtra(p); // Always call
  }
  public virtual void DoSomethingExtra(object p)
  {
      // Do nothing here
  }
}

---

编辑：既然您的问题已被编辑，我将向您提供与您的编辑相关的更多信息，特别是：

我意识到这个例子看起来很愚蠢（例如，为什么你不直接调用 DoSomethingExtra() 因为它没有做任何事情）。我向你保证我有一个合法的理由。有什么想法吗？

没有直接的方法可以确定当前实例是否覆盖了您的虚拟方法。这可能需要一些非常讨厌的、难以维护的代码，例如检查 通过反射声明类型的方法体看看有什么。

话虽这么说，我强烈质疑这里的设计目标。您的问题基本上是在询问一种违反里氏替换原则的具体方法，该原则是面向对象编程的核心原则。这将会降低你的代码的性能和可维护性......

Answer 2

反思是对这个问题的一个很好的回答。

using System.Reflection;

Type classType = typeof(ADerived);
MethodInfo method = classType.GetMethod("DoSomethingExtra");
if (method.DeclaringType == typeof(ABase))
    Console.WriteLine("DoSomethingExtra not overridden.");
else
    Console.WriteLine("DoSomethingExtra is overridden by " + method.DeclaringType.Name);

我希望您会发现这很有用。

一旦我实现了一个特殊的对象查看器，当对象未知时，我将使用 ToString() ，除非它没有被覆盖。

Answer 3

检查一下：检测是否使用反射覆盖了方法（C# )

反射是在运行时执行此操作的唯一方法。这应该带有健康警告，但是从面向对象编程的角度来看，它应该被视为非常糟糕的想法™。基类通常不应该知道或关心派生类是如何实现的。

Answer 4

有几种选择：

如果派生类必须实现DoSomethingExtra()，则将方法和类声明为抽象。这将迫使具体的派生类有一个实现。然后，您可以从基类代码中调用 DoSomethingExtra()，知道将存在一个实现。

abstract class ABase {

  public void DoSomething(object p)
  {
    p.Process();
    DoSomethingExtra(p);
  }

  public abstract void DoSomethingExtra(object p);
}

如果派生类可以实现该方法，则只需在基类中包含一个默认实现，如果没有可用的派生实现，就会调用该默认实现。

另一种选择是派生类可以设置一个标志，指示它们是否想要完成一些额外的操作：

class ABase {

  public virtual bool ShouldDoSomethingExtra { get { return false; } }

  public void DoSomething(object p)
  {
    p.Process();
    if(ShouldDoSomethingExtra)
      DoSomethingExtra(p);
  }
  public virtual void DoSomethingExtra(object p) { // Empty in base }
}

class ADerived {
  public override void DoSomethingExtra(object p)
  {
    p.ProcessMore();
  }

  public override bool ShouldDoSomethingExtra { get { return true; } }
}

不过，此解决方案有点脆弱，因为派生类可能会忘记在以下情况下重写该属性：覆盖该方法。我认为不做任何事情的基础实现是最简单的方法。

Answer 5

这是获得所需内容的最简单方法：

var isDerived = typeof(ADerived).GetMember("DoSomethingExtra", 
                   BindingFlags.NonPublic 
                 | BindingFlags.Instance 
                 | BindingFlags.DeclaredOnly).Length == 0;

更新：提供的链接这里比必要的更详细，但也不完整。上面的版本也适用于受保护的方法和虚拟属性（这就是为什么您使用 GetMember 而不是限制性更强的 GetMethod），这两个链接都没有解决。