1

我想从我的工作线程更新模态QProgressDialog。但是，如果我将对话框设置为模态，我的应用程序会崩溃（并且对话框没有显示任何进度）。如果我不这样做，一切都会好起来（但用户可以修改程序的其余部分，这可能会导致问题）。

我究竟做错了什么？

最小代码示例如下：

filereader qfr;    

QProgressDialog progress("Importing file.", "Cancel", 0, file_size);

connect(&qfr, &filereader::signalProgress, &progress, &QProgressDialog::setValue, Qt::QueuedConnection);

QThread worker_thread;

std::atomic<bool> success = false;

connect(&worker_thread, &QThread::started,
        [&]() {
    success = qfr.read_file(/* parameters */);
    worker_thread.quit();});

worker_thread.start();

//progress.setWindowModality(Qt::WindowModal); // Works only fine when this line is commented !!

while (worker_thread.isRunning()) {
    QApplication::processEvents();
    QThread::sleep(0);
}

progress.close();
4

2 回答 2

2

你的线程几乎毫无意义。它没有真正的目的。你也可以QApplication::processEvents在你的read_file方法中调用。但你不应该，打电话processEvents是不好的做法。

您应该做的是删除该while循环，并使您的进度对话框成为您班级的成员。我也不喜欢那个 lambda 的样子。我个人只会filereader::read_file用作插槽。

请注意，Qt::windowModal阻止输入到父窗口。您的进度对话框没有父级。所以你要么必须打电话progress->setModal(true)，要么progress.setWindowModality(Qt::ApplicationModal);。或为其设置父级。

这是一个小例子（它不是为您的应用程序量身定制的，但它应该为您指明正确的方向）：

#include <QtWidgets>

class Worker : public QObject
{
    Q_OBJECT
public:
    Worker(QObject *parent = nullptr) : QObject(parent){}
public slots:
    void simulateLongProcess()
    {
        for(int i = 0; i < 101; i++)
        {
            emit progressChanged(i);
            QThread::msleep(100);
        }
        emit finishedWorking(true);
    }
signals:
    void progressChanged(int progress);
    void finishedWorking(bool result);
};

class Widget : public QWidget
{
    Q_OBJECT
public:
    Widget(QWidget *parent = nullptr) : QWidget(parent)
    {
        setLayout(new QHBoxLayout);
        progress_dialog.setModal(true);
        progress_dialog.setAutoReset(false);
        progress_dialog.setCancelButton(nullptr);
        QThread *thread = new QThread(this);
        connect(thread, &QThread::started, &worker, &Worker::simulateLongProcess);
        connect(&worker, &Worker::finishedWorking, thread, &QThread::quit);
        connect(&worker, &Worker::progressChanged, &progress_dialog, &QProgressDialog::setValue);
        connect(&worker, &Worker::finishedWorking, &progress_dialog, &QProgressDialog::close);
        connect(&worker, &Worker::finishedWorking, this, &Widget::handleResult);
        QPushButton * start_button = new QPushButton("START");
        connect(start_button, &QPushButton::clicked, this, [=]
        {
            progress_dialog.show();
            thread->start();
        });
        layout()->addWidget(start_button);
        resize(400, 300);
    }

public slots:
    void handleResult(bool result)
    {
        // do something with the result
    }

private:
    QProgressDialog progress_dialog;
    Worker worker;
};

int main(int argc, char *argv[])
{
    QApplication a(argc, argv);
    Widget w;
    w.show();
    return a.exec();
}

#include "main.moc"
于 2018-04-04T12:21:55.427 回答
0

首先，如果你希望进度是模态的，为什么你需要在后台线程中执行文件的读取？您可以在主（GUI）线程中执行您的文件读取操作，只要您不时检查取消状态。

于 2018-04-04T09:03:37.650 回答