Czy to dobra praktyka mieć loggera jako Singletona?

Umieszczam instancję loggera w moim kontenerze dependency injection, który następnie wstrzykuje logger do klas, które go potrzebują.

Dobre pytanie. Wierzę, że w większości projektów logger jest singletonem.

• Użyj ServiceLocator (lub innego Dependency Injection kontenera, jeśli już używasz), który pozwala na współdzielenie loggera między usługami/klasami, w ten sposób możesz utworzyć instancję loggera lub nawet wiele różnych loggerów i udostępnić przez ServiceLocator, który oczywiście byłby singletonem, jakimś rodzajem inwersji loggera. Kontrola . Takie podejście zapewnia dużą elastyczność w procesie tworzenia i inicjalizacji loggera.
• Jeśli potrzebujesz loggera prawie wszędzie-zaimplementuj metody rozszerzeń dla typu Object, aby każda klasa mogła wywoływać metody loggera jak LogInfo(), LogDebug(), LogError()

Singleton to dobry pomysł. Jeszcze lepszym pomysłem jest użycie wzorca rejestru , który daje nieco większą kontrolę nad instancją. Moim zdaniem wzór Singletona jest zbyt zbliżony do zmiennych globalnych. W przypadku rejestru obsługującego tworzenie lub ponowne użycie obiektów jest miejsce na przyszłe zmiany reguł tworzenia instancji.

Sam rejestr może być klasą statyczną, która daje prostą składnię dostępu do dziennika:

Registry.Logger.Info("blabla");

Zwykły singleton nie jest dobrym pomysłem. To utrudnia wymianę rejestratora. Zazwyczaj używam filtrów dla moich loggerów(niektóre "hałaśliwe" klasy mogą rejestrować tylko ostrzeżenia / błędy).

Używam wzoru Singletona połączonego ze wzorem proxy dla logger factory:

public class LogFactory
{
    private static LogFactory _instance;

    public static void Assign(LogFactory instance)
    {
        _instance = instance;
    }

    public static LogFactory Instance
    {
        get { _instance ?? (_instance = new LogFactory()); }
    }

    public virtual ILogger GetLogger<T>()
    {
        return new SystemDebugLogger();
    }
}

Pozwala mi to stworzyć FilteringLogFactory lub po prostu SimpleFileLogFactory bez zmiany kodu (a więc spełniania Zasady Open/Closed).

Przykładowe rozszerzenie

public class FilteredLogFactory : LogFactory
{
    public override ILogger GetLogger<T>()
    {
        if (typeof(ITextParser).IsAssignableFrom(typeof(T)))
            return new FilteredLogger(typeof(T));

        return new FileLogger(@"C:\Logs\MyApp.log");
    }
}

I do korzystania z nowego fabryka

// and to use the new log factory (somewhere early in the application):
LogFactory.Assign(new FilteredLogFactory());

W klasie, która powinna się logować:

public class MyUserService : IUserService
{
    ILogger _logger = LogFactory.Instance.GetLogger<MyUserService>();

    public void SomeMethod()
    {
        _logger.Debug("Welcome world!");
    }
}

Istnieje iniekcja zależności W .NET. na podstawie tego, czego potrzebujesz, powinieneś użyć przechwytywania.

W tej książce znajduje się diagram pomagający zdecydować, czy użyć Injection konstruktora, property injection, method injection, Ambient Context, Interception.

Oto jak jeden z powodów używania tego diagramu:

Masz zależność czy jej potrzebujesz? - Need it Czy to przekrojowe zmartwienie? - Tak
1. potrzebujesz odpowiedzi? - Nie

Użyj Przechwytywania

Jeśli chcesz spojrzeć na dobre rozwiązanie do logowania proponuję spojrzeć na Google app engine z Pythonem, gdzie logowanie jest tak proste, jak import logging, a następnie można po prostu logging.debug("my message") lub logging.info("my message") co naprawdę trzyma to tak proste, jak powinno.

Java nie miaĹ 'a dobrego rozwiÄ ... zania do logowania ie log4j naleĹźy unikać, poniewaĹź praktycznie zmusza do uĹźycia singletonăłw ktĂłre jak tu odpowiedziaĺ' y sÄ ... "straszne" i miaĹ 'em straszne doĹ" wiadczenie z prăłbowaniem wykonania logowania wyĹ "wietlajÄ ... cego te same wyĹ" wietlajÄ ... ce kiedyś, gdy podejrzewam, że powodem podwójnego logowania było to, że mam jeden Singleton obiektu logowania w dwóch classloaderach na tej samej maszynie wirtualnej (!)

Przepraszam, że nie jestem tak specyficzny dla C#, ale z tego co widziałem rozwiązania z C# wyglądają podobnie do Javy gdzie mieliśmy log4j i też powinniśmy zrobić z niej singleton.

Dlatego bardzo spodobało mi się rozwiązanie z Gae / python, jest tak proste, jak to tylko możliwe i nie musisz się martwić o classloadery, uzyskanie podwójnej deklaracji logowania lub dowolnego wzorca projektowego w ogóle o to chodzi.

Mam nadzieję, że niektóre z tych informacji mogą być istotne dla Ciebie i mam nadzieję, że chcesz spojrzeć na i logowania rozwiązanie polecam zamiast tego dręczyć się na ile problem Singleton się podejrzewać ze względu na niemożność posiadania prawdziwego singleton, gdy musi być instanciating w kilku classloaders.

Innym rozwiązaniem, które osobiście uważam za najłatwiejsze, jest użycie klasy Static Logger. Można ją wywołać z dowolnej metody klasy bez konieczności zmiany klasy, np. add property injection itd. Jest dość prosty i łatwy w użyciu.

Logger::initialize ("filename.log", Logger::LEVEL_ERROR); // only need to be called once in your application

Logger::log ("my error message", Logger::LEVEL_ERROR); // to be used in every method where needed