qApp->processEvents();

[wysota]

Use the second method, it's beautiful
Your code from the first one is incorrect, anyway. You might employ a local event loop to make it work, but it's a bad idea.

How to use the code? Well... after the exec() call you will either have your reply in the reply object ready to read or a timeout will have occured. The "if" block is there to check which one occured. Then simply do your stuff.

[rmagro]

many thanks Wysota..

Just to better understand, you mean that the code that follows

while(http->currentId() != 0) 
{
    QCoreApplication::processEvents();
}

while(http->currentId() != 0) 
{
    QCoreApplication::processEvents();
}

To copy to clipboard, switch view to plain text mode 

is also incorrect ??

Then, about the second method, I would like you to suggest how to modify the first part of code. I well understand the "if block".

Is it correct like this ?

QHttp *http;
QEventLoop q;
QTimer tT;
 
tT.setSingleShot(true);
connect(&tT, SIGNAL(timeout()), &q, SLOT(quit()));
connect(http, SIGNAL(finished(QNetworkReply*)),&q, SLOT(quit()));
 
//**********************
id = http->request(header, b);
//**********************
 
tT.start(5000); // 5s timeout
q.exec();
 
if(tT.isActive()){
       //*****************
       result = http->readAll();
       //*****************
       tT.stop();
} else {
     // timeout
}

QHttp *http;
QEventLoop q;
QTimer tT;
    
tT.setSingleShot(true);
connect(&tT, SIGNAL(timeout()), &q, SLOT(quit()));
connect(http, SIGNAL(finished(QNetworkReply*)),&q, SLOT(quit()));

//**********************
id = http->request(header, b);
//**********************
		
tT.start(5000); // 5s timeout
q.exec();
    
if(tT.isActive()){
       //*****************
       result = http->readAll();
       //*****************
       tT.stop();
} else {
     // timeout
}

To copy to clipboard, switch view to plain text mode 

Many thanks in advance

Roby

[rmagro]

When I use the code:

QEventLoop q;
		QTimer tT;
 
		tT.setSingleShot(true);
		connect(&tT, SIGNAL(timeout()), &q, SLOT(quit()));
		connect(http, SIGNAL(done(QHttp*)),&q, SLOT(quit()));
 
		//**********************
		id = http->request(header, b);
		//**********************
 
		tT.start(5000); // 5s timeout
		q.exec();
 
		if(tT.isActive()){
			//*****************
			result = http->readAll();
			//*****************
			tT.stop();
		} else {
			// timeout
		}

QEventLoop q;
		QTimer tT;
		    
		tT.setSingleShot(true);
		connect(&tT, SIGNAL(timeout()), &q, SLOT(quit()));
		connect(http, SIGNAL(done(QHttp*)),&q, SLOT(quit()));
		 
		//**********************
		id = http->request(header, b);
		//**********************
		        
		tT.start(5000); // 5s timeout
		q.exec();
		    
		if(tT.isActive()){
			//*****************
			result = http->readAll();
			//*****************
			tT.stop();
		} else {
			// timeout
		}

To copy to clipboard, switch view to plain text mode 

I can't get to the "if" block, then result is always empty..

What wrongs ??

Roby

[rmagro]

OK Wysota

I made a mistake with the SLOT signal, which is done(bool) in case of http->request..

SO, the code:

QEventLoop q;
		QTimer tT;
 
		tT.setSingleShot(true);
		connect(&tT, SIGNAL(timeout()), &q, SLOT(quit()));
		connect(http, SIGNAL(done(bool)),&q, SLOT(quit()));
 
		//**********************
		id = http->request(header, b);
		//**********************
 
		tT.start(600000); // 6minuto timeout
		q.exec();
 
		if(tT.isActive()){
			//*****************
			result = http->readAll();
			//*****************
			tT.stop();
		} else {
			QMessageBox::warning(this, "TIME OUT", ("time out"));
			// timeout
		}

QEventLoop q;
		QTimer tT;
		    
		tT.setSingleShot(true);
		connect(&tT, SIGNAL(timeout()), &q, SLOT(quit()));
		connect(http, SIGNAL(done(bool)),&q, SLOT(quit()));
		 
		//**********************
		id = http->request(header, b);
		//**********************
		        
		tT.start(600000); // 6minuto timeout
		q.exec();
		    
		if(tT.isActive()){
			//*****************
			result = http->readAll();
			//*****************
			tT.stop();
		} else {
			QMessageBox::warning(this, "TIME OUT", ("time out"));
			// timeout
		}

To copy to clipboard, switch view to plain text mode 

Seem to be ok..

What's your opinin?
Many thanks in advance..

Roby

[wysota]

Just to better understand, you mean that the code that follows

while(http->currentId() != 0) 
{
    QCoreApplication::processEvents();
}

while(http->currentId() != 0) 
{
    QCoreApplication::processEvents();
}

To copy to clipboard, switch view to plain text mode 

is also incorrect ??

No, I mean the loop is incorrect, it would exit too soon.

Is it correct like this ?

QHttp *http;
QEventLoop q;
QTimer tT;
 
tT.setSingleShot(true);
connect(&tT, SIGNAL(timeout()), &q, SLOT(quit()));

QHttp *http;
QEventLoop q;
QTimer tT;
    
tT.setSingleShot(true);
connect(&tT, SIGNAL(timeout()), &q, SLOT(quit()));

To copy to clipboard, switch view to plain text mode 

Why use QHttp at all if you have a better mechanism available? What's wrong with QNetworkAccessManager?

[rmagro]

Ok ...I can try to adapt my code to use QNetworkAccessManager
but do you thing something is wrong with the code that follows:

QEventLoop q;
      QTimer tT;
 
        tT.setSingleShot(true);
        connect(&tT, SIGNAL(timeout()), &q, SLOT(quit()));
        connect(http, SIGNAL(done(bool)),&q, SLOT(quit()));
 
        //**********************
        id = http->request(header, b);
        //**********************
 
        tT.start(600000); // 6minuto timeout
        q.exec();
 
        if(tT.isActive()){
            //*****************
            result = http->readAll();
            //*****************
            tT.stop();
        } else {
            QMessageBox::warning(this, "TIME OUT", ("time out"));
            // timeout
        }

QEventLoop q;
      QTimer tT;
            
        tT.setSingleShot(true);
        connect(&tT, SIGNAL(timeout()), &q, SLOT(quit()));
        connect(http, SIGNAL(done(bool)),&q, SLOT(quit()));
         
        //**********************
        id = http->request(header, b);
        //**********************
                
        tT.start(600000); // 6minuto timeout
        q.exec();
            
        if(tT.isActive()){
            //*****************
            result = http->readAll();
            //*****************
            tT.stop();
        } else {
            QMessageBox::warning(this, "TIME OUT", ("time out"));
            // timeout
        }

To copy to clipboard, switch view to plain text mode 

Thanks

[rmagro]

Any suggestion to adapt the fallowing code to use the QNetworkAccessManager Class ??

QEventLoop q;
      QTimer tT;
 
        tT.setSingleShot(true);
        connect(&tT, SIGNAL(timeout()), &q, SLOT(quit()));
        connect(http, SIGNAL(done(bool)),&q, SLOT(quit()));
 
        //**********************
        id = http->request(header, b);
        //**********************
 
        tT.start(5000); 5s 
        q.exec();
 
        if(tT.isActive()){
            //*****************
            result = http->readAll();
            //*****************
            tT.stop();
        } else {
            QMessageBox::warning(this, "TIME OUT", ("time out"));
            // timeout
        }

QEventLoop q;
      QTimer tT;
            
        tT.setSingleShot(true);
        connect(&tT, SIGNAL(timeout()), &q, SLOT(quit()));
        connect(http, SIGNAL(done(bool)),&q, SLOT(quit()));
         
        //**********************
        id = http->request(header, b);
        //**********************
                
        tT.start(5000); 5s 
        q.exec();
            
        if(tT.isActive()){
            //*****************
            result = http->readAll();
            //*****************
            tT.stop();
        } else {
            QMessageBox::warning(this, "TIME OUT", ("time out"));
            // timeout
        }

To copy to clipboard, switch view to plain text mode 

[wysota]

Ok ...I can try to adapt my code to use QNetworkAccessManager
but do you thing something is wrong with the code that follows:

Yes, I do. There is no way to properly read the incoming data.

Any suggestion to adapt the fallowing code to use the QNetworkAccessManager Class ??

There is nothing to adapt, just take the original code using QNetworkAccessManager.