Загрузка файлов с QNetworkAccessManager на Android 6.0

@LetoLetoD · Регистрация: 24.01.2016

Author24 — интернет-сервис помощи студентам

filedownloader.h

C++ (Qt)

#ifndef FILEDOWNLOADER_H
#define FILEDOWNLOADER_H
 
#include <QObject>
#include <QByteArray>
#include <QNetworkAccessManager>
#include <QNetworkRequest>
#include <QNetworkReply>
#include <QDebug>
#include <QFile>
#include <QTemporaryFile>
 
class FileDownloader : public QObject
{
    Q_OBJECT
    QNetworkAccessManager m_WebCtrl;
    QString Type;
    QString m_data;
    QFile *pFile;
    QString Destination;
 
public:
    explicit FileDownloader(QString type, QUrl Url, QString Destination = "", QObject *parent = 0);
    virtual ~FileDownloader();
    QString downloadedData() const;
    QString getDestination() const;
 
signals:
    void downloaded(bool result);
 
private slots:
    void fileDownloaded(QNetworkReply* pReply);
 
};
 
#endif // FILEDOWNLOADER_H

filedownloader.cpp

C++ (Qt)

#include "filedownloader.h"
 
FileDownloader::FileDownloader(QString Type_, QUrl Url, QString Destination_, QObject *parent) :
    QObject(parent)
{   
    qDebug() << "FileDownloader constructor";
    Type = Type_;
    Destination = Destination_;
    if (Type == "toFile")
    {
        qDebug() << "FileDownloader constructor - 1";
        pFile = new QFile;
        pFile->setFileName(Destination);
        pFile->open(QIODevice::WriteOnly);
    }
    qDebug() << "FileDownloader constructor - 2";
    connect(&m_WebCtrl, SIGNAL (finished(QNetworkReply*)), this, SLOT (fileDownloaded(QNetworkReply*)));
    QNetworkRequest request(Url);
    qDebug() << "FileDownloader constructor - 3";
    if (QSslSocket::supportsSsl())
        request.setSslConfiguration(QSslConfiguration::defaultConfiguration());
    else
        qDebug() << "Error! Ssl unsupposed!";
    qDebug() << "FileDownloader constructor - 4";
    m_WebCtrl.get(request);
    qDebug() << "FileDownloader constructor - 5";
}
 
FileDownloader::~FileDownloader()
{
    qDebug() << "FileDownloader destructor";
}
 
void FileDownloader::fileDownloaded(QNetworkReply* pReply)
{
    qDebug() << "fileDownloaded";
    bool result;
    if (pReply->error() == QNetworkReply::NoError)
    {
        result = true;
        if (Type == "toFile")
        {
            pFile->write(pReply->readAll());
        }
        else
        {
            m_data = pReply->readAll();
        }
    }
    else
    {
        qDebug() << "fileDownloaded - error";
        result = false;
        m_data = pReply->errorString();
    }
    if (Type == "toFile" && pFile->isOpen())
    {
        pFile->close();
        pFile->deleteLater();
    }
    //pReply->deleteLater();
    emit downloaded(result);
}
 
QString FileDownloader::downloadedData() const
{
    return m_data;
}
 
QString FileDownloader::getDestination() const
{
    return Destination;
}

Используется этот класс в нескольких местах, проблемы возникают тут:

C++ (Qt)

void App::saveFileUrl(QString target)
{
    ...
    m_pFileDownloader = new FileDownloader("toFile", target, destination);
    connect(m_pFileDownloader, SIGNAL (downloaded(bool)), this, SLOT (SaveFileUrlDone(bool)));
}
 
void App::SaveFileUrlDone(bool result)
{
    qDebug() << "load icon finish";
    if (result)
    {
        Settings.setSetting("toolbarItemIconPath", "file://" + m_pFileDownloader->getDestination());
        emit signalQml("loadIconDone", "true");
    }
    else
    {
        qDebug() << m_pFileDownloader->downloadedData();
        emit signalQml("loadIconDone", "false");
    }
    delete m_pFileDownloader;
}

Проблема в следующем: при первом вызове файл скачивается и сохраняется, после я его использую - проблем нет. Но при втором обращении к функции saveFileUrl приложение крашится, лог такой:

Bash

FileDownloader constructor
FileDownloader constructor - 1
FileDownloader constructor - 2
FileDownloader constructor - 3
FileDownloader constructor - 4
W google-breakpad: ### ### ### ### ### ### ### ### ### ### ### ### ###
W google-breakpad: Chrome build fingerprint:
W google-breakpad: 1.0
W google-breakpad: 1
W google-breakpad: ### ### ### ### ### ### ### ### ### ### ### ### ###
F libc    : Fatal signal 11 (SIGSEGV), code 1, fault addr 0x0 in tid 12654 (QtThread)
I [MALI][Gralloc]: [+]r_hnd(0xd7b5e1e0), client(36), share_fd(35)
D GraphicBuffer: register, handle(0xd7b5e1e0) (w:1080 h:1920 s:1088 f:0x1 u:0x000b00)
D OpenGLRenderer: ProgramCache save to disk, size = 2

@Skjame · 22.12.2016, 10:05

Сообщение от LetoLetoD

pFile = new QFile;

Файл можно и в стэке создать, без выделения памяти, но конструкция с выделением в конструкторе, и освобождением в как-то мне не нравится. Если класс FileDownloader в стэке, или Вы его создаёте, как-то освобождение в деструкторе логичнее.
Ну или перед

C++ (Qt)

1	if (Type == "toFile")

сделайте

C++ (Qt)

1	pFile = Q_NULLPTR;

, а в деструкторе:

C++ (Qt)

if (pFile!=Q_NULLPTR){
    if (pFile->isOpened)
       pFile->close();
    delete pFile;
}

Кстати, раз вам нравится выводить отладочный вывод, то могу посоветовать добавить к выводу __FUNCTION__ и __LINE__.
Попробуйте, gcc давно эти директивы поддерживает.

@LetoLetoD · 23.12.2016, 11:44 **[ТС]**

Skjame, спасибо за замечания, все это тоже поправлю и буду использовать.

А моя основная проблема решилась, после того, как я перестроил FileDownloader и перестал его для каждой закачки создавать/удалять, теперь он живет все отведенное приложению время.

@LetoLetoD 5 / 5 / 6 Регистрация: 24.01.2016 Сообщений: 67
	23.12.2016, 11:44 [ТС]	3
	Skjame, спасибо за замечания, все это тоже поправлю и буду использовать. А моя основная проблема решилась, после того, как я перестроил FileDownloader и перестал его для каждой закачки создавать/удалять, теперь он живет все отведенное приложению время. 0