28
我需要通过字符串锁定一段代码。当然下面的代码是不可靠的：按字符串锁定。这是安全/理智吗？
lock("http://someurl")
{
//bla
}
所以我一直在做一个替代方案。我通常不是一个在这里发布大量代码的人，但是当谈到并发编程时，我对自己的同步方案有点担心，所以我提交了我的代码，询问是否理智这样或者是否有更直接的方法。
public class StringLock
{
private readonly Dictionary<string, LockObject> keyLocks = new Dictionary<string, LockObject>();
private readonly object keyLocksLock = new object();
public void LockOperation(string url, Action action)
{
LockObject obj;
lock (keyLocksLock)
{
if (!keyLocks.TryGetValue(url,
out obj))
{
keyLocks[url] = obj = new LockObject();
}
obj.Withdraw();
}
Monitor.Enter(obj);
try
{
action();
}
finally
{
lock (keyLocksLock)
{
if (obj.Return())
{
keyLocks.Remove(url);
}
Monitor.Exit(obj);
}
}
}
private class LockObject
{
private int leaseCount;
public void Withdraw()
{
Interlocked.Increment(ref leaseCount);
}
public bool Return()
{
return Interlocked.Decrement(ref leaseCount) == 0;
}
}
}
我会用这样的：
StringLock.LockOperation("http://someurl",()=>{
//bla
});
好到哪里去，或和好如初？
编辑
对于后代，这是我的工作代码。感谢所有的建议：
public class StringLock
{
private readonly Dictionary<string, LockObject> keyLocks = new Dictionary<string, LockObject>();
private readonly object keyLocksLock = new object();
public IDisposable AcquireLock(string key)
{
LockObject obj;
lock (keyLocksLock)
{
if (!keyLocks.TryGetValue(key,
out obj))
{
keyLocks[key] = obj = new LockObject(key);
}
obj.Withdraw();
}
Monitor.Enter(obj);
return new DisposableToken(this,
obj);
}
private void ReturnLock(DisposableToken disposableLock)
{
var obj = disposableLock.LockObject;
lock (keyLocksLock)
{
if (obj.Return())
{
keyLocks.Remove(obj.Key);
}
Monitor.Exit(obj);
}
}
private class DisposableToken : IDisposable
{
private readonly LockObject lockObject;
private readonly StringLock stringLock;
private bool disposed;
public DisposableToken(StringLock stringLock, LockObject lockObject)
{
this.stringLock = stringLock;
this.lockObject = lockObject;
}
public LockObject LockObject
{
get
{
return lockObject;
}
}
public void Dispose()
{
Dispose(true);
GC.SuppressFinalize(this);
}
~DisposableToken()
{
Dispose(false);
}
private void Dispose(bool disposing)
{
if (disposing && !disposed)
{
stringLock.ReturnLock(this);
disposed = true;
}
}
}
private class LockObject
{
private readonly string key;
private int leaseCount;
public LockObject(string key)
{
this.key = key;
}
public string Key
{
get
{
return key;
}
}
public void Withdraw()
{
Interlocked.Increment(ref leaseCount);
}
public bool Return()
{
return Interlocked.Decrement(ref leaseCount) == 0;
}
}
}
使用方法如下：
var stringLock=new StringLock();
//...
using(stringLock.AcquireLock(someKey))
{
//bla
}
它看起来过于专注于我，但很难说不知道它应该解决什么问题。这只是为了避免锁定在一个字符串？ – LukeH 2010-11-19 12:27:51
@LukeH，我试图写一个图像缓存多个客户端（超过100左右）可能同时请求图像。第一次命中将请求来自另一台服务器的映像，并且我希望缓存中的所有其他请求都被阻止，直到此操作完成，以便它们可以检索缓存的版本。在我看来，这将需要锁定这种性质，但如果有其他选择，我全都是耳朵。 – spender 2010-11-19 12:36:17
@Ani，是的，我知道这一点（正如我在文章中所述），如果我认为这种方法是一个游戏者，我会支撑防御。 – spender 2010-11-19 12:37:11