Question

根据传递给Factory类的泛型类型实例化对象的最有效方法是什么，例如：

public class LoggerFactory
{
    public static ILogger<T> Create<T>()
    {
        // Switch Statement?
        // Generic Dictionary?
        // EX.: if "T" is of type "string": return (ILogger<T>)new StringLogger();
    }
}

你会怎么做？哪个分支声明？等...

Answer 1

我认为最好保持简单，也许是这样的：

public static class LoggerFactory
{
    static readonly Dictionary<Type, Type> loggers = new Dictionary<Type, Type>();

    public static void AddLoggerProvider<T, TLogger>() where TLogger : ILogger<T>, new()
    {
        loggers.Add(typeof(T), typeof(TLogger));
    }

    public static ILogger<T> CreateLogger<T>()
    {
        //implement some error checking here
        Type tLogger = loggers[typeof(T)];

        ILogger<T> logger = (ILogger<T>) Activator.CreateInstance(tLogger);

        return logger;
    }
}

你只需为你想支持的每种类型调用AddLoggerProvider，可以在运行时扩展，它确保你明确地将一个接口的实现添加到库而不是某些对象，不是很快，因为Activator，但创建一个记录器无论如何都不会成为瓶颈。希望它看起来没问题。

用法：

// initialize somewhere
LoggerFactory.AddLoggerProvider<String, StringLogger>();
LoggerFactory.AddLoggerProvider<Exception, ExceptionLogger>();
// etc..

ILogger<string> stringLogger = LoggerFactory.CreateLogger<string>();

注意：每个ILogger<T>都需要Activator的无参数构造函数，但在add方法中也需要使用new()泛型约束。

Answer 2

我想我会这样做：

public class LoggerFactory<T>
{
    private static Dictionary<Type, Func<ILogger<T>>> LoggerMap = 
        new Dictionary<Type, Func<ILogger<T>>>
    {
        { typeof(string), 
            () => new StringILogger() as ILogger<T> },
        { typeof(StringWriter), 
            () => new StringWriterILogger() as ILogger<T> }
    };

    public static ILogger<T> CreateLogger()
    {
        return LoggerMap[typeof(T)]();
    }
}

你支付一些可读性价格（所有那些尖括号，sheesh），但正如你所看到的那样，它只能产生很少的程序逻辑。

Answer 3

虽然我通常会建议使用依赖注入框架，但您可以使用反射来实现某些内容，以便为实现相应ILogger接口的类型搜索可用类型。

我建议您仔细考虑哪些程序集将包含这些记录器实现以及您希望解决方案的可扩展性和防弹性。跨可用程序集和类型执行运行时搜索并不便宜。但是，这是一种在这种类型的设计中允许可扩展性的简单方法。它还避免了前期配置的问题 - 但是它要求只有单个具体类型实现特定版本的ILogger＆lt;＆gt;。界面 - 否则你必须解决一个模糊的情况。

您可能希望执行一些内部缓存，以避免在每次调用Create（）时执行反射的费用。

以下是一些您可以开始的示例代码。

using System;
using System.Linq;
using System.Reflection;

public interface ILogger<T> { /*... */}

public class IntLogger : ILogger<int> { }

public class StringLogger : ILogger<string> { }

public class DateTimeLogger : ILogger<DateTime> { }

public class LoggerFactory
{
    public static ILogger<T> Create<T>()
    {
        // look within the current assembly for matching implementation
        // this could be extended to search across all loaded assemblies
        // relatively easily - at the expense of performance
        // also, you probably want to cache these results...
        var loggerType = Assembly.GetExecutingAssembly()
                     .GetTypes()
                     // find implementations of ILogger<T> that match on T
                     .Where(t => typeof(ILogger<T>).IsAssignableFrom(t))
                     // throw an exception if more than one handler found,
                     // could be revised to be more friendly, or make a choice
                     // amongst multiple available options...
                     .Single(); 

        /* if you don't have LINQ, and need C# 2.0 compatibility, you can use this:
        Type loggerType;
        Type[] allTypes = Assembly.GetExecutingAssembly().GetTypes();
        foreach( var type in allTypes )
        {
            if( typeof(ILogger<T>).IsAssignableFrom(type) && loggerType == null )
                loggerType = type;
            else
                throw new ApplicationException( "Multiple types handle ILogger<" + typeof(T).Name + ">" );                   
        }

        */

        MethodInfo ctor = loggerType.GetConstructor( Type.EmptyTypes );
        if (ctor != null)
            return ctor.Invoke( null ) as ILogger<T>;

        // couldn't find an implementation
        throw new ArgumentException(
          "No mplementation of ILogger<{0}>" + typeof( T ) );
    }
}

// some very basic tests to validate the approach...
public static class TypeDispatch
{
    public static void Main( string[] args )
    {
        var intLogger      = LoggerFactory.Create<int>();
        var stringLogger   = LoggerFactory.Create<string>();
        var dateTimeLogger = LoggerFactory.Create<DateTime>();
        // no logger for this type; throws exception...
        var notFoundLogger = LoggerFactory.Create<double>(); 
    }
}

Answer 4

取决于您打算处理的类型数量。如果它很小（小于10）我会建议一个switch语句，因为它会更快更清晰。如果你想要更多，你会想要一个查找表（哈希映射，字典等），或一些基于反射的系统。

Answer 5

switch语句与字典 - 对于性能无关紧要，因为交换机被编译成字典。所以真的是可读性和灵活性。这个开关更容易阅读，另一方面，字典可以在运行时扩展。

Answer 6

您可以考虑使用依赖注入框架，例如Unity。您可以使用您的因子将返回的泛型类型对其进行配置，并在配置中执行映射。 Here's an example of that

Answer 7

1）我总是对人们记录日志的复杂性感到惊讶。对我来说似乎总是有点过分。如果log4net是opensource，我建议你去看看，事实上，你也可以使用它......

2）就个人而言，我试图尽可能避免类型检查 - 它违背了泛型。只需使用.ToString（）方法即可完成。

Answer 8

Hrm ......你实际上可以尝试更加聪明一点，具体取决于给定的运行时系统支持的内容。我实际上试图避免代码中的任何条件语句，特别是在多态和动态绑定代码中。那里有一个泛型类，为什么不使用呢？

例如，在Java中，您可以特别使用您在那里执行的静态方法：

public class LoggerFactory<T>
{
    public static ILogger<T> CreateLogger(Class<? extends SomeUsefulClass> aClass);
    {
        // where getLogger() is a class method SomeUsefulClass and its subclasses
        // and has a return value of Logger<aClass>.
        return aClass.getLogger();

        // Or perhaps you meant something like the below, which is also valid.
        // it passes the generic type to the specific class' getLogger() method
        // for correct instantiation. However, be careful; you don't want to get
        // in the habit of using generics as variables. There's a reason they're
        // two different things.

        // return aClass.getLogger(T);
    }
}

你会这样称呼它：

public static void main(String[] args)
{
    Logger = LoggerFactory.createLogger(subclassOfUsefulClass.class);
    // And off you go!
}

这避免了必须具有任何条件并且更加灵活：除了SomeUsefulClass的子类（或者实现记录器接口）的任何类都可以返回正确类型的记录器实例。

工厂根据通用类型C＃创建对象

8 个答案: