多线程Java JFrame在运行线程时陷入困境

Question

我有一个可能很简单的问题。很明显，因为我的程序没有做它应该做的

首先，我的代码是：

import java.awt.*;
import java.awt.event.*;

import javax.swing.*;
import javax.swing.plaf.nimbus.NimbusLookAndFeel;

public class Timer 
        extends JFrame
        implements ActionListener
{

    protected class TThread extends Thread{
        private boolean running = false;
        @Override
        public void run() {
            int timer = 10,
                index = 0;
            running = true;
            while(running){
                try {
                    out.setText(timer + " Secs");
                    timer--;
                    if(timer == 0){
                        if(index % 2 == 0){
                            timer = ti1;
                            out.setBackground(Color.red);
                        }else{
                            timer = ti2;
                            out.setBackground(Color.green);
                        }
                        index++;
                    }
                    sleep(1000L);
                } catch (InterruptedException e) {
                }
            }
        }
        @Override
        public void interrupt() {
            running = false;
        }
    }

    private static final long serialVersionUID = 1L;
    private JTextField t1 = new JTextField(),
                       t2 = new JTextField();
    private int ti1 = 0, ti2 = 0;
    private JLabel l1 = new JLabel("Zeit 1"),
                   l2 = new JLabel("Zeit 2"),
                   out = new JLabel("00 Secs", SwingConstants.CENTER);
    private JButton go = new JButton("Go"),
                    stop = new JButton("Stop");
    private JPanel cont = new JPanel();
    private TThread tr = new TThread();

    public Timer() {
        super("Timer");
        setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
        setSize(800, 600);
        setLayout(null);
        add(cont);
            cont.setBounds(0, 0, getWidth(), 200);
            cont.setLayout(new GridLayout(3, 2));
            cont.add(l1);
            cont.add(t1);
            cont.add(l2);
            cont.add(t2);
            cont.add(go);
                go.addActionListener(this);
            cont.add(stop);
                stop.addActionListener(this);
        add(out);
            out.setBounds(0, 200, getWidth(), getHeight()-200);
            out.setFont(new Font("Arial", Font.BOLD, 72));

        try {
            UIManager.setLookAndFeel(new NimbusLookAndFeel());
            SwingUtilities.updateComponentTreeUI(this);
        } catch (UnsupportedLookAndFeelException e) {
        }
    }

    public static void main(String[] args) {
        Timer t = new Timer();
        t.setVisible(true);
    }

    @Override
    public void actionPerformed(ActionEvent e) {
        if(e.getSource().equals(go)){
            ti1 = Integer.parseInt(t1.getText());
            ti2 = Integer.parseInt(t2.getText());
            tr.run();
        }else if(e.getSource().equals(stop)){
            tr.interrupt();
        }
    }
}

回到我的问题：
如果我运行程序并在输入一些数字后点击“开始”按钮，程序就会卡住。我认为这个问题是由TThread中的while循环引起的
这是相当长的一段时间，因为我上次使用线程，现在我搜索了很长时间，没有为我工作
希望有人能告诉我问题是什么，并能给出解决方案或一些如何解决问题的提示

问候
马克斯

Answer 1

您永远不会在后台线程中通过调用start()来运行线程。而是调用run()，它在当前线程而不是后台线程中运行它。要解决这个问题，请在线程对象上调用start()，而不是run()

所以不是：

tr.run();

而是：

tr.start();

---

其他问题：

• 通常，让类实现Runnable比扩展线程更好
• 您试图从后台线程进行Swing调用，这是不应该做的。几乎所有Swing方法调用都应该在Swing事件线程或EDT上完成。有关这方面的更多信息，请参阅：Concurrency in Swing
• 其中一个类名Timer与关键的Swing类javax.swing.Timer相同。为了避免混淆，我会重新命名你的类，尤其是如果你想使用Swing定时器的话
• 事实上，从您的代码来看，我认为您最好使用Swing计时器，而不是应用程序的后台线程。这是制作计时器动画的一种更简单的方法，并且可以更轻松地确保在Swing事件线程上进行Swing调用。请检查Swing Timer tutorial
• 您使用的是空布局。虽然对于新手来说，空布局通常是轻松创建复杂GUI的最佳方式，但随着您获得更多Swing GUI经验，您会发现事实恰恰相反。如果您使用布局管理器，通常是嵌套的JPanel，每个都有自己的布局管理器，那么您可以更轻松地创建复杂、美观的GUI，易于调整大小，在所有平台上都很好看，并且更易于维护、调试和增强