ManualResetEvent.WaitOne блокирует все нити

Question

я следующий код

ThreadPool.QueueUserWorkItem(new WaitCallback(DownloadAsync), apiMethod); 
downloadHandle.WaitOne(); 

Где DownloadAsync является

private void DownloadAsync(object _uri) 
     { 
      var url = _uri as string; 
      WebClient client = new WebClient(); 
      client.DownloadStringCompleted += new DownloadStringCompletedEventHandler(client_DownloadStringCompleted); 
      client.DownloadStringAsync(new Uri(GLOBALS.MAIN_API_URL + url, UriKind.Absolute)); 
     } 

void client_DownloadStringCompleted(object sender, DownloadStringCompletedEventArgs e) 
     { 
      result = e.Result; 
      downloadHandle.Set(); 
     } 

Так что моя проблема в том, что downloadHandle.Set() никогда не вызывается. Но я не понимаю, почему? Я создаю новый поток для DownloadAsync и downloadHandle.WaitOne() не должен его блокировать.

Мне нужно создать метод синхронизации вместо Async.

Спасибо!

UPD: Обновлен исходный код с вызовом Async.

Answer 1

client.DownloadString является синхронным методом, поэтому ваш обработчик никогда не будет вызван. Вы должны назвать асинхронную версию: client.DownloadStringAsync()

Вы можете узнать больше о DownloadStringAsync на msdn. Также разумно помещать код в блок try-catch и обрабатывать исключения, если вы полагаетесь на факт, что нужно вызвать некоторый код.

Ваш код может выглядеть следующим образом:

private void DownloadAsync(object _uri) 
{ 
    try 
    { 
     var url = _uri as string; 
     WebClient client = new WebClient(); 
     client.DownloadStringCompleted += new DownloadStringCompletedEventHandler(client_DownloadStringCompleted); 
     client.DownloadStringAsync(new Uri(GLOBALS.MAIN_API_URL + url, UriKind.Absolute)); 
    } 
    catch //appropriate exception 
    { 
     //Handle exception (maybe set downloadHandle or report an error) 
    } 
} 

void client_DownloadStringCompleted(object sender, DownloadStringCompletedEventArgs e) 
{ 
    result = e.Result; 
    downloadHandle.Set(); 
} 

Answer 2

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

Кстати, вам фактически не нужно использовать пул потоков здесь - вы можете вызвать DownloadAsync() в своем основном потоке, потому что он не блокируется.

Answer 3

Когда вы звоните downloadHandle.WaitOne(); от UI, вы блокируете UI thread, поэтому ThreadPool никогда не будет звонить. Переместить в фоне этого:

ThreadPool.QueueUserWorkItem(new WaitCallback(DownloadAsync), apiMethod); 
downloadHandle.WaitOne(); 

Answer 4

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

Почему это проблема?

Достаточно, DownloadAsync выполнен в фоновом режиме, благодаря threadpool. Тем не менее, класс WebClient всегда выполняет свой обратный вызов (то есть ваш метод client_DownloadStringCompleted) с использованием потока пользовательского интерфейса ... Но этот же поток блокируется вашим downloadHandle.WaitOne()!

Именно поэтому, когда вы накладываете таймаут на свой замок, метод client_DownloadStringCompleted получает магический кадр.

Как это исправить? Два решения.

1/Прекратить выполнение downloadHandle.WaitOne() в основной теме. Он блокирует пользовательский интерфейс, и ваше приложение перестает отвечать на запросы, то есть никогда хорошая вещь.

2/Использовать HttpWebRequest вместо WebClient.HttpWebRequest выполняет обратный вызов в том же потоке, который начал загрузку, поэтому у вас не будет этой проблемы с блокировкой.