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