Hashtable to Dictionary<> syncroot .

Question

Hashtables have a syncroot property but generic dictionaries don't. If I have code that does this:

lock (hashtable.Syncroot)
{
....
}

How do I replicate this if I am removing the hashtable and changing to generic dictionaries?

Answer 1

If you are going strictly for compatability then Bryan is correct. This is the best way to maintain your current semantics on top of a Dictionary.

Expanding on it though. The reason the SyncRoot property was not directly added to the generic dictionary is that it's a dangerous way to do synchronization. It's only slighly better than "lock(this)" which is very dangerous and prone to deadlocks. Here are a couple of links that speak to why this is bad.

• http://blogs.msdn.com/bclteam/archive/2005/03/15/396399.aspx
• http://blogs.msdn.com/ricom/archive/2006/05/01/587750.aspx

Answer 2

The new thinking behind SyncRoot is that it was a mistake in the original design. If the only thing to lock is the dictionary and it's private, you can lock it or another object that serves as the synchronization object. The latter technique is useful when the state you are protecting is more than just the dictionary.

// used as you would have used SyncRoot before
object _syncLock = new object();
Dictionary<string, int> numberMapper = new Dictionary<string, int>();

// in some method...
lock (_syncLock)
{
    // use the dictionary here.
}

Answer 3

var dictionary = new Dictionary<int, string>();

lock(((ICollection) dictionary).SyncRoot)
{
    // ...
}

Answer 4

If the hashtable/dictionary isn't public, you could just lock the dictionary object itself.