調用內部的新線程是構造函數

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

恕我直言，不要這樣做。在建設過程中，您允許參考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 
} 
} 

服務器基準可能是，即使沒有異常發生空。這是因爲線程將使用創建的runnable並調用run方法，即使您的類的構造函數尚未完成。

Answer 4

一對夫婦更充足的理由從構造分裂Thread.start（）：

1. 如果你想使用一些其他的框架/系統運行的線程，比如java.util中。 concurrent.Executor，你可以這樣做。

2. 如果您想要中斷該線程，則需要對其進行引用。在單獨的代碼行中創建線程使得這更具有慣例/習慣用法。例如

   線程rememberMe = new Thread（server）.start（）;

在你的原代碼，服務器可能有一個字段記住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(){ 
} 
}