C# Добавление кода в событие FormClosing отключает X на MdiParent

Question

У меня есть два фоновых потока и поток для обработки минимизации, выполняющийся в моем приложении WinForms. Когда программа закрывается, я использую этот метод.

private void MyForm_Closing(object sender, FormClosingEventArgs e) 
    { 
     if(labelsUpdaterThread.IsAlive == true) 
      labelsUpdaterThread.Abort(); 
     if(printNotifyThread.IsAlive == true) 
      printNotifyThread.Abort(); 
     if(minimizeThread.IsAlive == true) 
      minimizeThread.Abort(); 
    } 

labelsUpdaterThread and printNotifyThread все время работает. MinimizeThread, как вы могли догадаться, работает только при свертывании родительской формы. Вот моя проблема:

Когда методы thread.abort вызываются в моем методе выше, «X» в верхнем правом углу моей формы MdiParent ничего не делает. Щелчок по нему не влияет.

Когда методы thread.abort НЕ вызывается в моем методе выше, закрытие MdiParent иногда вызывает исключения, поскольку потоки все еще пытаются получить доступ к ресурсам на MdiParent, которые больше не доступны, хотя они являются фоновыми потоками!

Я не уверен, почему это происходит, не имеет для меня никакого смысла. Заранее благодарю за любую помощь!

Answer 1

Я согласен с Paul Alexander's answer в том, что вы должны никогда вызова Thread.Abort, это ужасный способ обработки синхронизации.

Кроме того, здесь есть ужасное разделение. Нити не должны напрямую обращаться к ресурсам в форме. Должно быть какое-то абстракционное/разделяемое состояние между ними, которое изменяется и считывается обеими сторонами (поток и форма, поэтому убедитесь, что экземпляр потокобезопасен).

Если вы не смогли внести эти изменения, то в методе Close вызовите методы Thread.Abort в другом потоке с инструкцией try/catch вокруг каждого. Запишите ошибку (-ы) где-то как минимум.

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

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

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

Оттуда вы можете ввести надлежащий механизм отмены сотрудничества (cooperative cancellation, which Task supports in .NET 4.0, если это доступно для вас).

Answer 2

Вызовы Abort, вероятно, бросают исключения. Убедитесь, что указатели действительны, и потоки остаются действительными (не удаляются), прежде чем вызывать прерывание.

И, в визуальной студии, откройте Debug \ Exceptions ... и установите проверку в столбце 'throw' для всех исключений, чтобы вы видели, что что-то пошло не так.

Answer 3

Сначала удалите вызовы на .Abort() и никогда не используйте их снова. Нити никогда не должны прерываться, вызывая Abort. В основном это приводит к сбою вашего потока и не дает ему возможности выпустить любые ресурсы должным образом или освободить любые системные дескрипторы. Вместо этого создайте ManualResetEvent и проверьте это в своих потоках. Когда событие установлено, они должны прекратиться.

Резьба1

while(! _stopEvent.WaitOne(0)) 
{ 
    ...do my thready work 

} 

Тогда при закрытии

private void MyForm_Closing(object sender, FormClosingEventArgs e) 
{ 
    _stopEvent.Set(); 
    labelsUpdaterThread.Join(); 
    ... 
} 

Если вы не заботитесь, если потоки прекратить должным образом на применении существуют, просто установите IsBackground = true, и они будут автоматически прекращается, когда приложение выходы.