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

        }

 

你可能感兴趣的:(C#)