c#-4.0 - C# 中的 Activator.CreateInstance 和泛型方法

Question

interface IExecutor
{
   void Execute();
}

class Executor2<T> where T : IExecutor
{
    public void Execute()
    {
        var ex = (T)Activator.CreateInstance(typeof(T));
        ex.Execute();
    }
}

这是面试时的一个问题。他们告诉我，有时这段代码会失败(导致异常)，并且至少有 3 个原因可能导致问题。目前尚不清楚有哪些异常(exception)情况。但是Execute方法创建得很好，它的实现没有错误。

有人对此有什么建议吗？

编辑:至少有 3 个原因可能导致问题。这些原因是什么？

Answer 1

从表面上看，我可以看到一些问题。

1. 代码无法编译，但我可以忽略它并使其编译。
2. 该代码并没有按照您的想法进行操作。

解释2:你指定一个类型T在类定义中，约束为 IExecutor ，但您随后定义了另一种类型 T在方法级别没有约束。这无法编译。

如果我修复此问题并删除 <T>从该方法的定义我可以看到它失败的许多原因，而没有太多警告:

1. ex为空。
2. 类型 T没有定义公共(public)无参数构造函数。
3. 也许它无法加载包含 T 的 DLL .

Jakub 发现:

1. T可以是一个接口(interface)(无构造函数)。
2. T可能是一个抽象类，它们不允许直接创建实例。

可以使用空检查来防止第一个 if (ex != null)可以使用另一个通用约束来防止第二个 new() :

class Executor2<T> where T : IExecutor, new()
{
}

显然，您也可以修改代码以包含异常日志记录。这可能有助于找出实际问题，而不是盲目地摸索:

public void Execute<T>()
{
    try
    {
        var ex = (T)Activator.CreateInstance(typeof(T));
        ex.Execute();
    }
    catch (Exception ex)
    {
        Log(ex); // Mystical logging framework.
        throw;
    }
}

考虑到我不明白这个问题，这是我能拼凑出来的唯一答案。

如果我在面试中被问到这个问题，我可能会说我无法说出所有 3 个代码，但我知道如何更改代码以使其更易于维护，并告诉我出了什么问题。然后我可能会因为问毫无意义的面试问题而走开。