由于异常，代码在线程池中执行了无限次

Question

我听说在 ASP.NET 中使用 ThreadPool 不好，但是我使用它是为了自我教育。我的目标是确定 Application_Error 事件是否被触发（在 Global.asax 中处理） - 我的答案是：不，它不会被触发。

但我观察到了一些奇怪的事情。我编写的线程只是将任务排队到线程池中。该任务旨在随机抛出错误。但我观察到我经常收到错误 - 数量超过了数量。我已经多次将任务排队了。我另一个担心的是，甚至 System.Diagnostics.Trace.WriteLine() 也没有将消息记录到我的输出窗口（Visual Studio）。为什么会有这种奇怪的行为？

using System;
using System.Threading;
namespace ThreadPoolDemo.Web
{
    public partial class _default : System.Web.UI.Page
    {
        protected void Page_Load(object sender, EventArgs e)
        {

        }

        protected void Button1_Click(object sender, EventArgs e)
        {
            Thread t = new Thread(createthreads);
            t.IsBackground = true;
            t.Start();
        }

        void createthreads()
        {
            Thread.Sleep(10 * 1000);
            int i;
            System.Diagnostics.Trace.WriteLine("Queueing items");
            for (i = 0; i < 1; i++)
                ThreadPool.QueueUserWorkItem(new WaitCallback(ErrorTask), null);
            System.Diagnostics.Trace.WriteLine("End Queueing items");
        }

        void ErrorTask(object obj)
        {
            Random generator = new Random();
            int value = generator.Next(1);            
            if (value == 0)
                throw new Exception("Sample exception thrown");
            else
                System.Diagnostics.Trace.WriteLine("Processed thread");
        }

    }
}

Answer 1

您的 ErrorTask 有两个问题。第一个是每次调用该方法时都会初始化一个新的 Random 实例。默认的 Random 构造函数使用来自 Environment.TickCount 的值来生成随机数生成器，这对于连续线程来说可能是相同的。因此，您将获得多个线程的相同随机序列。

不过，更大的问题是 generator.Next(1) 将总是返回 0。Random.Next(int max) 生成一个随机数数字 N，使得 0 <= N 最大。因此，您的 ErrorTask 将为每个线程抛出异常。

我不知道每次调用 ErrorTask 时如何或为何会多次抛出该异常。这似乎不可能。

我建议进行以下修改：

private Random generator = new Random();

void ErrorTask(object obj)
{
    int value;
    lock (generator)
    {
        value = generator.Next(2);
    }
    if (value == 0)
        throw new Exception("Sample exception thrown");
    else
        System.Diagnostics.Trace.WriteLine("Processed thread");
}

generator 现在具有类范围，并且仅初始化一次。 锁的作用是防止多个线程尝试同时生成一个数字。如果没有锁，随机数生成器可能会被损坏，并且每次调用都会开始返回 0。我将 generator.Next 参数更改为 2，这样就可以获得数字 0 和 1。