QNetworkAccessManager и QThread

[ecspertiza]

Подскажите пожалуйста как их подружить, если я организовываю скачивание в главном потоке то всё отлично, если пытаюсь сделать закачку в вторичном потоке то всё плохо вот исходник

Код:

#include <QtCore/QCoreApplication>

#include <QNetworkAccessManager>
#include <QNetworkReply>
#include <QDebug>
#include <QFile>
#include <QThread>


#define USAGE QT_TRANSLATE_NOOP("threadDownload::main", "usage: %s <link> <file name>")

class ThreadHttpDownload : public QThread
{
    Q_OBJECT

    public:
        ThreadHttpDownload(QObject *parent,QString link);

        QNetworkReply *reply;
        QFile file;

        QString link;
        void run();

    private:

    private slots:
        void readyRead();
        void replyFinished(QNetworkReply*);
};

ThreadHttpDownload::ThreadHttpDownload(QObject *parent,QString _link)
        :QThread(parent)
{
     link = _link;
}

void ThreadHttpDownload::run()
{

    file.setFileName("download.file");
    file.open(QIODevice::WriteOnly);

     QNetworkRequest request;  // Формируем запрос
     request.setUrl(QUrl(link));


     QNetworkAccessManager manager;
     manager.moveToThread(this);

     reply = manager.get(request); //Отправляем запрос
     reply->moveToThread(this);

     connect(&manager, SIGNAL(finished(QNetworkReply*)),
         this, SLOT(replyFinished(QNetworkReply*)));
     connect(reply,SIGNAL(readyRead()),this,SLOT(readyRead()));

     exec();
}


void ThreadHttpDownload::readyRead()
{
        qDebug() << reply->size();
        file.write(reply->readAll());
}

void ThreadHttpDownload::replyFinished(QNetworkReply *reply)
{
    if (reply->error())
    {
        qDebug() << "error download" << reply->errorString();
        QCoreApplication::quit();
    }else
    {
        qDebug() << "download complete";
        file.close();
        QCoreApplication::quit();
    }
}

int main(int argc, char *argv[])
{
    if (argc < 2)
    {
        qDebug(USAGE, argv[0]);
        return 0;
    }

    QCoreApplication a(argc, argv);

        ThreadHttpDownload *th = new ThreadHttpDownload(0,argv[1]);
        th->start();

    return a.exec();
}

#include <main.moc>

вот в этом куске всё падает

file.write(reply->readAll());

где я туплю , уже несколько дней разобраться немогу

[SABROG]

У меня ничего не падает.

[ecspertiza]

Уменя Qt 4.5.0 MinGW или gcc Windows Vista падает с такой ошибкой

Код:

C:\threadDownload4\debug>threadDownload5.exe http://127.0.0.1/AdobePhotoshopCS4.
rar
32768
32768
0
32768
32768
ASSERT: "d->ref == 1" in file tools\qlistdata.cpp, line 123

This application has requested the Runtime to terminate it in an unusual way.
Please contact the application's support team for more information.

или просто молча, ещё под OpenSUSE 11.1 собирал падает с ошибкой сигментирования.

может Qt обновить...

[SABROG]

У меня не упала видимо потому, что я в release собирал. Завтра посмотрю еще раз.

[ecspertiza]

Ну вобщем обновил я и Qt до 4.5.2 и MinGW вместе сним, пересобрал софт всё тоже. Пробовал в обоих сборках и в debug и release.

[ecspertiza]

Да забыл сказать падает с файлами большого размер я пытаюсь скачать 2гб.

[ecspertiza]

Кажется справился, помог метод "Мюнхаузена" В конструкторе потока прописал moveToThread(this) и вроде как заработало, SABROG, спасибо в твоём блоге прочитал )))

[SABROG]

Вообще странно всё это:

Код:

[New thread 4676.0x1218]
[New thread 4676.0x13d8]
[New thread 4676.0x12b8]
[New thread 4676.0x150]
2572
4380
5840
7300
7300
0
7300
7300
19020
0
0
0
0
0
0
0
0
0
0
0
4380
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
18980
14600
5840
7300
5840
2920
7460
4380
2920
7300
0
4380
1460
0
6488
7300
7300
5840
1460
7300
5840
4380
0
0
0
2920
2920
4380
0
4380
0
5840
7300
7300
5840
0
5840
1460
0
0
0
0
4380
2920
2920
4380
2920
2920
0
7300
5840
7300
7300
5840
0
5840
1460
4380
0
4380
2920
4380
2920
2920
4380
2920
4380
warning: Heap corruption detected at 01628E80

warning: Heap corruption detected at 01624518

warning: Heap corruption detected at 01624518

warning: HEAP[networkthread.exe]:
warning: HEAP: Free Heap block 1624510 modified at 1624520 after it was freed


Program received signal SIGTRAP, Trace/breakpoint trap.
0x7c90120f in ntdll!DbgUiConnectToDbg () from C:\WINDOWS\system32\ntdll.dll

Попробую троллям отрепортить.

[ecspertiza]

   Вообще странно всё это:

Немогу несогласиться, если троли чего дельного напишут, расскажи ,а то интересно уже))))

[SABROG]

Они где-то на 3-4 моих предыдущих багрепорта вообще ничего не ответили 

[SABROG]

В общем такая тема:

Hi Sabrog

Your ThreadHttpDownload object belongs to the main thread. So the slot ThreadHttpDownload::readyRead() above is run in the main thread, while the QNetworkAccessManager and QNetworkReply are running in the thread.

That means the signal readyRead() is queued before delivered to your object. That's why you get the zeroes in size.

And you're calling readAll() from a thread that is not reply's thread. That's a big violation of the QObject guidelines. That's why you're getting the crashes.

This is not a Qt bug.
--
Thiago Macieira - thiago.macieira (AT) nokia.com
  Senior Product Manager - Nokia, Qt Software
     Sandakerveien 116, NO-0402 Oslo, Norway

[ecspertiza]

Пасиба, тоесть на сколько я понимаю moveToThread(this) в конструкторе правельный выход, но вот всётаки интересно почему readReady() работает в основном потоке а не в вторичном, как то это неправельно

[SABROG]

Чет я сам понять не могу. Явно же делаем moveToThread на reply, значит должно работать.

[Ymilij]

Спасибо, эта тема помогла, наконец, разобраться с QNetworkAccessManager.