Hat Klasse mit statischen Methoden, die alle dieselbe Eigenschaft haben, die als schlecht angesehen wird?

Question

Zum Beispiel:

class A 
    { 
    ... 
    //class variables and methods 

    public static A GetAWithKey (DatabaseConnection conn, string key) {...} 
    public static A[] GetSomeAWithKeys (DatabaseConnection conn, string keys[]) {...} 
    public static bool IsAWithThisPropertyExists (DatabaseConnection conn, string thisProperty) {...} 
    } 

Problem ist - ruft in Code alle von ihnen gleiche Instanz Database arbeiten, die ich an alle Methoden übergeben.

Es scheint schlau DatabaseConnection als statische Variable der Klasse A zu machen, aber dann, wie initialisiere ich es richtig? Oder gibt es eine andere gute Lösung?

Answer 1

Ok, nachdem ich darüber nachgedacht habe, entschied ich, dass die beste Lösung denselben Wert von externer Quelle bekommen wird, wie @Xaqron erwähnt - Singleton Instanz. Das erfordert eine neue Referenz mit Singleton zu A Scope, aber immer noch scheint es, es ist die beste. Beispiel

class A 
{ 
... 
//class variables and methods 

public static A GetAWithKey (string key) 
{ 
    DatabaseConnection conn = DataBase.GetConnection(); 
    ... 
} 
} 

Answer 2

Mit statischen ist eine Anomalie in gut OO sowieso. Also "allein mit statischen" ist oft schlecht (es sei denn, Sie verstehen wirklich, was/warum Sie es verwenden).

Und die Tatsache, dass alle Ihre Methoden nehmen, dass DBConnection conn Parameter ziemlich deutlich zeigt an, dass Sie stattdessen ändern möchten, dass connParameter ein gewöhnlicher Feld dieser Klasse (wo man Dependency Injection verwenden würde, um sicherzustellen, Sie können alle Ihre Methoden vernünftig testen.