QNetworkAccessManager请求

Question

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

这是类代码：

requester.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

requester.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(应答、信号(错误(QNetworkReply：：NetworkError))，SLOT(get_reply_error(QNetworkReply::NetworkError)))；QByteArray res = reply->readAll()；

您应该在构造函数中与已完成的信号管理一起执行此操作。

( 3) QNetworkAccessManager *经理；

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

4) QByteArray res =res>readAll()；

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

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

您需要确保编码，所以我建议编写如下内容：

QString data =QString：：fromUtf8 8(答复->readAll())；

或

QString data =QString：：fromLatin1 1(答复->readAll())；

或

QString data =QString：：data Local8Bit(答复->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，插槽(MainWindow：：ret_out(QString)).

1)对于请求者类来说，堆上似乎没有可用的MainWindow对象。

2) MainWindow::ret_out(QString)是错误的.就像你也不使用其他插槽一样。你应该把MainWindow:：scope放在那里..。

Answer 2

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

在第一种情况下，您没有消息循环运行，因此没有处理信号的机会。但是QMessageBox::exec运行它自己的消息循环，在本例中信号被删除。

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

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