QThread&QProcess Опции Подписка на тему Сообщить другу Версия для печати Скачать тему Подписка на этот форум Режимы отображения Переключить на: Древовидный Стандартный Переключить на: Линейный

[Azazello]

Всем привет
Подскажите пожалуйста, что не так в коде ниже, и как можно сделать вывод процесса, запущенного в отдельном потоке в textEdit (как я понял, надо сначала передавать данные в основной поток)
Задача - запуск из GUI приложения процесса, в отдельном потоке и вывод полученной, в результате его выполнения, информации в textEdit.

mainwindow.cpp

#include "mainwindow.h"
#include "ui_mainwindow.h"

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

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


void MainWindow::on_pushButton_clicked()
{
    ServerThread th;
    th.start();
}

mainwindow.h

#ifndef MAINWINDOW_H
#define MAINWINDOW_H

#include <QtGui/QMainWindow>
#include <QProcess>
#include <QString>
#include "thread.h"

namespace Ui
{
    class MainWindow;
}

class MainWindow : public QMainWindow
{
    Q_OBJECT

public:
    MainWindow(QWidget *parent = 0);
    ~MainWindow();

private:
    Ui::MainWindow *ui;

private slots:
    void on_pushButton_clicked();
};

#endif // MAINWINDOW_H

thread.cpp

#include "thread.h"
#include "mainwindow.h"
#include <QProcess>
#include <QTextCodec>
#include <QDebug>

void ServerThread::run()
{
    process = new QProcess(this);
    process->setProcessChannelMode(QProcess::MergedChannels);
    connect(process, SIGNAL(readyReadStandardOutput()), this, SLOT(output()));
    process->start("uname");
    exec();
}
void ServerThread::output()
{
    QByteArray bytes = process->readAllStandardOutput();     //как передать вывод из созданного потока в основной?
}

thread.h

#ifndef THREAD_H
#define THREAD_H
#include <QThread>

class QProcess;
 
class ServerThread : public QThread
{
    Q_OBJECT
 
public:
     void run();
 
public slots:
     void output();
 
signals:
 
private:
    QProcess *process;
    
};

#endif // THREAD_H

На данный момент при попытке компиляции, программа вылетает. Если трассировать вручную, то отладчик пишет:

QObject: Cannot create children for a parent that is in a different thread.
(Parent is ServerThread(0xbfffd8c4), parent's thread is QThread(0x804f818), current thread is ServerThread(0xbfffd8c4)
QThread: Destroyed while thread is still running

[Litkevich Yuriy]

Cannot create children for a parent that is in a different thread.

нельзя создавать дочерний объект, находящийся в другом, от родителя, потоке

//как передать вывод из созданного потока в основной?

в дополнительном потоке посылать сигнал. А в основном создать слот и соединить его с этим сигналом

[Azazello]

нельзя создавать дочерний объект, находящийся в другом, от родителя, потоке

Перевести-то я перевел, но только не понял, что этим мне хочет сказать компилятор...

в дополнительном потоке посылать сигнал. А в основном создать слот и соединить его с этим сигналом

Направление понял, буду пробовать. Спасибо.

[Litkevich Yuriy]

Перевести-то я перевел, но только не понял, что этим мне хочет сказать компилятор...

ну, схематично вот о чём:

QObject parentObject; // находится в одном потоке.
...
QObject child(parentObject); // находится в другом потоке

так делать нельзя

[Azazello]

А, все, дошло ) Спасибо!

[Azazello]

Пытаюсь сделать вывод из дочернего потока в textEdit основного:

Добавил:
mainwindow.cpp

void MainWindow::on_pushButton_clicked()
{
    ServerThread th;
    connect(&th,SIGNAL(updateOutput(QString)),this,SLOT(setOutput(QString)));
    th.start();
}

mainwindow.h

private slots:
    void setOutput(QString msg);

Функцию run() переделал:

void ServerThread::run()
{
    QProcess process;
    process.setProcessChannelMode(QProcess::MergedChannels);
    connect(&process, SIGNAL(readyReadStandardOutput()), this, SLOT(output()));
    process.start("uname");
    exec();
}

thread.cpp

void ServerThread::output()
{
    QByteArray bytes = process->readAllStandardOutput();
    emit updateOutput(bytes);
}

thread.h

signals:
     void updateOutput(QString msg);

По нажатию на кнопку, прога вылетает. Я вот думаю, может процесс не успевает сделать вывод? Но ведь по идее для этого вызов идет через emit, или я что-то не правильно понимаю?

[Litkevich Yuriy]

private slots:

у тебя слот не доступен для других объектов, он ЛИЧНЫЙ (private) данного класса. сделай его открытым (public)

[Azazello]

у тебя слот не доступен для других объектов, он ЛИЧНЫЙ (private) данного класса. сделай его открытым (public)

Юрий, спасибо за ответ.
Но, к сожалению, ситуацию это не меняет. Я пробовал объявлять его в public slots. Поток создается, но после этого приложение вылетает с "The program has unexpectedly finished."
Пробовал также запускать через отладчик, чтобы понять, в чем причина, но кроме того, что создается поток больше ничего не ясно...

[igor_bogomolov]

На сколько я понимаю, setOutput(QString) будет вызван в дочернем потоке, а в этом слоте по всей видимости идет обращение к GUI. С GUI нельзя работать вне основного потока, поэтому и получаем вылет.

Попробуй сделать следущие изменения в коде

connect(&th,SIGNAL(updateOutput(QString)),this,SLOT(setOutput(QString)), Qt::QueuedConnection);

private slots:
    void setOutput(const QString &msg);

signals:
     void updateOutput(const QString &msg);

[Litkevich Yuriy]

, Qt::QueuedConnection)

по идее Qt сама выбирает тип соединения и указывать явно не нужно.

[igor_bogomolov]

по идее Qt сама выбирает тип соединения и указывать явно не нужно.

Иногда нужно. MainWindow и ServerThread созданы с контексте одного потока поэтому м/у ниму тип соединения будет Qt::DirectConnection

Можно в принципе воспользоваться moveToThread, тогда тип соединения автоматически установится правильным

[Azazello]

igor_bogomolov, Litkevich Yuriy, большое спасибо за ответы.
Игорь, я попробовал сделать как Вы посоветовали, но, к сожалению, и это не помогло. Возможно, я просто что-то упустил, какую-нибудь мелоч. Выкладываю весь проект, может это внесет ясность, а то я уже и не знаю, что можно сделать. Перечитал уже сотню другую страниц форумов, ассистента и книжек, а результата никакого...

[igor_bogomolov]

Блин, слепой я. Ты же ServerThread на стеке создаёшь. После исполнения функции он у тебя удаляется.
Сделай так

void MainWindow::on_pushButton_clicked()
{
    ServerThread *th = new ServerThread;
    connect(th,SIGNAL(updateOutput(QString)),this,SLOT(setOutput(QString)));
    th->start();
}

[Azazello]

Действительно, поставив создание потока на обработчик кнопки, я не учел то, что он не останется по выходу из функции. Теперь ясно, что означало "QThread: Destroyed while thread is still running".
Отладчиком стали видится данные вывода в QByteArray, но упорно продолжает вылетать с unexpectedly finished... И трассировать что-то нормально не выходит, не понятно ничего

Я уже и, как советовали, выность ф-цию наружу пробовал,

MainWindow::MainWindow(QWidget *parent)
    : QMainWindow(parent), ui(new Ui::MainWindow)
{
        m_serverThread = new ServerThread();
        connect( m_serverThread, SIGNAL( updateOutput( QString ) ), this, SLOT( setOutput( QString ) ) );
        ui->setupUi( this );
}

объявляя класс в mainwindow.h и инициируя запуск в обработчике кнопки:

m_serverThread->start();

и все равно результат тот же

[igor_bogomolov]

Держи тестовый пример. Он правда для Windows, поэтому если ты делаешь для Linux строчки

    QTextCodec::setCodecForTr(QTextCodec::codecForName("system"));
    QTextCodec::setCodecForCStrings(QTextCodec::codecForName("IBM 866"));

могут быть лишними, просто закоментируй


---Добавлено--------------
Кратко, проблема просто в невнимательности. Ты делаешь доступ к несуществующему объекту

QByteArray bytes = process->readAllStandardOutput();

Объекта process у тебя не существует, т.к. ты его не создал. То что ты создаешь в run

void ServerThread::run()
{
    QProcess process;

это совершенно другой объект доступа до которого ты не имеешь.

P.S. Мой пример тоже не без недостатков, но для демонстрации сойдёт

Прикрепленные файлы

 test.zip ( 3.47 килобайт ) Кол-во скачиваний: 207

 

[Azazello]

Кратко, проблема просто в невнимательности.

И почему я так и думал Это всегда было моей основной проблемой.

Игорь, спасибо тебе громадное. И не только за пример, но и за все ответы, которые мне очень помогли не просто тупо переписать, а именно понять свою ошибку. Мой самый искренний респект тебе!