QtConcurrent::run Access violation reading location

bagipro

Oh no, it's some system function

Its my functions
@void MainWindow::incLastLink()
{
lastLink++;
ui->label_7->setNum(lastLink);
}

void MainWindow::incGoodLinks()
{
goodLinks++;
ui->label_5->setNum(goodLinks);
}@

SGaist

Since it's pointer related, try to replace delete manager with
@manager->deleteLater();@

And maybe check that reply is a valid pointer (could be something related to it since it's the only other pointer visible in your code).

On a related note, I don't know for sure but I think It's your job to delete reply.

bagipro

How i can check reply pointer on valid? I didn't find some method like isValid. Or check like
@reply != NULL@

SGaist

Since it's a pointer:

@if (reply)@

Unless something is wrong, reply should either be NULL or pointing to a valid QNetworkReply (AFAIK, It shouldn't be NULL, but it's better to check to ensure that's not that that is causing you the access violation)

bagipro

I edit the code
@QString MainWindow::doRequest(QString url)
{
QString out;
QTimer timer;
QNetworkAccessManager* manager = new QNetworkAccessManager;
QEventLoop loop;
QObject::connect(&timer,SIGNAL(timeout()),&loop,SLOT(quit()));
QObject::connect(manager,SIGNAL(finished(QNetworkReply*)),&loop,SLOT(quit()));
QNetworkReply* reply = manager->get(QNetworkRequest(QUrl(url)));
timer.start(TIMEOUT);
loop.exec();
if(!timer.isActive())
{
reply->abort();
}
else
{
if(!reply) ::Beep(1000,1000);
if(reply->error() == QNetworkReply::NoError)
{
out = QString::QString( reply->readAll() );
}
reply->close();
}
reply->deleteLater();
manager->deleteLater();
return out;
}@

And i got error
!http://clip2net.com/clip/m0/1373183493-clip-9kb.png(1)!

So, i never heard beep

SGaist

I would check reply sooner, you might be calling abort on an invalid reply (though it would be strange)

bagipro

I edit
@loop.exec();
if(!reply)
{
::Beep(10000,10000);
exit(123456);
}@
But i got same violation error
Maybe I need to create exeptions?

SGaist

Are you sure everything you are accessing is valid (like links[i]) ?

bagipro

I think yes, i get errors in the middle of work process

bagipro

I runned it from cdb debugger. I see many errors, its showed at startup
@:0: предупреждение: Exception at 0x7c812aeb, code: 0x6ba: RPC server unavailable, flags=0x1 (execution cannot be continued) (first chance)

:0: предупреждение: Exception at 0x7c812aeb, code: 0x6ba: RPC server unavailable, flags=0x1 (execution cannot be continued) (first chance)

:0: предупреждение: Exception at 0x7c812aeb, code: 0x406d1388: Startup complete, flags=0x0 (first chance)

:0: предупреждение: Exception at 0x7c812aeb, code: 0x6ba: RPC server unavailable, flags=0x1 (execution cannot be continued) (first chance)

:0: предупреждение: Exception at 0x7c812aeb, code: 0x6a6: Invalid RPC server, flags=0x1 (execution cannot be continued) (first chance)@

So, on my pc RPC service is runned

SGaist

Then are you sure your server is running and that you can make as much connection that you are trying ?

bagipro

Oh, i haven't server, i have only this code.
And i don't click any buttons, i only run application and see errors

SGaist

Then I would :

First step: ensure the server is responding
Second step: try to access the server with one query
Third step: try with several queries in a row
Forth step: Try to parallelize your queries

That might include refactoring your code so that for example you'll use QtConcurrent::map to call your function with all the queries from a QList rather than accessing them yourself

bagipro

I have more than 100 servers, i can't do it ;(
But when i use 1 thread all works correctly. Sometimes some servers are disabled

SGaist

Then I would refactor the code so that it doesn't use a while loop. Create a QList of queries and either use map or mapped (depending on wether you return a value or not, I think you do return a value)

That would leave the entire parallelization to Qt and you'll only have to worry about retrieving the data.

bagipro

!http://clip2net.com/clip/m0/1373740625-clip-47kb.png(1)!
I launched it from debugger and i see it >_>
So many bugs, its increases on 5-10 every second

SGaist

Did you try starting with only one thread ? And then parallelizing with two and only then more ?

bagipro

Yes, i started in one thread. I found the bug in doRequest method.
I don't know, but he's somewhere in filing the request
But why? I do all correctly

SGaist

In that case, you can always use a request known to work for testing that part of your code while debugging the request build part

bagipro

I get critical exeption and warnings in
@loop.exec();@

And many warnings in
@QNetworkReply* reply = manager->get(QNetworkRequest(QUrl(url)));@