QFuture<void> Thread is not Starting

KroMignon

@Ketan__Patel__0011 said in QFuture<void> Thread is not Starting:

i was already try with global

I don't want to blame you, but what are your C/C++ skills?
I would also not use globals but class members.

Ketan__Patel__0011

@jsulm

Because I want to access it anywhere in application

that's why I am declare QFuture<void> Globally

jsulm

@Ketan__Patel__0011 said in QFuture<void> Thread is not Starting:

Because I want to access it anywhere in application

Sounds like bad design...

Ketan__Patel__0011

@jsulm

so what is the correct way ?

help me for it

jsulm

@Ketan__Patel__0011 Why do you need access to these variables everywhere in your program?
Make these variables members of the class and if other parts of the application REALLY need access to them implement getter methods returning pointers to these variables...

Ketan__Patel__0011

@jsulm
I am declare QFuture<void> variable is my class, means it is my class member

jsulm

@Ketan__Patel__0011 Do you mean you changed MainWindow::StartFunctions() ? Because in the code you posted before both variables were local variables.

Ketan__Patel__0011

@jsulm

Yes i changed

jsulm

@Ketan__Patel__0011 And does it work now or not?

Ketan__Patel__0011

@jsulm
Sorry to say it's not working

Ketan__Patel__0011

Let Me Share My Whole Code

mainwindow.h

namespace Ui {
class MainWindow;
}

class MainWindow : public QMainWindow
{
    Q_OBJECT

public:
    explicit MainWindow(QWidget *parent = nullptr);
    ~MainWindow();
private:
    void Function_1();
    void Function_2();
    void StartFunctions();
private slots:
    void on_BtnStart_clicked();
    void on_BtnStop_clicked();
private:
     bool CheckBool_1;
     bool CheckBool_2;
    Ui::MainWindow *ui;
    QFuture<void> future_1 ;
    QFuture<void> future_2 ;
};

mainwindow.cpp

void MainWindow::StartFunctions()
{
future_1 = QtConcurrent::run(this, &MainWindow::Function_1); /// This Function Is Started Successfully

future_2 = QtConcurrent::run(this, &MainWindow::Function_2); /// This Function in not Starting For Processing
}

void MainWindow::on_BtnStart_clicked() //// START
{
     StartFunctions();
}

void MainWindow::on_BtnStop_clicked() //// Stop
{
    CheckBool_1 = false;
        if(future_1 ->isRunning())  
        {
             future_1->setPaused(true);
             future_1->cancel();
        }

    CheckBool_2 = false;
        if(future_2->isRunning())  
        {
             future_2->setPaused(true);
             future_2->cancel();
        }
}

KroMignon

@Ketan__Patel__0011 said in QFuture<void> Thread is not Starting:

Sorry to say it's not working

What do you mean with "not working"?
As far as I can see from your code, on Function_1() start, you should see "START:_Function_1" on debug output and "START:_Function_2" on Function_2() start.

Do you have any of those message on debug console?

Ketan__Patel__0011

@KroMignon

i didn't get "START:_Function_2" this message on my debug console.

KroMignon

@Ketan__Patel__0011 said in QFuture<void> Thread is not Starting:

i didn't get "START:_Function_2" this message on my debug console.

And when you invert the lines in StartFunctions() to start Function_2() before Function_1()?

KroMignon

@Ketan__Patel__0011

Take from QConcurrent::run() documentation:

Note that function may not run immediately; function will only be run once a thread becomes available.

I would suggest you to use a dedicated thread pool to ensure to functions would be started in parallel.
Add a QThreadPool member to your MainWindow class and ensure it has enough threads for you parallel works:

   pool.setMaxThreadCount(2);

And change your code:

void MainWindow::StartFunctions()
{
    future_1 = QtConcurrent::run(&pool, this, &MainWindow::Function_1); 
    future_2 = QtConcurrent::run(&pool, this, &MainWindow::Function_2); 
}

Another improvement would be to check function is not already running:

void MainWindow::StartFunctions()
{
    if(!future_1.isRunning())
        future_1 = QtConcurrent::run(&pool, this, &MainWindow::Function_1); 
    if(!future_2 .isRunning())
        future_2 = QtConcurrent::run(&pool, this, &MainWindow::Function_2); 
}

Ketan__Patel__0011

@KroMignon

Thanks for Your replay

single Threadpool member for both QFuture<void> ?

JonB

@KroMignon said in QFuture<void> Thread is not Starting:

pool.setMaxThreadCount(2);

@Ketan__Patel__0011
It's interesting. If that is indeed needed (I don't doubt it is), https://doc.qt.io/qt-5/qthreadpool.html#maxThreadCount-prop :

The default maxThreadCount is QThread::idealThreadCount().

https://doc.qt.io/qt-5/qthread.html#idealThreadCount :

Returns the ideal number of threads that can be run on the system. This is done querying the number of processor cores, both real and logical, in the system. This function returns 1 if the number of processor cores could not be detected.

OOI, how many cores do you actually have? What OS are you?

Ketan__Patel__0011

@JonB

Thanks for Reply

I have 4 Core in My System
And I am using Windows 10

JonB

@Ketan__Patel__0011
It would be interesting to hear/confirm what static int QThread::idealThreadCount() returns on your system?

But to resolve your issue, you should be following what @KroMignon said above, pool.setMaxThreadCount(2);. Confirm that allows your Function_2() to run?

Ketan__Patel__0011

@JonB said in QFuture<void> Thread is not Starting:

static int QThread::idealThreadCount()

Where i call this function ?