Question

Is it correct to create a thread and call its start() method inside a class’ constructor as done here?

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, do not do this. You’re allowing the this reference to escape during construction.