Что не так с этим кодом блокировки?

Question

static void Main(string[] args) 
    { 
      Test c = new Test(); 
      Thread oThread = new Thread(new ThreadStart(c.Lock)); 
      oThread.Start(); 

      Thread oThread2 = new Thread(new ThreadStart(c.AfterLock)); 
      oThread2.Start(); 

      Console.ReadLine(); 

    } 



public class Test 
{ 
    public Dictionary<string, string> dic = new Dictionary<string, string>(); 

    public void Lock() 
    { 
     lock (((IDictionary)dic).SyncRoot) 
     { 
      for (var i = 3; i < 200; i++) 
      { 
       Console.WriteLine(i.ToString()); 
       dic.Add(i.ToString(), i.ToString()); 
      } 
     } 

    } 
    public void AfterLock() 
    {    
     Console.WriteLine(dic["100"]); 
    } 
} 

AfterLock бросает исключение: Данный ключ не присутствовал в словаре

выпо был заблокирован первым потоком? почему afterlock не дождался блокировки первых потоков?

Answer 1

1. Вам нужно заблокировать вокруг доступа к объекту как при чтении данных, а также при записи данных. Синхронизация только записи небезопасны.

2. Ничего не мешает вашему методу AfterLock от фактического запуска после вашего метода Lock. Он может так же легко запускаться до него, либо, поскольку вы неправильно синхронизируете чтение, они могут даже работать параллельно. Вам нужно добавить механизм синхронизации, чтобы предотвратить запуск этого метода после записи, если его выполнение зависит от первой записи, как это явно делается здесь.

Answer 2

Вы должны заблокировать выделенный объект, а не тот, который вы пытаетесь использовать.

class Example 
{ 
    private object cacheLock; 

    public static void Load() 
    { 
     // . . . 

     lock (cacheLock) 
     { 
      CacheTable = (ThreadSafeDictionary<String, 
       ThreadSafeDictionary<Object, Object>>)CacheTableTmp.Clone(); 
     } 
    }