从计时器事件中止线程

时间:2016-07-07 13:54:28

标签: c# multithreading timer threadabortexception thread-abort

我有Timer,如果需要太多时间,则必须取消Thread

System.Timers.Timer timer_timers = new System.Timers.Timer();
Thread thread = new Thread(startJob);
thread.Name = "VICTIM_THREAD";

启动Thread方法时,我启动Timer并将当前Thread作为参数传递给事件。

public void startJob()
{
    Debug.WriteLine("Name: " + Thread.CurrentThread.Name);
    timer_timers.Elapsed += (sender, e) => T_Elapsed(sender, e, Thread.CurrentThread);
    timer_timers.Interval = 5000;

    // Start simulation process
    while (true)
    {
        Thread.Sleep(700);
        Debug.WriteLine("Thread: " + Thread.CurrentThread.Name + " ALIVE: " + thread.IsAlive);
    }            
}

计时器事件:

private void T_Elapsed(object sender, ElapsedEventArgs e, Thread currentThread)
{
    // EDIT: show the correct NAME! of the thread
    Debug.WriteLine("Name: " + currentThread.Name);

    System.Timers.Timer tim = sender as System.Timers.Timer;

    currentThread.Abort();  // <-- this line throws exception

    if (tim != null)
    {
        tim.Stop();
    }

}

Abort电话会引发异常:

  

&#39;无法评估表达式,因为代码已优化或本机框架位于调用堆栈顶部&#39;

并且线程仍然存在。 如果我在startJob()之前启动计时器并直接传递线程,它可以正常工作。

public void startThread()
{
    timer_timers.Elapsed += (sender, e) => T_Elapsed(sender, e, thread);
    timer_timers.Interval = 5000;

    timer_timers.Start();
    thread.Start();
}
public void startJob()
{
    // Start simulation process
    while (true)
    {
        Thread.Sleep(700);
        Debug.WriteLine("Thread: " + Thread.CurrentThread.Name + " ALIVE: " + thread.IsAlive);
    }            
}

问题:为什么Thread.CurrentThread版本不起作用?是因为我还必须中止计时器线程?我在这里缺少什么?

答案我发现这个例外,如thisthis来自不同的背景,并没有真正帮助我理解为什么。

编辑:我知道这是中止或取消线程的错误方法。它应该做的工作是打开一个SerialPort。但每次约200次,线程将永远不会返回,我需要杀死它,永远不要回避后果。模拟while循环可能是一个不好的例子。

2 个答案:

答案 0 :(得分:5)

如评论中所述,您不应该使用Abort。即使你这样做,这也是你使用它的方式的问题:

定时器不会在你的线程上运行。它们在线程池线程上运行。因此,lambda中使用的Thread.CurrentThread将成为该线程池线程。

如果你想中止创建定时器的线程,你应该做的是:在lambda之外的变量中捕获线程。

Thread myThread = Thread.CurrentThread;
timer_timers.Elapsed += (sender, e) => T_Elapsed(sender, e, myThread);

但是你应该找到另一种方法来更优雅地终止你的线程,或者重新编写你的代码而不需要显式线程。

答案 1 :(得分:3)

请勿拨打Thread.Abort。以下是如何正确地做到这一点:

var tokenSource = new CancellationTokenSource();
var token = tokenSource.Token;

var t = Task.Run(() =>
{
    while (!token.IsCancellationRequested)
    {
        Console.Write(".");
        Thread.Sleep(500);
    }
}, token);

var timer = new System.Timers.Timer();
timer.Interval = 5000;
timer.Elapsed += (s, e) => tokenSource.Cancel();
timer.Enabled = true;

您的代码似乎在第二种情况下工作的原因是您在调用T_Elapsed之前捕获了该线程。在第一种情况下,只有在调用时间Elapsed事件时才请求当前线程(此时它不是调用线程,它是被调用者)。