Java TimerTask调用cancel()方法后无法停止的问题排查

问题重现 #

启动按钮触发Timer运行，点击暂停按钮控制台能打印pressed pause，但Timer持续计数无法停止。相关代码如下：

启动逻辑代码：

Timer timer = new Timer();
TimerTask task = new TimerTask() {
    public void run() {
        intCounter.incrementCount();
        System.out.println(intCounter.counterNum);
    };
};

暂停按钮代码：

if (e.getSource() == pauseButton) {
    System.out.println("pressed pause");
    task.cancel();
    timer.cancel();
}

尝试自定义TimerTask的cancel方法后问题依旧：

Timer timer = new Timer();
TimerTask task = new TimerTask() {
    private volatile Thread thread;

    public void run() {
        thread = Thread.currentThread();
        intCounter.incrementCount();
        System.out.println(intCounter.counterNum);
    }

    public boolean cancel() {
        Thread thread = this.thread;
        if (thread != null) {
            thread.interrupt();
        }
        return super.cancel();
    }
};

核心原因 #

问题出在Timer和TimerTask的引用失效：

• 如果每次点击启动按钮时都在方法内部创建Timer和TimerTask的局部实例，暂停按钮中调用的task和timer引用的是旧的实例，而非当前正在运行的Timer对象，导致cancel()调用对实际运行的Timer完全无效。
• 你修改的自定义cancel方法属于冗余操作，Java原生的TimerTask.cancel()已经能取消当前任务，Timer.cancel()也会终止整个Timer线程池，核心问题不在任务取消逻辑，而在于引用的正确性。

解决方案 #

将Timer和TimerTask声明为类成员变量，确保启动和暂停操作的是同一个实例：

1. 类成员变量声明：

private Timer timer;
private TimerTask task;

2. 启动按钮逻辑：

if (e.getSource() == startButton) {
    // 先停止已有运行的Timer，避免重复启动
    if (timer != null) {
        task.cancel();
        timer.cancel();
    }
    // 创建新的任务和Timer
    task = new TimerTask() {
        @Override
        public void run() {
            intCounter.incrementCount();
            System.out.println(intCounter.counterNum);
        }
    };
    timer = new Timer();
    // 示例：以1秒为间隔重复执行任务
    timer.scheduleAtFixedRate(task, 0, 1000);
}

3. 暂停按钮逻辑：

if (e.getSource() == pauseButton) {
    System.out.println("pressed pause");
    if (timer != null) {
        task.cancel();
        timer.cancel();
        // 置空引用，方便下次启动重新创建实例
        timer = null;
        task = null;
    }
}

额外说明 #

• Timer.cancel()会终止Timer的线程并丢弃所有已安排的任务，无需额外手动中断线程。
• 若需要更灵活的定时任务控制，推荐使用ScheduledExecutorService替代Timer，它支持多线程任务调度，API也更健壮。

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