Log4Net und Speicherverbrauch

Question

Ich sehe, dass LogNet typischerweise mit einer statischen Variablen instanziiert wird:

static logger = LogManager.GetLogger(frame.GetMethod().DeclaringType); 

Dumme Frage, aber:

1) Ist das nicht ein bisschen ärgerlich --- dh. Ich muss dieses Chaos eines Anrufs in jedem Logging-Stück meines Codes haben.

2) Generiert dies nicht viel Speicherplatz Overhead --- dh. Jeder einzelne meiner Klassen wird einer separaten Protokollierungsklasse zugeordnet. 300 Klassen, die die Protokollierung verwenden, erzwingen die Verwendung von 300 statischen Protokollvariablen? Erscheint merkwürdig.

Answer 1

Sie brauchen/müssen keine statische Variable verwenden. Typischerweise wird meine Klassen etwas aussehen

public class Foo 
{ 
    private ILog Logger; 

    public Foo() 
    { 
     Logger = LogManager.GetLogger(GetType()); 
    } 

    //rest of class here 

} 

Dieser ziemlich sauber ist und bis ich ein Speicherproblem haben, gibt es keinen Punkt in der frühen Optimierung der Speichernutzung der Anwendung. Allerdings führen nicht alle meine Klassen Protokollierung durch, also haben nur diejenigen, die dies tun/müssen, dieses Bit in ihnen. Wenn jeder Ihrer Klassen protokolliert wird, dann klingt es so, als würden Sie zu viel Logging machen, entweder das oder Sie haben es mit einer ziemlich großen Anwendung zu tun (basierend auf der Tatsache, dass Sie 300 Klassen in das Log schreiben). Wenn dies der Fall ist, dann sind 300 Instanzen einer Protokollierungsklasse wahrscheinlich kein signifikanter Teil des Speicherbedarfs.

Sie können den Logger stattdessen als benannte Instanz ILog Logger = LogManager.GetLogger("MyNamedLogger"); einrichten und diese "benannte" ILog-Instanz über mehrere Klassen hinweg verwenden. Beachten Sie, dass Sie aufgrund der Art und Weise, wie die Protokollmeldungen im Allgemeinen ausgeschrieben werden, den Nutzen der protokollierten Nachrichten verlieren. Das könnte man auf verschiedene Arten kompensieren.

Answer 2

log4net unterhält ein Repository von Loggern, Sie müssen sie nicht statisch machen. Es empfiehlt sich, einen benannten Logger pro Klasse zu verwenden. Sie müssen sich keine Gedanken über den "Overhead" machen, wenn Sie ein paar hundert Logger im Speicher haben. Wenn das ein Problem wäre, müssten Sie wahrscheinlich die Protokollierung vollständig entfernen.

Das Protokollieren ist unordentlich, d. H. Es stopft Ihren Code mit Dingen, die nicht wirklich Teil des Geschäfts sind. Aspect Orient-Programmierung könnte Ihnen helfen, there.