新しいスレッドを内部で呼び出すのがコンストラクタ

Question

スレッドを作成し、start（）メソッドをクラスのコンストラクタ内でここで行うように呼び出すのは正しいですか？

public class Server implements Runnable { 

    private ServerSocket server; 

    public Server(int port) { 
     try { 
      //Opens a new server 
      server = new ServerSocket(port); 
     } catch (IOException ioe) { 
      ioe.printStackTrace(); 
     } 

     new Thread(this, "Server").start(); 
    } 

    @Override 
    public void run() { 
    } 
} 

Answer 1

IMHO、これを行わないでください。建設中にthisの参照がエスケープできるようにしています。

Answer 2

Server s = new Server(); 
Thread t = new Thread(s, "Server").start(); 

はもっとテスト可能です。これにより、サーバーのインスタンスを作成し、スレッドを生成することなくそのメソッドをテストすることができます。確かに

Answer 3

は、あなたのコードは、それをやってイマイチが、あなたのコードは次のように見えたものとします。

public Server(int port) 
{  
    new Thread(this, "Server").start(); 

    try 
    { 
     //Opens a new server 
     server = new ServerSocket(port); 
    } 
    catch (IOException ioe){ ioe.printStackTrace(); } 

} 
@Override  
public void run(){ 
    if(server == null)throw new NullPointerException();// this may happen 
} 
} 

サーバーの参照も例外が発生していないにもかかわらず、nullの場合もあります。これは、スレッドが作成された実行可能ファイルを使用し、クラスのコンストラクタが終了していなくてもrunメソッドを呼び出すためです。

Answer 4

カップルコンストラクタからThread.start（）を分割するために、より良いな理由：

1. あなたは今まで、このようなjava.utilのようスレッドを実行するためにいくつかの他のフレームワーク/システムを使用したい場合。あなたはそうするかもしれない。

2. スレッドを中断したい場合は、そのスレッドへの参照が必要です。別のコード行でスレッドを作成すると、これはやや日常的な/慣用的なものになります。例えば

   スレッドrememberMe =新しいスレッド（サーバー）.start（）;

元のコードでは、ServerにはmyThreadを記憶するフィールドがありますが、そうではありませんでした。

Answer 5

public class Server implements Runnable 
{ 
private ServerSocket server; 

/** 
* Because the constructor is private, the only way to instantiate a Server is through 
* the static factory method. 
* If there are any instantiation problems, the static factory method will fail in 
* first line, before it is put into a thread. 
* It will be put into a thread before being released. 
**/  
public static Server startServer (int port) 
{ 
    Server server = new Server (port) ; 
    new Thread (server , "Server") . start () ; 
    return server ; 
} 

private Server(int port) 
{  
    try 
    { 
     //Opens a new server 
     server = new ServerSocket(port); 
    } 
    catch (IOException ioe){ ioe.printStackTrace(); } 

// don't release me into the wild yet! 
// new Thread(this, "Server").start(); 
} 
@Override  
public void run(){ 
} 
}