Creating Dynamic Locks at Runtime in ASP.NET

Question

Are the following assumptions valid for this code? I put some background info under the code, but I don\'t think it\'s relevant.

Assumption 1: Since th

Answer 1

Your current solution looks pretty good. The two things I would change:

1: UnlockOnValue needs to go in a finally block. If an exception is thrown, it will never release its lock.

2: LockOnValue is somewhat inefficient, since it does a dictionary lookup twice. This isn't a big deal for a small dictionary, but for a larger one you will want to switch to TryGetValue.

Also, your assumption 3 holds - at least for now. But the Dictionary contract makes no guarantee that the Keys property always returns the same object. And since it's so easy to not rely on this, I'd recommend against it. Whenever I need an object to lock on, I just create an object for that sole purpose. Something like:

private static Object _lock = new Object();