0

Возможно изобретаю велосипед, хочется узнать все ли корректно (возможно у моей реализации не все учтено), или можно лучше, проще?

Есть ф-ция MainWindow::someSlot, где определенную часть (MainWindow::veryLongFunc) необходимо делать отдельным потоком, чтобы интерфейс не зависал.

#mainwindow.h

#ifndef MAINWINDOW_H
#define MAINWINDOW_H

#include <QMainWindow>

namespace Ui {
    class MainWindow;
}

class MainWindow : public QMainWindow
{
    Q_OBJECT    
public:
    explicit MainWindow(QWidget *parent = nullptr);
    ~MainWindow();

    int otherFields() const;
    void setOtherFields(int otherFields);

public slots:
    void someSlot();
private:
    void veryLongFunc();
    Ui::MainWindow *ui;
    int _otherFields;
};

#endif // MAINWINDOW_H

mainwindow.cpp

#include "mainwindow.h"
#include "ui_mainwindow.h"
#include <QtConcurrent>
#include <future>

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

void MainWindow::someSlot() {
    /* some code */
    QFuture<void> t = QtConcurrent::run([this] { veryLongFunc(); });
    while(!t.isFinished()) {
        QApplication::processEvents();
    }
    /* some code */
}

void MainWindow::veryLongFunc() {
    /* very long func */
}

int MainWindow::otherFields() const
{
    return _otherFields;
}

void MainWindow::setOtherFields(int otherFields)
{
    _otherFields = otherFields;
}

MainWindow::~MainWindow() { delete ui; }
10
  • 2
    Не совсем вкурил зачем вам мьютекс, но тем не менее, в qt есть свой механизм для работы в таких ситуациях - QtConcurrent. Вот простейший механизм "опоточивания" функции doc.qt.io/qt-5/qtconcurrent.html#run 30 апр 2020 в 13:49
  • @AlexanderChernin исправил вроде. На счет мьютекса понял что не нужен. А строчка while (_flagThread) насколько корректна?
    – Kto To
    30 апр 2020 в 14:03
  • 1
    кстати если не надо ждать, завершения потока, то можно переменную и while убрать, а в veryLongFunc сделать сигнал...
    – eri
    30 апр 2020 в 14:18
  • 2
    while это ожидание завершения? тогда уж что-то типа QFuture<void> t = QtConcurrent::run([this] { veryLongFunc(); });t.waitForFinished(); 30 апр 2020 в 14:23
  • 1

1 ответ 1

2

Не надо делать while(!t.isFinished()) {} - вы тормозите главный поток приложения (со всем его GUI) до тех пор, пока поток функции не выполнит свою работу.

Как я указал в комментарии, вам нужно воспользоваться классом QFutureWatcher, который просигнализирует нам об завершении выполнения функции в потоке.

Заведите объект класса QFutureWatcher в MainWindow:

class MainWindow : ... {
private:
    QFutureWatcher<void> _watcher;
}

В конструкторе MainWindow:

MainWindow::MainWindow():... {
    connect(&_watcher, &QFutureWatcher<void>::finished, [&](){
        qDebug() << "VeryLongFunction's job is done!";
    });
}

Далее:

void MainWindow::someSlot() {
    QFuture<void> t = QtConcurrent::run([this] { veryLongFunc(); });
    _watcher.setFuture(t);
}

Таким образом, вы избежите отсутствия отклика от GUI во время работы ОченьДлиннойФункции

2
  • защитить другие поля от изменений, пока выполняется долгая ф-ция нужно?
    – Kto To
    1 мая 2020 в 8:10
  • 1
    @KtoTo при параллельном доступе к ним на изменение - мьютексом, или атомарностью. Если только на чтение, то ничего делать не надо 1 мая 2020 в 8:45

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.