Metody statyczne w instancji klasy - dobre, złe lub zależne?

Question

Mam starą klasę strażniczą - to stałe lub statyczne metody, typowe dla klasy użytkowej.

Jednak ostatnio zacząłem używać NLog - więc moi strażnicy mogą teraz logować się i rzucać. Chodzi o to, ze nlog że każda klasa wywołujący (gdzie przebywa strażnik) tworzy własny rejestrator, więc zamiast metody takie jak to:

public static void NotNull<T>(T obj, string param) 
{ 
    if (obj.Equals(null)) 
     throw new ArgumentNullException(param); 
} 

Mam metodę z podpisem jak ten:

public static void NotNull<T>(T obj, string param, Logger logger, LogLevel logLevel) 
{ 
} 

Teraz wszystkie moje metody zawierają dwa takie same parametry związane z rejestratorem, więc prawie zdecydowałem, że wstrzyknięcie zależności byłoby lepszym rozwiązaniem, przekazując rejestrator do konstruktora i przechodząc do metody.

Pytanie, które mam, opiera się na moim niedoświadczeniu - moja nowa klasa nie będzie statyczna, ale czy powinienem pozostawić metody w środku jako statyczne?

Answer 1

Wygląda na to, że nie trzeba w ogóle przechodzić w rejestratorze. To dobrze, a nie wbrew powszechnej praktyce, aby mieć statyczne pole rejestratora (spójrz na this answer), aby było ono współużytkowane we wszystkich instancjach klasy. Rozważ:

public static class Utils 
{ 
    private static readonly ILog Log = LogManager.GetLogger(typeof(Utils)); 

    public static void NotNull<T>(T obj, string param) 
    { 
     Log.Debug("Huston, we got a null."); 
     if (obj.Equals(null)) 
      throw new ArgumentNullException(param); 
    } 
}