内部状態の静的メンバーのみを持つクラス

Question

私はここに自分のデザインを持っていないと感じ、私は2つのテクニックの間にこだわっています。私は、データベースに接続を渡すクラスを作成しようとしています。コードは以下の通りです：私が行うとき

public final class DBUtil { 

    private static String databaseDriver = null; 
    private static String databaseConnectionString = null; 
    private static String databaseUser = null; 
    private static String databasePassword = null; 
    private static String serverName = null; 
    private static ComboPooledDataSource dataSource = null; 

    private DBUtil() { 
    throw new AssertionError(); 
    } 

    private static void getParameters() { 
    final Properties configFile = new Properties(); 
    try { 
     configFile.load(DBUtil.class.getClassLoader().getResourceAsStream("my.properties")); 
     if (configFile.containsKey("databaseConnectionString") && configFile.containsKey("databaseUser") && configFile.containsKey("databasePassword") && configFile.containsKey("databaseDriver")) { 
     DBUtil.databaseConnectionString = configFile.getProperty("databaseConnectionString"); 
     DBUtil.databaseDriver = configFile.getProperty("databaseDriver"); 
     DBUtil.databaseUser = configFile.getProperty("databaseUser"); 
     DBUtil.databasePassword = configFile.getProperty("databasePassword"); 
     } 
     else { 
     // Properties file not configured correctly for database connection 
     } 
    } 
    catch (IOException e) {} 
    } 

    public static Connection getDatabaseConnection() { 
    if (Strings.isNullOrEmpty(databaseConnectionString) || Strings.isNullOrEmpty(databaseUser) || Strings.isNullOrEmpty(databasePassword) || Strings.isNullOrEmpty(databaseDriver)) { 
     DBUtil.getParameters(); 
    } 
    dataSource = getDataSource(); 
    int retryCount = 0; 
    Connection connection = null; 
    while (connection == null) { 
     try { 
     connection = dataSource.getConnection(); 
     } 
     catch (SQLException sqle) {} 
    } 
    return connection; 
    } 

    private static ComboPooledDataSource getDataSource() { 
    if (dataSource == null) { 
     dataSource = new ComboPooledDataSource(); 
     try { 
     dataSource.setDriverClass(databaseDriver); 
     dataSource.setJdbcUrl(databaseConnectionString); 
     dataSource.setUser(databaseUser); 
     dataSource.setPassword(databasePassword); 
     } 
     catch (PropertyVetoException pve) {} 
    } 
    return dataSource; 
    } 

    public static void cleanUpDataSource() { 
    try { 
     DataSources.destroy(dataSource); 
    } 
    catch (SQLException sqle) {} 
    } 
} 

FindBugsのはIncorrect lazy initialization and update of static fieldをreturingさ：

if (dataSource == null) { 
     dataSource = new ComboPooledDataSource(); 
     ... 

何かアドバイスは大歓迎します。私はシングルトンパターンと単純な静的メソッドのセットからなるクラスの間のどこかにここでこだわっているかのように感じます。

もっと一般的に、これはDAOへのデータベース接続を引き渡す良い方法ですか？

多くのありがとうございます。

Answer 1

この方法は、2つの並列実行（同時に2つのスレッドを呼び出す）を避けるため、​​である必要があります。 2つのデータソース（T2はT1のオブジェクトの作成をオーバーライドする際に依存する）のいずれかでメソッドを呼び出す

T1 if (datasource == null) YES 
T2        if (datasource == null) YES 
T1 datasource = new Datasource... 
T2        datasource = new Datasource(...); AGAIN! 

とT1とT2：例外がこれを実行する二つのスレッドにつながる可能性が同期していない

を追加しました

。

揮発性ジャンマルクは、あなたがvolatileとしてデータソースのフィールドを宣言すべき提案@として。このキーワードは、スレッドが変数のスレッドローカルコピーを使用しないようにします（スレッドが古いキャッシュされた値を読み取ると問題を引き起こす可能性があります）。

私はこのような場合は、別のメソッド呼び出しの間で発生したかどうかわからないんだけど、かとsyncrhonized取引であれば、確かに良いでしょう:)

Answer 2

そのコードに問題があなたをmultitheadingの存在下での可能性があることです多くの場合、ComboPooledDataSourceの複数のインスタンスが作成され、dataSourceのインスタンスが異なる時刻にあることがあります。

これは、メソッドがほぼ同時に複数のスレッドによって呼び出された場合に発生する可能性があります。

Thread 1: if (dataSource == null) { // condition is true 
Thread 2: if (dataSource == null) { // condition is true 
Thread 1: dataSource = new ComboPooledDataSource(); 
Thread 2: dataSource = new ComboPooledDataSource(); 

同時実行の問題を解決する簡単な方法は、同期を追加することです：

はのはdataSourceがnullで、次のように2つのスレッドの実行がインターリーブされているとしましょう。

Answer 3

getDataSourceメソッドが同期されていないため、Findbugsが苦情を申し立てています。現在のコードでは、2つの同時スレッドがgetDataSourceを呼び出して、別のDataSourceオブジェクトに取得することが可能です。

http://findbugs.sourceforge.net/bugDescriptions.html#LI_LAZY_INIT_UPDATE_STATIC

を述べたように、あなたが問題を持つべきではありません複数のスレッドを持っていない場合：FindBugsのは不平を言っている理由

Answer 4

です。