Das deutschsprachige Forum rund um Qt
https://www.qtforum.de/
Code: Alles auswählen
// In der vererbten QThread Klasse "WorkerThread":
void run()
{
Worker worker;
worker.connect(.....);
exec();
}Code: Alles auswählen
// In der instanzierenden Klasse, dessen Object auf einem anderen Thread liegt:
void erstelleThread()
{
this->workerThread = new WorkerThread(this);
}Code: Alles auswählen
// In der instanzierenden Klasse, dessen Object auf einem anderen Thread liegt:
void erstelleThread()
{
this->workerThread = new QThread(this);
this->worker = new Worker();
this->worker->connect(...);
this->worker->moveToThread(this->workerThread);
}Was soll ein Parent machen, dessen child komplett(also mit konstruktor/destruktor) in nem anderen thread erstellt wurde ?Was sagt ihr dazu?
Deshalb die Einschraenkungen halt, das es funktioniert. Ich geb Dir recht, zum QT Style wuerde die sache mit dem parent-child beziehungen threadübergreifend sicher passen !Andernfalls ist QThread eine riesige und problematische Lücke des "Qt Coding Standards für C++".
Kann es sein das du mit Threads noch ned allzuviel gemacht hasst ?Ich sehe keinen Grund
Intressant wirds erst, wenn du threads von aussen kontrolliert beenden musst(ohne terminate) ... Wenn Du threads synchronisieren musst.
der quit macht nur sinn, wenn der Thread in ner eventloop laeuft. also im run muss nen exec() aufgerufen werden.1. QThread muss in seinem destructor als Allererstes lediglich den quit() slot aufrufen (per signal) und mit this.wait() darauf warten, dass sein thread beendet ist.
Code: Alles auswählen
QMyThread::run()
{
QMySignalReceiverIrgendwas sigreceiver;
connect(this,SIGNAL(terminated(),&sigreceiver,SLOT(makeBlueScreen()));
exec();
/// da sigreceiver lokal aufm Stack war, wird es hier automatisch ungueltig, die connections werden soweiso getrennt durch den DTOR ... alles super in ordnung.
}
Code: Alles auswählen
void MyWidget::prepareWork()
{
QThread *context = new QThread();
mWorker = new Worker();
mWorker->moveToThread(context);
// Beispiel: auf Button-Klick beginnt die Action:
connect(aPushButton, SIGNAL(clicked()),
mWorker, SLOT(startWork()));
// Den Thread brauchen wir nach getaner Arbeit nicht mehr:
connect(mWorker, SIGNAL(workingDone()),
context, SLOT(quit()));
// Context soll nach Beendigung Selbstmord begehen:
connect(context, SIGNAL(finished()), context, SLOT(deleteLater()));
// das waers.. start:
context->start();
}
Code: Alles auswählen
if (!mWorker->isRunning()) {
mWorker->deleteLater();
mWorker = NULL;
}
.. Meist verwende ich einen geblockten Mutex, welcher von der Workerinstanz freigegeben wird, sobal die Arbeit erledigt wurde. Da kann dann z.B. auch mit "tryLock()" gearbeitet werden, so dass die GUI nicht einfriert. Aber im Prinzip:Code: Alles auswählen
void MyWidget::closeEvent(....)
{
if (mWorker) {
..
mWorker->shutdown();
.. // diverse Varianten denkbar mit mehrmaligen Nachfragen beim Benutzer oder Abbruch des Close-Events..
}
}
void Worker::shutdown()
{
// setze ein Flag oder sende ein Qt-Signal
mMutex.lock(); // warten auf Beenden...
// viele Varianten denkbar mit tryLock und Timeout und bool-Returnwert.
}
Code: Alles auswählen
this->workerThread = new WorkerThread(this); Oft kannst du nich mal was dafuer ...Ich gehe nicht von der übelsten Anwendung aus
Ja das stimmt.. es gibt Anwendungen wo schwierig sind in der Aufteilung (Terminierungspunkte). Aber dann bin ich jeweils knallhart: ein Terminieren eines Threads gibt's nicht.. Je nach Applikation entscheide ich mich dann für: a) der Anwender kann nicht abbrechen und muss warten, oder b) ich nehme kein Multithread sondern Multiprozess-Konzept mit IPC... ein Prozess kann dann problemlos gekillt werdenRHBaum hat geschrieben: oder der user gewinnt, und du musst den thread terminieren, mit allen konsequenzen.
Naja.. also eigentlich ist es ja egal, ob ich einen Thread nur runterfahren oder runterfahren+löschen muss.. nach "mWorker->shutdown()" noch ein "delete mWorker" ist ja keine grosse Sache.Sebastian Barth hat geschrieben: Eines hast du jedoch vergessen: Wer und wann deleted das Workerobjekt? Es hat ja kein QObject parent.
Code: Alles auswählen
QThread *context = new QThread();
mWorker = new Worker();
mWorker->moveToThread(context);
// Wenn der Kontext gelöscht wird, soll auch der Worker gelöscht werden:
mWorker->setParent(context);
// Den Thread-Kontext brauchen wir nach getaner Arbeit nicht mehr:
connect(mWorker, SIGNAL(workingDone()),
context, SLOT(quit()));
// Context soll nach Beendigung Selbstmord begehen:
connect(context, SIGNAL(finished()), context, SLOT(deleteLater()));
// weitere Signal-Connections..
// los gehts
context->start();