Implementazione e utilizzo del wrapper logger per log4net

Question

Questa domanda è correlata alla risposta di Steven - here. Ha proposto un ottimo wrapper per il logger. Io incollare il suo codice qui sotto:

public interface ILogger 
{ 
    void Log(LogEntry entry); 
} 

public static class LoggerExtensions 
{ 
    public static void Log(this ILogger logger, string message) 
    { 
     logger.Log(new LogEntry(LoggingEventType.Information, 
      message, null)); 
    } 

    public static void Log(this ILogger logger, Exception exception) 
    { 
     logger.Log(new LogEntry(LoggingEventType.Error, 
      exception.Message, exception)); 
    } 

    // More methods here. 
} 

Quindi, la mia domanda è qual è il modo corretto per creare implementazione che proxy per log4net? Dovrei semplicemente aggiungere un altro metodo di estensione Log con il parametro type e quindi creare un interruttore all'interno? Utilizzare un metodo log4net diverso in caso di LoggingEventType?

E seconda domanda, qual è il modo migliore per utilizzarlo più avanti nel codice?

perché ha scritto:

(...) è possibile creare facilmente un'implementazione ILogger (...) e configurare vostro contenitore DI iniettare in classi che hanno un ILogger nella loro costruttore .

Significa che ogni classe che registrerà sth (quindi praticamente ogni), dovrebbe avere ILogger nel suo costruttore?

Answer 1

Quindi, la mia domanda è qual è il modo corretto per creare l'implementazione che proxy a log4net?

si dovrebbe creare qualcosa di simile:

public class Log4netAdapter : ILogger 
{ 
    private readonly log4net.ILog m_Adaptee; 

    public Log4netAdapter(log4net.ILog adaptee) 
    { 
     m_Adaptee = adaptee; 
    } 

    public void Log(LogEntry entry) 
    { 
     //Here invoke m_Adaptee 
     if(entry.Severity == LoggingEventType.Debug) 
      m_Adaptee.Debug(entry.Message, entry.Exception); 
     else if(entry.Severity == LoggingEventType.Information) 
      m_Adaptee.Info(entry.Message, entry.Exception); 
     else if(entry.Severity == LoggingEventType.Warning) 
      m_Adaptee.Warn(entry.Message, entry.Exception); 
     else if(entry.Severity == LoggingEventType.Error) 
      m_Adaptee.Error(entry.Message, entry.Exception); 
     else 
      m_Adaptee.Fatal(entry.Message, entry.Exception); 
    } 
} 

Questo vuol dire che ogni classe che registrerà esimo (in modo praticamente ogni), dovrebbe avere ILogger nel suo costruttore?

Come ho capito dalla risposta di Stevens: Sì, dovresti farlo.

qual è il modo migliore per utilizzarlo più avanti nel codice?

Se si utilizza un contenitore DI, poi basta usare il contenitore per mappare DI ILogger-Log4netAdapter. È inoltre necessario registrare log4net.ILog o semplicemente fornire un'istanza del logger log4net al contenitore DI per iniettarlo nel costruttore Log4netAdapter.

Se non si utilizza un contenitore DI, vale a dire, si utilizza Pure DI, poi si fa qualcosa di simile:

ILog log = log4net.LogManager.GetLogger("MyClass"); 

ILogger logging_adapter = new Log4netAdapter(log); 

var myobject = new MyClass(other_dependencies_here, logging_adapter);