How to know thread is running from different button ?

SGaist

Remove MyThread * from your constructor.

jsulm

@Hiloshi You should really read more carefully what others are writing. What @SGaist said is:

MainWindow::MainWindow(QWidget *parent) :
    QMainWindow(parent),
    ui(new Ui::MainWindow)
{
    ui->setupUi(this);   
    T3Thread = new MyThread; // Remove MyThread* - else you are declaring a local variable which is completely unrelated to the one in your header!
    T3Thread->start();
}

And after this change think about where to start the thread: in the code you posted you're doing it twice.

Hiloshi

Very happy to learn from here. Thanks everyone.

Hiloshi

I want to study in depth in this question. In the original question, I have a running T3Thread, mainwindow cannot reach T3Thread, thanks for @a-normal-worker 's suggestion, we can make T3Thread as a class member of mainwindow then mainwindow have ability to access T3Thread.

Can we use signal/slot instead of using class member ?

SGaist

What exactly do you want to do with your thread ?

Hiloshi

Dear @SGaist ,

I am thinking to create a breathing LED in my thread, "run" function is keep turn on/off LED.
I want to put one button on mainwindow to start the thread.

I do not want this button to be pressed twice ( or thread will create twice), so in the button I put:

void MainWindow::on_pushButton_3_clicked()
{
    if(T3Thread->isRunning()==false)
    {
        MyThread  *T3Thread = new MyThread;  // local variable
        T3Thread->start;
    }
    else
        qDebug()<<"T3 thread is running";
}

If I do not want to use class member, can I use signal/slot to know T3Thread is running or not ?

Thanks.

J.Hilk

@Hiloshi
ok, I think we talk at cross here.

Here's a basic setup to check if your thread is running when you press a Button, if its not running the button will start the thread:

//in your header:
example.h

#include <QThread>
#include <qDebug>

private:
QThread *thread;

//in your source file

//Constructor
{
    ui->setupUi(this);   
    thread = new QThread;
....
}


void on_pushButton_clicked(){
   if(thread->isRunning())
      qDebug() << "Thread is running";
   else{
      qDebug() << "Thread started";
      thread->start(); 
   }
}

//Destructor
{
  if(thread->isRunning()){
   thread->quit();
   thread-> wait();
  }
...
  delete ui;
}

SGaist

What is the problem with using a member variable to handle that thread object properly ?

Hiloshi

@SGaist , There is no problem using member variable. I just thinking if it possible to use signal/slot to reach the same purpose.

SGaist

There could be ways however you are talking about creating an object somewhere without any reference to it and still want to interact with it. That's bad by design.

Hiloshi

@SGaist , Understood, thanks for the help.