Swing小游戏while循环单指令执行异常问题求助

这个问题的核心在于Swing的单线程模型和Java的线程可见性优化，咱们一步步拆解：

为什么不加System.out.println()就不工作？ #

1. 线程可见性优化：当你的while循环里只有对mapa.todasComidas()的判断时，JVM可能会进行激进的优化——它会默认认为这个方法的返回值在循环周期内不会改变，于是直接缓存结果，不再去实际读取mapa的最新状态。这样哪怕游戏中食物已经被吃完，循环也不会察觉到变化。
2. System.out.println()的隐藏作用：这个方法内部其实包含了同步操作（PrintStream的println是带锁的），而同步操作会强制JVM刷新线程的缓存，让它重新读取mapa.todasComidas()的真实值。同时，这个输出操作会让线程短暂让步，给Swing的事件调度线程（EDT）留出处理UI更新的时间，让mapa的状态有机会被正确更新。

另外还有一个关键问题：你把这个while循环放在了JFrame的构造函数里，而Swing的组件创建和UI操作必须在EDT中执行。这个无限循环会直接阻塞EDT，导致所有UI事件（比如玩家操作、组件重绘）都无法被处理，mapa的状态自然也无法得到更新。

正确的解决方法 #

方案1：使用Swing Timer（推荐，符合Swing规范） #

Swing Timer专门用来在EDT中定时执行任务，既不会阻塞UI线程，又能定期检查游戏状态：

public class ejecutor extends JFrame {
    public ejecutor(){
        mapa mapa = new mapa();
        setSize(665,690);
        add(mapa);
        setResizable(false);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setLocationRelativeTo(null);
        setVisible(true);

        // 创建一个每隔100ms检查一次的Timer
        Timer gameCheckTimer = new Timer(100, e -> {
            if(mapa.todasComidas()){
                // wLabel();
                ((Timer)e.getSource()).stop(); // 停止Timer
                System.out.println("You finished!!");
            }
        });
        gameCheckTimer.start();
    }

    // main方法确保在EDT中启动UI
    public static void main(String[] args) {
        SwingUtilities.invokeLater(() -> new ejecutor());
    }
}

方案2：将游戏循环放到独立线程（需注意线程安全） #

如果你坚持用while循环，要把它放到单独的线程中，同时保证共享变量的线程可见性：

public class ejecutor extends JFrame {
    // 用volatile修饰，确保线程间可见性
    volatile boolean ejecutando = true;

    public ejecutor(){
        mapa mapa = new mapa();
        setSize(665,690);
        add(mapa);
        setResizable(false);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setLocationRelativeTo(null);
        setVisible(true);

        // 启动独立线程处理游戏循环
        new Thread(() -> {
            while(ejecutando){
                if(mapa.todasComidas()){
                    ejecutando = false;
                    // 注意：Swing UI操作必须在EDT中执行
                    SwingUtilities.invokeLater(() -> {
                        // wLabel();
                        System.out.println("You finished!!");
                    });
                }
                // 让线程短暂休眠，避免占用过多CPU
                try {
                    Thread.sleep(100);
                } catch (InterruptedException ex) {
                    Thread.currentThread().interrupt();
                }
            }
        }).start();
    }

    public static void main(String[] args) {
        SwingUtilities.invokeLater(() -> new ejecutor());
    }
}

关键注意事项 #

• 永远不要在EDT中执行长时间运行的任务（比如你的原始while循环），否则会导致UI卡顿甚至无响应。
• 对多个线程共享的变量（比如ejecutando），要用volatile修饰或者通过同步块/方法保证可见性。
• 所有Swing组件的更新操作必须在EDT中执行，否则可能出现UI异常。

内容的提问来源于stack exchange，提问作者Emanuel Hegedűs