我是否需要在以下c#代码中使用锁定?

时间:2012-12-21 23:03:49

标签: c# multithreading thread-safety

在下面的代码中我使用两个线程来共享sane资源,在这个示例中它是queue所以我需要使用locken-queueing or dequeuing如果是,那么为什么因为程序似乎工作正常。

class Program
{   
    static Queue<string> sQ = new Queue<string>();

    static void Main(string[] args)
    {
        Thread prodThread = new Thread(ProduceData);
        Thread consumeThread = new Thread(ConsumeData);
        prodThread.Start();
        consumeThread.Start();
        Console.ReadLine();
    }

    private static void ProduceData()
    {
        for (int i = 0; i < 100; i++)
        {
            sQ.Enqueue(i.ToString());               
        }
    }

    private static void ConsumeData()
    {
        while (true)
        {
            if (sQ.Count > 0)
            {
                string s = sQ.Dequeue();
                Console.WriteLine("DEQUEUE::::" + s);
            }
        }
    }
}

1 个答案:

答案 0 :(得分:8)

是的,System.Collections.Generic.Queue<T>对于同时写入和读取不是线程安全的。您需要在enquing或dequing之前锁定同一对象,或者如果您使用的是.NET 4 / 4.5,请使用System.Collections.Concurrent.ConcurrentQueue<T>类并使用TryDequeue方法。

到目前为止您当前实施并未导致问题的原因是Thread.Sleep(500)调用(不是您应该在生产代码中使用的内容),这意味着prodThread没有在consumeThread读取队列时写入队列,因为读取操作的时间少于500毫秒。如果删除Thread.Sleep赔率,它会在某个时刻抛出异常。