Aplikacja wielowątkowa Qt nie kończy się po zamknięciu

Question

Piszę prosty program Qt do przechwytywania wideo z kamery (przy użyciu OpenCV). Używam obiektu QThread, który wykonuje pętle, przechwytuje obrazy i przesyła je do obiektu MainWindow. Działa to tak, jak powinno.

Problem polega na tym, że po zamknięciu aplikacja (tj. Naciśnięcie przycisku "X") zatrzymuje przechwytywanie kamery i gui znika. Ale program nadal działa w tle. Otrzymuję również ostrzeżenie w danych wyjściowych aplikacji:

QThread: Zniszczone, gdy wątek nadal działa.

Jak całkowicie zatrzymać aplikację po jej zamknięciu?

main.cpp

#include <QApplication> 
#include "application.h" 

using namespace cv; 

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

    Application app; 
    app.init(); 

    return a.exec(); 
} 

application.h

#include "mainwindow.h" 
#include "camerathread.h" 
#include "mathandler.h" 
#include "tools.h" 
#include "opencv2/core/core.hpp" 

#ifndef APPLICATION 
#define APPLICATION 

class Application : public MatHandler{ 
    MainWindow w; 
    CameraThread ct; 
public: 
    Application() { 
     w.setFixedSize(800,600); 
    } 

    void init() { 
     ct.setMatHandler(this); 
     ct.start(); 
     w.show(); 
    } 

    void handleMat(cv::Mat mat) { 
     QImage qImage = toQImage(mat); 
     w.setImage(qImage); 
    } 
}; 

#endif // APPLICATION 

camerathread

#include <QThread> 
#include "mathandler.h" 
#include "opencv2/highgui/highgui.hpp" 

#ifndef CAMERATHREAD 
#define CAMERATHREAD 

class CameraThread : public QThread { 
    MatHandler *matHandler; 
public: 
    ~CameraThread() { 
    } 

    void setMatHandler(MatHandler *h) { 
     matHandler = h; 
    } 

private: void run() { 
     cv::VideoCapture vc(0); 

     if (vc.isOpened()) { 
      for(;;) { 
       cv::Mat img; 
       vc >> img; 
       matHandler->handleMat(img); 
      } 
     } 
    } 
}; 

#endif // CAMERATHREAD 

programu C. Onsists więcej kodu niż to, ale tylko uwzględnione kod Myślę, że jest istotna dla pytania. W razie potrzeby opublikuję resztę.

Answer 1

Występują poważne problemy z kodem. Przede wszystkim w biegu funkcji niekończące się pętli spowoduje swój wątek, aby nigdy nie przestać, chyba że uda Ci zakończenie wątku przez siebie jak:

while(!finishThread) 
{ 
    cv::Mat img; 
    vc >> img; 
    matHandler->handleMat(img); 
} 

Należy ustawić finishThread do true gdy aplikacja będzie się zamykać. Po prostu podaj miejsce, w którym ustawisz wartość dla finishThread. Gdy chcesz zakończyć wątek, wyemituj sygnał, który jest podłączony do tego gniazda z wartością true. Potem czekać na gwint do końca prawidłowo przez kilka sekund i zmusić go do wypowiedzenia, jeśli nie ukończył:

emit setThreadFinished(true); //Tell the thread to finish 
if(!ct->wait(3000)) //Wait until it actually has terminated (max. 3 sec) 
{ 
    ct->terminate(); //Thread didn't exit in time, probably deadlocked, terminate it! 
    ct->wait(); //We have to wait again here! 
} 

Również nie należy bezpośrednio wywołać handleMat funkcji z innego wątku. Może to spowodować awarię aplikacji lub doprowadzić do niezdefiniowanego zachowania. Do tego należy użyć mechanizmu sygnału/szczeliny. Podłącz sygnał z wątku do tego gniazda i emituj sygnał wzdłuż argumentu za każdym razem, gdy chcesz go wywołać.

Inną kwestią jest to, że lepiej jest wyprowadzić klasę z QObject i użyć moveToThread. Można to zrobić w konstruktorze klasy:

th = new QThread(); 

this->setParent(0); 
this->moveToThread(th); 

QObject::connect(th,SIGNAL(started()),this,SLOT(OnStarted())); 
QObject::connect(th,SIGNAL(finished()),this,SLOT(OnFinished())); 

th->start(); 

Twoje inicjalizacji i zakończenia zadania powinny być wykonane w OnStarted() i OnFinished() szczelin odpowiednio. Możesz mieć funkcję pracującą, w której uruchamiasz powtarzalną operację.

Również w destruktorze klasy Application zakończyć wątek w sposób podobny do podanego.