コンストラクタでのJavaの安全なパブリケーション、 "this"リファレンスが漏れないようにする

Question

以下のコードが完全にスレッドセーフであり、 "this"リファレンスがリークしないかどうかについて、私は感謝していますか？私がやろうとしているのは基本的にExecutorServiceを使ってバックグラウンドスレッドで別のサービスをブートストラップ/初期化することです。

私は、クラスが完全に構築される前に "this"参照をリークするので、コンストラクタからスレッドを開始することは悪い習慣であるとどこかから読んだので、少し気になります。

public class MyService { 

private final ExecutorService executorService; 
private volatile AnotherService anotherService; 
private volatile boolean isReady = false; 


public MyService(final ExecutorService executorService) { 
    this.executorService = executorService; 
    start(); 
} 

private void start() { 
    executorService.submit(new Runnable() { 
     @Override 
     public void run() { 
      try { 
       anotherService = init(); 
       isReady = true; 
      } catch (Exception e) { 
       // do nothing, just retry later 
      } 
     } 
    }); 
} 

private AnotherService init() { 
    // some code to initialize 
    return AnotherServiceBootstrap.getInstance().bootstrap(); 
} 

// some other methods in class 

} 

事前に感謝します。

Answer 1

私はあなたのコードを持つすべてのリスクを持っていないと思う -

1. その記事で説明が与えられthis question
2. を参照してください誰かが最後のメンバだけにアクセスすることができれば、あなたが持っていた唯一のリスクがあります（executorService）が生成されます。しかし、変数がそのようにアクセスされる唯一の場所は、作成して実行系サービスに送信するスレッド内のものです。そして、submit（）メソッドが実行される前に、あなたが作成したスレッドが実行される前に、あなたのコンストラクターが今までに行ったすべての変更を示すfuture-before保証を取得します。私はあなたがカバーされていると信じています。