Как в C# осуществить выход из потока (с другого класса)?

Question

[zoke]

В общем, такая суть. Мы имеет поток (Thread), который запускает функцию Start, а она, в свою очередь, работает в цикле и запускает функцию из другого класса - Check. Нам нужно сделать завершение потока из функции Check как можно проще. Потоков у нас будет около 80. Можно сделать возврат значения и останавливать с помощью break, но код будет запутанный, потому что у нас не одна точка выхода должна быть. Посоветуйте оптимальное решение.

class Job
    {
      public void Start ()
      {
           While(true)
           {
               //...иной код
               new Job2().Check();
              //...иной код
           }
      }
    }

И другой класс

class Job2
    {
      public void check ()
      {
           //...иной код
               if(a == true)
               {
                       //здесь должен быть выход из потока.
               }
            //...иной код
           }
      }
    }

Answer 1

[Aleksandr S]

Мне кажется задача изначально возможно так спроектирована, что получаются запутанные решения. Первая мысль про отмену выполнения потока была тоже сделать на базе CToken, но получается токен нужно передавать еще и в метод check (оттуда же вызываем отмену в данном варианте), что лично мне не не очень нравится.

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

С другой стороны смотрите, получается метод check что-то проверяет. Значит должен возвращать результат проверки (true/ false). По результатам проверки должно приниматься решение о том, прекратить выполнение или продолжить. Мне кажется надо пересмотреть решение задачи, чтобы логика была более очевидной, а не вбросами исключений в разных местах кода.

Answer 2

[_ _]

Как насчет выбросить исключение, во внешнем цикле его поймать и там выйти?

Answer 3

[Артем Воронов]

А чем не устраивает CancellationToken?

Answer 4

[Grigory Perepechko]

У вас каша в архитектуре. Опишите задачу без ваших догадок касательно реализации и будет намного проще предложить вам более адекватное решение.

Не совсем понимаю зачем вам явное управление потоками. Посмотрите в сторону Task, там для возврата значений и отмены есть удобные механизмы.

Answer 5

[Антон]

Можно попробовать вызывать метод Abort у текущего потока.

Thread.CurrentThread.Abort();

Comments

[Grigory Perepechko]

Никогда так не делайте. Поток должен сам завершать свою работу.

[Антон]

@GrigoryPerepechko, на сколько я понимаю опасность данного метода заключается в асинхронном выбрасывании исключения (неизвестно что будет выполнятся в потоке во время вызова метода аборт). В данном же случае, оно выбрасывается синхронно, по этому я не вижу препятствий для его использования. Или есть еще какие то проблемы?

Answer 6

[gleb_kudr]

Если есть возможность использовать .net 4.5 то лучше всего делать все с помощью Async конструкций (если использование тредов не оправдано особой необходимостью). Обычный выход с потока передается через Invoke, только придется хранить ссылку на родительский поток.