VPF::Как остановить поток? - Форум программистов

Warchief

Дата 18.12.2008, 04:24 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 52
Регистрация: 24.5.2007

Репутация: нет
Всего: 3

Добрый день!

помогите разобраться с потоками. есть приложение, в котором при нажатии на баттон, стартует поток:

Код


 t = new Thread(new ThreadStart(delegate
            {
                MakeWork();
            }));
         
            t.Start();

Поток удачно стартует и работает. В теле функции MakeWork() есть следующий код,

Код


Thread.Sleep(int.Parse(textGetWaiter.Text));

который усыпляет поток. В приложении есть кнопка, которая должна прерывать поток, пытаюсь делать так:

Код


t.Join();

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

t.interrupt(), помогает, НО при закрытии приложения(когда Thread.Sleep(int.Parse(textGetWaiter.Text)); ещё не отработал) во время выполнения потока t, приложение пытается прочитать значение Thread.Sleep(int.Parse(textGetWaiter.Text)); и естественно не найдя его на форме, валится с ошибкой.

jonie

Дата 18.12.2008, 08:33 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 5613
Регистрация: 21.8.2005
Где: Владимир

Репутация: 6
Всего: 118

есть Abort. по идее сгенерирует ThreadAbortException при достижении первой IL инструкции.... этот Exception делает rethrow в блоке catch, если не изменить явным образом то поведение. Однако этот Exception не является аварийным событием, так что сама программа не завершается

--------------------

Что-то не поняли? -> Напейтесь до зеленых человечков... эта сверхцивилизация Вам поможет...

mihryak

Дата 18.12.2008, 18:37 (ссылка)

(голосов:1)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 731
Регистрация: 28.4.2007
Где: С-Пб

Репутация: 10
Всего: 36

посмотри в сторону EventWaitHandle
усыплять поток можно через

Код


public virtual bool WaitOne(
    int millisecondsTimeout,
    bool exitContext
)

при выходе из приложения нужно будет послать сигнал и вызвать thread.Join(), что разбудит поток и дождётся его корректного завершения

Warchief

Дата 18.12.2008, 20:12 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 52
Регистрация: 24.5.2007

Репутация: нет
Всего: 3

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

PashaPash

Дата 18.12.2008, 20:37 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1233
Регистрация: 3.1.2008

Репутация: 9
Всего: 49

Warchief, вызывай вместо Sleep WaitOne c таймаутом 30 секунд.

--------------------

Bird's-eye View?

mihryak

Дата 18.12.2008, 21:01 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 731
Регистрация: 28.4.2007
Где: С-Пб

Репутация: 10
Всего: 36

Цитата(Warchief @ 18.12.2008, 21:12

)

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

ну тогда в моём решении между сигналом и tread.Join() делай thread.Abort()

mihryak

Дата 18.12.2008, 22:47 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 731
Регистрация: 28.4.2007
Где: С-Пб

Репутация: 10
Всего: 36

Код


    public partial class Form1 : Form
    {
        private readonly AutoResetEvent resetEvent = new AutoResetEvent(false);
        private readonly Thread thread;

        public Form1()
        {
            InitializeComponent();
            thread = new Thread(ThreadWork);
        }

        private void button1_Click(object sender, EventArgs e)
        {
            thread.Start();
        }

        private void ThreadWork()
        {
            SomeWork();
            SetText("Sleeping...");
            resetEvent.WaitOne(int.Parse(textBox1.Text), false);
            SomeWork();
        }

        private void SomeWork()
        {
            for (int i = 0; i < 10000; i++)
            {
                SetText(i);
            }
        }

        private void SetText(object value)
        {
            Invoke(new MethodInvoker(delegate() { textBox2.Text = value.ToString(); }));
        }

        private void Form1_FormClosing(object sender, FormClosingEventArgs e)
        {
            if (thread.ThreadState == ThreadState.Running || thread.ThreadState == ThreadState.WaitSleepJoin)
            {
                resetEvent.Set();
                thread.Abort();
            }
            thread.Join();
        }
    }

можешь ещё обратить внимание на свойство Thread.IsBackground

PashaPash

Дата 19.12.2008, 01:27 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1233
Регистрация: 3.1.2008

Репутация: 9
Всего: 49

mihryak, а зачем в этом варианте Abort? достаточно if (resetEvent.WaitOne(...)) return;

--------------------

Bird's-eye View?

mihryak

Дата 19.12.2008, 01:37 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 731
Регистрация: 28.4.2007
Где: С-Пб

Репутация: 10
Всего: 36

Цитата(PashaPash @ 19.12.2008, 02:27

)

mihryak, а зачем в этом варианте Abort? достаточно if (resetEvent.WaitOne(...)) return;

это на тот случай, если

Код

resetEvent.WaitOne(int.Parse(textBox1.Text), false);

уже прошёл, пошла остальная работа, и её надо тоже остановить
но проверка результата WaitOne и досрочный неаварийный выход - стоящая штука, не подумал как-то
_____________

ещё раз напомню про IsBackground
если не требуется особенная обработка завершения потока, то это свойство - самое то, что нужно, рантайм сам всё грамотно прибьёт

Это сообщение отредактировал(а) mihryak - 19.12.2008, 01:49

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| .NET для новичков \| Следующая тема »