イベントメソッドのスレッドオブジェクト：クラスメンバーを使用するか、ローカルオブジェクトを作成しますか？

Question

私はしばしば呼び出されるイベントメソッドを持つクラスを持っています。メソッド名がonClickedであるとします。メソッドの中で、私はいくつかのものを行うスレッドを作成します。

まず、イベントメソッド内にスレッドオブジェクトを作成してみました。

public class MyService { 

    //Event method 
    public void onClick() { 
     new Thread(new Runnable() { 

      @Override 
      public void run() { 
       // do some stuff here 
      } 
     }).start(); 
    } 
} 

しかし、onClickのは、多くの場合、イベントメソッドが呼び出されたときに毎回作成された新しいスレッドオブジェクトがメモリ空間やパフォーマンスの問題を作成した場合、私は心配している、と呼ばれるであろうから：？コードは次のようですか

代わりに、私はこのようなクラスのメンバスレッドソリューションを作成することができます

public class MyService { 
    private Thread myThread; 

    //Event method 
    public void onClick() { 
     myThread = new Thread(new Runnable() { 

      @Override 
      public void run() { 
        // do some stuff here 
      } 
     }); 
     myThread.start(); 
    } 
} 

をので、どのコードが良いですか？ 1位か2位？それとももっと良い解決策がありますか？

Answer 1

OnClick()のスレッドを再初期化せずに2番目のコードを使用することはできません。これは、スレッドが実行を終了すると、「死んでいる」ためです。したがって、start()へのその後の呼び出しは失敗し、IllegalThreadStateExceptionになります。

完了後にGCがスレッドを削除するので、最初のコードを使用する方がよいでしょう。

Answer 2

毎回onclickという新しいスレッドを作成しているので、両方のソリューションが同じだと思います。 2番目のスレッドは、新しいスレッドが作成されてグローバル変数myThreadに割り当てられ、startが呼び出される前に、別のonclickイベントが入ってきて、前のグローバル変数を再割り当てすることがありません。 私はリソースが不足のを避けるために、ここで許可されるスレッドの数を制御するために

ThreadPoolExecutor

を見て、あなたを提案します。

Answer 3

ExecutorServiceを作成して初期化してから、runnableインターフェイスを実装する独自のクラスのインスタンスを作成できます。それは、プールの数（サイズ）を指定することによって、スレッドが多すぎるのを避けることです（ExecutorService：スレッドプールの修正）。それは良い方法です。

私が言ったように、小さなリファクタリングを行い、独自のクラスイベントを作成してRunnableを実装することができます。ポイントはコンストラクタを定義することです。