Прекращение/соединение нити в C#

Question

Я не могу убить свою нить на C#. Кажется, что программа застревает в бесконечном цикле события FormClosing.

EDIT // Я пытаюсь закончить поток и закрыть всю программу при запуске события FormClosing.

Вот код:

public partial class Form1 : Form 
{ 
    private Thread thread; 
    private volatile bool threadRunning = true; 

    public Form1() 
    { 
     InitializeComponent(); 
    } 

    private void Loop() 
    { 
     Console.WriteLine(threadRunning); 
     while (threadRunning) 
     { 
      MethodInvoker mi = delegate { timeLabel.Text = TimeWriterSingleton.Instance.OutputTime(); }; 
      Invoke(mi); 
     } 
    } 

    private void Form1_Load(object sender, EventArgs e) 
    { 
     thread = new Thread(Loop); 
     thread.Start(); 
    } 

    private void Form1_FormClosing(object sender, FormClosingEventArgs e) 
    { 
     threadRunning = false; 
     thread.Join(); 
    } 
} 

Answer 1

Я решил переключиться на использование таймера. Код теперь выглядит так, и приложение работает:

public partial class Form1 : Form 
{ 
    private System.Timers.Timer timer; 

    public Form1() 
    { 
     InitializeComponent(); 

     timer = new System.Timers.Timer(60000); 
    } 

    private void Form1_Load(object sender, EventArgs e) 
    { 
     timeLabel.Text = TimeWriterSingleton.Instance.OutputTime(); 
     timer.Elapsed += TimerElapsed; 
     timer.Enabled = true; 
    } 

    private void TimerElapsed(object sender, ElapsedEventArgs e) 
    { 
     timeLabel.Text = TimeWriterSingleton.Instance.OutputTime(); 
    } 
} 

Answer 2

Я собирался предложить нечто более сложное, но то, что вы делаете, это довольно просто, так вот простой ответ.

Вместо темы Thread просто используйте поток ThreadPool, чтобы выполнить свою работу и избавиться от этого thread.Join() в вашем закрытии формы.

Нити ThreadPool являются фоновыми потоками и автоматически уничтожаются, когда приложение закрывается.

Ваша проблема заключается в том, что вы пытаетесь Invoke методом на потоке пользовательского интерфейса, но когда вы не выходя thread.Join() эффективно блокирует поток пользовательского интерфейса до ваших выходов резьбы рабочих. Итак, ты ДОЗЫВАЙТЕ себя. Добро пожаловать в мир многопоточности.

Кроме того, если вы обнаружите, что используете Thread, вы, вероятно, делаете что-то неправильно. Вы должны использовать Tasks. Это проще, и с помощью токенов отмены вы можете участвовать в совместном аннулировании.

Answer 3

Ваш Join заблокировал поток GUI, а ваш Invoke в другом потоке ждет, пока ваш поток GUI обработает делегат.

Быстрое исправление будет состоять в том, чтобы использовать BeginInvoke вместо Invoke, таким образом отправляя вместо отправки оконного сообщения.

В качестве альтернативы, не присоединяйтесь. Цель этого кода состоит в том, чтобы очистить себя, почему вы заботитесь, когда нить умирает?

Третьим исправить было бы просто потянуть нить, либо через Thread.Abort, либо Environment.Exit. Он может пропустить некоторую очистку, но ваш конкретный код не должен заботиться, и в любом случае нужно выйти.

Edit: рабочий код, используя BeginInvoke следующим образом:

private void Loop() 
    { 
     while (threadRunning) 
     { 
      BeginInvoke(new MethodInvoker(() => timeLabel.Text = DateTime.Now.ToString())); 
      Thread.Sleep(100); 
     } 
    } 

    private void Form1_FormClosing(object sender, FormClosingEventArgs e) 
    { 
     threadRunning = false; 
     thread.Join(); 
    } 

Проблема с исходным кодом является то, что он работает так же быстро, как ваш процессор позволяет, наполняя очередь сообщений в точке, где GUI поток не может поддерживать. Обновление элементов управления Windows очень дорого, по сравнению с простое добавление числа в очередь. Поэтому я добавил паузу между обновлениями пользовательского интерфейса, чтобы дышать потоком графического интерфейса.

Для спутников, мне было бы любопытно, почему вы это делаете. Ничто из того, что я сказал, фактически неверно.

Answer 4

Фактически использование BeginInvoke() - неплохая идея. Это может выглядеть так:

private void Form1_Load(object sender, EventArgs e) 
{ 
    thread = new Thread(() => Loop(this)); 
    thread.Start(); 
} 

private void Loop(Form1 form) 
{ 
    while (threadRunning && !form.IsDisposed) 
    { 
     MethodInvoker mi = delegate() { timeLabel.Text = /* Some text */ ; }; 
     BeginInvoke(mi); 

     // Let sleep some time... 
     Thread.Sleep(1); 
    } 
} 

private void Form1_FormClosing_1(object sender, FormClosingEventArgs e) 
{ 
    threadRunning = false; 
    thread.Join(); 
}