Java GUI自动鼠标移动程序启停功能故障排查求助

你的代码存在几个核心问题导致功能失效：

• 事件调度线程（EDT）阻塞：automaticMouseMoverStart在EDT中执行（按钮点击事件由EDT处理），方法内部的while(true)循环和Thread.sleep会完全卡住GUI线程，导致Stop按钮状态永远无法更新，程序失去响应。
• 赋值而非判断：while(isPressed = false)是赋值操作（将false赋值给isPressed），而非条件判断，这会导致循环逻辑完全错误。
• Stop按钮无监听器：未给Stop按钮添加状态变化监听器，且仅在Start按钮点击时获取一次Stop按钮状态，后续状态变化无法被感知。
• 资源重复创建：每次循环都创建新的Robot和Random实例，属于不必要的资源浪费。

以下是修复后的完整代码，解决了上述所有问题：

package Übung;

import java.awt.BorderLayout;
import java.awt.Robot;
import java.util.Random;
import javax.swing.BorderFactory;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JToggleButton;

public class GUIMouseMover extends JFrame {
    private JButton btnStart;
    private JToggleButton btnStop;
    private static final int MOVE_INTERVAL = 5000;
    private static final int MAX_X = 400;
    private static final int MAX_Y = 400;
    // volatile保证多线程下状态可见性
    private volatile boolean isRunning = false;
    private Thread moverThread;
    private Robot robot;
    private Random random;

    public static void main(String[] args) {
        java.awt.EventQueue.invokeLater(GUIMouseMover::new);
    }

    public GUIMouseMover() {
        buildAndShowGUI();
        try {
            robot = new Robot();
            random = new Random();
        } catch (Exception e) {
            e.printStackTrace();
        }
    }

    private void buildAndShowGUI() {
        setTitle("Mouse-Mover");
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        JPanel panel = new JPanel(new BorderLayout());
        panel.setBorder(BorderFactory.createEmptyBorder(5, 5, 5, 5));

        btnStart = new JButton("Start");
        btnStart.addActionListener(e -> startMouseMover());

        btnStop = new JToggleButton("Stop");
        btnStop.addActionListener(e -> toggleStop());

        panel.add(btnStart, BorderLayout.NORTH);
        panel.add(btnStop, BorderLayout.SOUTH);

        add(panel);
        setResizable(false);
        pack();
        setLocationRelativeTo(null);
        setVisible(true);
    }

    private void startMouseMover() {
        if (isRunning) return; // 防止重复启动多个线程
        isRunning = true;
        btnStop.setSelected(false); // 重置Stop按钮状态
        moverThread = new Thread(() -> {
            while (isRunning) {
                robot.mouseMove(random.nextInt(MAX_X), random.nextInt(MAX_Y));
                try {
                    Thread.sleep(MOVE_INTERVAL);
                } catch (InterruptedException e) {
                    // 线程被中断时退出循环
                    Thread.currentThread().interrupt();
                    break;
                }
            }
            isRunning = false;
            btnStop.setSelected(false);
        });
        moverThread.start();
    }

    private void toggleStop() {
        isRunning = !btnStop.isSelected();
        if (!isRunning && moverThread != null) {
            moverThread.interrupt(); // 中断后台线程，快速停止任务
        }
    }
}

• 后台线程执行任务：鼠标移动逻辑放在单独的后台线程中，避免阻塞EDT，保证GUI始终响应。
• volatile控制变量：isRunning用volatile修饰，确保多线程下状态的可见性，Stop按钮点击时能及时更新运行状态。
• Stop按钮监听器：给Stop按钮添加点击监听器，实时切换运行状态，并通过中断线程快速停止任务。
• 资源预初始化：Robot和Random在构造方法中初始化一次，避免重复创建。
• 避免重复启动：startMouseMover方法中先判断是否已在运行，防止多次点击Start创建多个线程。
• 面向对象重构：去掉不必要的静态成员，使用实例变量和方法，符合Java面向对象规范。

• 添加状态提示标签，显示当前程序运行状态，让用户更直观了解情况。
• 允许用户自定义移动间隔和坐标范围，替代固定常量。
• 优化异常处理，当Robot创建失败时（如权限不足），给用户弹出友好提示，而非仅打印堆栈信息。

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