为什么我在TimerTask中出现此错误?

时间:2013-08-17 18:36:54

标签: java android multithreading timertask

08-17 18:26:44.540: E/AndroidRuntime(2405): FATAL EXCEPTION: Scheduler
08-17 18:26:44.540: E/AndroidRuntime(2405): java.lang.IllegalThreadStateException: Thread already started.
08-17 18:26:44.540: E/AndroidRuntime(2405):     at java.lang.Thread.start(Thread.java:1045)
08-17 18:26:44.540: E/AndroidRuntime(2405):     at com.example.simpledownloader.scheduler.Scheduler.run(Scheduler.java:21)
08-17 18:26:44.540: E/AndroidRuntime(2405):     at java.util.Timer$TimerImpl.run(Timer.java:284)  

所以,我有一个TimerTask,它通过暴力强制检查所有待处理的下载 下载全部Thread存储在Vector中。只要TimerTask看到线程准备就绪,就会在其上调用start()。这是代码:

代码:

package com.example.simpledownloader.scheduler;

import java.util.TimerTask;

import android.util.Log;

import com.example.simpledownloader.sharable.Sharable;
import com.example.simpledownloader.task.Task;

public class Scheduler extends TimerTask {
//------------------------------------------------------------------------------
    @Override
    public void run() {
        if(Sharable.getShouldLook()){ // Should we look for pending tasks ?
            for(Task each: Sharable.downloads){ // Yes, then loop
                if(each.isAlive()==false){ // Thread is not running ?
                    Log.v("SCHEDULER", "NOT RUNNING");
                    if(each.getReadiness() == true){ // Is it ready ?
                        each.start(); // Start !
                        break; // stop looping
                    }
                }
            }
        }
    }
//------------------------------------------------------------------------------
}  

为什么我一直收到这个错误?
Timer设置如下:

Sharable.schedulerTimer.scheduleAtFixedRate(Sharable.schedulerTask, 1000, 1000); // start scheduler

1 个答案:

答案 0 :(得分:1)

如果isAlive()返回false,则在启动线程之前进行检查。有两种方法可能无法阻止您获取IllegalThreadStateException

  1. 其他一些主题是在致电isAlive()和致电start()

  2. 之间启动下载主题
  3. 下载主题正在终止,但未从Sharable.downloads集合中删除。

  4. 我怀疑后者更有可能,所以这就是我开始寻找的地方。或者,如果您希望在完成后将线程保留在下载集合中,则可能需要尝试each.getState() == Thread.State.NEW而不是each.isAlive() == false