Как приостановить или убить дочерний поток в C#?

Question

[Артем Иванцов]

У меня есть дочерний поток, который что-то там делает в бесконечном цикле.
В какой-то момент времени мне надо:
1. Убить дочерний поток. Сейчас я делаю нечто подобное

private Thread _thread;
        private Boolean _isWorking;
        private void DoWork()
        {
            while(_isWorking)
            {
                Console.WriteLine("SomeWork");
                Thread.Sleep(1000);
            }
        }

        public void Start()
        {
            _thread = new Thread(DoWork);
            _isWorking = true;
            _thread.Start();
        }

        public void Stop()
        {
            _isWorking = false;
            while (_thread.ThreadState != ThreadState.Stopped)
                Thread.Sleep(100);
        }

Я знаю про метод Abort, но он может вызвать утечку ресурсов. Может быть есть более разумный метод без постоянной проверки флага?
2. Заморозить дочерний поток на бесконечно большее время. Можно как вариант сделать еще одну проверка флага и если она проходит, то войти в AutoResetEvent. А что бы разбудить поток, то сбрасываем AutoResetEvent. Еще есть вариант делать бесконечный сон, а для пробуждения использовать Interrupt. Но тут опять же выходит постоянная проверка флага.

Answer 1

[mayorovp]

Для проверки на завершение внутри потока хорошо использовать ManualResetEvent, на нем же и ожидая:

private void DoWork()
        {
            do
            {
                Console.WriteLine("SomeWork");
            } while (!closeEvent.WaitOne(1000));
        }

Ну я для того, чтобы дождаться завершения потока, существует метод Join

Comments

[Артем Иванцов]

Это хуже вариант. Тут производительность раз в 8 ниже, чем при проверке флагов.

private Thread _thread1, _thread2;

        private Boolean _isWorking;
        private ManualResetEvent _exitEvent;
        
        private Stopwatch sw1, sw2;
        private UInt32 counter1, counter2;
        private void DoWork1()
        {
            counter1 = 0;
            sw1 = new Stopwatch();
            sw1.Start();
            while(_isWorking)
            {
                counter1++;
                if (sw1.ElapsedMilliseconds > 1000)
                {
                    Console.WriteLine("DoWork1: " + counter1);
                    counter1 = 0;
                    sw1.Restart();
                }
            }
        }

        private void DoWork2()
        {
            counter2 = 0;
            sw2 = new Stopwatch();
            sw2.Start();
            do
            {
                counter2++;
                if (sw2.ElapsedMilliseconds > 1000)
                {
                    Console.WriteLine("DoWork2: " + counter2);
                    counter2 = 0;
                    sw2.Restart();
                }
            } while (!_exitEvent.WaitOne(0));
        }

        public void Start()
        {
            _thread1 = new Thread(DoWork1);
            _isWorking = true;
            _thread1.Start();

            _thread2 = new Thread(DoWork2);
            _exitEvent = new ManualResetEvent(false);
            _thread2.Start();
        }

        public void Stop()
        {
            _isWorking = false;
            _exitEvent.Set();

            _thread1.Join();
            _thread2.Join();
        }

[mayorovp]

Артем Иванцов: Производительность чего?

Не забывайте, что вы между итерациями цикла ждете секунду. Так какая разница, ждать безусловно или на событии?

[Артем Иванцов]

mayorovp: Я секунду в примере пихнул просто так. Можно сказать, имитирую некий код, который работает некое время. В проекте же уже нет никакой задержки.

[mayorovp]

Артем Иванцов: Тогда приведите больше сведений о схеме работы потока. Разные потоки имеет смысл завершать разными методами.

[Артем Иванцов]

mayorovp: Все тоже самое только вместо этого
Console.WriteLine("SomeWork");
Thread.Sleep(1000);
код получения данных с внешнего устройства.

[mayorovp]

Артем Иванцов: Как происходит взаимодействие с внешним устройством? Если это файловый объект, то можно закрыть его - а в потоке поймать исключение.

[Артем Иванцов]

mayorovp: АЦП и цифровые порты. Команды просто чтения данных с портов.

[mayorovp]

Артем Иванцов: не знаю, что такое цифровой порт - но если это файл, то его можно просто закрыть.

[Артем Иванцов]

mayorovp: Это порт, который в памяти представлен, как Byte, где каждый бит соответствует значению на входе платы. К сожалению драйвер платы написан криво, потом отключить плату, что бы она сгенерировала бы мне ошибку при чтении не могу :-)

[mayorovp]

Артем Иванцов: Я так понимаю, вы в цикле считываете по некоторому адресу значение. И даже, предположим, это считывание происходит без пауз (если там есть паузы - см. первое решение).

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

Answer 2

[AxisPod]

Поглядите на BackgroundWorker. Ну и в текущем примере, вместо ожидания остановки используйте метод Thread.Join(), текущий поток будет ждать пока ваш останавливаемый поток не завершится.

Comments

[Артем Иванцов]

Там мне потребуется постоянно проверять установлен или нет CancellationPending.

[AxisPod]

И в чём проблема? Или вы не хотите использовать правильный механизм предназначенный для задачи именно потому, что вам не хочется проверять флаг? Или в треде while с проверкой нормально, а вот в BackgroundWorker нет?

[Артем Иванцов]

AxisPod: Мне и там и там кажется костыльно :-)

[AxisPod]

А вы как-то иначе хотите? Убить поток вместо нормального завершения по вашему нормальное поведение? Вы же не выходите из функций в случае нормального завершения бросая исключение.

[Артем Иванцов]

AxisPod: Ну так я для этой цели и сделал вопрос. Вдруг иначе можно сделать. Хотя как просто заморозить поток изве мне в обще ничего не предложили(

Answer 3

[Nubzilo]

А вот как советует останавливать поток MSDN - How to: Create and Terminate Threads

Comments

[Виталий Пухов]

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