使用Ninject填充Log4Net依赖项

Question

我在我的应用程序中使用Ninject作为DI容器。为了松散地耦合到我的日志库，我使用的界面是这样的：

public interface ILogger 
    { 
     void Debug(string message); 
     void Debug(string message, Exception exception); 
     void Debug(Exception exception); 

     void Info(string message); 
     ...you get the idea 

，我的实看起来像这样

public class Log4NetLogger : ILogger 
    { 
     private ILog _log; 

     public Log4NetLogger(ILog log) 
     { 
      _log = log; 
     } 

     public void Debug(string message) 
     { 
      _log.Debug(message); 
     } 
     ... etc etc 

样本类与日志记录依赖

public partial class HomeController 
    { 
     private ILogger _logger; 

     public HomeController(ILogger logger) 
     { 
      _logger = logger; 
     } 

在实例化Log4Net的实例时，应该给它一个它将要记录的类的名称。这对Ninject来说是一个挑战。

的目标是，实例化的HomeController时，Ninject应该以一个“HomeController的”

这里的“名”实例ILog的是我的配置

public class LoggingModule : NinjectModule 
    { 
     public override void Load() 
     { 
      Bind<ILog>().ToMethod(x => LogManager.GetLogger(GetParentTypeName(x))) 
       .InSingletonScope(); 

      Bind<ILogger>().To<Log4NetLogger>() 
       .InSingletonScope(); 
     } 

     private string GetParentTypeName(IContext context) 
     { 
      return context.Request.ParentContext.Request.ParentContext.Request.Service.FullName; 
     } 
    } 

然而，“姓名”，也就是被传递给ILog不是我所期待的。我无法找出任何韵律或理由，有时它是正确的，大部分时间不是这样。我看到的名字是OTHER类的名字，它们也依赖于ILogger。

Answer 1

Ninject.Extension.Logging扩展已经提供了您正在实施的所有功能。包括对log4net，NLog和NLog2的支持。

https://github.com/ninject/ninject.extensions.logging

---

而且要使用以下内容作为记录器类型：

context.Request.ParentRequest.ParentRequest.Target.Member.DeclaringType 

否则，你将获得的服务类型，而不是实现类型的记录。

Answer 2

ILog和​​的范围需要是暂态的，否则它只会重用它创建的第一个记录器。感谢@Meryln Morgan-Graham帮助我找到。

Answer 3

Bind<ILog>().ToMethod(x => LogManager.GetLogger(GetParentTypeName(x))) 
      .InSingletonScope(); 

您当前正在辛格尔顿范围结合，因此只有一个记录器创建将使用创建的第一个名字。而是使用InTransientScope()

Answer 4

我个人没有兴趣抽象掉我的记录器，所以我的实现模块直接引用log4net.dll，我的构造函数根据需要请求ILog。

要做到这一点，利用Ninject v3的一个在线注册看起来像这样在我的static void RegisterServices(IKernel kernel)结束：

 kernel.Bind<ILog>().ToMethod(context=> 
      LogManager.GetLogger(context.Request.Target.Member.ReflectedType)); 
     kernel.Get<LogCanary>(); 
    } 

    class LogCanary 
    { 
     public LogCanary(ILog log) 
     { 
      log.Debug("Debug Logging Canary message"); 
      log.Info("Logging Canary message"); 
     } 
    } 

为了便于诊断日志记录问题的，我坚持下，在一开始就得到一个非-DI从动消息太：

public static class NinjectWebCommon 
{ 
    public static void Start() 
    { 
     LogManager.GetLogger(typeof(NinjectWebCommon)).Info("Start"); 

其中产量上的应用程序的起动时的以下操作：

<datetime> INFO MeApp.App_Start.NinjectWebCommon   - Start 
<datetime> DEBUG MeApp.App_Start.NinjectWebCommon+LogCanary - Debug Logging Canary message 
<datetime> INFO MeApp.App_Start.NinjectWebCommon+LogCanary - Logging Canary message 

Answer 5

也许我的回答是迟，但我使用这种格式：

private static void RegisterServices(IKernel kernel) 
    { 
     kernel.Bind<ILog>() 
      .ToMethod(c => LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType)) 
      .InSingletonScope(); 
    } 

Answer 6

我喜欢在我自己的接口包裹log4net的的想法。我不想依赖于Ninject的实现，因为对我来说，这意味着我在整个应用程序中依赖于Ninject，我认为这与依赖注入的目的完全相反。与第三方服务脱钩。所以我拿了原来的海报代码，但我改变了下面的代码来使它工作。

private string GetParentTypeName(IContext context) 
    { 
     var res = context.Request.ParentRequest.ParentRequest.Service.FullName; 
     return res.ToString(); 
    } 

我有打电话给ParentRequest.ParentRequest这样，当我打印布局％记录器将打印调用log4net的日志方法而不是log4net的类称为日志方法，该方法的类。

Answer 7

对于所有的你，还在寻找正确的答案，正确的实施是：

public class LoggingModule : NinjectModule 
{ 
    public override void Load() 
    { 
     Bind<ILog>().ToMethod(x => LogManager.GetLogger(x.Request.Target.Member.DeclaringType)); 

     Bind<ILogger>().To<Log4NetLogger>() 
      .InSingletonScope(); 
    } 
} 

重视：

x.Request.Target.Member.DeclaringType