Java 检查空无限循环与dosomething infite循环中的选项

我需要输入

System.out.println（“选项-”+Option）在while循环中运行，否则程序在运行StartUp obj=new StartUp（）后冻结

选项
是启动类中的静态int，由Actionlistener更改
选项
中的值由ActionListener更改，但while循环似乎不起作用

但是如果我把
System.out.println（“选项-”+Option）内部while循环，它可以工作<为什么

我使用这个while循环是因为
newpworld.Splash（）.load（）
有
线程.sleep（）
，如果从ActionListener（在UI线程中）调用新的JFrame，它有
线程

谢谢
如果初始输入时选项不是1或2，该循环将执行什么操作？您只是无缘无故地消耗CPU周期，等待其他线程执行某些操作

添加print语句会带来一点非CPU消耗延迟，在这种情况下，可能要设置“option”的线程会运行

（FWIW，“option”可能需要声明为volatile，如果您希望更改对其他线程可见）

这不是一个好的设计。我不能告诉你足够的上下文来告诉你你应该做什么，但是需要某种像样的通知机制。但这应该能回答你的“为什么”问题。
你的问题是：


您正在调用一个“紧密”循环，它占用CPU并阻止其他代码运行。
System.out.println（…）
语句添加了减慢这个循环的代码，将CPU从紧循环的钳口中释放出来，允许其他线程运行，这就是问题的根源
话虽如此，您的编码方法仍然不好，因为您使用
while（true）
循环来代替对事件的响应，这就是Swing GUI的编码方式
您声明这样做的原因是while循环中的一位代码调用了
线程。sleep
，并且如果在Swing事件线程上（例如在ActionListener中）调用此代码，将阻塞事件线程，冻结GUI——全部为真
但你的解决方案是错误的。正确的解决方案不是在main方法的
while（true）
循环中调用它，而是从后台线程调用
线程。sleep
，例如在a（link is to tutorial）的
doInBackground（）方法中，或者更好的是使用a（同样，link is to tutorial）代替
线程。睡眠
。这将允许您的代码暂停一些代码，而不会阻塞Swing事件线程

如果需要显示对话框（子）窗口，另一个选项是使用模态JDialog显示窗口，同时阻止与主GUI窗口的交互，直到对话框窗口不再可见

对于更详细和全面的解决方案，请再次考虑创建和发布您的程序与您的问题。
例如，以下是我的最小可复制示例：

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

不管用println延迟循环是否有助于它的工作，我能给你的主要建议是：完全摆脱循环。您使用的是一个事件驱动的GUI库，因此不需要循环，相反，您应该编写代码，根据程序状态（对象的关键字段持有的值）的行为响应事件。@DontKnowMuchBut越来越好，我无法在ActionListener中直接调用这些方法，因为例如
new PWorld.Splash（）.load()；
使用的线程会干扰ActionListenserNo的UI线程，事实并非如此。可以在ActionListeners中创建新线程，事实上我一直都在这样做。这个问题看起来是一个类型问题，您可能问错了问题。正确的问题是如何在没有inte的情况下将此代码正确集成到GUI中使用GUI功能，并且要回答这个问题，您需要告诉更多的细节，也许还需要发布一个有效的。您可能希望了解和改进这个问题，告诉这些细节，因为我认为您的整个方法需要更改，包括摆脱while循环。示例代码发布在我的答案中。请检查它，运行它，如果需要，请进行评论有任何问题我不能直接在ActionListener中调用这些方法，因为例如
newpWorld.Splash（）.load（）
使用
线程。sleep
会干扰ActionListenser的UI线程，并且不会绘制新的框架。所以我发现ActionListener是一个接口。我猜你应该实现这个接口，然后对选项的任何更改做出响应。也就是说，你的设计是颠倒的。你不调用它；它在召唤你。谢谢你@DontKnowMuchBut好转了，@SameerGupta:不，我不需要看。MadProgrammer忘记了比我所知道的更多的挥杆动作，他应该把事情做好。
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;
    }
}