反射或动态调度

Question

我正在编写一个抽象文件解析器（C#），它由两个具体解析器扩展。两者都需要执行多项检查。目前抽象解析器中有一个 validate 方法，它使用反射来调用名称以 'test' 开头的所有方法。这样，添加检查就像添加名称以“test”开头的方法一样简单。

最近我对反射的使用有一些评论，并且最好使用动态调度。我向您提出的问题是，为什么不使用反射以及您将如何实现它？另外我应该如何使用动态调度来解决这个问题？

    public bool Validate()
    {
        bool combinedResult = true;
        Type t = this.GetType();
        MethodInfo[] mInfos = t.GetMethods();

        foreach (MethodInfo m in mInfos)
        {
            if (m.Name.StartsWith("Check") && m.IsPublic)
            {
                combinedResult &= (bool)m.Invoke(this, null);
            }
        }
        return combinedResult;
    }

Answer 1

为此，您应该使用常规的 OOP 而不是反射。您的抽象类是否公开了像 Validate 这样的抽象方法。每个解析器都必须实现它。在Validate中，每个解析器都会调用相关的Check方法来完成工作。

Answer 2

有效的代码没有任何问题......直到有人来维护它。特别是，这种约定需要非常仔细地记录下来，因为你的类将如何做它所做的事情并不是立即显而易见的。

（不过，顺便说一句，为此使用反射会相当慢。）

最明显的方法可能是使用由子类实现的抽象基方法 bool Validate() 。然后子类有例如

public override bool Validate()
{
    return TestFirst() && 
        TestSecond() &&  
        TestThird();
 }

虽然这看起来很庞大，但发生的事情立即显而易见。它还使 Validate() 的单元测试变得轻而易举。

也可以让 Testxxx() 方法在构造函数中向超类注册自身，以便自动调用它们 - 但这会增加工作量，而且可能不太容易维护。

如果您确实想通过反射来完成此操作，请考虑使用属性标记 Testxxx() 方法，然后对这些方法进行反射。然后你的代码仍然可读。

Answer 3

据我了解，动态调度适用于需要根据参数类型确定要调用的方法的情况。在你的情况下，你调用不带参数的方法，所以我不确定动态调度与此有什么关系。

我喜欢你处理快速而肮脏的事情的方法。对于生产质量的代码，我发现您的方法存在以下潜在问题：

• 您不检查参数类型和返回类型，因此如果您或某人添加了方法string CheckWithWrongReturnType，那么您的代码就会中断。
• 每次调用此函数时，它都会调用 GetMethods() 并遍历列表。这可能是低效的。将此列表缓存在数组中可能会更好。

为了避免反射，我将创建一个委托并让每个类返回一个委托列表。

delegate bool Validator();

bool F1() { return true;  }
bool F2() { return false; }

List<Validator> validators = new List<Validator>(F1, F2);

// 然后在主类中你可以这样做：

foreach(Validator v in validators)
{
   combinedResult &= v();
}