多线程和Java Swing问题

Question

嗨，我有一个工作正常的GUI应用程序。 我创建了一个套接字服务器 当我在程序中创建Server类的新对象时，GUI应用程序停止响应。

这是我的服务器类。 如果我做

Server s = new Server();

在我的主应用程序中它停止工作。 我应该如何添加它？ 制作一个新帖子？ 我试过了

Thread t = new Thread(new Server());
t.start();

但问题仍然存在。 拜托，我将非常感谢你的帮助。

package proj4;

import java.net.*; 
import java.io.*; 

public class Server implements Runnable { 
    ServerSocket       serverSocket = null;
    Socket             clientSocket = null;
    ObjectOutputStream out          = null;
    ObjectInputStream  in           = null;
    int                port;
    static int         defaultPort  = 30000;
    boolean            isConnected  = false;
    Thread             thread;
    DataPacket         packet       = null;

    public Server(int _port) {
        try {
            serverSocket = new ServerSocket(_port);
            serverSocket.setSoTimeout(1000*120);  //2 minutes time out     
            isConnected = true;
            System.out.println("server started successfully");
            thread = new Thread(this);
            thread.setDaemon(true);
            //thread.run();
        } catch (IOException e) {
            System.err.print("Could not listen on port: " + port);
            System.exit(1);
        }
        try {
            System.out.println("Waiting for Client");
            clientSocket = serverSocket.accept();
            System.out.println("Client Connected");
            thread.run();
        } catch (IOException e) {
            System.err.println("Accept failed.");
            System.exit(1);
        }
        try {
            out = new ObjectOutputStream(clientSocket.getOutputStream());
            System.out.println("output stream created successfully");
        } catch (IOException e) {
            e.printStackTrace();
        }
        try {
            in = new ObjectInputStream(clientSocket.getInputStream());
            System.out.println("input stream created successfully");
        } catch (IOException e) {
            e.printStackTrace();
        }
    }

    public Server() {
        this(defaultPort); //server listens to port 30000 as default
    }

    public void run() {
        System.out.println("Thread running, listening for clients");//debugging purposes
        while (isConnected) {
            try {
                packet = this.getData();
                Thread.sleep(0);
            } catch(InterruptedException e) {
                e.printStackTrace();
            }
        }
    } 

    public DataPacket getData() {
        try {
            packet = (DataPacket)in.readObject();
        } catch (Exception ex)  {
            System.out.println(ex.getMessage());
        }
        return packet;
    }

    public void sendData(DataPacket dp) {
        try {
            out.writeObject(dp);
        } catch (IOException e) {
            e.printStackTrace();
        } 
        try {
            out.flush();
        } catch (IOException e) {
            e.printStackTrace();
        }
    }

    public void closeConnection() throws IOException {
        out.close(); 
        in.close(); 
        clientSocket.close(); 
        serverSocket.close(); 
    }
} 

Answer 1

您的Server构造函数可能无限期地阻塞在accept() 。

关于Swing程序的两件事：

1. 永远不要在Swing事件线程中执行任何长任务，并且，
2. 除非将正在使用的方法明确记录为线程安全，否则永远不要操作Swing事件线程之外的任何Swing对象。

这意味着如果服务器是从Swing事件线程启动的 - 也就是说，如果它是为响应按钮单击等而启动的 - 那么是的，您必须为您的Server对象生成另一个线程。 否则，您保证在您的线程退出之前阻止Swing事件线程。

您说即使您为服务器生成另一个线程，您的应用程序仍然停止响应？ 确保您正在调用Thread.start()而不是run() ，否则您将意外地通过在您自己的Thread中运行“new Thread”来阻止自己。

笔记：

1. 我看到你做了一个Thread.sleep(0); 在你的run（）循环中。 这不能保证做任何事情。 如果你有一台CPU机器，这可以公平地实现为无操作，允许同一个线程继续运行。
2. 你真的希望isConnected是volatile - 否则不能保证除了改变它之外的任何线程都会看到对这个变量的更改。
3. 您没有在任何地方将isConnected为false，因此您的run()将一直运行，直到JVM停止或直到该Thread接受RuntimeException。
4. 不鼓励在构造函数中启动Threads。 （参见Java Concurrency In Practice 。）
5. 在进入Thread的run()方法之前，您不希望在ServerSocket上accept它！ 否则，您的构造函数将阻止等待连接，并且不会将控制权返回给事件线程！
6. 您的构造函数中包含以下代码：

你的代码是：

thread = new Thread(this);
thread.setDaemon(true);
//thread.run();

当你没有注释掉thread.run()时，你没有开始新的线程！ 要做到这一点，你需要做thread.start() 。 相反，你在调用构造函数的同一个线程中运行这个新的Thread（它永远不会停止，因为上面的原因＃3）。 现在编写代码的方式，所有IOExceptions都会被记录，但是否则会被吞噬。 您可能希望在任何IOException以及closeConnection()中将isConnected为false 。

Answer 2

问题是您的Server构造函数是阻塞的。 cosntructor不应该进行任何阻塞调用（实际上它应该尽可能少）。 阻塞调用应该由run（）或run（）调用。

另请注意，您创建一个新的Thread（）是一个似乎没有任何意义的构造函数。

Answer 3

您的服务器类看起来似乎合理（我实际上没有编译它，但它看起来似乎合情合理。）如果您的GUI变得无法响应，这几乎意味着GUI线程无法获得控制权。 要了解原因，我们真的需要查看创建它的代码 - 它在哪里以及它是如何工作的。

你的基本想法是正确的，尽管你可以缩短它

(new Thread(new Server()).start();

但这肯定不是问题。

这是一个想法：在您创建和启动服务器时，在调用后直接添加打印或日志记录语句，即，

(new Thread(new Server()).start();
System.err.println("Got here!");

看看你有没有看到这里！ 信息。 如果没有，那么你就是阻止了GUI线程。

Answer 4

您可以使用Java 6中提供的SwingWorker。如果您需要在以前版本的Java中使用它，您可以始终使用：

https://swingworker.dev.java.net/

SwingWorker可能是更清洁的方法。 你可以在这里找到更多关于它的信息：

http://java.sun.com/products/jfc/tsc/articles/threads/threads2.html

然后，您可以编写新的ServerSwingWorker内部类来包装该功能或使用匿名内部类。

Answer 5

做的小细节

    new Thread(new Server())

因为“new Server（）”仍在实际的Thread中执行，所以无济于事。

它应该是这样的：

    Thread t = new Thread(new Runnable() {
        @Override
        public void run() {
            new Server();
        }
    });
    t.start();