Почему мой цикл не позволяет изменить цвет меток?

Question

Без цикла do мой код работает нормально. Как только я помещаю его в цикл do или while, код не обновляет статус цвета. Есть идеи? Из того, что я собрал из Интернета, мои петли написаны правильно.

using System; 
using System.Collections.Generic; 
using System.ComponentModel; 
using System.Data; 
using System.Drawing; 
using System.Linq; 
using System.Text; 
using System.Threading.Tasks; 
using System.Windows.Forms; 
using System.Net.NetworkInformation; 
using System.Threading; 



namespace SystemsUpDown 
{ 
    public partial class MainForm : Form 
    { 

     public MainForm() 
     { 
      InitializeComponent(); 
     } 

     bool ContinuePing = false; 


     private void QuitButton_Click(object sender, EventArgs e) 
     { 
      this.Close(); 
     } 

     private void StartButton_Click_1(object sender, EventArgs e) 
     { 
      Ping ping = new Ping(); 
      ContinuePing = true; 
      do 
      { 
       try ///ping google 
       { 
        PingReply reply = ping.Send("8.8.8.8"); 

        if (reply.Status == IPStatus.Success) 
        { 
         GoogleStatusLabel.BackColor = Color.Green; 
        } 

       } 
       catch 
       { 
        GoogleStatusLabel.BackColor = Color.Red; 
       } 

       try ///ping Yahoo! 
       { 
        PingReply reply = ping.Send("www.yahoo.com"); 

        if (reply.Status == IPStatus.Success) 
        { 
         YahooStatusLabel.BackColor = Color.Green; 
        } 

       } 
       catch 
       { 
        YahooStatusLabel.BackColor = Color.Red; 
       } 

       try ///ping Reddit.com 
       { 
        PingReply reply = ping.Send("www.reddit.com"); 

        if (reply.Status == IPStatus.Success) 
        { 
         RedditStatusLabel.BackColor = Color.Green; 
        } 

       } 
       catch 
       { 
        RedditStatusLabel.BackColor = Color.Red; 
       } 

       try ///ping Chive 
       { 
        PingReply reply = ping.Send("www.chive.com"); 

        if (reply.Status == IPStatus.Success) 
        { 
         ChiveStatusLabel.BackColor = Color.Green; 
        } 

       } 
       catch 
       { 
        ChiveStatusLabel.BackColor = Color.Red; 
       } 

       try ///ping CNN 
       { 
        PingReply reply = ping.Send("www.cnn.com"); 

        if (reply.Status == IPStatus.Success) 
        { 
         CNNStatusLabel.BackColor = Color.Green; 
        } 

       } 
       catch 
       { 
        CNNStatusLabel.BackColor = Color.Red; 
       } 

      } while (ContinuePing); 

     } 

     private void StopButton_Click(object sender, EventArgs e) 
     { 
      GoogleStatusLabel.BackColor = Color.Yellow; 
      ChiveStatusLabel.BackColor = Color.Yellow; 
      CNNStatusLabel.BackColor = Color.Yellow; 
      RedditStatusLabel.BackColor = Color.Yellow; 
      YahooStatusLabel.BackColor = Color.Yellow; 
      ContinuePing = false; 

     } 
    } 
} 

Answer 1

Try принуждать обновление метки после того, как вы изменили свой цвет:

GoogleStatusLabel.Refresh(); 

Answer 2

Ваш процесс непрерывно работает в другом потоке, чем в потоке пользовательского интерфейса, который не получает время для обновления пользовательского интерфейса Предметы.

Использование Update/Обновить, чтобы заставить UI некоторое время обновления, или использовать сон, чтобы нить UI некоторое время

Answer 3

Это, вероятно, потому, что петли работать достаточно быстро, и поэтому обновления не приемлемы.

Попробуйте добавить некоторый сон в петле, как

//Sleep for two seconds. You can add this at end of loop. 
//Or, sleep for 2 secs after pinging one site. 
// 2000 miliseconds = 2 seconds. 
System.Threading.Thread.Sleep(2000); 

, а затем проверить.

Я хотел бы предложить вам использовать BackgroundWorker. Поскольку этот цикл while в основном потоке будет висит в форме главного окна.

BackgroundWorker подходит для таких периодических обновлений элементов управления формой окна, чтобы главное окно не зависало. Пожалуйста, обратитесь к следующей ссылке - https://msdn.microsoft.com/en-us/library/cc221403(v=vs.95).aspx

Answer 4

Отметить обработчик Click() с async, а затем бросить свой цикл в await Task.Run() с анонимным делегатом. Обновление пользовательского интерфейса с помощью this.Invoke(), который я сделал ниже с вспомогательным методом:

private async void StartButton_Click_1(object sender, EventArgs e) 
    { 
     await Task.Run(() => 
     { 
      Ping ping = new Ping(); 
      ContinuePing = true; 
      do 
      { 
       try ///ping google 
       { 
        PingReply reply = ping.Send("8.8.8.8"); 

        if (reply.Status == IPStatus.Success) 
        { 
         SetLabelColor(GoogleStatusLabel.BackColor, Color.Green); 
        } 

       } 
       catch 
       { 
        SetLabelColor(GoogleStatusLabel.BackColor, Color.Red); 
       } 

       try ///ping Yahoo! 
       { 
        PingReply reply = ping.Send("www.yahoo.com"); 

        if (reply.Status == IPStatus.Success) 
        { 
         SetLabelColor(YahooStatusLabel.BackColor, Color.Green); 
        } 

       } 
       catch 
       { 
        SetLabelColor(YahooStatusLabel.BackColor, Color.Red); 
       } 

       try ///ping Reddit.com 
       { 
        PingReply reply = ping.Send("www.reddit.com"); 

        if (reply.Status == IPStatus.Success) 
        { 
         SetLabelColor(RedditStatusLabel.BackColor, Color.Green); 
        } 

       } 
       catch 
       { 
        SetLabelColor(RedditStatusLabel.BackColor, Color.Red); 
       } 

       try ///ping Chive 
       { 
        PingReply reply = ping.Send("www.chive.com"); 

        if (reply.Status == IPStatus.Success) 
        { 
         SetLabelColor(ChiveStatusLabel.BackColor, Color.Green); 
        } 

       } 
       catch 
       { 
        SetLabelColor(ChiveStatusLabel, Color.Red); 
       } 

       try ///ping CNN 
       { 
        PingReply reply = ping.Send("www.cnn.com"); 

        if (reply.Status == IPStatus.Success) 
        { 
         SetLabelColor(CNNStatusLabel, Color.Green); 
        } 

       } 
       catch 
       { 
        SetLabelColor(CNNStatusLabel, Color.Red); 
       } 

      } while (ContinuePing); 
     }); 
    } 

    private void SetLabelColor(Label lbl, Color clr) 
    { 
     this.Invoke((MethodInvoker)delegate { 
      lbl.BackColor = clr; 
     }); 
    }