在空的无限循环与dosomething中野循环中检查选项

Question

public static void main(String... s) {
StartUp obj = new StartUp();

while(true) {
//System.out.println("Option - " + option);
if(option == 1) {
option = 0;
obj.setVisible(false);
obj.dispose();
new Test();
break;
}else if(option == 2) {
option = 0;
obj.setVisible(false);
obj.dispose();
new PWorld.Splash().load();
break;
}
}
}

我需要把System.out.println("Option - " + option);放在while循环中才能工作，否则程序在运行StartUp obj=new StartUp((后会冻结；

option是StartUp类中的静态int，由Actionlistener更改ActionListener更改了option中的值，但while循环似乎不起作用。

但如果我在while循环中放入System.out.println("Option - " + option);，它就可以工作了为什么

我使用这个while循环是因为new PWorld.Splash().load();有Thread.sleep()，并且在这个answere中，如果从具有Thread的ActionListener(在UI线程中(调用，则不会绘制新的JFrame。

谢谢

Answer 1

您的问题是：

• 你在叫一个"紧密的"；循环，占用CPU并阻止其他代码运行。System.out.println(...)语句添加了减缓这个循环的代码，将CPU从紧密循环的钳口中释放出来，允许其他线程运行，这就是问题的根源
• 话虽如此，但您的编码方法并不好，因为您使用while (true)循环来代替对事件的响应，这就是Swing GUI的编码方式
• 您指出，原因是while循环中的一位代码调用了Thread.sleep，并且如果在Swing事件线程上(例如在ActionListener中(调用此代码，则会阻塞事件线程，冻结您的GUI——全部为true
• 但你的解决方案是错误的。正确的解决方案不是在主方法中的while (true)循环中调用它，而是从后台线程调用Thread.sleep，例如在SwingWorker的doInBackground()方法中(链接指向教程(，或者更好的是，使用SwingTimer(同样，链接指向指南(代替Thread.sleep。这将允许您的代码在不阻塞Swing事件线程的情况下暂停一些代码
• 如果需要显示对话框(子(窗口，另一种选择是使用模式JDialog显示窗口，同时阻止与主GUI窗口的交互，直到对话框窗口不再可见

要获得更详细、更全面的解决方案，请再次考虑创建并发布您的问题的最小、可复制示例程序。

例如，这里是我的最小，可复制的例子：

import java.awt.Dialog.ModalityType;
import java.awt.Dimension;
import java.awt.Color;
import java.awt.GridBagLayout;
import java.awt.GridLayout;
import java.awt.Window;
import javax.swing.*;
public class MinReproExample {
public static void main(String[] args) {
SwingUtilities.invokeLater(() -> {
Startup startup = new Startup();
startup.showStartUp();
Option option = startup.getOption();
if (option == Option.TEST) {
JOptionPane.showMessageDialog(null, "Test selected", "Selection", JOptionPane.DEFAULT_OPTION);
} else if (option == Option.PWORLD) {
PWorld pworld = new PWorld();
pworld.showSplash();
}   
});
}
}

class Startup {
private JDialog startupDialog;
private Option option = null;

public Startup() {
ButtonGroup buttonGroup = new ButtonGroup();
JPanel optionsPanel = new JPanel(new GridLayout(1, 0, 10, 10));
optionsPanel.setBorder(BorderFactory.createTitledBorder("Options"));
for (final Option op : Option.values()) {
JRadioButton rBtn = new JRadioButton(op.getText());
rBtn.setActionCommand(op.getText());
optionsPanel.add(rBtn);
buttonGroup.add(rBtn);
rBtn.addActionListener(e -> {
option = op;
Window window = SwingUtilities.getWindowAncestor(optionsPanel);
window.dispose();
});
}

startupDialog = new JDialog(null, "Select Option", ModalityType.APPLICATION_MODAL);
startupDialog.setDefaultCloseOperation(JDialog.DISPOSE_ON_CLOSE);
startupDialog.add(optionsPanel);
startupDialog.pack();
startupDialog.setLocationRelativeTo(null);
}

public void showStartUp() {
if (startupDialog != null) {
startupDialog.setVisible(true);
}
}

public Option getOption() {
return option;
}
}

class PWorld {
private static final Color ROBINS_EGG_BLUE = new Color(0, 204, 204);
private JDialog pworldSplashDialog;
private JFrame mainPWorldFrame;
public PWorld() {
JLabel splashLabel = new JLabel("Splash Window", SwingConstants.CENTER);
JPanel splashPanel = new JPanel(new GridBagLayout());
splashPanel.add(splashLabel);
splashPanel.setBackground(Color.PINK);
splashPanel.setPreferredSize(new Dimension(300, 250));
pworldSplashDialog = new JDialog(null, "Splash", ModalityType.MODELESS);
pworldSplashDialog.setDefaultCloseOperation(JDialog.DISPOSE_ON_CLOSE);
pworldSplashDialog.add(splashPanel);
pworldSplashDialog.pack();
pworldSplashDialog.setLocationRelativeTo(null);
JLabel mainLabel = new JLabel("Main GUI Window", SwingConstants.CENTER);
JPanel mainPanel = new JPanel(new GridBagLayout());
mainPanel.add(mainLabel);
mainPanel.setBackground(ROBINS_EGG_BLUE);
mainPanel.setPreferredSize(new Dimension(500, 350));
mainPWorldFrame = new JFrame("Main PWorld GUI");
mainPWorldFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
mainPWorldFrame.add(mainPanel);
mainPWorldFrame.pack();
mainPWorldFrame.setLocationRelativeTo(null);
}
public void showSplash() {
int timerDelay = 2000; // two second delay
Timer timer = new Timer(timerDelay, e -> {
if (pworldSplashDialog != null && pworldSplashDialog.isVisible()) {
pworldSplashDialog.dispose();
showMainPWorldFrame();
}
});
timer.setRepeats(false);
timer.start();
pworldSplashDialog.setVisible(true);
}
private void showMainPWorldFrame() {
mainPWorldFrame.setVisible(true);
}
}

// options to choose from
enum Option {
TEST("Test"), PWORLD("PWorld");
private String text;
private Option(String text) {
this.text = text;
}
public String getText() {
return text;
}
}

Answer 2

如果初始输入的选项不是1或2，这个循环应该怎么做？您只是在无理由地消耗CPU周期，等待其他线程执行某些操作。

添加print语句会注入一点非CPU烧录延迟，在这种情况下，可能要设置"option"的线程会运行。

(FWIW，如果您希望更改对其他线程可见，则可能需要声明"option"为volatile(。

这不是一个好的设计。我对上下文的了解不足以告诉您应该做什么，但需要某种像样的通知机制。但这应该能回答你"为什么？"的问题。