シングルトンデザイン＆新しいオブジェクト内部のそれここに具体的なシナリオでは

Question

、

私はマネージャーがUserMangerは、設計によって、ユーザー

このマネージャーは、「シングルトン」で、正しいのためのCRUDを処理するために使用されると呼ばれています。

しかし、更新方法では、私はロジック

public User update (User u) 
{ 

    // This line is problematic? 
    User u1 = new User(); 

    copy(u,u1); 
    //Some logic 
    dao.update(u); 

} 

が問題シングルトンマネージャ内の任意の新しいオブジェクトを作成するのでしょうか？特に、非常に並行したシステムの場合。

は

public User update (User u, User u1) 
{ 

    copy(u,u1); 
    //Some logic 
    dao.update(u); 
} 

は私の問題を解決するのでしょうか？

Answer 1

シングルトンメソッド内にオブジェクトを作成しても問題は発生しません。結局のところ、スレッド間で状態を共有しているわけではありません。

あなたはシングルトン自身の中の状態を持っていた場合は、潜在的な問題を持っているでしょう - しかし、あなたは示されてきたすべてがインスタンス変数を変更しない、ローカル変数を作成しています。メソッドを呼び出すたびに、独自の完全に独立したローカル変数のセットがあります。 2つのスレッドはどちらも同じメソッドを実行できますが、お互いのローカル変数は表示されません。

Answer 2

号しかし、あなたはグローバル変数として

User u1; 

を置けば、それは問題が発生します。

しかし、私は質問があります。なぜあなたはこのようにコードしていませんか？

public User update (User u) 
{ 

    //Some logic 
    User u = dao.update(u); 
    return u; 
    // OR return dao.update(u); 
} 

ここにコピーする必要はありません。このメソッドでは、コピーに2倍のUserオブジェクトが割り当てられる必要があります。

Answer 3

システムを理解していないと、ここに記載されているデザインパターンについて質問する必要があります。 SingletonとSomethingManagerはどちらもアンチパターンです。

、本質的に何をしている（あなただけのUserManagerのために、より説明的な名前を検討する必要があるかもしれませんが、それはむしろ必要な場合、私は非常に「グローバル」としてそれを下に釘よりも、それを注入するSpringようIoCフレームワークを使用することをお勧めしますシングルトンは）。