[SOLUCIONADO] Error en Qt/C++ trabajando con hilos

[susi141]

Hola,

Estoy trabajando en Qt Creator con C++, tengo tres hilos, por ahora,el principal, uno para tomar una data y otro para subir al servidor.

Utilizo un Api para conectarme a la base de datos, por lo que cree una clase en c++, Api, donde delcaro todos los metodos que necesito, por ahora esto ha funcionado muy bien, pero ahora requiero hacer el llamado de un metodo en los otros hilos.

Entonces primero intente pasar por parametro la instancia api, que tiene toda la información de la session, cuando intento hacer esto me da un error que finaliza el programa inexperadamene justo cuando hace el llamdo del metodo con el request a la api de crear un registro.

Luego intente otra forma, con signals y slots, funciona para el primera llamado que hago, que al inicial el hilo, tengo un while que se repite constantemente, hay un delay de 1 minuto, luego hace lo que tiene que hacer y vuelve a hacer otro llamado un metodo de la Api, cuando hace esto me genera este error:

Código Qt:

QEventLoop::exec: instance 0x15719fb0 has already called exec()
 

Estoy super estancada en este error y no he podido avanzar mas. Dejo por aqui el codigo de donde se me presenta el error:

task es la clase principal que muestra una ventana, al precionar un boton en task comienza a correr un hilo CaptureThread, y desde CaptureThread llamo al tercer hilo UploadS3.

task.h

Código Qt:

#ifndef TASK_H
#define TASK_H
 
#include <QMainWindow>
#include <QSystemTrayIcon>
#include "capturethread.h"
#include "api.h"
#include <QJsonArray>
 
namespace Ui {
class Task;
}
 
class Task : public QMainWindow
{
    Q_OBJECT
 
public:
    explicit Task(Api* api, int projectid, QWidget *parent = 0);
    ~Task();
     Api* currentapi;
     QJsonArray tasks;
public slots:
      void iconActivated(QSystemTrayIcon::ActivationReason reason);
private slots:
      void on_btn_SwitchProject_clicked();
 
      void on_btnStart_clicked();
 
      void on_btnStop_clicked();
 
      // slots for the trayicon
      void logout();
      void resume();
      void suspend();
      void update();
      void log();
      void clog();
 
private:
    Ui::Task *ui;
    QSystemTrayIcon* trayIcon;
    QMenu* trayIconMenu;
    QAction* resumeAction;
    QAction* suspendAction;
    QAction* logoutAction;
    QAction* openTask;
    QAction* timeLoggedAction;
    CaptureThread* capThread;
 
};
 
#endif // TASK_H
 

task.cpp

Código Qt:

#include "task.h"
#include "ui_task.h"
#include "projects.h"
#include <QMessageBox>
#include <QMenu>
#include <QDebug>
#include <QJSonArray>
#include <QDesktopWidget>
#include <QNetworkCookieJar>
 
Task::Task(Api *api, int projectid, QWidget *parent) :
    QMainWindow(parent),
    ui(new Ui::Task)
{ 
   .........
 
    capThread = new CaptureThread(0);
}
 
 
void Task::on_btnStart_clicked()
{
    .....
        connect (capThread, SIGNAL(createLog()),this,SLOT(log()));
        connect (capThread->uploads3, SIGNAL(closelog()),this,SLOT(clog()));
 
        hide();
        
        capThread->flagpause = false;
        capThread->resume();
        capThread->start();
    }
}
 
....
 
void Task::update(){
    timeLoggedAction->setText("Time Logged: "+QString::number(capThread->countTime)+" min");
}
 
void Task::log(){
    currentapi->createLog(currentapi->jsonUser["id"].toInt(),currentapi->jsonSession["id"].toInt());
}
 
void Task::clog(){
    currentapi->closeLog(currentapi->jsonUser["id"].toInt(),currentapi->jsonLog["id"].toInt(),"capThread->url", "capThread->url");
}
 

CaptureThread.cpp

Código Qt:

CaptureThread::CaptureThread(QObject *parent):
    QThread(parent)
{
    this->countTime = 0;
    this->url = " ";
    this->uploads3 = new UploadToS3();
}
 
void CaptureThread::run(){
 
    while (true) {
           if (!flagpause){
               emit createLog();
            }
           this->msleep(60000);
           QDesktopWidget* dw = QApplication::desktop();
           QPixmap pixmap = QPixmap::grabWindow( dw->winId(), 0, 0, dw->width(), dw->height() );
 
           QDir mDir(QDir::homePath());
           QString format = "jpg";
           QString filename = "/screenshot_" + QString::number(QDate::currentDate().year())+ "_"
                   +QString::number(QDate::currentDate().month())+ "_"
                   +QString::number(QDate::currentDate().day())+ "_"
                   +QString::number(QTime::currentTime().hour())+ "_"
                   +QString::number(QTime::currentTime().minute())+ "_"
                   +QString::number(QTime::currentTime().second());
           QString mPath = QDir::homePath() + "/user";
           if (!mDir.exists(mPath))
                mDir.mkpath(mPath);
           pixmap.save(mPath+filename+".jpg", format.toLatin1().constData());
           this->url = mPath+filename+".jpg";
           uploads3->start();
           sync.lock();
           this->countTime ++;
           emit counTimeChange(this->countTime);
           if(flagpause)
                 pauseCond.wait(&sync);
           sync.unlock();
       }
}
 
void CaptureThread::pause() {
        sync.lock();
        flagpause = true;
        sync.unlock();
}
 
void CaptureThread::resume() {
        sync.lock();
        flagpause = false;
        sync.unlock();
        pauseCond.wakeAll();
}
 

Por ultimo UploadToS3.cpp solo hace emit closelog(); en el run().

No veo a que se debe ese error de Eventloop, por favor si alguien pudiera orientarme un poco, para resolver este error o si es que lo estoy haciendo mal, me pudieran indicar la forma correcta de manejar hilos en Qt.

[eferion]

Si quieres que te ayudemos tienes que intentar poner el código mínimo y suficiente para reproducir el problema... Mirar código sin poder compilarlo, depurarlo y trastearlo no es la mejor forma de encontrar errores.

[susi141]

De acuerdo, entonces a ver si asi es mas entendible.

Este es el error que estoy obteniendo.

Código Qt:

QEventLoop::exec: instance 0x15719fb0 has already called exec()
 

Al hacer esto en mi programa:

al oprimir un boton en la ventana principal:

Código Qt:

connect (capThread, SIGNAL(createLog()),this,SLOT(log()));
connect (capThread->uploads3, SIGNAL(closelog()),this,SLOT(clog()));
capThread->start();
 

capThread es una variable de tipo CaptureThread.

los slots

Código Qt:

void Task::log(){
    currentapi->createLog(currentapi->jsonUser["id"].toInt(),currentapi->jsonSession["id"].toInt());
}
 
void Task::clog(){
    currentapi->closeLog(currentapi->jsonUser["id"].toInt(),currentapi->jsonLog["id"].toInt(),"capThread->url", "capThread->url");
}
 

en el metodo run del hilo hago esto:

Código Qt:

while (true){
           if (!flagpause){
               emit createLog();
            }
           this->msleep(60000);
           uploads3->start();
}
 

uploads3 es una variable de tipo UploadS3.

en el segunfo hilo hago esto:

Código Qt:

emit closelog();
 

[eferion]

Iniciado por susi141

De acuerdo, entonces a ver si asi es mas entendible.

No se trataba de que explicases mejor o peor tu problema... sino de la imposibilidad de ver el código para localizar el problema:

Iniciado por eferion

Mirar código sin poder compilarlo, depurarlo y trastearlo no es la mejor forma de encontrar errores.

Seguimos en las mismas. Sigues sin poner una unidad de código compilable que reproduzca el error y yo sigo sin tener código que pueda compilar y probar.

[susi141]

Para que tu pudieras compilar y correrlo tendria que pasarte todo el proyecto y darte claves de acceso. No puedo simplemente extraer el parte del codigo porq no funcionaria, te generaria otros problemas.

Me podrias indicar o facilitarme un link para el manejo correcto de los hilos en Qt. Porq el problema ocurre solo cuando utilizo hilos diferentes, al probar en el hilo principal no me genera problemas.

[eferion]

http://doc.qt.io/qt-5/thread-basics.html

[susi141]

Gracias por tus respuestas, ya logre resolver el problema.