我试图找出如何在一个简单的Qt gui应用程序中使用线程

时间:2012-10-16 17:34:26

标签: c++ multithreading qt user-interface

在我的简单示例代码中,有一个启动和停止按钮以及一个计数器。当您按下开始时,会创建一个线程并使计数器递增,直到您按下停止。 click事件都在工作,但是当从dialog.cpp调用它时,线程本身不会启动,并且计数器永远不会递增。任何想法为什么??

代码来自这个人的教程,正如他在这里所做的那样,他的工作:http://www.voidrealms.com/viewtutorial.aspx?id=79

dialog.h

#ifndef DIALOG_H
#define DIALOG_H

#include "mythread.h"
#include <QDialog>

namespace Ui {
class Dialog;
}

class Dialog : public QDialog
{
    Q_OBJECT

public:
    explicit Dialog(QWidget *parent = 0);
    ~Dialog();
    MyThread *mThread;

private:
    Ui::Dialog *ui;

public slots:
    void onNumberChanged(int);

private slots:
    void on_pushButton_clicked();
    void on_pushButton_2_clicked();
};

#endif // DIALOG_H

dialog.cpp

#include "dialog.h"
#include "ui_dialog.h"

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

    mThread = new MyThread(this);
    connect(mThread,SIGNAL(NumberChanged(int)), this, SLOT(onNumberChanged(int)));
}

Dialog::~Dialog()
{
    delete ui;
}

void Dialog::onNumberChanged(int Number){
    ui->label->setText(QString::number(Number));
}

void Dialog::on_pushButton_clicked()
{
    mThread->start();
}

void Dialog::on_pushButton_2_clicked()
{
    mThread->Stop = true;
}

mythread.h

#ifndef MYTHREAD_H
#define MYTHREAD_H

#include <QThread>

class MyThread : public QThread
{
    Q_OBJECT
public:
    explicit MyThread(QObject *parent = 0);
    void run();
    bool Stop;

signals:
    void NumberChanged(int);

public slots:

};

#endif // MYTHREAD_H

mythread.cpp

#include "mythread.h"
#include <QtCore>

MyThread::MyThread(QObject *parent) :
    QThread(parent)
{
}


void MyThread::run() {

    for (int i = 0; i < 100000; i++) {
        QMutex mutex;
        mutex.lock();
        if (this->Stop) break;
        mutex.unlock();

        emit NumberChanged(i);
    }
}

谢谢!

1 个答案:

答案 0 :(得分:2)

在查看您链接的网站的示例代码后,我发现至少有两个问题:

1)正在使用Stop成员变量未初始化,将构造函数更改为应该解决您的主要问题:

MyThread::MyThread(QObject *parent) :
    QThread(parent),
    Stop(false)
{
}

2)Stop变量永远不会被重置,所以按下开始/停止按钮只能工作一次。如果break语句也重置了标志,它将更好地工作:

if (this->Stop)
{
    Stop = false;
    break;
}