Sollte der Logger privat statisch sein oder nicht?

Sollte der Logger als statisch deklariert werden oder nicht? Normalerweise habe ich zwei Arten von Deklarationen für einen Logger gesehen:

    protected Log log = neuer Log4JLogger (aClass.class);

oder

    privates statisches Protokoll log = neuer Log4JLogger (aClass.class);

Welches sollte verwendet werden? Was sind die Vor- und Nachteile von beiden?

Der Vorteil des nicht statischen Formulars besteht darin, dass Sie es in einer (abstrakten) Basisklasse wie folgt deklarieren können, ohne sich Sorgen machen zu müssen, dass der richtige Klassenname verwendet wird:

protected Log log = new Log4JLogger(getClass());

Der Nachteil ist jedoch offensichtlich, dass für jede Instanz der Klasse eine ganz neue Logger-Instanz erstellt wird. Dies ist zwar an sich nicht teuer, verursacht jedoch einen erheblichen Overhead. Wenn Sie dies vermeiden möchten, möchten Sie staticstattdessen das Formular verwenden. Der Nachteil ist jedoch, dass Sie es in jeder einzelnen Klasse deklarieren und in jeder Klasse darauf achten müssen, dass der richtige Klassenname während der Erstellung des Loggers verwendet wird, da er getClass()nicht im statischen Kontext verwendet werden kann. In der durchschnittlichen IDE können Sie jedoch eine Vorlage für die automatische Vervollständigung erstellen. ZB logger+ ctrl+space.

Wenn Sie den Logger hingegen von einer Factory beziehen, die wiederum die bereits instanziierten Logger zwischenspeichern kann, führt die Verwendung des nicht statischen Formulars nicht zu einem so hohen Overhead. Log4j hat zum Beispiel eine LogManagerfür diesen Zweck.

protected Log log = LogManager.getLogger(getClass());

Früher dachte ich, dass alle Logger statisch sein sollten. aber dieser Artikel bei wiki.apache.org jedoch einige wichtige Speicherprobleme in Bezug auf Classloader-Lecks . Wenn Sie einen Logger als statisch deklarieren, wird verhindert, dass die deklarierende Klasse (und die zugehörigen Klassenladeprogramme) in J2EE-Containern, die einen gemeinsam genutzten Klassenladeprogramm verwenden, als Müll gesammelt werden. Dies führt zu PermGen-Fehlern, wenn Sie Ihre Anwendung genügend oft erneut bereitstellen.

Ich sehe keine Möglichkeit, dieses Problem mit dem Classloader-Leck zu umgehen, außer Logger als nicht statisch zu deklarieren.

Der wichtigste Unterschied besteht darin, wie sich dies auf Ihre Protokolldateien auswirkt: In welche Kategorie fallen Protokolle?

• Bei Ihrer ersten Wahl landen die Protokolle einer Unterklasse in der Kategorie der Oberklasse. Das scheint mir sehr kontraintuitiv zu sein.

• Es gibt eine Variante Ihres ersten Falls:

  protected Log log = neuer Log4JLogger (getClass ());

  In diesem Fall gibt Ihre Protokollkategorie an, an welchem ​​Objekt der protokollierte Code gearbeitet hat.

• Bei Ihrer zweiten Wahl (private statische) ist die Protokollkategorie die Klasse, die den Protokollierungscode enthält. Normalerweise also die Klasse, die das tut, was protokolliert wird.

Ich würde diese letzte Option wärmstens empfehlen. Es hat diese Vorteile im Vergleich zu den anderen Lösungen:

• Es besteht eine direkte Beziehung zwischen dem Protokoll und dem Code. Es ist leicht herauszufinden, woher eine Protokollnachricht stammt.
• Wenn jemand die Protokollierungsstufen anpassen muss (was pro Kategorie erfolgt), liegt dies normalerweise daran, dass er an bestimmten Nachrichten interessiert ist (oder nicht), die von einer bestimmten Klasse geschrieben wurden. Wenn die Kategorie nicht die Klasse ist, die die Nachrichten schreibt, ist es schwieriger, die Ebenen zu optimieren.
• Sie können sich bei statischen Methoden anmelden
• Logger müssen nur einmal pro Klasse initialisiert (oder nachgeschlagen) werden, also beim Start, anstatt für jede erstellte Instanz.

Es hat auch Nachteile:

• Es muss in jeder Klasse deklariert werden, in der Sie Nachrichten protokollieren (keine Wiederverwendung von Protokollierern der Oberklasse).
• Sie müssen darauf achten, dass Sie beim Initialisieren des Loggers den richtigen Klassennamen eingeben. (Aber gute IDEs erledigen das für Sie).

Verwenden Sie die Umkehrung der Steuerung und übergeben Sie den Logger an den Konstruktor. Wenn Sie den Logger innerhalb der Klasse erstellen, werden Sie mit Ihren Unit-Tests eine teuflische Zeit haben. Sie schreiben Unit-Tests, nicht wahr?