Монитор Ждите кеширования, ... это хорошая практика?

Question

Я только что написал код, а затем я узнал, что есть некоторые проблемы с monitor.wait, заставляя меня выполнять операцию внутри замков, я хотел бы сейчас, если это хороший способ сохранить поток ожидания, ....

Я не уверен, что thread.join выполнит эту работу, так как в моем приложении много потоков, и каждая из них выполняет определенную работу, которую они могут завершить в течение времени ...

вот мой код:

public static class TaskManager 
{ 
    private static readonly object UpdateLock = new object(); 
    private static readonly object WaitLock = new object(); 

    private static readonly LiaisonDb _db = new LiaisonDb(); 
    private static List<liaQueue> _liaQueueList = new List<liaQueue>(); 
    private static DateTime _lastUpdate = new DateTime(); 

    public static liaQueue GetTask(string sessionType) 
    { 
     liaQueue task; 
     lock (UpdateLock) 
     { 
      if (_lastUpdate < DateTime.Now.AddSeconds(-5)) 
      { 
       Thread t = new Thread(UpdateCache) {IsBackground = true}; 
       t.Start(); 
       lock (WaitLock) 
       { 
        Monitor.Wait(WaitLock); 
       } 

       _lastUpdate = DateTime.Now; 
      } 
      task = _liaQueueList 
       .FirstOrDefault(w => w.Stat == 0 
            && w.Type != null 
            || string.Equals(w.Type, sessionType)); 
     } 
     return task; 
    } 

    private static void UpdateCache() 
    { 
     try 
     { 
      _liaQueueList = _db.liaQueue.Where(w => w.Stat == 0).ToList(); 
     } 
     finally 
     { 
      lock (WaitLock) 
      { 
       Monitor.Pulse(WaitLock); 
      } 
     } 
    } 
} 

Как вы видите, я положил два замка, и один из них только для monitor.wait, держать нить ждет ответа ...

я думаю, что я также должен возвращает нуль, а кэш становится обновилась? ...

Answer 1

От MSDN

Если два потока с помощью Pulse и подождите чтобы взаимодействовать, это может привести к тупиковой ситуации.

Итак, нет. Ваша реализация не будет лучшей практикой.

Мне кажется, что GetTask должен обновлять кеш в фоновом потоке, а затем блокировать вызывающий поток до тех пор, пока кеш не будет обновлен, а затем вернет первую задачу в соответствии с критерием выбора.

Поскольку вызывающий поток будет блокировать (ждать) для обновления кеша, я не совсем понимаю, как использовать фоновый поток в первую очередь. Если целью является предотвращение одновременного обновления нескольких потоков вызовов для параллельной работы кеша, используйте только инструкцию lock (UpdateLock).

Если вы хотите запустить кеш в фоновом потоке в любом случае (и дождаться его), попробуйте вместо этого использовать библиотеку задач. Но на самом деле я не знаю.

lock (UpdateLock) 
{ 
    if (_lastUpdate < DateTime.Now.AddSeconds(-5)) { 
    Task.Run(() => { 
     _liaQueueList = _db.liaQueue.Where(w => w.Stat == 0).ToList(); 
    }).Wait(); 

    _lastUpdate = DateTime.Now; 
    } 
} 

return _liaQueueList.FirstOrDefault(w => w.Stat == 0 && w.Type != null || string.Equals(w.Type, sessionType));