Wie Logger mit Google Guice injizieren

Question

Normalerweise definiere ich Logger wie folgt aus:

private static final Logger logger = LoggerFactory.getLogger(MyClass.class); 

Aber wenn @Inject verwenden, müssen wir nicht statisch und nicht-final-Feld verwenden, wie:

@Inject 
private Logger logger; 

dh Logger wird in jeder Instanz dieser Klasse erstellt, auch Logger ist veränderbar. Kann es eine Möglichkeit geben, Logger statisch zu machen? Auch, wie ich Logger an bestimmte Klasse binden kann (ich benutze das Klassenobjekt beim Erstellen von Logger-Objekt von der Fabrik LoggerFactory.getLogger(MyClass.class);, wie Logger auf die gleiche Weise mit injizierenden erstellen?)?

Answer 1

Bitte überprüfen Sie die Custom Injections auf Guice Wiki, gibt es eine komplette Log4J Beispiel.

EDIT: Sie können entweder ein static Feld oder ein final Feld für Ihren Logger verwenden, aber kein static final eins. Dies ist ein Java limitation.

auch vorsichtig sein, that:

final Felder Injizieren wird nicht empfohlen, weil die injizierte Wert nicht auf andere Threads sichtbar sein können.

haben, die nicht getestet, aber der Code in dem Artikel sollte für statische Felder gut funktionieren, obwohl Sie es durch das Loswerden von MembersInjector verbessern könnten und in den TypeListener alle es tun (da ein statisches Feld sein muss nur einmal einstellen).

Mit requestStaticInjection() werden Sie gezwungen, alle Ihre Klassen in einer Moduldatei aufzulisten - keine gute Idee, da Sie bald vergessen werden, eine hinzuzufügen.

OTOH Wenn Sie nur JUL unterstützen möchten, könnten Sie besser die built-in support verwenden (wie von Jeff erwähnt, nahm ich an, Sie wollten keine allgemeine Antwort, da Sie JUL nicht speziell in Ihrer Frage erwähnt haben).

Answer 2

Wenn Sie Ihre Anwendung für die Abhängigkeitsinjektion entwerfen, besteht die beste Vorgehensweise darin, statische Felder und Methoden so weit wie möglich zu vermeiden. Dies ist so, dass Ihre Abhängigkeiten klarer sind und es so einfacher ist, Ihre realen Abhängigkeiten während der Tests und während der Entwicklung Ihrer Anwendung durch andere Instanzen zu ersetzen. Das ideale Verhalten besteht daher darin, statische Methoden und Felder (einschließlich Logger) so weit wie möglich zu vermeiden.

Guice, jedoch erlaubt Markierung Felder statisch und requesting injection of static fields when the Injector is created. Die Felder müssen veränderbar bleiben - final bedeutet nicht "endgültig außer Guice".

public class YourClass { 
    @Inject static Logger logger; 

    /* ... */ 
} 

public class YourModule extends AbstractModule { 
    @Override public void configure() { 
    /* YourClass.logger will work once you create your Injector. */ 
    requestStaticInjection(YourClass.class); 
    } 
} 

Guice bietet automatisch java.util.logger.Logger instances für Sie mit dem Klassennamen in sie eingebettet ist, aber das ist nur wegen a special case coded into Guice. Wenn Sie einen speziellen Logger haben wollen, wie in this SO question, müssen Sie die benutzerdefinierten Injektionen untersuchen, mit denen Jakub verbunden hat - aber wenn das ganze Ziel darin besteht, die Logger-Erstellung zu zentralisieren, können Sie sie einfach umgestalten in eine statische Fabrik auch außerhalb von Guice.

@LogToFile 
public class YourClass { 
    private static final Logger logger = YourLoggerFactory.create(YourClass.class); 

    /* ... */ 
} 

public class YourLoggerFactory { 
    private YourLoggerFactory { /* do not instantiate */ } 

    public Logger create(Class<?> clazz) { 
    if (clazz.getAnnotation(LogToFile.class) != null) { 
     return someImplementation(new File(...)); 
    } else { 
     return someOtherImplementation(); 
    } 
    } 
}