java - 我没有收到任何错误，但数字不会添加到 JTextField

标签 java swing event-handling jframe

我正在尝试用 Java 创建一个基本上可以制作假彩票的程序。它允许您快速选择 5 个随机数，或者允许您选择自己的随机数。快速选择和个人按钮工作正常。一旦您单击快速选择，它就会在 5 个随机框中显示 5 个随机数。现在的问题是，当单击播放按钮时，不会显示中奖号码，程序也不会跟踪需要多少年和多少张图纸，如果我从 6 个号码中获得 3 个，它不会显示，所以在。请记住，程序运行时不会出现任何错误。

这是我制作 gui 的第一个类的代码:

public class LotteryInterface extends JFrame {

LotteryEvent hello = new LotteryEvent(this);

//set up row 1 
JPanel row1 = new JPanel();
ButtonGroup option = new ButtonGroup();
JCheckBox quickpick = new JCheckBox("Quick Pick", false);
JCheckBox personal = new JCheckBox("Personal", true);
//set up row 2 
JPanel row2 = new JPanel();
JLabel numbersLabel = new JLabel("Your Picks", JLabel.RIGHT);
JTextField[] numbers = new JTextField[6];
JLabel winnersLabel = new JLabel("Winners", JLabel.RIGHT);
JTextField[] winners = new JTextField[6];
//set up row 3
JPanel row3 = new JPanel();
JButton play = new JButton("Play");
JButton stop = new JButton("Stop");
JButton reset = new JButton("Reset");
//set up row 4
JPanel row4 = new JPanel();
JLabel got3label = new JLabel("3 of 6", JLabel.RIGHT);
JTextField got3 = new JTextField("0");
JLabel got4Label = new JLabel("4 of 6", JLabel.RIGHT);
JTextField got4 = new JTextField("0");
JLabel got5label = new JLabel("5 of 6", JLabel.RIGHT);
JTextField got5 = new JTextField("0");
JLabel got6label = new JLabel("6 of 6", JLabel.RIGHT);
JTextField got6 = new JTextField("0", 10);
JLabel drawingsLabel = new JLabel("Drawings", JLabel.RIGHT);
JTextField drawings = new JTextField();
JLabel yearsLabel = new JLabel("Years", JLabel.RIGHT);
JTextField years = new JTextField();

public LotteryInterface() {
    super("Lottery Interface");
    setLookAndFeel();
    setSize(550, 400);
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    GridLayout layout = new GridLayout(0, 1, 10, 10);
    setLayout(layout);

    personal.addItemListener(hello);
    quickpick.addItemListener(hello);
    play.addActionListener(hello);
    stop.addActionListener(hello);
    reset.addActionListener(hello);

    FlowLayout layout1 = new FlowLayout(FlowLayout.CENTER, 10, 10);
    option.add(quickpick);
    option.add(personal);
    row1.setLayout(layout1);
    row1.add(quickpick);
    row1.add(personal);
    add(row1);

    GridLayout layout2 = new GridLayout(2, 7, 10, 10);
    row2.setLayout(layout2);
    row2.add(numbersLabel);
    for (int i = 0; i < 6; i++) {
        numbers[i] = new JTextField();
        row2.add(numbers[i]);
    }
    row2.add(winnersLabel);
    for (int i = 0; i < 6; i++) {
        winners[i] = new JTextField();
        winners[i].setEditable(false);
        row2.add(winners[i]);
    }
    add(row2);

    FlowLayout layout3 = new FlowLayout(FlowLayout.CENTER, 10, 10);
    row3.setLayout(layout3);
    row3.add(stop);
    row3.add(play);
    row3.add(reset);
    add(row3);

    GridLayout layout4 = new GridLayout(2, 3, 20, 10);
    row4.setLayout(layout4);
    row4.add(got3label);
    got3.setEditable(false);
    row4.add(got3);
    row4.add(got4Label);
    got4.setEditable(false);
    row4.add(got4);
    row4.add(got5label);
    got5.setEditable(false);
    row4.add(got5);
    row4.add(got6label);
    got6.setEditable(false);
    row4.add(got6);
    row4.add(drawingsLabel);
    drawings.setEditable(false);
    row4.add(drawings);
    row4.add(yearsLabel);
    years.setEditable(false);
    row4.add(years);
    add(row4);


    setVisible(true);

}

private void setLookAndFeel() {


}

public static void main(String args[]) {


    LotteryInterface yeah = new LotteryInterface();

    }

}

这是让 gui 做事的第二个类:

public class LotteryEvent extends JFrame implements ActionListener, ItemListener, Runnable {

LotteryInterface gui;
Thread playing;


public LotteryEvent(LotteryInterface in ) {
    gui = in ;
}

@Override
public void actionPerformed(ActionEvent event) {
    String command = event.getActionCommand();
    if (command.equals("Play")) {
        startPlaying();
    }
    if (command.equals("Stop")) {
        stopPlaying();
    }
    if (command.equals("Reset")) {
        clearAllFields();
    }

}

void startPlaying() {
    playing = new Thread(this);
    playing.start();
    gui.play.setEnabled(false);
    gui.stop.setEnabled(true);
    gui.reset.setEnabled(false);
    gui.quickpick.setSelected(false);
    gui.personal.setEnabled(false);

}

void stopPlaying() {
    gui.play.setEnabled(true);
    gui.stop.setEnabled(false);
    gui.reset.setEnabled(true);
    gui.quickpick.setSelected(true);
    gui.personal.setEnabled(true);
    playing = null;

}

void clearAllFields() {
    for (int i = 0; i < 6; i++) {
        gui.winners[i].setText(null);
        gui.numbers[i].setText(null);
    }

    gui.got3.setText("0");
    gui.got4.setText("0");
    gui.got4.setText("0");
    gui.got5.setText("0");
    gui.got6.setText("0");

    gui.years = null;
    gui.drawings = null;


}

@Override
public void itemStateChanged(ItemEvent event) {
    Object item = event.getItem();
    if (item == gui.quickpick) {
        for (int i = 0; i < 6; i++) {
            int pick;
            do {
                pick = (int) Math.floor(Math.random() * 50 + 1);

            } while (numberGone(pick, gui.numbers, i));
            gui.numbers[i].setText("" + pick);
        }
    } else {

        for (int i = 0; i < 6; i++) {
            gui.numbers[i].setText(null);
        }
    }

}

void addOneToField(JTextField field) {
    int num = Integer.parseInt("0" + field.getText());
    num++;
    field.setText("" + num);
}

boolean numberGone(int nums, JTextField[] pastNums, int count) {
    for (int i = 0; i < count; i++) {
        if (Integer.parseInt(pastNums[i].getText()) == nums) {
            return true;
        }
    }
    return false;
}

boolean matchedOne(JTextField win, JTextField[] allpicks) {
    for (int i = 0; i < 6; i++) {
        String winText = win.getText();
        if (winText.equals(allpicks[i].getText())) {
            return true;
        }
    }
    return false;
}

@Override
public void run() {
    Thread thisThread = new Thread();
    while (playing == thisThread) {
        addOneToField(gui.drawings);
        int draw = Integer.parseInt(gui.drawings.getText());
        float numYears = (float) draw / 104;
        gui.years.setText("" + numYears);

        int matches = 0;
        for (int i = 6; i < 6; i++) {
            int ball;
            do {
                ball = (int) Math.floor(Math.random() * 50 + 1);
            } while (numberGone(ball, gui.winners, i));
            gui.winners[i].setText("" + ball);
            if (matchedOne(gui.winners[i], gui.numbers)) {
                matches++;
            }

        }
        switch (matches) {
            case 3:
                addOneToField(gui.got3);
                break;
            case 4:
                addOneToField(gui.got4);
                break;
            case 5:
                addOneToField(gui.got5);
                break;
            case 6:
                addOneToField(gui.got6);
                gui.stop.setEnabled(false);
                gui.play.setEnabled(true);
                playing = null;
        }
        try {
            Thread.sleep(100);
        } catch (InterruptedException e) {

        }

    }

    }

}

最佳答案

好吧，问题很简单，解决方案不是那么...

当您调用 startPlaying 时你做...

playing = new Thread(this);

但是当run被调用，然后你这样做......

Thread thisThread = new Thread();
while (playing == thisThread) {

问题是，thisThread和 playing永远不相等。

更好的解决方案可能是使用 AtomicBoolean作为运行状态标志并检查它......但是......

Swing 不是线程安全的，您正试图从事件调度线程的上下文之外更新 UI

更好的解决方案是实际使用 Swing Timer ，这将允许您设置一个定期延迟的重复回调，这将在事件调度线程的上下文中得到通知。

另一个好处是，Swing Timer有一个 stop它自己的方法...

参见 Concurrency in Swing和 How to use Swing Timers了解更多详情。

正如已经指出的那样，不需要 LotteryEvent从 JFrame 扩展，事实上，我认为它甚至不应该实现 ActionListener或 ItemListener , 但那就是我

更新了例子

此外，这 for (int i = 6; i < 6; i++) {错了……

import java.awt.EventQueue;
import java.awt.FlowLayout;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.ItemEvent;
import java.awt.event.ItemListener;
import javax.swing.ButtonGroup;
import javax.swing.JButton;
import javax.swing.JCheckBox;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JTextField;
import javax.swing.Timer;
import javax.swing.UIManager;
import javax.swing.UnsupportedLookAndFeelException;

public class LotteryInterface extends JPanel {

    LotteryEvent hello = new LotteryEvent(this);

    //set up row 1 
    JPanel row1 = new JPanel();
    ButtonGroup option = new ButtonGroup();
    JCheckBox quickpick = new JCheckBox("Quick Pick", false);
    JCheckBox personal = new JCheckBox("Personal", true);
    //set up row 2 
    JPanel row2 = new JPanel();
    JLabel numbersLabel = new JLabel("Your Picks", JLabel.RIGHT);
    JTextField[] numbers = new JTextField[6];
    JLabel winnersLabel = new JLabel("Winners", JLabel.RIGHT);
    JTextField[] winners = new JTextField[6];
    //set up row 3
    JPanel row3 = new JPanel();
    JButton play = new JButton("Play");
    JButton stop = new JButton("Stop");
    JButton reset = new JButton("Reset");
    //set up row 4
    JPanel row4 = new JPanel();
    JLabel got3label = new JLabel("3 of 6", JLabel.RIGHT);
    JTextField got3 = new JTextField("0");
    JLabel got4Label = new JLabel("4 of 6", JLabel.RIGHT);
    JTextField got4 = new JTextField("0");
    JLabel got5label = new JLabel("5 of 6", JLabel.RIGHT);
    JTextField got5 = new JTextField("0");
    JLabel got6label = new JLabel("6 of 6", JLabel.RIGHT);
    JTextField got6 = new JTextField("0", 10);
    JLabel drawingsLabel = new JLabel("Drawings", JLabel.RIGHT);
    JTextField drawings = new JTextField();
    JLabel yearsLabel = new JLabel("Years", JLabel.RIGHT);
    JTextField years = new JTextField();

    public LotteryInterface() {
        GridLayout layout = new GridLayout(0, 1, 10, 10);
        setLayout(layout);

        ItemListener itemListener = new ItemListener() {

            @Override
            public void itemStateChanged(ItemEvent e) {
                Object item = e.getItem();
                if (item == quickpick) {
                    for (int i = 0; i < 6; i++) {
                        int pick;
                        do {
                            pick = (int) Math.floor(Math.random() * 50 + 1);

                        } while (numberGone(pick, numbers, i));
                        numbers[i].setText("" + pick);
                    }
                } else {

                    for (int i = 0; i < 6; i++) {
                        numbers[i].setText(null);
                    }
                }

            }
        };

        personal.addItemListener(itemListener);
        quickpick.addItemListener(itemListener);
        play.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                hello.start();
            }
        });
        stop.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                hello.stop();
            }
        });
        reset.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                hello.stop();
                clearAllFields();
            }
        });

        FlowLayout layout1 = new FlowLayout(FlowLayout.CENTER, 10, 10);
        option.add(quickpick);
        option.add(personal);
        row1.setLayout(layout1);
        row1.add(quickpick);
        row1.add(personal);
        add(row1);

        GridLayout layout2 = new GridLayout(2, 7, 10, 10);
        row2.setLayout(layout2);
        row2.add(numbersLabel);
        for (int i = 0; i < 6; i++) {
            numbers[i] = new JTextField();
            row2.add(numbers[i]);
        }
        row2.add(winnersLabel);
        for (int i = 0; i < 6; i++) {
            winners[i] = new JTextField();
            winners[i].setEditable(false);
            row2.add(winners[i]);
        }
        add(row2);

        FlowLayout layout3 = new FlowLayout(FlowLayout.CENTER, 10, 10);
        row3.setLayout(layout3);
        row3.add(stop);
        row3.add(play);
        row3.add(reset);
        add(row3);

        GridLayout layout4 = new GridLayout(2, 3, 20, 10);
        row4.setLayout(layout4);
        row4.add(got3label);
        got3.setEditable(false);
        row4.add(got3);
        row4.add(got4Label);
        got4.setEditable(false);
        row4.add(got4);
        row4.add(got5label);
        got5.setEditable(false);
        row4.add(got5);
        row4.add(got6label);
        got6.setEditable(false);
        row4.add(got6);
        row4.add(drawingsLabel);
        drawings.setEditable(false);
        row4.add(drawings);
        row4.add(yearsLabel);
        years.setEditable(false);
        row4.add(years);
        add(row4);

    }

    void clearAllFields() {
        for (int i = 0; i < 6; i++) {
            winners[i].setText(null);
            numbers[i].setText(null);
        }

        got3.setText("0");
        got4.setText("0");
        got4.setText("0");
        got5.setText("0");
        got6.setText("0");

        years = null;
        drawings = null;

    }

    boolean numberGone(int nums, JTextField[] pastNums, int count) {
        for (int i = 0; i < count; i++) {
            if (Integer.parseInt(pastNums[i].getText()) == nums) {
                return true;
            }
        }
        return false;
    }

    public static void main(String args[]) {

        EventQueue.invokeLater(new Runnable() {
            @Override
            public void run() {
                try {
                    UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
                } catch (ClassNotFoundException | InstantiationException | IllegalAccessException | UnsupportedLookAndFeelException ex) {
                    ex.printStackTrace();
                }

                JFrame frame = new JFrame("Testing");
                frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
                frame.add(new LotteryInterface());
                frame.pack();
                frame.setLocationRelativeTo(null);
                frame.setVisible(true);
            }
        });
    }

    public class LotteryEvent {

        LotteryInterface gui;
        private Timer timer;

        public LotteryEvent(LotteryInterface in) {
            gui = in;
            timer = new Timer(100, new ActionListener() {
                @Override
                public void actionPerformed(ActionEvent e) {
                    addOneToField(gui.drawings);
                    int draw = Integer.parseInt(gui.drawings.getText());
                    float numYears = (float) draw / 104;
                    gui.years.setText("" + numYears);

                    int matches = 0;
                    for (int i = 0; i < 6; i++) {
                        int ball;
                        do {
                            ball = (int) Math.floor(Math.random() * 50 + 1);
                        } while (numberGone(ball, gui.winners, i));
                        gui.winners[i].setText("" + ball);
                        if (matchedOne(gui.winners[i], gui.numbers)) {
                            matches++;
                        }

                    }
                    switch (matches) {
                        case 3:
                            addOneToField(gui.got3);
                            break;
                        case 4:
                            addOneToField(gui.got4);
                            break;
                        case 5:
                            addOneToField(gui.got5);
                            break;
                        case 6:
                            addOneToField(gui.got6);
                            gui.stop.setEnabled(false);
                            gui.play.setEnabled(true);
                            stop();
                    }
                }
            });
        }

        public void stop() {
            timer.stop();
        }

        public void start() {
            timer.start();
        }

        void addOneToField(JTextField field) {
            int num = Integer.parseInt("0" + field.getText());
            num++;
            field.setText("" + num);
        }

        boolean matchedOne(JTextField win, JTextField[] allpicks) {
            for (int i = 0; i < 6; i++) {
                String winText = win.getText();
                if (winText.equals(allpicks[i].getText())) {
                    return true;
                }
            }
            return false;
        }

    }

}

关于java - 我没有收到任何错误，但数字不会添加到 JTextField，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/29620031/

上一篇：java - 使用 JUnit 和 JSONassert 比较 JSON 响应

下一篇：java - 在特定字符串后向字符添加单引号

相关文章：

swing - scala 的鼠标事件有效吗？如何？

java - getConstructor() 返回一个未实现的构造函数

java - 让一个字符串代表一个数组

java - 我们什么时候应该修改通过引用传递的对象？

c# - 对不同语言/体系结构/操作系统中相对较小的数字取对数

java - 使用 Graphics2D 为 2 个椭圆着色

java - SwingWorker 中的类型参数有哪些？

jquery - 如何在不多次执行函数的情况下将转换结束事件绑定(bind)到多个元素？

c#-4.0 - 响应式(Reactive)扩展处理事件一次

c# - 在动态表的 RadioButton CheckedChanges 中添加新的事件处理程序

©2024 IT工具网  联系我们