静的メソッドを持つクラスを持っていますか？そのすべてが同じプロパティを持ち、悪いとみなされますか？

Question

例えば：

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) {...} 
    } 

問題である - それらの全ては、私は、コード内のすべてのメソッド呼び出しに渡すDatabaseConnectionの同じインスタンスを動作します。

DatabaseConnectionをクラスAの静的変数にするのは賢明ですが、正しく初期化するにはどうすればいいですか？それとも別の良い解決策がありますか？

Answer 1

[OK]をいくつか考えた後、私は最高のソリューションは@ Xaqronのような外部のソースから同じ価値を得ることになると決めた - シングルトンインスタンス。それはスコープにシングルトンを持つ新しい参照を与える必要がありますが、それはまだ最高のものです。 静的を使用した例

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

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

Answer 2

はとにかく良いOOで異常です。だから、「静的を使って単独で」は、（あなたが本当に何を/なぜ使っているのかを本当に理解していない限り）しばしば悪いです。

し、すべてのメソッドがDBConnection connパラメータは、あなたの代わりにあなたがそれを保証するために、依存性注入を使用し、そのクラスの普通フィールド（connパラメータことを変更することを、かなり明確に示すことを取るという事実すべてのメソッドを合理的にユニットテストすることができます）。