Question

尝试重新启动线程时，我不时会收到System.Threading.ThreadStateException。有问题的代码如下：

// Make sure the thread is done stopping
while (this.mThread.ThreadState == ThreadState.Running)
{ 
    Thread.Sleep(0);
}
// Respawn a thread if the current one is stopped or doesn't exist
if (this.mThread == null || this.mThread.ThreadState == ThreadState.Stopped)
{ 
    this.mThread = new Thread(new ParameterizedThreadStart(Monitor)); }
// Start the thread
if (check)
{ 
    this.mThread.Start(60000); 
}
else
{   
    this.mThread.Start(0); 
}

所以有两个问题 - 这是正确的做事方式，而且，是否有办法防止错误发生？

Answer 1

一个线程可能同时处于多个状态，因此ThreadState属性实际上是可能状态的位图。因此，只用一个状态来测试相等性就不会给你正确的结果。您需要执行以下操作：

if((mThread.ThreadState & ThreadState.Running) != 0)

但是，检查线程状态是不对的。我并不完全清楚你想要实现的目标，但我猜你正在等待一个线程在重新启动之前终止。在这种情况下，你应该这样做：

mThread.Join();
mThread = new Thread(new ParameterizedThreadStart(Monitor));
if(check)
    mThread.Start(60000);
else
    mThread.Start(0);

虽然如果你描述了你想要更详细解决的问题，我几乎可以肯定会有更好的解决方案。等待一个线程结束只是为了重新启动它对我来说似乎并不高效。也许你只需要某种线程间的沟通？

约翰。

Answer 2

问题是你有代码首先检查它是否应该创建一个新的线程对象，以及另一段代码确定是否启动线程对象。由于竞争条件和类似的事情，您的代码可能最终尝试在现有线程对象上调用.Start。考虑到您没有在 check 变量后面发布详细信息，我们无法知道可能会触发此行为的原因。

您应该重新组织代码，以确保只在新对象上调用.Start。简而言之，您应该将Start方法放入与创建新线程对象的if语句相同的if语句中。

就个人而言，我会尝试重新组织整个代码，这样我就不需要创建另一个线程了，而是将代码包装在一个循环中的线程对象中，以便线程继续运行。

Answer 3

抛出ThreadStateException是因为您尝试启动一个不处于可启动状态的线程。最可能的情况是它已经在运行，或者已经完全退出。

可能会发生一些事情。首先，线程可能已从Running转换为StopRequested，但尚未完全停止，因此您的逻辑不会创建新线程，并且您正在尝试启动刚刚运行或即将运行的线程完成运行（两者都不是重启的有效状态）。

另一种可能性是线程被中止。中止的线程进入Aborted状态，而不是Stopped状态，当然也无法重启。

真的，唯一可以“重启”的活着的线程是暂停的。您可能希望使用此条件：

if (this.mThread == null || this.mThread.ThreadState != ThreadState.Suspended)

尝试重新启动线程时发生ThreadStateException

3 个答案: