2013-08-17 93 views
1
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()。这里是代码:为什么我的TimerTask中出现这个错误?

代码:

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 
+0

* Sharable *类包含所有在活动中共享的单例实例。 –

+0

线程死后不能再次启动。 – aim

+0

@aim你的意思是它被打断了吗? –

回答

1

启动线程,如果isAlive()返回false之前,您正在检查。有两种方法,这可能不足以阻止你获得IllegalThreadStateException

  1. 其他某些线程开始调用isAlive()和调用start()的下载线程,或

  2. 下载线程正在终止,不会从您的Sharable.downloads集合中删除。

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

+0

是的,我已经添加了一个方法到名为getIsReadyForDownload()的'Task',它是'synchronized' –

+0

同步方法是不够的 - 如果你有多个线程可能会启动你的下载线程,你必须确保所有可能启动它们的代码都在同一个对象上同步(最简单的就是下载线程本身),直到线程刚刚启动之后,它才检查'isReadyForDownload'或'isAlive'。 – Jules

相关问题