设计：管理通过 Activator.CreateInstance(...) 调用的程序集

Question

我目前正在研究优化 C# (.NET 3.5) 应用程序框架，目前我正在研究使用 Activator.CreateInstance 通过接口调用程序集方法的部分代码。作为示例，我有以下功能齐全的代码：

private object InvokeAssembly(string assemblyPath, string assemblyType, string data)
{
    Assembly assembly = Assembly.LoadFrom(assemblyPath);
    Type type = assembly.GetType(assemblyType, true, true);
    IMyInterface assemblyInterface = (IMyInterface)Activator.CreateInstance(type);

    return assemblyInterface.DoSomething(data);
}

问题是，这是一个好的设计吗？特别是考虑到这种特定方法每分钟被调用 100 次，那么下面的单例模式会是更好的设计吗？

改进的示例代码，由 JaredPar 提供

Dictionary<string, IMyInterface> assemblyCache = new Dictionary<string, IMyInterface>();

private object InvokeAssembly(string assemblyPath, string assemblyType, string data)
{
    var cacheKey = assemblyPath + "*" + assemblyType;

    IMyInterface assemblyInterface;

    if (!this.assemblyCache.TryGetValue(cacheKey, out assemblyInterface))
    {
        Assembly assembly = Assembly.LoadFrom(assemblyPath);
        Type type = assembly.GetType(assemblyType, true, true);
        assemblyInterface = (IMyInterface)Activator.CreateInstance(type);
        assemblyCache[cacheKey] = assemblyInterface;
    }

    return assemblyInterface.DoSomething(data);
}

在第二个示例中，接口创建一次并重用，而不是每个请求创建/收集一次。

第二种方法似乎是更好的解决方案，但我很高兴被告知这没有什么区别。

还值得一提的是，这个方法并且只有这个方法对Assembly进行操作

Answer 1

第二种解决方案性能更好，但有一个错误。它缓存 IMyInterface 实例并在方法调用之间重用它，而无需检查是否将相同的程序集路径和类型传递到方法中。使用不同 path 或 type 值的连续调用将导致使用错误的 IMyInterface 实例。缓存模式需要意识到这一点。

实现此目的的一种方法是使用 Dictionary 来缓存类型。

Dictionary<string, IMyInterface> _map = new Dictionary<string, IMyInterface>();

private object InvokeAssembly(string assemblyPath, string assemblyType, string data)
{
  var key = assemblyPath + "*" + assemblyType;
  IMyInterface data;
  if (!_map.TryGetValue(key, out data)) {
    Assembly assembly = Assembly.LoadFrom(assemblyPath);
    Type type = assembly.GetType(assemblyType, true, true);
    data = (IMyInterface)Activator.CreateInstance(type);
    _map[key] = data;
  }
  return data;
}

Answer 2

第二种解决方案肯定会表现得更好。程序集被缓存，因此不需要每次都从磁盘重新加载它，但 GetType 调用和创建实例仍然存在开销。

Answer 3

我认为第二种方法总体上较差——通过缓存以这种方式创建的实例，您将面临由该接口的实现维护的任何状态的风险，以一种潜在不可预测的方式与程序的其余部分进行交互。

我认为是 Activator 调用导致了性能问题？毕竟，CreateInstance 并不是本书中最快的东西——更好的方法是创建和缓存相关类型的构造函数的轻量级委托，然后调用它们。您可以通过 DynamicMethod 来做到这一点——在 Google 上搜索有关“工厂反射动态方法”的信息，这样您会得到很多结果，这是其中之一。