经验 C#手动同步的滥用实例

刚刚到新公司发现很多同学在用AutoResetEvent作线程同步,有的人用AutoResetEvent代替了C#的lock,

public void Func2()
{
Console.WriteLine("Func2 WaitOne");
autoResetEvent.WaitOne();
 Thread.Sleep(1000); //do Something
Console.WriteLine("Func2 Set");
autoResetEvent.Set();
}

这种写法我还是真是第一次见到。所以不是很确定,结果我研究了一下果然杯具了,重入后直接死锁了。

示例:

class Program
{


static void Main(string[] args)
{
var p = new Program();

p.Func1();/这样会死锁

} AutoResetEvent autoResetEvent
= new AutoResetEvent(true); public void Func1() { Console.WriteLine("Func1 WaitOne"); autoResetEvent.WaitOne(); Thread.Sleep(1000); Func2(); Console.WriteLine("Func1 Set"); autoResetEvent.Set(); } public void Func2() { Console.WriteLine("Func2 WaitOne"); autoResetEvent.WaitOne(); Thread.Sleep(1000); Console.WriteLine("Func2 Set"); autoResetEvent.Set(); } }

正确的写法还是用lock或者直接Monitor(try-finally暂时省略),看来这周需要花大功夫整改了。

private readonly object lockObj = new object();

        public void Func3()
        {
            Console.WriteLine("Func3 WaitOne");
            Monitor.Enter(lockObj);
            Thread.Sleep(1000);
            Func4();
            Console.WriteLine("Func3 Set");
            Monitor.Exit(lockObj);
        }

        public void Func4()
        {
            Console.WriteLine("Func4 WaitOne");
            Monitor.Enter(lockObj);
            Thread.Sleep(1000);
            Console.WriteLine("Func4 Set");
            Monitor.Exit(lockObj);
        }

 

posted @ 2012-04-18 23:17  kiminozo  阅读(403)  评论(0编辑  收藏  举报