2

我在使用 QNetworkAccessManager 从获取请求中获得回复时遇到了一个奇怪的问题。

这是类的代码:

请求者.h

#ifndef REQUESTER_H
#define REQUESTER_H

#include <QtNetwork/QNetworkAccessManager>
#include <QtNetwork/QNetworkReply>
#include <QtCore/QtCore>
#include <QVector>
#include <QObject>
#include <QMessageBox>

class Requester : public QObject
{
    Q_OBJECT
    public:
        explicit Requester(QObject *parent = 0);
        ~Requester();
        QString get_last_reply();
        void send_request();
    private:
        QNetworkAccessManager *manager;
        QVector<QString> replies;
    private slots:
        void get_reply(QNetworkReply *reply);
        void get_reply_error(QNetworkReply::NetworkError err);
};

#endif // REQUESTER_H

请求者.cpp

#include "requester.h"

Requester::Requester(QObject *p)
        : QObject(p)
        , manager(new QNetworkAccessManager)
{
    QObject::connect(manager, SIGNAL(finished(QNetworkReply*)), this, SLOT(get_reply(QNetworkReply*)));
}

Requester::~Requester() {
    delete manager;
}

void Requester::get_reply(QNetworkReply *reply) {
    QObject::connect(reply, SIGNAL(error(QNetworkReply::NetworkError)), this, SLOT(get_reply_error(QNetworkReply::NetworkError)));
    QByteArray res = reply->readAll();
    QString data = res.data();
    replies.push_back(data);
    QObject::disconnect(reply, SIGNAL(error(QNetworkReply::NetworkError)), this,  SLOT(get_reply_error(QNetworkReply::NetworkError)));
    reply->deleteLater();
}

void Requester::get_reply_error(QNetworkReply::NetworkError err) {
    QMessageBox msg;
    msg.setText(QString::number(err));
    msg.setStandardButtons(QMessageBox::Discard);
    msg.exec();
}

QString Requester::get_last_reply() {
    if(!(replies.isEmpty())) {
        QString res =  replies.back();
        replies.pop_back();
        return res;
    }
    return "";
}

void Requester::send_request() {
    QNetworkRequest request;
    request.setUrl(QUrl("http://127.0.0.1"));
    request.setRawHeader("User-Agent", "Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.17 (KHTML, like Gecko) Chrome/24.0.1312.60 Safari/537.17");
    manager->get(request);
    //QMessageBox *msg = new QMessageBox;
    //msg->exec();
}

回复是通过这个函数写在 textEdit 中的

void MainWindow::ret_out(QString str) {
    ui->out->setText(str);
}

现在 main.cpp

int main(int argc, char *argv[])
{
    QApplication a(argc, argv);
    MainWindow w;
    Requester req;
    req.send_request();
    //QMessageBox *msg = new QMessageBox;
    //msg->exec();
    QString buf = req.get_last_reply();
    w.show();
    w.ret_out(buf);
    return a.exec();
}

使用此代码我有空的文本编辑。但是如果取消注释

QMessageBox *msg = new QMessageBox;
msg->exec();

在 Requester::send_request 或 main.cpp 然后 textEdit 包含服务器答案。

新更新

现在我有了这段代码,它可以工作了。也许这不是最好的变化,但我会非常乐意听取您的建议:)

void Requester::send_request(QUrl url) {
    QEventLoop loop;
    loop.connect(manager, SIGNAL(finished(QNetworkReply*)), this, SLOT(get_reply(QNetworkReply*)));
    loop.connect(this, SIGNAL(done()), &loop, SLOT(quit()));
    QNetworkRequest request;
    request.setUrl(url);
    request.setRawHeader("User-Agent", "Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.17 (KHTML, like Gecko) Chrome/24.0.1312.60 Safari/537.17");
    manager->get(request);
    loop.exec(QEventLoop::AllEvents);
    loop.disconnect(manager, SIGNAL(finished(QNetworkReply*)), this, SLOT(get_reply(QNetworkReply*)));
    loop.disconnect(this, SIGNAL(done()), &loop, SLOT(quit()));
}
4

2 回答 2

2

正如我之前在评论中所写,这段代码有几个问题,但主要问题是最后一个,所以如果你不关心其他建议,请跳到那个。

1) QVector 回复;

你应该考虑QStringList在这里使用。

2) QObject::connect(reply, SIGNAL(error(QNetworkReply::NetworkError)), this, SLOT(get_reply_error(QNetworkReply::NetworkError))); QByteArray res = reply->readAll();

您应该在完成信号管理的同时在构造函数中执行此操作。

3) QNetworkAccessManager *manager;

不需要在堆上分配这个对象,而且还有额外的负担,比如在类构造函数中手动构造,在类析构函数中手动删除。您可以简单地在堆栈上分配此对象而不会出现问题,这将导致代码稍微简单一些。

4) QByteArray res = reply->readAll();

您可能在这里不需要临时变量。

5) QString 数据 = res.data();

你需要确定编码,所以我建议写这样的东西:

QString 数据 = QString::fromUtf8(reply->readAll());

或者

QString 数据 = QString::fromLatin1(reply->readAll());

或者

QString 数据 = QString::fromLocal8Bit(reply->readAll());

6)回复.push_back(数据);

这不是 Qt 风格。你可以考虑这个:

回复。附加(数据);

7) w.ret_out(buf);

您在进入 Qt 事件循环之前要求这样做。当您的插槽处理程序被调用完成时,您应该使用 setText() 编写 ui 元素。试试这个:

void Requester::get_reply(QNetworkReply *reply) {
    QObject::connect(reply, SIGNAL(error(QNetworkReply::NetworkError)), this, SLOT(get_reply_error(QNetworkReply::NetworkError)));
    QByteArray res = reply->readAll();
    QString data = res.data();
    replies.push_back(data);
    QObject::disconnect(reply, SIGNAL(error(QNetworkReply::NetworkError)), this,  SLOT(get_reply_error(QNetworkReply::NetworkError)));
    reply->deleteLater();

    ui->out->setText(data); // You will need access the "ui" here obviously, or the main window needs access to the Requester object, and the signal-slot has to be connected in there.
}

您还可以通过实例化它来设置“本地” QEventLoop,但最好将它留在主应用程序事件循环中,即return a.exec();.

请注意,QtNetwork API 是异步的,它需要一个事件循环才能工作。它不是阻塞的,因为如果没有为最终用户提供额外的工作线程管理,这对于 UI 应用程序来说是不幸的。这就是神秘QDialog::exec()代码“修复”您的代码的原因,因为那是一个事件循环。但是,在这种特殊情况下,这似乎不是一个很好的解决方案,所以我建议使用前面提到的主应用程序事件循环,就像在这种情况下的绝大多数应用程序一样。

在您新上传的代码中,有两个问题:

...主窗口,SLOT(主窗口::ret_out(QString))...

1)您的请求者类在堆上似乎没有 MainWindow 对象。

2) MainWindow::ret_out(QString) 是错误的......就像你不使用其他插槽一样。您应该将 MainWindow:: 范围放在那里...

于 2013-10-06T08:21:38.980 回答
0

您错误地使用了信号和插槽。默认情况下,QObject::connect方法具有 Qt::AutoConnection 连接类型,这意味着当信号从不同的线程发出时,它会被发布到对象线程消息队列并在消息被泵送时执行。QNetworkRequest 是异步的,并且在不同的线程中运行,所以这似乎是你的情况。

在第一种情况下,您没有运行消息循环,因此没有机会处理信号。但是 QMessageBox::exec 运行它自己的消息循环,在这种情况下,信号被传递。

这意味着您可以在 QApplication::exec 之后发送请求,或者在 QObject::connect() 中使用 Qt::DirectConnection 标志。在第二种情况下,您应该注意使您的请求者线程安全。

最后一个问题是繁忙循环中的崩溃。我不知道为什么它会在没有调用堆栈的情况下发生,但我确信它会在您修复代码后消失。

于 2013-10-06T08:07:23.573 回答