Закрытие потока, как правильно это сделать? Опции

[AD]

Что-то не удается закрыть поток:

/// Класс параллельного потока для чтения в автоматическом режиме параметров
class ReadAutoThread: public QThread
{
Q_OBJECT

protected:
         virtual void run();
};

void ReadAutoThread::run()
{
        connect(imitTsl, SIGNAL(thread_stop()), SLOT(quit()));
/// imitTsl - указатель на главное окно программы и соответственно основной поток

    for(;;)
    {
        /// что-нибудь да делаем.
    }
}

а в главном окне программы есть следующий сигнал:

/// Класс главного окна программы
class ImitatorTSL: public QMainWindow, public Ui::ImitatorTSLClass
{
    Q_OBJECT

signals:
void thread_stop();
}

/// Изменение текущей вкладки
void ImitatorTSL::changeTab(int index)
{
    ////

    if(index != 2 && _curr_page == 2)
        emit thread_stop();
      ////
}

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

[SABROG]

В основной программе в слоте переключения вкладок я имитирую этот сигнал.

И слот на него выполняется тоже в основной программе, если ты не делал thread->moveToThread(thread);

В этот бесконечный цикл все-равно попадает.

В этом бесконечном цикле тогда надо проверять флаг типа if (finished) return;
А в слоте его тогда надо устанавливать. Неужели бесконечный цикл нельзя заменить на exec()? И выполнять задачи в слотах по таймеру или по сигналам из внешнего потока?

[AD]

В этом бесконечном цикле тогда надо проверять флаг типа if (finished) return;
А в слоте его тогда надо устанавливать. Неужели бесконечный цикл нельзя заменить на exec()? И выполнять задачи в слотах по таймеру или по сигналам из внешнего потока?

А что за флаг? Т.е. в слоте устанавливать его, а в потоке в цикле проверять?

[SABROG]

В этом бесконечном цикле тогда надо проверять флаг типа if (finished) return;
А в слоте его тогда надо устанавливать. Неужели бесконечный цикл нельзя заменить на exec()? И выполнять задачи в слотах по таймеру или по сигналам из внешнего потока?

А что за флаг? Т.е. в слоте устанавливать его, а в потоке в цикле проверять?

Ну да, по-старинке объявить некую переменную bool terminate = false. В слоте его выставлять, а в бесконечном цикле проверять. Только тут как обычно 2 косяка возникает - использование sleep(), чтобы не грузить проц бесконечным циклом и возможная долгая пауза перед завершением потока, если опрашиваемое устройство не отвечает долгое время и не дает проверить флаг terminate.

[igor_bogomolov]

connect(imitTsl, SIGNAL(thread_stop()), SLOT(quit()));

void QThread::quit () [slot]

Tells the thread's event loop to exit with return code 0 (success). Equivalent to calling QThread::exit(0).

This function does nothing if the thread does not have an event loop.

See also exit() and QEventLoop.

Смотри

void QThread::terminate () [slot]

Сам я никогда этой функцией не пользовался, но вроде, то что надо

P.S. ИМХО. Мне не очень нравится когда коннекты раскидывают по всей программе. Программы становится труднее читать. Зачем он вынесен в run?
На мой взгляд, конекты лучше делать там, где создается экземпляр класса.

Сообщение отредактировал igor_bogomolov - 23.6.2009, 17:36

[AD]

Смотри

void QThread::terminate () [slot]

Не самая хорошая функция - аварийное завершение потока, а то и программы!

void QThread::terminate () [slot]
Terminates the execution of the thread. The thread may or may not be terminated immediately, depending on the operating systems scheduling policies. Use QThread::wait() after terminate() for synchronous termination.
When the thread is terminated, all threads waiting for the thread to finish will be woken up.
Warning: This function is dangerous and its use is discouraged. The thread can be terminate at any point in its code path. Threads can be terminated while modifying data. There is no chance for the thread to cleanup after itself, unlock any held mutexes, etc. In short, use this function only if absolutely necessary.
Termination can be explicitly enabled or disabled by calling QThread::setTerminationEnabled(). Calling this function while termination is disabled results in the termination being deferred, until termination is re-enabled. See the documentation of QThread::setTerminationEnabled() for more information.

[igor_bogomolov]

Не самая хорошая функция

На самом деле, даже в документации не рекомендуют ее использовать.
quit тебе тоже не помошник, т.к. нет eventLoop.

Значит совет SABROG-а, остается единственным решением в данном случае.

[Litkevich Yuriy]

connect(imitTsl, SIGNAL(thread_stop()), SLOT(quit()));

В твоей программе РОДИТЕЛЬ - окно, я думаю в нём создаётся объект потока ДЕТЁНЫШ.

Дак почему ты в детёныше соеденяешь сигнал родителя с самим детёнышем, это откуда такая практика?

[AD]

В твоей программе РОДИТЕЛЬ - окно, я думаю в нём создаётся объект потока ДЕТЁНЫШ.

Дак почему ты в детёныше соеденяешь сигнал родителя с самим детёнышем, это откуда такая практика?

Дак ты сам говорил, что неважно, в каком направлении соединять сигнал со слотом! Да и потом - работает! Исключений не вызывает, что еще надо! ))

Сообщение отредактировал AD - 23.6.2009, 23:05

[Litkevich Yuriy]

Дак ты сам говорил, что неважно, в каком направлении

направление - не место.