Javaの同時実行 - 監視スレッド

Question

とシングルトンデザイン私は、コンストラクタが終了する前に、私は、監視スレッドにシングルトンインスタンスをリークしていますので、

public final class HandlerCache { 
    //the cache maintains a handler per thread 
    private final Map<Thread, Handler> cache = new ConcurrentHashMap<>(); 

    private final Thread monitor; 

    private static final HandlerCache INSTANCE = new HandlerCache(); 

    private HandlerCache() { 
     monitor = new Thread() { 
     //periodically monitor cache and close handlers when a thread has died 
     } 
     monitor.start() 
    } 

    public static HandlerCache getInstance() { 
     return INSTANCE; 
    } 

    public Handler getHandler() throws Exception { 
     final Thread thread = Thread.currentThread(); 
     Handler handler = cache.get(thread); 

     if (!(handler == null)) 
      return handler; 

     handler = HandlerFactory.get(getHandlerFromName(thread.getName())); 
     cache.put(thread, handler); 
     return handler; 
    } 

} 

シングルトンクラスを持って、より良い方法は何ですか？

キャッシュをvolatileにすると問題が解決されますか？

Answer 1

HandlerCacheコンストラクタ内でスレッドを開始するのではなく、最初にHandlerCacheを構築してからスレッドを開始する静的関数を使用してインスタンスを初期化できます。

Answer 2

user2677485で説明したように、ThreadLocalを使用してinitialValueメソッドを実装する必要があります。もう一つのポイントは、Handlerの実装でfinalizeメソッドを実装する必要があるため、GCによって再利用されているときにこのメソッドが呼び出され、リソースをクリーンアップできることです。

コードは、次のようなもののように簡略化することができます。

public class HandlerCache { 

    private static final handlers = new ThreadLocal<Handler>() { 
     protected Handler initializeValue() { 
       return HandlerFactory.get(...); 
     } 
    }; 

    public static Handler getHandler() { 
     return handlers.get(); 
    } 

}