AC
Alexandru Codreanu22 июня 2020 г. 2:04

Qt / C++ QProcess получение результата вывода данных архиватора 7z и вывод в QProgressBar

QProcess, Qt, QThread, qProgressBar

Доброго времени суток.
Пишу проект для архивации и администрировании баз данных используя архиватор 7z.
При написании проекта возникли вопросы при работе с классом QProcess .
Есть форма на которой есть таблица QTableView и модель QStandardItemModel в которую помещаю данные о базах данных (путь, наименование архива, статус БД и прогресс бар).

При действии Выполнить происходит следующее:
- проверки на существование основного файла БД и т.д.
- запуск процесса 7z + аргументы
- считывание данных вывода процесса 7z в QByteArray
- преобразование данных вывода в int для определения процента выполнения и отрисовки в QProgressBar

Суть вопроса - при выборе только одной БД процесс архивации происходит нормально (отрабатывает и прогресс бар), но как только выбираю более одной БД выводит ошибку что данный процесс уже запущен.
Фрагмент кода:

void MainWindow::onExecCopy()
{
    // проверим если установлена модель
    if (!ui->tableView->model()){
        insertMessageInListMessage("Не заполнена таблица с базами данных !!!\n"
                                   "Для продолжения необходимо выбрать одно из действий:\n"
                                   " - указать путь к папке с базами данных (кнопка \"Папка с базами данных 1С\")\n"
                                   " - определить базы данных пользователя (кнопка \"Определить базы 1С\")", ICON_MSG_ERROR);
        return;
    }

    // проверим если указан каталог где будут храниться архивы
    if (ui->lineEditPathBases->text().isEmpty()){
        insertMessageInListMessage("Не указан каталог где будут храниться архивы с базами данных.", ICON_MSG_ERROR);
        ui->tabWidget->setCurrentIndex(1);
        return;
    }

    // устанавливаем делегат для презентации прогресса
    ui->tableView->setItemDelegateForColumn(COLUMN_PROGRESS, new ProgressBarDelegate(this));

    // перебор строк
    for (int row = 0; row < ui->tableView->model()->rowCount(); row ++) {

        // переменные для формирования имени архива и путь к папке где будет храниться архивы
        QVariant checkState = model->data(model->index(row, 0), Qt::CheckStateRole);           // для определения помечен объект или нет
        QString _path = model->data(model->index(row, 0), Qt::DisplayRole).toString();         // путь к БД
        QString strDateTime = QDateTime::currentDateTime().toString("dd.MM.yyyy_hh.mm.ss");    // время и дата для архива
        QString nameBase = model->data(model->index(row, 1), Qt::DisplayRole).toString() + "_" + strDateTime; // имя архива 7z

        // проверка если базу необходимо архивировать
        if (checkState == Qt::Checked){

            // проверяем если есть папка и файл
            if (!QFile::exists(QDir::fromNativeSeparators(_path) + "/1Cv8.1CD")){ // преобразуем разделители в "/"
                qDebug() << "Не найден путь к БД: " + QDir::fromNativeSeparators(_path) + "/1Cv8.1CD";
                insertMessageInListMessage("Отсутствует основной файл \"1Cv8.CD\" базы данных в папке - " +
                                           QDir::fromNativeSeparators(_path), ICON_MSG_WARNNING);
                continue; // продолжаем
            }

            // устанавливаем значение текущуй строки
            m_currentRow = row;

            // устанавливаем элемент для вывода прогресса в модель
            QStandardItem *itemProgresBase = new QStandardItem("0");
            itemProgresBase->setFlags(itemProgresBase->flags() ^ Qt::ItemIsEditable);
            model->setItem(row, COLUMN_PROGRESS, itemProgresBase);

            /*** QStringList arguments; *************************************
             **
             ** arguments << "a";          // добавить к архиву
             ** arguments << "-bsp1";      // Установить поток вывода для строки вывода / ошибки / хода выполнения
             ** arguments << "-ssw";       // принудительная упаковка файлов, которые в данный момент открыты для записи
             ** arguments << "-m0=lzma2";  // метод сжатия
             ** arguments << "-mx9";       // высокая степень сжатия (7), ультра высокая степень сжатия (9)
             ** arguments << "-r0";        // исключения, которые будут прописаны
             **
             ****************************************************************/

            // переменные для старта выполнения архивации и пути к архивам
            QString strExec7za = "7z.exe a -bsp1 -ssw -m0=lzma2 -mx9 -r0 ";      // команды 7za
            QString strPath    = "\"" + _path + "\\1Cv8.1CD\" ";                 // путь и файл БД
            QString strFolder  = ui->lineEditPathBases->text();                  // путь к папке где будут храниться архивы
            QString nameArhive = "\"" + strFolder + "\\" + nameBase + ".7z\" ";  // имя архива 7z

            // подключаемся и считываем вывод
            connect(process, &QProcess::readyReadStandardOutput, this, &MainWindow::_readyReadStandardOutput);
            connect(process, QOverload<int, QProcess::ExitStatus>::of(&QProcess::finished),
                    this, QOverload<int>::of(&MainWindow::processFinished));

            // стартуем процесс 7z
            process->start(strExec7za + nameArhive + strPath);

        }
    }
}

int MainWindow::getPercentFromOutput(QByteArray processData)
{
    if (!processData.isEmpty()){

        QString strCurrnet = QString(processData);       // преобразуем QByteArray в QString

        QRegExp reg("[0-9]{1,}\\%");                     // маска поиска
        int indexLast = strCurrnet.lastIndexOf(reg) - 1; // находим последнюю позицию -1 (-1 это "%")

        if (indexLast != -1){
            QString subString = strCurrnet.mid(indexLast, 2);
//            qDebug() << subString;
            return subString.toInt(); //возвращаем процент выполения
        }
    }
    return 0;
}

void MainWindow::_readyReadStandardOutput()
{
    // получаем процент выполнения
    m_currentProgress = getPercentFromOutput(process->readAllStandardOutput());
    auto item = model->item(m_currentRow, COLUMN_PROGRESS);
    item->setData(m_currentProgress, Qt::DisplayRole);
}

Попыталься следующим образом, но блокируется весь интерфейс и не обновляется процесс выполнения архивации в ячейки.

            if (process->waitForStarted(-1)){
                while (process->waitForReadyRead(-1)) {
                    int outPrecent = getPercentFromOutput(process.readAllStandardOutput());
                }
            }

Понимаю, что основной процесс(или поток - если это правильное выражение) уже запущен.
Как быть в данной ситуации ?

Рекомендуем хостинг TIMEWEB
Рекомендуем хостинг TIMEWEB
Стабильный хостинг, на котором располагается социальная сеть EVILEG. Для проектов на Django рекомендуем VDS хостинг.

Вам это нравится? Поделитесь в социальных сетях!

10
Александр Панюшкин
  • 22 июня 2020 г. 11:09

доброе утро
я с телефона, поэтому могу ошибиться
я правильно понимаю, что у вас один процесс, который вы создаете в конструкторе?
Если так - тогда ответ на поверхности - создавайте для каждой архивации отдельный процесс.
А так вы и коннеты задваиваете (при назатии на кнопку у вас создаются дополнительные коннекты, а старые не убиваются)

    Да вы правы у меня только один процесс.
    Первоначально хочу добиться чтобы архивация происходила поочередно (архивация 1БД -> архивация 2БД и т.д.).
    На будущее хочу добавить настройку "Количество потоков" для одновременной архивации, но это не сейчас.
    Насколько я понимаю мне необходимо класс QTimer для периодического повторение считывания _readyReadStandardOutput() , но тогда вопрос - необходимо установить значение интервала запуска в миллисекундах. Как узнать значение интервала ?

      Александр Панюшкин
      • 22 июня 2020 г. 12:24

      Я бы добавил отслеживание сигнала QProcess:finished(). Ну, и ошибки отслеживать - QProcess::readyReadStandardError().
      И перенёс все коннекты в место определения переменной.
      Тогда и костыли в виде QTimer будут не нужны.

      А вообще, я бы вынес выполнение всех процессов в отдельный класс. И в него отправлял бы задачи. И уже в этом классе реализовывал задуманное о кол-ве потоков. Т.е. отделил бы одно от другого. Но это можно сделать на следующем этапе.

        По поводу отслеживание сигнала QProcess:finished() и QProcess::readyReadStandardError() согласен что нужно добавить.
        Также согласен по поводу отдельного класса для выполнения всех процессов и кол.потоков (на будущее) - удобнее будет.
        Не понимаю:

        И перенёс все коннекты в место определения переменной.

          Александр Панюшкин
          • 22 июня 2020 г. 12:48

          У вас при нажатии на кнопку вызывается функция onExecCopy() (поправьте, если не прав). В этой функции вы создаёте коннекты.
          Т.е. каждый раз, когда вы нажимаете кнопку, создаются дополнительные коннекты, что приводит множественному вызову слотов.
          Коннекты имеет смысл устанавливать там же, где вы создаёте (определяете) переменную (по крайней мере, я стараюсь делать так).

            При нажатии кнопки "Выполнить" вызывается функция onExecCopy()
            В функции происходит перебор строк таблицы (цикл):
            - проверка если БД помечена для архивации

            checkState == Qt::Checked

            - запуск процесса архивации только для БД которая прошла проверку
            - создание коннектов для считывания вывода
            Я нажимаю только один раз кнопку Выполнить .
            У меня создаются коннекты внутри цикла и мне нужно дождаться выполнения считывания архивации (к примеру БД1), а после этого продолжить цикл для проверки следующей базы если она помечена для архивации и т.д.

              AC
              • 22 июня 2020 г. 21:05

              Я понимаю что для каждой архивации необходимо создать отдельный процесс (это легче сделать и на будущее), но я хочу добиться следующего(в процессе переборки строк в цикле):
              1. инициализация процесса архивации БД1 и не продолжать цикл до тех пор пока QProcess::finished - что-то наподоби функции process->waitForReadyRead(-1) (которая блокирует весь интерфейс программы и не обновляется процесс выполнения архивации)
              2. инициализация процесса архивации БД2 ... и т.д

              По поводу ...

              Коннекты имеет смысл устанавливать там же, где вы создаёте (определяете) переменную (по крайней мере, я стараюсь делать так).

              ... вы правы исправил.

                Александр Панюшкин
                • 22 июня 2020 г. 21:15

                Создайте массив, в который добавляйте задания (например, в виде адресов на базы данных).
                Добавляйте туда сколько нужно заданий - одной или десять, или сто тысяч.

                Создайте метод, который будет брать элемент из массива и удалять его.
                И полученное значение отправлять в процесс.
                Когда процесс завершился - метод вызывается снова (из слота, который у вас вызывается по QProcess::finished). И так до тех пор, пока массив с заданиями не опустеет.

                Так вы сможете отправлять задания в любой момент (даже тогда, когда уже что-то архивируется - они просто встанут в очередь).

                Ну, и с некоторыми доработками вы сможете добавить возможность архивации в несколько процессов.

                И, повторюсь, этот функционал лучше вынести в отдельную сущность, в отдельный класс.

                Надеюсь, понятно объяснил.

                  Спасибо, объяснили очень понятно.
                  Огромное спасибо за идею с массивом.
                  Думаю что должно выйти (в теории все ОК).
                  Как только реализую отпишусь и выложу код.

                    Александр Панюшкин
                    • 22 июня 2020 г. 23:44

                    удачи! :)

                      Комментарии

                      Только авторизованные пользователи могут публиковать комментарии.
                      Пожалуйста, авторизуйтесь или зарегистрируйтесь
                      Ua

                      Qt - Тест 001. Сигналы и слоты

                      • Результат:84баллов,
                      • Очки рейтинга4
                      Ua

                      Qt - Тест 001. Сигналы и слоты

                      • Результат:42баллов,
                      • Очки рейтинга-8
                      ОК

                      Qt - Тест 001. Сигналы и слоты

                      • Результат:47баллов,
                      • Очки рейтинга-6
                      Последние комментарии
                      ИМ
                      Игорь Максимов22 ноября 2024 г. 21:51
                      Django - Урок 017. Кастомизированная страница авторизации на Django Добрый вечер Евгений! Я сделал себе авторизацию аналогичную вашей, все работает, кроме возврата к предидущей странице. Редеректит всегда на главную, хотя в логах сервера вижу запросы на правильн…
                      Evgenii Legotckoi
                      Evgenii Legotckoi31 октября 2024 г. 23:37
                      Django - Урок 064. Как написать расширение для Python Markdown Добрый день. Да, можно. Либо через такие же плагины, либо с постобработкой через python библиотеку Beautiful Soup
                      A
                      ALO1ZE19 октября 2024 г. 17:19
                      Читалка fb3-файлов на Qt Creator Подскажите как это запустить? Я не шарю в программировании и кодинге. Скачал и установаил Qt, но куча ошибок выдается и не запустить. А очень надо fb3 переконвертировать в html
                      ИМ
                      Игорь Максимов5 октября 2024 г. 16:51
                      Django - Урок 064. Как написать расширение для Python Markdown Приветствую Евгений! У меня вопрос. Можно ли вставлять свои классы в разметку редактора markdown? Допустим имея стандартную разметку: <ul> <li></li> <li></l…
                      d
                      dblas55 июля 2024 г. 20:02
                      QML - Урок 016. База данных SQLite и работа с ней в QML Qt Здравствуйте, возникает такая проблема (я новичок): ApplicationWindow неизвестный элемент. (М300) для TextField и Button аналогично. Могу предположить, что из-за более новой верси…
                      Сейчас обсуждают на форуме
                      f
                      firstlunoxod15 февраля 2025 г. 13:46
                      Рисование на QGraphicsScene при зажатой кнопке мыши Подскажите, пожалуйста! Как данный класс можно дополнить, чтобы созданные объекты можно было перемещать мышкой по сцене?
                      Дмитрий
                      Дмитрий3 февраля 2025 г. 16:24
                      Создание deb-пакета. Как создать ярлык на рабочем столе после установки собственного deb-пакета? Всем привет. Сделал свой deb-пакет с программой. Всё устанавливается и работает. Ставлю по пути /usr/bin/my_application. Как для пользователя при установке пакета сразу создать ярлык на раб…
                      NW
                      Nayo Wai30 января 2025 г. 19:22
                      не запускается компьютер!!! Не запускается компьютер (точнее работает блок , но сам монитор вообще жесть)В общем я ничего с интернета не скачивала в последнее время. На компе никаких левых пр…
                      n
                      nkly3 января 2025 г. 12:52
                      Нужно запретить перемещение только некоторых итемов, остальные перемещать можно. Вопрос решен. Узнать QModelIndex элемента на который мы перетаскиваем другой элемент, можно с помощью функции indexAt(event->position().toPoint()) представления QTreeViev вызываемой в переопр…
                      M
                      Marsel17 августа 2023 г. 0:26
                      OAuth2.0 через VK, получение email Спасибо большое за помощь и простите за то что отнял время своей невнимательностью.

                      Следите за нами в социальных сетях