Correct usage of QThread

[MarekR22]

Rename InventoryThread to InventoryWorker or something similar.
You really don't need QThreadEx.

MainWindow::MainWindow(QWidget *parent) :
QMainWindow(parent),
ui(new Ui::MainWindow)
{
    ui->setupUi(this);
 
    invThread = new QThread(this);
    worker = new InventoryWorker; // parent is forbidden here
    worker->moveToThread(invThread);
    connect(ui->start,SIGNAL(clicked()), 
               invThread, SIGNAL(startTimer())); // add slot startTimer in InventoryWorker
    invThread->start();
}
 
MainWindow::~MainWindow() {
    worker->deleteLater();
    invThread->terminate();
    invThread->waitFor(3000);
}
 
InventoryWorker::InventoryWorker(QObject *parent, int h, QListWidget * lst) : QObject(parent)
{
    handle = h;
    list = lst;
    timer = new QTimer(this);
    list->connect(timer,SIGNAL(timeout()),SLOT(clear()));
    connect(timer,SIGNAL(timeout()),list,SLOT(clear()));
    timer->setSingleShot(false);
}
 
InventoryWorker::startTimer() {
    timer->start(1500);
}

MainWindow::MainWindow(QWidget *parent) :
QMainWindow(parent),
ui(new Ui::MainWindow)
{
    ui->setupUi(this);

    invThread = new QThread(this);
    worker = new InventoryWorker; // parent is forbidden here
    worker->moveToThread(invThread);
    connect(ui->start,SIGNAL(clicked()), 
               invThread, SIGNAL(startTimer())); // add slot startTimer in InventoryWorker
    invThread->start();
}

MainWindow::~MainWindow() {
    worker->deleteLater();
    invThread->terminate();
    invThread->waitFor(3000);
}

InventoryWorker::InventoryWorker(QObject *parent, int h, QListWidget * lst) : QObject(parent)
{
    handle = h;
    list = lst;
    timer = new QTimer(this);
    list->connect(timer,SIGNAL(timeout()),SLOT(clear()));
    connect(timer,SIGNAL(timeout()),list,SLOT(clear()));
    timer->setSingleShot(false);
}

InventoryWorker::startTimer() {
    timer->start(1500);
}

To copy to clipboard, switch view to plain text mode 

[Patrik]

How does your code call InventoryWorker:: DoInventory()? To put it simply what I need is:

every time ui->start emits clicked() then run InventoryWorker:: DoInventory() in a different thread

[MarekR22]

This code I gave you is not complete. I wrote it just to show you a pattern. I don't know what you code do I don't need to.

[Patrik]

Ok, after reading it twice I got it. Tank you all, you've been really helpful