QNetworkAccessManager help needed! Class problem...

[TCB13]

Hi,

I'm trying to download a page from a server to my QT program, but I've been searching ways to do it and they don't really work a lot. I'm not an expert in QT, so be kind

I ear some suggestions under StackOverflow about how to use QNetworkAccessManager. So here it is my codes so far working (almost) correctly.

http2.cpp

#include "http2.h"
 
http2::http2(QObject *parent) :
    QObject(parent)
{
    m_manager = new QNetworkAccessManager(this);
    connect(m_manager,SIGNAL(finished(QNetworkReply*)),this,SLOT(httpdown(QNetworkReply*)));
 
    QNetworkRequest request;
    request.setUrl(QUrl("http://localhost/test.php"));
    request.setRawHeader("User-Agent", "MyOwnBrowser 1.0");
 
    m_manager->get(request);
}
 
void http2::httpdown(QNetworkReply* result)
{
 
       QByteArray data= result->readAll();
       QString str(data);
 
       qDebug() <<  str;
 
}

#include "http2.h"

http2::http2(QObject *parent) :
    QObject(parent)
{
    m_manager = new QNetworkAccessManager(this);
    connect(m_manager,SIGNAL(finished(QNetworkReply*)),this,SLOT(httpdown(QNetworkReply*)));

    QNetworkRequest request;
    request.setUrl(QUrl("http://localhost/test.php"));
    request.setRawHeader("User-Agent", "MyOwnBrowser 1.0");

    m_manager->get(request);
}

void http2::httpdown(QNetworkReply* result)
{

       QByteArray data= result->readAll();
       QString str(data);

       qDebug() <<  str;

}

To copy to clipboard, switch view to plain text mode 

http2.h

#ifndef HTTP2_H
#define HTTP2_H
 
#include <QObject>
#include <QDebug>
#include <QtNetwork>
#include <QNetworkReply>
 
class http2 : public QObject
{
    Q_OBJECT
public:
    explicit http2(QObject *parent = 0);
 
signals:
 
public slots:
    void httpdown(QNetworkReply* result);
private:
    QNetworkAccessManager* m_manager;
 
};
 
#endif // HTTP2_H

#ifndef HTTP2_H
#define HTTP2_H

#include <QObject>
#include <QDebug>
#include <QtNetwork>
#include <QNetworkReply>

class http2 : public QObject
{
    Q_OBJECT
public:
    explicit http2(QObject *parent = 0);

signals:

public slots:
    void httpdown(QNetworkReply* result);
private:
    QNetworkAccessManager* m_manager;

};

#endif // HTTP2_H

To copy to clipboard, switch view to plain text mode 

Now if I call it directly under main.cpp like this:

main.cpp

#include <QtCore/QCoreApplication>
#include "tcpserver.h"
#include "http2.h"
 
int main(int argc, char *argv[])
{
    QCoreApplication a(argc, argv);
 
    http2 h; // --> Working!!
 
    tcpserver mServer;
 
    return a.exec();
}

#include <QtCore/QCoreApplication>
#include "tcpserver.h"
#include "http2.h"

int main(int argc, char *argv[])
{
    QCoreApplication a(argc, argv);

    http2 h; // --> Working!!

    tcpserver mServer;

    return a.exec();
}

To copy to clipboard, switch view to plain text mode 

It works fine. However if I call it inside the tcpserver class like this:

tcpserver.cpp

#include "tcpserver.h"
#include "protocol.h"
#include "http2.h"
 
QTextStream in(stdin);
 
tcpserver::tcpserver(QObject *parent) :
    QObject(parent)
{
    server = new QTcpServer(this);
 
    [ ... Other Server Stuff ... ]
 
    http2 h; // --> Not Working :(
 
}

#include "tcpserver.h"
#include "protocol.h"
#include "http2.h"

QTextStream in(stdin);

tcpserver::tcpserver(QObject *parent) :
    QObject(parent)
{
    server = new QTcpServer(this);

    [ ... Other Server Stuff ... ]

    http2 h; // --> Not Working :(

}

To copy to clipboard, switch view to plain text mode 

The signal never happens... http2::httpdown(QNetworkReply* result) is never called, so I don't get the page contents.

What's wrong? :S

Thanks.

[TCB13]

I've a solution to the problem, new code inside the class:

tcpserver.cpp

#include "tcpserver.h"
#include "protocol.h"
#include "http2.h"
 
QTextStream in(stdin);
 
tcpserver::tcpserver(QObject *parent) :
    QObject(parent)
{
    server = new QTcpServer(this);
 
    [ ... Other Server Stuff ... ]
 
    // http2 h; // --> OLD non working code
    http2 *h = new http2(this); // --> New working
 
}

#include "tcpserver.h"
#include "protocol.h"
#include "http2.h"

QTextStream in(stdin);

tcpserver::tcpserver(QObject *parent) :
    QObject(parent)
{
    server = new QTcpServer(this);

    [ ... Other Server Stuff ... ]

    // http2 h; // --> OLD non working code
    http2 *h = new http2(this); // --> New working

}

To copy to clipboard, switch view to plain text mode 

This way it works fine.

[squidge]

This important thing is: Do you know WHY it didn't work before but does now?

[TCB13]

This important thing is: Do you know WHY it didn't work before but does now?

Well... I guess the problem is I was allocating it locally (instead of dynamically) so when tcpserver ended the other class was ended too.

Anyway, just a quick question: How can I keep the program waiting for data on http2 before returning to the tcpserver class?

This should be like a checkpoint were I need to get the data from the server and then keep running the tcpserver and use that data there. (So I can't just create a signal and trow the rest of the tcpserver into a slot...)

Usually and because I'm a C guy and new to QT/C++ and classes, I would do it like the following code, but it doesn't work...:

#include "http2.h"
bool httpdonne = false;
QByteArray finaldata;
 
http2::http2(QObject *parent, QByteArray url, QByteArray data) :
    QObject(parent)
{
    url.append(data);
 
    m_manager = new QNetworkAccessManager(this);
    connect(m_manager,SIGNAL(finished(QNetworkReply*)),this,SLOT(httpdown(QNetworkReply*)));
 
    QNetworkRequest request;
    request.setUrl(QUrl(url));
    request.setRawHeader("User-Agent", "MyOwnBrowser 1.0");
 
    m_manager->get(request);
 
    while ( httpdonne == false ) {
 
    }
 
    finaldata.append("HTTP: ");
    qDebug() << finaldata;
 
}
 
QByteArray http2::httpdown(QNetworkReply* result)
{
    QByteArray data = result->readAll();
    finaldata = data;
    httpdonne = true;
    return data;
}

#include "http2.h"
bool httpdonne = false;
QByteArray finaldata;

http2::http2(QObject *parent, QByteArray url, QByteArray data) :
    QObject(parent)
{
    url.append(data);

    m_manager = new QNetworkAccessManager(this);
    connect(m_manager,SIGNAL(finished(QNetworkReply*)),this,SLOT(httpdown(QNetworkReply*)));

    QNetworkRequest request;
    request.setUrl(QUrl(url));
    request.setRawHeader("User-Agent", "MyOwnBrowser 1.0");

    m_manager->get(request);

    while ( httpdonne == false ) {

    }

    finaldata.append("HTTP: ");
    qDebug() << finaldata;

}

QByteArray http2::httpdown(QNetworkReply* result)
{
    QByteArray data = result->readAll();
    finaldata = data;
    httpdonne = true;
    return data;
}

To copy to clipboard, switch view to plain text mode 

The code is always stuck in the while loop and nothing else happens.

Thanks a lot!

[TCB13]

some help pleasse!!!

[wysota]

Anyway, just a quick question: How can I keep the program waiting for data on http2 before returning to the tcpserver class?

I really would advise against that.

[TCB13]

It's the only way I can do it...

Because that's a checkpoint I need to check if the info is correct, otherwise I need to stop the server and clean some stuff in other classes.

So I need the program to stay on http2 until it has an answer and then return it to tcpserver class...

I can't just throw the rest of the program in a new custom slot and activate it by a signal... it doesn't make any sense in the case... it's just "wait for the data, go or quit"

The solution I made is not working the program is always in the while loop and nothing happens then.

Thanks.

[wysota]

It's the only way I can do it...

No, it's not, you just have to change the way you think.

Because that's a checkpoint I need to check if the info is correct, otherwise I need to stop the server and clean some stuff in other classes.

It doesn't mean your program has to "wait" before "returning to the tcpserver class". Especially since you're in a constructor.

So I need the program to stay on http2 until it has an answer and then return it to tcpserver class...

No, you don't. You only need to execute code B after code A is complete. It doesn't mean you have to wait for anything.

I can't just throw the rest of the program in a new custom slot and activate it by a signal...

Why not?

The solution I made is not working the program is always in the while loop and nothing happens then.

It won't work. You don't have an event loop running, no networking operations can ever complete this way.

[TCB13]

No, it's not, you just have to change the way you think.

Ok, then so help me here a little with logic...

It doesn't mean your program has to "wait" before "returning to the tcpserver class". Especially since you're in a constructor.

No, you don't. You only need to execute code B after code A is complete. It doesn't mean you have to wait for anything.

Why not? - "throw the rest of the program in a new custom slot and activate it by a signal..."

I'll have to do this type of "checkpoints" when the server starts, a client connects and more other places.. and I don't want to have my QT server interacting directly with my MySQL db.

If I throw the rest of the tcpserver start in another slot I'll be like "breaking" my program in more parts and I'm adding more complexibility to this. Because usually in C and my AVR/PIC stuff I'm used to run one thing at the time and call a function to do something and the periodically run checks to see if the sockets are still ok and stuff like that. Here it's like everything is happening at the same time..

Also, each time I will need to do something on the server I will need to "split" the current function into two in order to implement the slot/signal thing. Or can I do that all on the http2 side and leave everything on the tcpserver like it is? - How?

And.. if a client sends a message that I haven't already checked in the DB he will be able to send another and it will be processed right way. I'm usually more comfortable with the "one thing at a time" approach.

It won't work. You don't have an event loop running, no networking operations can ever complete this way.

Got it

Thanks.

[wysota]

I'll have to do this type of "checkpoints" when the server starts, a client connects and more other places.. and I don't want to have my QT server interacting directly with my MySQL db.

That's ok. But your code doesn't need to be synchronous.

If I throw the rest of the tcpserver start in another slot I'll be like "breaking" my program in more parts

That's a good thing.

and I'm adding more complexibility to this.

No, that's only your impression. Look at it that you simplify each step of the process by limiting it to a short sequence of operations.

Because usually in C and my AVR/PIC stuff I'm used to run one thing at the time and call a function to do something and the periodically run checks to see if the sockets are still ok and stuff like that. Here it's like everything is happening at the same time..

Yes. You are used to synchronous approach and here is an asynchronous approach. Synchronous approach wastes too much resources, asynchronous is better (however it seems more complicated if you're not used to it).

Also, each time I will need to do something on the server I will need to "split" the current function into two in order to implement the slot/signal thing.

That's not entirely correct but it is better to stick to such approach if you don't feel confident with Qt. One method triggers a request and another handles the response. You can write a simple dispatcher method that will handle all responses by calling appropriate methods.

And.. if a client sends a message that I haven't already checked in the DB he will be able to send another and it will be processed right way.

It depends how you implement it. The proper approach would be to block the user interface for the duration of executing the request.

I'm usually more comfortable with the "one thing at a time" approach.

Unfortunately it makes your user interface freeze and there is no way to abort an already running request.