java.lang.IllegalStateException: TimerTask 已被调度：Android 中 Timer 和 TimerTask 的合理使用

Question

我编写了一个连接到服务器并向他发送 ping 命令的应用程序，服务器用 pong 命令回答。

我想实现连接超时机制。我认为它会如下：

• 客户端发送 ping 并使用 timertask 和延迟启动计时器
• 当客户端收到 pong 时，timertask 被取消。

另外，我想优化内存。因此，不要在每次发送 ping 命令时重新创建 TimerTask。我尝试下面的代码：

private final Timer mSystemLogoutTimer = new Timer();
private final TimerTask mLogoutTask = new TimerTask() {

    @Override
    public void run() {
        mMessageInterface.onConnectionTimeout();
        cancel();
    }
};

private void ping() {
    sendRequest(RequestBuilder.formPing());
    mSystemLogoutTimer.schedule(mLogoutTask, CoreConst.PING_ANSWER_DELAY);
}

private void onPong() {
    mLogoutTask.cancel();
}

但在尝试第二次安排 TimerTask 时出现以下错误：

java.lang.IllegalStateException: TimerTask is scheduled already
at java.util.Timer.scheduleImpl(Timer.java:572)
at java.util.Timer.schedule(Timer.java:459)

我不明白，因为我在 TimerTask 上调用了cancel()。

请告诉我我做错了什么。 谢谢你的回答！

Answer 1

TimerTask.cancel() 不一定会阻止任务的执行。根据the SDK documentation，实际阻止执行时返回true，否则返回false。

看起来像这样，您的代码在第一次返回 true 时会发生这种情况，但不是第二次，导致在随后调用 Timer.schedule() 时抛出 IllegalStateException。

您应该检查TimerTask.cancel() 的返回码，并在返回false 时重新创建您的TimerTask：TimerTask 已被烧毁，无法在该阶段重复使用。