经验 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);
        }