以下代码都是asp.net,我能预见到,如果多个客户端并发访问下面这个页面,asp.net从线程池中分配一个空闲的线程去服务这个请求。也就是可能存在多个线程并发访问log.txt文件。
- C# code
protected void Page_Load(object sender, EventArgs e) { string path = Server.MapPath("~/log.txt"); for (int i = 0; i < 100000; i++) { System.IO.File.AppendAllText(path, i.ToString()); } }
所以,我想那当然的加上了lock
- C# code
object locker = new object(); protected void Page_Load(object sender, EventArgs e) { string path = Server.MapPath("~/log.txt"); lock (locker) { for (int i = 0; i < 100000; i++) { System.IO.File.AppendAllText(path, i.ToString()); } } }
结果发现,出现文件被其它进程占用的IOException.
这时,我又想,ReaderWriterLock锁应该是解决此类问题的。所以又改成了这样的代码:
- C# code
protected void Page_Load(object sender, EventArgs e) { string path = Server.MapPath("~/log.txt"); System.Threading.ReaderWriterLock rwl = new System.Threading.ReaderWriterLock(); rwl.AcquireWriterLock(1000); for (int i = 0; i < 100000; i++) { System.IO.File.AppendAllText(path, i.ToString()); } rwl.ReleaseWriterLock(); }
结果,涛声依旧。。。我有几点不明白:
1\.net设计准则中说,可以确保所有静态方法是线程安全的,为什么这里却出现这种情况。
2、如果一定要采用这个静态方法,有什么办法可以确保多个线程访问不会出现以上问题。谢谢。
------解决方案--------------------------------------------------------
你应该先看下msdn 中关于lock 的部分
然后和自己程序比对下 就会找到原因。
------解决方案--------------------------------------------------------
static object locker = new object();
你这个锁应该是个静态的,才能保证唯一吧
------解决方案--------------------------------------------------------
Best practice is to define a private object to lock on, or a private static object variable to protect data common to all instances
------解决方案--------------------------------------------------------
顶着,LZ 精神可嘉
------解决方案--------------------------------------------------------
明显,你对于线程安全理解有问题.
15楼给解释了.