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();
        }
    }
}

回到我的问题：
如果我运行程序并在输入一些数字后点击“Go”按钮，程序就会卡住。我认为问题是由TThread中的while循环引起的自从我上次使用Threads已经很长一段时间了，现在我搜索了很长时间，没有任何对我有用... 希望有人可以告诉我问题是什么，并可以提供解决方案或一些提示如何解决问题。

问候
最大

Answer 1

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

所以不是：

tr.run();

而是：

tr.start();

其他问题：

你通常最好让类实现Runnable而不是扩展Thread。
您正在尝试从后台线程进行Swing调用，这不应该完成。几乎所有的Swing方法调用都应该在Swing事件线程或EDT上完成。有关详情，请参阅：Concurrency in Swing。
您的一个类名称Timer与一个关键的Swing类javax.swing.Timer类相同。我会重命名你的课程以避免混淆，特别是如果你想使用Swing Timer。
事实上，鉴于您的代码，我认为使用Swing Timer代替应用程序的后台线程会更好。这是一种更简单的方法来执行计时器动画，并且更容易确保在Swing事件线程上进行Swing调用。请查看Swing Timer tutorial。
您使用的是null布局。对于新手来说，空布局通常似乎是轻松创建复杂GUI的最佳方式，因为当您获得更多Swing GUI体验时，您会发现实际情况恰恰相反。如果您使用布局管理器，通常嵌套JPanels，每个都有自己的布局管理器，您可以更轻松地创建复杂，漂亮的GUI，轻松调整大小，在所有平台上看起来都很好，并且更容易维护，调试和增强。

Java JFrame在运行线程时遇到困难

1 个答案: