为什么每次执行 ActionListener 时我的 GUI 窗口都会冻结？

Question

每当单击 GUI 窗口上的按钮并且服务器接收到消息并将其发送到连接到同一端口的另一个客户端客户端 2 时，我都会通过客户端 1 向服务器发送一条字符串消息“5”作为客户端 1。当单击按钮时，客户端 2 确实收到字符串消息“5”，但问题是一旦单击按钮，客户端 1 的整个 GUI 窗口就会冻结（无法单击任何事物）。客户端 2 也是如此，它执行相同的操作，即在单击按钮时发送字符串消息“2”，但在单击按钮时 GUI 窗口也会冻结。为什么 GUI 窗口冻结？

代码sn-p：

BufferedReader buffR;
Socket sock = new Socket(nameHost, portNum); 
PrintWriter printW = new PrintWriter(sock.getOutputStream()); 
this.buffR = new BufferedReader(new InputStreamReader(sock.getInputStream()));

button.addActionListener(new ActionListener(){
    public void actionPerformed(ActionEvent ae){
        printW.println("5");
        printW.flush(); 
    }
});

已编辑 - 附加代码片段（接收部分）： 这是线程类中的一个可运行方法以及上面的代码 sn-p。

public void run() { 

    try{
        while(true){

            String line = br.readLine(); 
            int update = Integer.parseInt(line);
            current-= update; //int current = 0 and update is the number received from the other client, which is "5"

            JLabel newNumber = new JLabel(current); //Assign the newNumber JLabel with the current number

            jp.remove(number); //Remove the JLabel 'number' which initially shows "50" 
            jp.revalidate();
            jp.repaint();
            jp.add(newNumber); //Replace the JLabel 'number' with 'newNumber'

            if(update == 0){
                JOptionPane.showMessageDialog(jf, "Reached 0", "Message", JOptionPane.INFORMATION_MESSAGE);
                System.exit(0);
        }
    }

    } catch (IOException ioe){
        System.out.println("ioe in ChatClient.run: " + ioe.getMessage());
    }
}

Answer 1

您的while (true) 正在 Swing 事件线程上被调用，从而占用了线程并阻止它执行绘制 GUI 和与用户交互的操作。此外，您要多次添加 ActionListener，这几乎没有意义，因为按下按钮时将调用每个 ActionListener。

解决办法：

1. 摆脱不必要且危险的 while 循环。
2. 阅读Swing Threading，然后努力遵守规则。

---

请注意，如果您需要连续读取 BufferedReader 对象，那么可以，将读取的内容放在 while (true) 循环中，但不要添加任何 ActionListener 或对 Swing 对象的状态进行任何更改从循环内部。相反，我会使用 SwingWorker 对象，将我的 while (true) 循环在其 doInBackground() 方法中，以便它确保在后台线程中运行，然后我会使用 SwingWorker 的发布更新 GUI 的显示/处理方法对，以便以线程安全的方式将字符串推送到 GUI 中。我上面给你的链接将帮助你实现这一目标。

---

编辑
您的代码可能类似于：

设置按钮的 ActionListener

  // add this ActionListener once and only once
  button.addActionListener(new ActionListener() {
     @Override
     public void actionPerformed(ActionEvent e) {
        // get the line of text from the GUI
        String text = myTextField.getText();

        // send the text out to the socket via the PrintWriter
        printW.println(text);
     }
  });

请注意，最好使用 AbstractAction，然后使用同一个对象设置 JButton 和 JTextArea 的 Action。

事情的接受方面将涉及使用 SwingWorker 来允许我们使用阻塞代码buffR.readLine(); 而不会阻塞 GUI。请注意，doInBackground() 方法中的所有代码都是在后台线程中完成的，并且在 Swing 事件线程、EDT（事件调度线程）之外完成：

  // use a <Void, String> SwingWorker since I want to publish a String
  new SwingWorker<Void, String>() {
     protected Void doInBackground() throws Exception {
        while (true) {

           // read in a String from the BufferedReader in background thread
           String line = buffR.readLine();

           // publish String so it can be used on Swing event thread, the EDT 
           publish(line);
        }            
     };

     // this code is called in the Swing event thread, the EDT
     protected void process(java.util.List<String> chunks) {
        for (String line : chunks) {
           // display text in my JTextArea
           myTextArea.append(line + "\n");
        }
     };
  }.execute();