Java条件语句实现鼠标控制自动点击器启停逻辑异常问题

• 你把点击执行逻辑直接写在了setHeld状态修改方法里，这个方法只会在状态变更的瞬间执行一次，执行完就退出，根本不会在鼠标按住的整个时间段里持续触发点击。
• 内层嵌套的if(held != true)是空代码块，没有任何停止逻辑，完全是无效代码。
• 没有做多线程状态同步：如果后续加循环逻辑，不加可见性修饰的话，线程读不到最新的held值，还是会出现点击停不下来的问题。
• 类名取Random会和JDK自带的java.util.Random类重名，极易引发类加载冲突；且每次调用方法都新建Robot实例属于不必要的资源开销。

核心思路是把状态修改和点击执行逻辑解耦：

1. 鼠标按下、抬起的监听器只负责修改held状态值，不掺杂任何点击执行代码。
2. 单独开一个守护线程，死循环检测held的实时值：值为true就执行点击+间隔，值为false就短休眠避免CPU空转。
3. held字段加volatile修饰，保证多线程下的状态可见性，避免线程读不到最新状态。
4. Robot实例在类构造阶段一次性初始化，不用反复新建。

修正后的可运行代码：

import java.awt.*;
import java.awt.event.MouseEvent;

public class AutoClicker {
    // volatile保证多线程下状态修改对所有线程可见
    private volatile boolean held;
    private final Robot robot;
    // 自动点击间隔，单位毫秒，可按需调整
    private static final int CLICK_DELAY = 100;

    public AutoClicker() throws AWTException {
        this.held = false;
        this.robot = new Robot();
        // 初始化时启动后台点击线程
        initClickWorkThread();
    }

    private void initClickWorkThread() {
        Thread clickWorker = new Thread(() -> {
            while (!Thread.currentThread().isInterrupted()) {
                if (held) {
                    // 执行一次左键点击
                    robot.mousePress(MouseEvent.BUTTON1_DOWN_MASK);
                    robot.delay(20);
                    robot.mouseRelease(MouseEvent.BUTTON1_DOWN_MASK);
                    robot.delay(CLICK_DELAY);
                } else {
                    // 未激活状态下短休眠，避免CPU空转占满资源
                    robot.delay(10);
                }
            }
        });
        // 设为守护线程，主程序退出时线程自动终止，避免残留进程
        clickWorker.setDaemon(true);
        clickWorker.start();
    }

    // 鼠标事件监听器只需要调用这个方法更新状态即可
    public void setHeld(boolean held) {
        this.held = held;
    }

    // 测试入口
    public static void main(String[] args) throws AWTException, InterruptedException {
        AutoClicker autoClicker = new AutoClicker();
        // 模拟鼠标按下，启动自动点击
        autoClicker.setHeld(true);
        // 保持点击3秒
        Thread.sleep(3000);
        // 模拟鼠标抬起，停止自动点击
        autoClicker.setHeld(false);
    }
}

实际使用时，不需要修改类内部逻辑，只需要在自己的全局鼠标监听器里做两个绑定：

• 监听到mousePressed（鼠标按下）事件时，调用autoClicker.setHeld(true)
• 监听到mouseReleased（鼠标抬起）事件时，调用autoClicker.setHeld(false)

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