Java中actionPerformed方法内循环失效：JFrame椭圆绘制需求

核心问题总结 #

原代码存在多个关键错误，导致功能无法正常运行：

• 字符串比较错误：使用==比较字符串内容，Java中==比较的是对象引用而非字符串值，应改用equals()方法。
• EDT线程阻塞：actionPerformed运行在Swing事件调度线程（EDT）中，直接写while循环会卡死整个界面，既无法响应Stop按钮，也无法触发重绘。
• 绘图计算逻辑错误：(int)Math.random()*2000的运算优先级问题，会先把Math.random()转为int（结果为0）再乘2000，导致椭圆位置永远在左上角，需改为(int)(Math.random()*2000)。
• 缺少绘制控制开关：没有变量标记绘制状态，无法通过Stop按钮终止绘制流程。
• 绘图方法误用：重写JFrame的paint方法时错误调用super.paintComponents(g)，且未使用fillOval实现填充椭圆，也未设置随机颜色。

修正后的完整代码 #

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

public class RandomOvalDrawer extends JFrame implements ActionListener {
    private JButton startBtn, stopBtn;
    private DrawingPanel drawingPanel;
    private Timer drawTimer;
    private boolean isRunning = false;

    public RandomOvalDrawer() {
        setTitle("随机椭圆绘制器");
        setDefaultCloseOperation(EXIT_ON_CLOSE);
        setSize(1500, 1000);
        setLayout(new BorderLayout());

        // 控制按钮面板
        JPanel controlPanel = new JPanel();
        startBtn = new JButton("Start");
        stopBtn = new JButton("Stop");
        stopBtn.setEnabled(false);

        startBtn.addActionListener(this);
        stopBtn.addActionListener(this);

        controlPanel.add(startBtn);
        controlPanel.add(stopBtn);
        add(controlPanel, BorderLayout.NORTH);

        // 绘图面板
        drawingPanel = new DrawingPanel();
        add(drawingPanel, BorderLayout.CENTER);

        // 初始化Timer，每100ms触发一次绘制
        drawTimer = new Timer(100, e -> {
            drawingPanel.repaint();
        });

        setVisible(true);
    }

    @Override
    public void actionPerformed(ActionEvent ae) {
        if ("Start".equals(ae.getActionCommand())) {
            isRunning = true;
            startBtn.setEnabled(false);
            stopBtn.setEnabled(true);
            drawTimer.start();
        } else if ("Stop".equals(ae.getActionCommand())) {
            isRunning = false;
            startBtn.setEnabled(true);
            stopBtn.setEnabled(false);
            drawTimer.stop();
        }
    }

    // 自定义绘图面板，负责绘制椭圆
    private class DrawingPanel extends JPanel {
        @Override
        protected void paintComponent(Graphics g) {
            super.paintComponent(g);
            if (!isRunning) return;

            // 设置随机颜色
            Color randomColor = new Color(
                    (int)(Math.random()*256),
                    (int)(Math.random()*256),
                    (int)(Math.random()*256)
            );
            g.setColor(randomColor);

            // 随机位置与大小
            int x = (int)(Math.random() * getWidth());
            int y = (int)(Math.random() * getHeight());
            int width = (int)(Math.random() * 200 + 50); // 50-250的宽度
            int height = (int)(Math.random() * 200 + 50); // 50-250的高度

            // 绘制填充椭圆
            g.fillOval(x, y, width, height);
        }
    }

    public static void main(String[] args) {
        // 在EDT中启动Swing程序
        SwingUtilities.invokeLater(RandomOvalDrawer::new);
    }
}

关键修改说明 #

1. 使用Swing Timer替代while循环：Timer在EDT中定时触发重绘，不会阻塞界面，同时可通过start()/stop()方法控制绘制启停。
2. 添加绘制状态开关isRunning：标记当前是否处于绘制状态，绘图面板根据该变量决定是否绘制新椭圆。
3. 修正字符串比较：用equals()方法比较按钮的动作命令。
4. 使用JPanel作为绘图容器：重写paintComponent方法（Swing推荐的绘图方式），避免直接操作JFrame的paint方法。
5. 实现随机颜色与填充椭圆：通过Color类生成随机RGB颜色，使用fillOval绘制填充椭圆。
6. 修正位置计算：基于绘图面板的宽高计算随机位置，确保椭圆在界面内。
7. 优化界面布局：使用BorderLayout替代绝对布局，让界面元素自适应窗口大小。

内容的提问来源于stack exchange，提问作者harsha lohana