Не убивается поток Qt3.3.8 VC2005

[Примерный ученик]

Попробовал

Код:

parVF->exit()

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

[sadhu]

3 Qt остался на работе, но в 4 имееться дополнительная "защита от дурака", дабы получить возможность убивать нити методом terminate надо в любом месте программы вызвать защищённый статический метод
void setTerminationEnabled ( bool enabled = true ).
Если в 3 аналога нет , то завтра если не запамятую гляну , т.к как раз похожую ситуацию мне и предстоит в ближайшем будующем реализовать

[Fat-Zer]

нет, в 3-их такой защиты нет... в 4-х даже после вызова этого метода поток постоянно выполняющийся не убивается...

[sadhu]

В третьих пока могу посмотреть только документацию, кстати возможно стоит просто дождаться когда процесс прерветься ( QThread::wait() ) ибо "The thread may or may not be terminated immediately, depending on the operating system's scheduling policies." .

Что имееться в виду под

в 4-х даже после вызова этого метода поток постоянно выполняющийся не убивается...

Спецом проверил ща и убиваеться в любом случае и при

Код:

...
TestThread::run()
{
while(true){полезная_нагрузка_дабы_не_убило_оптимизацией;}
}
...

и при

Код:

...
TestThread::run()
{
полезная_нагрузка;
exec();
}
...

[Fat-Zer]

О_о я тоже только сегодня днём пробовал...
Не убивается, мой пример:

Код:

// widget.h
#ifndef WIDGET_H
#define WIDGET_H

#include <QWidget>
#include <QThread>

class Thread: public QThread
{
public:
	Thread()
	{
		setTerminationEnabled(1);
	}

	void run()
	{
		int i;
		while(1) i++;
	}

};

class Widget : public QWidget
{
Q_OBJECT
public:
	explicit Widget(QWidget *parent = 0);
public slots:
	void terminateThread();
private:
	Thread *thr;
};

#endif // WIDGET_H


// widget.cpp
#include <QPushButton>
#include "widget.h"

Widget::Widget(QWidget *parent)
	:QWidget(parent)
{
	QPushButton *btn=new QPushButton(this);
	thr = new Thread();
	thr->start();
	connect(btn,SIGNAL(clicked()),
			this,SLOT(terminateThread()));
}

void Widget::terminateThread()
{
	thr->terminate();
	thr->wait();
}



// main.cpp
#include <QApplication>
#include "widget.h"

int main(int argc, char *argv[])
{
	QApplication a(argc, argv);

	Widget w;
	w.show();
	return a.exec();
}

Тоже самое, если setTerminationEnabled(1) ставить где-либо ещё...

[Примерный ученик]

Самое неприятное, что метод

Код:

Qt::HANDLE currentThread () 

возвращает код GUI-потока.
Так что и системными средствами не убить зависший поток

[Примерный ученик]

Пробовал ставить минимальный приоритет...
Не помогает
Как жрал 99%, так стал 96% жрать и все

[brankovic]

С qthread не знаю что, но сама постановка нехорошая:

Проблема

1. допустим таки убили вы тред. В результате утечка памяти (в момент postMessage). Кроме того, убивание malloc в середине вызова может привести вообще к падению или завису malloc в другом треде. Зачем вам такая программа?

Варианты решения

2. вставить в calc проверку флага, по которому тред культурно выходит (предлагали уже выше). Пусть написано другим разработчиком, если код доступен, вставьте флаг. Это простой путь.

3. можно запустить calc в отдельном процессе. Не знаю, насколько просто ipc в кьют, но принципиально это возможно. Это долгий путь. Но устойчивость получится выше.

[Waryable]

Согласен с последним автором. Убиение потока "топором" - это всегда не хорошо. В данном случае полагаться на то, что система разрулит то, что вы не смогли разрулить своим кодом не безопасно. Тем более, что ввести флаг, разрешающий/запрещающий дальнейшую работу, довольно просто.

[Примерный ученик]

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

[brankovic]

Если доступа действительно нет, то единственный надёжный вариант -- запустить в отдельном процессе. В документации есть некий QProcess, например. На вид то, что нужно.

При его убиении я ничего не теряю

Что значит, "я ничего не теряю"? Вот микрософт нам пишет:

http://msdn.microsoft.com/en-us/library/ms686717%28v=vs.85%29.aspx


суть:

TerminateThread is used to cause a thread to exit. When this occurs, the target thread has no chance to execute any user-mode code. DLLs attached to the thread are not notified that the thread is terminating. The system frees the thread's initial stack.

    Windows Server 2003 and Windows XP/2000:  The target thread's initial stack is not freed, causing a resource leak.

TerminateThread is a dangerous function that should only be used in the most extreme cases. You should call TerminateThread only if you know exactly what the target thread is doing, and you control all of the code that the target thread could possibly be running at the time of the termination. For example, TerminateThread can result in the following problems:

    * If the target thread owns a critical section, the critical section will not be released.
    * If the target thread is allocating memory from the heap, the heap lock will not be released.
    * If the target thread is executing certain kernel32 calls when it is terminated, the kernel32 state for the thread's process could be inconsistent.
    * If the target thread is manipulating the global state of a shared DLL, the state of the DLL could be destroyed, affecting other users of the DLL.

Обратите внимание на замечание про 2003/2000/XP, память будет течь независимо от вашего кода или от реализации calc.

Даже если течь не смущает или платформа другая, всё равно. Не имея кода calc, как вы можете быть уверены, что там не вызывается malloc? Один прерванный malloc, это на 99% либо падение, либо зависание в _любом_ треде! С таким же успехом "оператор" может снимать программу на ctrl-alt-del и запускаться заново.

Итого: причина, по которой QThread::terminate не работает под windows в том, что terminate там вообще не нужен. Но тот же метод вполне безопасно реализован в linux и, возможно, ещё на каких-то платформах, только поэтому он продолжает существовать.