Question

我简化了我的代码，与你们分享。现在它有3个班级。 JFrame，JInternalFrame和一个Thread。 JFrame创建JInternalFrame，JInternalFrame创建Thread。我的线程中的构造函数需要一个JInternalFrame的实例来弄乱它的按钮。所以在我创建了我的InternalFrame的界面后，我创建并启动了我的Thread。不幸的是，整个JFrame冻结了，代码将完全保留在线程中。我已经试图摆脱我的线程中的while（true）循环。然后，JFrame仅冻结我的睡眠功能所需的时间。

长话短说：我怎么能让我的线程实际运行并行？没有我的主代码等待线程中的任何睡眠时间或类似的东西。

这是我的3个班级。提前抱歉任何愚蠢的noob错误。我还在学习。

的JFrame

package nachgestellt;

import java.awt.*;
import java.awt.event.*;
import java.util.Random;
import javax.swing.*;

public class Langtons extends JFrame implements ActionListener {
    private JDesktopPane desk;
    private JPanel panelButtons;
    JMenuBar jmb;
    JMenu file;
    JMenuItem open, exit;
    JSlider slider;
    int xInt, yInt;

    Random randomGenerator = new Random();

    JLabel xLabel, yLabel, speed, test;
    JButton start, stop, addAnt;
    JTextField xField, yField;

    public Langtons() {
        //Desktop
        desk = new JDesktopPane();
        getContentPane().add(desk, BorderLayout.CENTER);

        xField = new JTextField();
        yField = new JTextField();

        start = new JButton("Fenster erstellen");

        panelButtons = new JPanel();
        panelButtons.setLayout(new GridLayout());
        ;

        panelButtons.add(start);
        panelButtons.add(xField);
        panelButtons.add(yField);

        start.addActionListener(this);
        add(panelButtons, BorderLayout.NORTH);
        setSize(new Dimension(1100, 900));
        setVisible(true);
    }

    public void actionPerformed(ActionEvent e) {
        if (e.getActionCommand().equals("Fenster erstellen")) {
            xInt = Integer.parseInt(xField.getText());
            yInt = Integer.parseInt(yField.getText());
            addChild(new kind(this, xInt, yInt), this.getSize().width, this.getSize().height);
        }
    }


    public void addChild(JInternalFrame kind, int x, int y) {
        kind.setSize(370, 370);
        kind.setLocation(randomGenerator.nextInt(x - kind.getSize().height), randomGenerator.nextInt(y - kind.getSize().height - 100));
        kind.setDefaultCloseOperation(JInternalFrame.DISPOSE_ON_CLOSE);
        desk.add(kind);
        kind.setBackground(Color.blue);
        kind.setVisible(true);

    }

    public static void main(String[] args) {
        new Langtons();
    }

}

的JInternalFrame

package nachgestellt;

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.GridLayout;
import java.util.ArrayList;

import javax.swing.*;


public class kind extends JInternalFrame {

    Langtons hFenster;
    static int nr = 0;

    JPanel panelButtonsKind;

    ArrayList<JButton> jbArray = new ArrayList<JButton>();

    JPanel panelSpielfeld;

    int posAmeise, aktuellesIcon;

    public kind(Langtons la, int x, int y) {
        super("Kind " + (++nr), true, true, true, true);
        hFenster = la;

        panelSpielfeld = new JPanel();

        panelSpielfeld.setLayout(new GridLayout(y, x));

        jbArray.add(new JButton());

        for (int i = 1; i <= (x * y); i++) {
            jbArray.add(new JButton(Integer.toString(i)));
            panelSpielfeld.add(jbArray.get(i));
            jbArray.get(i).setBackground(Color.WHITE);
        }

        jbArray.get(((x / 2) * y) - y / 2).setBackground(Color.GREEN);

        posAmeise = (((x / 2) * y) - y / 2);

        setLayout(new BorderLayout());

        panelButtonsKind = new JPanel();
        panelButtonsKind.setLayout(new GridLayout(1, 3));

        add(panelButtonsKind, BorderLayout.NORTH);
        add(panelSpielfeld, BorderLayout.CENTER);
        setVisible(true);

        this.repaint();
        thread a = new thread(this);
        a.start();
        System.out.println("Test Message After Creation and Start of Thread");
    }

    public void changeColor() {
        if (jbArray.get(posAmeise).getBackground().equals(Color.GREEN))
            jbArray.get(posAmeise).setBackground(Color.WHITE);
        else
            jbArray.get(posAmeise).setBackground(Color.GREEN);
    }
}

发

package nachgestellt;

import java.awt.Color;

public class thread extends Thread {

    kind k;

    public thread(kind kk) {
        this.k = kk;
        run();
    }

    public void run() {
        super.run();

        while (true) {
            if (k.jbArray.get(k.posAmeise).getBackground().equals(Color.GREEN)) {
                try {
                    sleep(1000);

                    k.changeColor();
                    System.out.println("Test Message Thread!");
                } catch (InterruptedException e) {
                    e.printStackTrace();
                }

            } else if (k.jbArray.get(k.posAmeise).getBackground()
                    .equals(Color.WHITE)) {
                try {
                    sleep(1000);
                    k.changeColor();
                } catch (InterruptedException e) {
                    e.printStackTrace();
                }
            }
        }
    }
}

Answer 1

代码中最直接的问题是：

public thread(kind kk) {
    this.k = kk;
    run();
}

您不想直接调用run()方法。您想在某个主题上调用start()。
你不应该从构造函数中启动一个线程（这会导致问题）
你应该找到一个比小写线程更好的名字。（不是每个人说的错误，只是命名不好）。

此外，在您的run方法中，您有：

public void run() {
    super.run();
}

超级调用是不必要的，您应该添加@Override注释以提高代码的清晰度：

@Override
public void run() {
   //your code
}

Answer 2

Swing既是单线程又不是线程安全的，你不应该在Event Dispatching Thread中做任何事情，它可能会阻塞它或运行很长一段时间。同样，您不应该尝试从EDT外部更新UI。

有关详细信息，请参阅Concurrency in Swing。

Thread#run只会在当前主题的上下文中调用Thread的{{1}}方法，这不是特别有用，您应该使用{ {1}}。

但是，在你说的情况下，你应该使用Swing run而不是Thread#start。计时器将以指定的延迟间隔设置对已注册Timer的回调，该延迟间隔在EDT的上下文中执行，从而可以安全地从内部更新UI。

有关详细信息，请参阅How to use Swing Timers

当线程中的while（true）循环时，JFrame会冻结

的JFrame

的JInternalFrame

发

2 个答案: