Ist es eine gute Praxis, die statische Methode in der Singleton-Klasse zu verwenden?

Question

Ich arbeite gerade an einer großen Codebasis, und ich mache etwas Refactoring darauf.

Die Hauptklasse I in gearbeitet ist ein Singletonklasse A, und es gibt einige Methoden, die

beispielsweise auf der statischen Methoden aus der Klasse B. abhängig sind:

public class A{ 

    public static A getInstance() {}; 

    method1() { 
     B.dosomething1(); 
    } 

    method2() { 
     dosomething2(); 
    } 
} 

method1 dann hängt eine statische Methode, während Methode2 nicht ist.

wenn ich method1 verwenden müssen, ist es gute Praxis zu schreiben:

A.getInstance().method1(); 

oder ist es besser method1 als statisch zu deklarieren, und rufen:

A.method1(); 

und für alle anderen nicht-statische Methode, Anruf A.getInstance().method2()?

Danke.

Answer 1

Ich würde statischen Kontextcode von Instanzkontextcode trennen. In Ihrem speziellen Fall halte ich beide Methoden für statisch.

public class A { 
    public static void method1(){ 
     B.doSomething(); 
    } 
    public static void method2(){ 
     doMoreThings(); 
     //Incorrect: object.doSomething(); 
    } 
} 

Answer 2

Sie sollten Singletons nur bei Bedarf verwenden. Das heißt, statische Methoden werden zum Zeitpunkt des Ladens der Klasse in den Speicher geladen und hängen außerdem von B ab. Daher sollten Sie es meiner Meinung nach vermeiden, die Methode statisch zu markieren. Sie sollten Methoden nur dann statisch machen, wenn es notwendig ist, da mit steigender Anzahl der statischen Methoden beim Laden von Klassen eine erhebliche Menge an Speicher statischen Dingen zugewiesen wird.