Завершение QThread

[skydion]

Собственно вопрос, как правильно прекращать работу треда...?
У меня запускается тред на сканирование файлов, ну все это дело долгое,
поэтому нужно из интерфейса завершать тред, сейчас делал просто

Код:

bool MainWindow::stopScan()
{
if (thread)
{
	thread->terminate();
	thread->wait(500);
	qDebug("Thread terminated");
	
	delete thread;
}
}

почитал про треды и там есть предупреждение

]void QThread::terminate ()
Warning: This function is dangerous and its use is discouraged.

[hackoff]

thread->exit(0);
thread->deleteLater();

[skydion]

я так понимаю Вашим кодом заменить этот код?

thread->terminate();
thread->wait(500);

а как проверить завершился ли поток?
чтобы сделать изменения в интерфейсе? разблокировать заблокировать кнопки?
раньше я делал через перехват сигнала terminated (), а теперь finished () ловить?

[skydion]

теперь вот така фигня вылазит...

QObject::killTimers: timers cannot be stopped from another thread чем лечить?

Код:

MainWindow::MainWindow
{
    thread = new ScanThread(this);
}

void MainWindow::scanVocabulary(void)
{
    if (thread)
    {
	thread->start(QThread::NormalPriority);

	ui->pushButtonStart->setDisabled(true);
	ui->pushButtonStop->setDisabled(false);

	qDebug("scanVocabulary: Scan thread started");
    }
}

void MainWindow::stopScan()
{
    if (thread)
    {
	thread->exit(0);
	thread->deleteLater();

	ui->pushButtonStart->setDisabled(false);
	ui->pushButtonStart->setChecked(false);
    }
}

ScanThread::~ScanThread()
{
    if (!isRunning())
	return;

    if (text.count())
    {
	drv->beginTransaction();
	query.addBindValue(text);
	query.execBatch(QSqlQuery::ValuesAsRows);

	le = query.lastError();
	if (le.type() == QSqlError::NoError)
	    drv->commitTransaction();
	else
	    qDebug() << "~ScanThread: [" << le.text() << "]";

	query.finish();
    }

    wait(500);
}

[alexman]

Замени

Код:

thread->exit(0);
thread->deleteLater();

на

Код:

thread->wait();
delete thread;

[alexman]

Кстати,

Код:

void MainWindow::stopScan()
{
    if (thread)
    {
	thread->exit(0);
	thread->deleteLater();

	ui->pushButtonStart->setDisabled(false);
	ui->pushButtonStart->setChecked(false);
    }
}

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

[skydion]

Замени

Код:

thread->exit(0);
thread->deleteLater();

на

Код:

thread->wait();
delete thread;

вначале у меня так было... смотри начало треда...

[skydion]

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

как он может знать? какой сигнал ему послать?
я его хочу принудительно остановить, притом корректно (мне еще при завершении треда надо буфер скинуть в БД)

[alexman]

Из потока нельзя принудительно завершать другой поток!
terminate() -> 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.

Тебе нужно как-нибудь уведомить о завершении другой поток из главного; например, послать сигнал canceled(), по сигналу сделать cancel = true, а далее

void run ()
{
    while(...) {
    ...
        if ( cancel ) {
            return;
        }
    ...
    }
}

[alexman]

Ну а далее в главном потоке уже можно вызвать

thread->wait();
delete thread;

[Igors]

как он может знать? какой сигнал ему послать?
я его хочу принудительно остановить, притом корректно (мне еще при завершении треда надо буфер скинуть в БД)

Принудительно не надо - можно нарваться на неприятности. Нитка завершится когда она выйдет из своего метода run - Вам надо каким-то образом сказать ей когда выходить. Напр. если нитка крутит EventLoop, сделайте ей quit. Если нитка занимается чисто расчетами - сделайте ей флажок типа FlagStop и проверяйте его в расчетах. После этого цивильно ждите завершения: thread->wait(); а можно и подсесть на сигнал нитки terminated()

[skydion]

спасибо Вам, щас все осмыслю и буду ваять, много инфы навалилось 

[ieroglif]

Код:

ThreadClass * thrd = new ThreadClass();
connect(thrd ,SIGNAL(finished()),thrd ,SLOT(deleteLater()));
thrd->start();

ну а внутри треда держать переменную булевскую и создать слот по её изменению.

[skydion]

Всем спасибо, все решилось положительно

[Djavdet]

как конкретно?