スレッドコード...これに何か、私はアプリケーションサーバーの起動時に自動的に発射サーブレットを持っており、その初期化中に（）は、Java 1.4

Question

を使用する必要があり、私は別のスレッドを作ってるんだ：

init(){ new FooThread().start() } 

FooThread（）内の

、私は定期的にDBの値のステータスをチェックし、値に応じてWebサービスコールを行いたいと思います。これらの2つのタスクが完了すると、スレッドをスリープして一定の期間待ってから繰り返します。 このサイクルはちょうど永遠に続くでしょう。

FooThread：それは競合状態と不定動作につながる可能性があるため、

public class FooThread implements Runnable{ 
    Thread t; 

    FooThread(){ 
    t = new Thread(this, "FooThread"); 
    } 

public void start(){ 
    t.start(); 
} 

    public void run() { 
    try{ 
     while(true){ 
     //do the db check, then conditionally do the web services call 
     logger.info("*** calling sleep() ***"); 
     Thread.sleep(50000); 
     logger.info("*** now awake ***"); 
     } 
    } catch (InterruptedException e) { 
     System.out.println("*** FooThread interrupted"); 
    } 
    } 
} 

Answer 1

あなたのコンストラクタ内のスレッドを起動しないでください。代わりに、オブジェクトを作成した後にstart()メソッドを呼び出します。

EDIT： コンストラクタが完了する前に、あなたはthisポインタを公開する可能性があるので、それは悪いです

理由があります。

Answer 2

Threadからサブクラス化する場合は、tをFooThreadに新しく作成する必要はありません。 FooThreadのインスタンスは、すでにスレッドです。コンストラクタでstart()を呼び出す必要はありません。

init()内部FooThreadの適切な使用は、次のようになります。あなたがFooThreadのコンストラクタの上にラインアウトコードで

FooThread ft = new FooThread(); 
ft.start();       // calls run() inside FooThread 

はrun()は正常に見える、任意のロジックを必要としません。

Answer 3

あなたがやっていることは安全でないとみなされます。この参照がコンストラクタをエスケープすると、部分的に構築されたオブジェクトをスレッドクラスに送ることができます。あなたのクラスとrunメソッドは、このように見えた場合はどのようなあなたの例では

：

public class FooThread implements Runnable{ 
    Thread t; 

    private int someInt; 
    private Object someObject; 

    FooThread(){ 
    t = new Thread(this, "BBSThread"); 
    t.start(); 
    someInt = 10; 
    someObject = new Object(); 
    } 

    public void run() { 
    System.out.println(this.someInt); 
    System.out.println(this.someObject);  
    } 
} 

それはsomeIntは0を出力し、SomeObjectのがnull印刷していることが可能です。あなたのオブジェクトは技術的に構築されていますが、完成していません。