C# во время использования цикла

Question

У меня довольно общий вопрос с циклом C# while.

Этот код должен продолжать выполняться только после того, как сеанс RDP действительно отключен.

Когда свойство Connected изменено на 0, это означает, что соединение сеанса RDP действительно завершено. Когда свойство равно 1, оно все еще подключено и соединение еще не завершено.

Кто-нибудь видит что-то внутренне плохое в отношении этого кода? Есть ли лучший способ сделать это?

private void Reconnect() 
{ 
    rdp1.Disconnect(); // force the RDP session to disconnect 
    while (rdp1.Connected == 1) // true as long as RDP is still connected 
    { 
     // do nothing 
    } 
    rdp1.Connect(); // execute this code after while loop is broken 
} 

/****************************************** ********************/

Вот последний код, который я использовал для ответа Джеймса. Счетчик достаточен в качестве таймаута для моей цели.

  int i = 0; 
      rdp1.Disconnect(); 
      while (rdp1.Connected == 1) 
      { 
       if (i == 1000 * 10) break; 
       else Thread.Sleep(100); 
       i++; 
      } 
      rdp1.Connect(); 

Answer 1

Вы должны сделать что-то в теле цикла, или он будет потреблять весь ваш процессор (по крайней мере для одного ядра). Обычно в этом типе цикла вы некоторое время будете спать, используя System.Threading.Thread.Sleep(100) или что-то в этом роде. Sleep берет количество миллисекунд, чтобы подождать, прежде чем снова проверить условие while. В идеале, объект RDP имел бы мьютекс или событие или что-то, что вы могли бы просто заблокировать до тех пор, пока оно не будет отключено, но меня это не удивило бы, если бы они это оставили.

РЕДАКТИРОВКА: Как указал Бен, всегда хорошо иметь выход из петли. Что-то вроде этого (вашего указанного ответа будет зависеть от скорости процессора, которые могли бы нарушить в будущем, когда процессоры гораздо быстрее):

DateTime stop = DateTime.UtcNow.AddSeconds(30); 
while (rdp1.Connected) 
{ 
    if (DateTime.UtcNow > stop) throw new ApplicationException ("RDP disconnect timeout!"); 
    System.Threading.Thread.Sleep (100); 
} 

Конечно, вы, вероятно, хотите, чтобы указать таймаут с константой, только для чтения TimeSpan или динамически настраиваемый TimeSpan, а не волшебное число, и вы должны, вероятно, иметь конкретный класс исключений для этого случая.

Answer 2

Установите тайм-аут для целей

private void Reconnect() 
{ 
    timeOut = false; 
    new System.Threading.Thread(new System.Threading.ThreadStart(setTimeout)).Start(); 
    rdp1.Disconnect(); 
    while (rdp1.Connected == 1 && !timeOut); 
    rdp1.Connect(); 
} 

bool timeOut = false; 

void setTimeout() 
{ 
    System.Threading.Thread.Sleep(7000); 
    timeOut = true; 
}