c++ - 使用 QNetworkAccessManager 请求

Question

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

这是类的代码:

请求者.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();
}

使用这段代码，我得到了空的 textEdit。但是如果取消注释

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()));
}

Answer 1

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

1) QVector回复；

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

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

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

3) QNetworkAccessManager *管理器;

没有必要在堆上分配这个对象，而且在类构造函数中手动构造，在类析构函数中手动删除它也有额外的负担。您可以简单地在堆栈上分配此对象而不会出现问题，这样代码会稍微简单一些。

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

你可能不需要这里的临时变量。

5) QString data = res.data();

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

QString data = QString::fromUtf8(reply->readAll());

或

QString data = QString::fromLatin1(reply->readAll());

或

QString data = QString::fromLocal8Bit(reply->readAll());

6) replies.push_back(数据);

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

replies.append(数据);

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() 代码“修复”您的代码的原因，因为那是一个事件循环。然而，在这种特殊情况下，这似乎不是一个很好的解决办法，所以我建议使用前面提到的主应用程序事件循环，就像在这种情况下可能存在的绝大多数应用程序一样。

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

...MainWindow, SLOT(MainWindow::ret_out(QString))...

1) 您的请求者类在任何可用的堆上似乎都没有 MainWindow 对象。

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