2

Решил пощупать приложение с окнами, и тут возникла проблема. Есть код:

Random rnd = new Random();
 for (int i = 0; i < 9; i++)
 {
    int value = rnd.Next(1, 6);
    switch (value)
    {
      case 1:
         label3.Text = "";
         break;
      case 2:
         label3.Text = "";
         break;
      case 3:
         label3.Text = "";
         break;
      case 4:
         label3.Text = "";
         break;
      case 5:
         label3.Text = "";
         break;

    }
    Thread.Sleep(2000);
    label4.Text = Convert.ToString(i);
  }

Однако Thread.Sleep(2000); не работает. Когда я запускаю программу, она как будто игнорирует слип, и в label.4 сразу выводится 8. Почему так и как выйти из данной ситуации?

SKProCH
  • 954
  • 7
  • 23
  • В каком методе запускаете этот код? По идее, Sleep должен отработать в UI-потоке. Т.е. ваше окно зависнет на 16 секунд и выведет 8 в label4. – Vlad Jul 25 '17 at 06:20
  • @Vlad А можно ли это обойти, что бы рандомилось и отображалось каждые 2 секунды? – SKProCH Jul 25 '17 at 06:23
  • @Regent Действия разные, я просто выпилил текст который должен отображаться. А на счет case 6 ступил. valie - очепятка. – SKProCH Jul 25 '17 at 06:49
  • @SKProCH а изменение текста label3 вообще имеет отношение к вопросу? Или вопрос только про label4? Если не имеет, то этот switch имеет смысл убрать из кода в вопросе. – Regent Jul 25 '17 at 06:51
  • @Regent label4 это отображение того, сколько раз уже выполнился for. – SKProCH Jul 25 '17 at 06:54
  • @SKProCH да, я вижу по коду. Мой вопрос в том, действительно ли в коде вопроса нужен label3. – Regent Jul 25 '17 at 07:03

1 Answers1

6

Ничего странного. Когда вы меняете текст надписи - ее регион в окне инвалидируется и в очередь сообщений посылается сообщение WM_PAINT. Но это сообщение может быть обработано только после того, как ваш цикл завершит работу.

Есть три способа исправить такую ситуацию. Первый - старый - вызов Application.DoEvents() внутри цикла, чтобы "разгрести" накопившиеся сообщения:

for (int i = 0; i < 9; i++)
{
  Thread.Sleep(2000);
  label4.Text = Convert.ToString(i);
  Application.DoEvents();
}

Второй способ - объявить метод асинхронным (async) и использовать await Task.Delay для ожидания:

async void button1_Click (object sender, EventArgs e) {
  for (int i = 0; i < 9; i++)
  {
    await Task.Delay(2000);
    label4.Text = Convert.ToString(i);
  }
}

Не забудьте только установить обработчик события Application.ThreadException при использовании async void-метода.

Третий способ - можно вынести цикл в отдельный поток.

Pavel Mayorov
  • 58,537
  • Application.DoEvents()? Нуууууу. Окно ж зависнет на две секунды. – VladD Jul 25 '17 at 10:15