QNetworkAccessManager within QThread

[Tomdarkness]

Hey,

I'm trying to make an application that is designed to grab data from a website in a thread but for some reason, despite having no errors in compile/run, the slot never seems to get fired.

Heres my code:

void itemthread::run() {
    int i;
    QTextStream out(stdout, QIODevice::WriteOnly);
    for(i=1; i < 10; i++) {
        out << i;
        QNetworkAccessManager *manager = new QNetworkAccessManager();
        connect(manager, SIGNAL(finished(QNetworkReply*)), this, SLOT(gotData(QNetworkReply *)));
 
        QString url_string = QString("http://www.someurl.com?arg=%1").arg(i);
        QUrl url(url_string);
 
        QNetworkRequest request;
        request.setUrl(url);
        request.setRawHeader("User-Agent", "Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1)");
 
        manager->get(request);
    }
}

void itemthread::run() {
    int i;
    QTextStream out(stdout, QIODevice::WriteOnly);
    for(i=1; i < 10; i++) {
        out << i;
        QNetworkAccessManager *manager = new QNetworkAccessManager();
        connect(manager, SIGNAL(finished(QNetworkReply*)), this, SLOT(gotData(QNetworkReply *)));

        QString url_string = QString("http://www.someurl.com?arg=%1").arg(i);
        QUrl url(url_string);

        QNetworkRequest request;
        request.setUrl(url);
        request.setRawHeader("User-Agent", "Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1)");

        manager->get(request);
    }
}

To copy to clipboard, switch view to plain text mode 

"itemthread" is a class that extends QThread. My slot "gotData" never seems to be fired, no matter how long you leave the program running. I had to remove this from new QNetworkAccessManager because it was causing a error during runtime ("QObject: Cannot create children for a parent that is in a different thread.").

How can I request URLs while inside a QThread?

Thanks,

Tom

[pitonyak]

My guess is that your thread does not have an event loop. Try a simple experiment and create a QEventLoop and call exec on it.

Also, I assume that you are taking care of delete the allocated network managers and such somewhere.

[weaver4]

Make sure that you have gotData(QNetworkReply *) is defined in your *.h file as a slot.

If you have it defined as a standard function the compiler will not give you an error but it will not work.

Something like:

slot:
void gotData(QNetworkReply *trigger);

BTW: I found this out the hard way.

[hassinoss]

I have the same problem, when i put the code in the constructor the slot is working but when i put it in the run method it doesn't work , someone know why ?

[anda_skoa]

someone know why ?

Yes, easy, you have done something wrong.

Cheers,
_

[hassinoss]

This is my code

Manager::Manager(QWidget *parent)
{
	m_url.setUrl("http://time.jsontest.com/");
	m_request.setUrl(m_url);
	connect(&m_networkManager, SIGNAL(finished(QNetworkReply*)), this, SLOT(onResult(QNetworkReply*)));*/
	m_networkManager.get(m_request); 
}
 
void Manager::onResult(QNetworkReply* reply)
{
	if(reply)
	{
		QString data = (QString) reply->readAll();
 
		......;
	}
}
 
void Manager::run()
{
	while(true)
	{
 
	}
}

Manager::Manager(QWidget *parent)
{
	m_url.setUrl("http://time.jsontest.com/");
	m_request.setUrl(m_url);
	connect(&m_networkManager, SIGNAL(finished(QNetworkReply*)), this, SLOT(onResult(QNetworkReply*)));*/
	m_networkManager.get(m_request); 
}

void Manager::onResult(QNetworkReply* reply)
{
	if(reply)
	{
		QString data = (QString) reply->readAll();

		......;
	}
}

void Manager::run()
{
	while(true)
	{
              
	}
}

To copy to clipboard, switch view to plain text mode 

whene i put the code in the constructor in the run method it doesn't work , there is any problem with doing this

[anda_skoa]

Well, assuming that "Manager" is a QThread, it has a weird parent (QWidget can't live in a secondary thread).

Also the networkmanager is created in the context of the thread running the manager constructor, so it will do its event processing in that thread, not in the context of the "Manager" thread.

Your run() method also looks suspiciously like it doesn't run the thread's event loop, but since the code is missing this is just a wild guess.

Cheers,
_

[hassinoss]

I changed the run method with a method with a loop for with 10 loops, and i see that it wait the end of the loop and it enter after that to the onResult method 10 times.

[anda_skoa]

So your loop enters event processing by calling exec() and each time your slot is invoked you quit() the event loop and continue in your outer loop?

Why not just count the invocations and quit on the last one?

Cheers,
_

[hassinoss]

No what i want is for each loop the onResult must be invoked, but im my case its invoked just until the end of the loop.

[anda_skoa]

Your posting is missing the code for the loop. Again.

What do you need the thread for anyway?

Cheers,
_