QFile сохраняет файлы при выходе из программы, вообще не сохраняет файлы ~ 500 МБ

Question

QFile сохраняет файлы при выходе из программы, вообще не сохраняет файлы ~ 500 МБ

Да, ребята, это снова я. У меня есть следующий код:

void MainWindow::on_startButton_clicked()
{
QNetworkAccessManager *nam = new QNetworkAccessManager(this);
QNetworkReply *re = nam->get(QNetworkRequest(QUrl("http://somesite/ai/iai.jpg")));
QEventLoop loop;
QObject::connect(reply, SIGNAL(readyRead()), &loop, SLOT(quit()));
loop.exec();
ui->dbgOut->insertHtml("<font color='green'>OK</font><br>");
ui->dbgOut->insertHtml("##################################");
//save
QFile file("C:\\a.bin");
file.open(QIODevice::WriteOnly);
file.write(re->readAll());
file.close();
}

И у меня две проблемы:

Когда я нажимаю на кнопку, он загружает файлы, но не записывает их на жесткий диск. Я могу ждать 5, 10 минут и ничего. В настоящее время весь файл хранится в памяти программы. Если я закрою свою программу, они будут сохранены на диске.
Большие файлы (~ 500 МБ) вообще не сохраняются. Когда я закрываю свою программу, она мгновенно падает.

Как я могу отредактировать его, чтобы моя программа сохраняла загруженные файлы «в реальном времени»?

1

c++qeventloop qfile qnetworkaccessmanager qt5

Решение

Другие решения

Других решений пока нет …

Источник

Accepted Answer

Невзаимодействие потому что re->readAll() на устройстве с неизвестным размером происходит блокировка вызова. Он будет продолжать читать, пока запрос не будет завершен.

Проблема с большими файлами связана с увеличением байтового массива, в котором находится файл. В какой-то момент ваш байтовый массив будет, скажем, 400 МБ, а затем он должен будет увеличиться до 2-кратного размера, поэтому вам придется держать ~ 1 ГБ сразу, двумя большими блоками, и из-за фрагментации адресного пространства и запрос на выделение не удастся, и ваша программа вылетит

Небольшие изменения в вашем коде приводят к желаемому поведению: вы начинаете читать и писать сразу и просто связываете два:

class MainWindow {
...
// You shouldn't be creating a new network access manager for each request.
QScopedPointer<QNetworkAccessManager> m_nam;
...
};

void MainWindow::handleError(const QNetworkReply *) { ... }
void MainWindow::handleError(const QFile *) { ... }
void MainWindow::on_startButton_clicked()
{
// Lazily create the network access manager once.
if (!m_nam) m_nam.reset(new QNetworkAccessManager);
// The file is destructed, and thus flushed and closed, when the
// last shared pointer to this file is destructed.
QSharedPointer<QFile> output(new QFile("C:\\a.bin"));
if (!output->open(QIODevice::WriteOnly)) return;
QNetworkReply *reply = m_nam->get(QNetworkRequest(QUrl("http://somesite/ai/iai.jpg")));
// The lambda syntax creates a functor object that holds a copy
// of the reply pointer and the output file pointer.
// This functor will be destructed when the reply is destructed.
QObject::connect(reply, &QIODevice::readyRead, [this, output, reply]{
QByteArray data(reply->bytesAvailable(), Qt::Uninitialized);
qint64 in = reply->read(data.data(), data.size());
qint64 out = output->write(in);
if (in < 0) {
handleError(reply);
reply->deleteLater();
}
else if (out != in) {
handleError(output.data());
reply->deleteLater();
}
});
// The reply will self-destruct when finished, thus deleting the file
// instance.
QObject::connect(reply, &QNetworkReply::finished, reply, &QObject::deleteLater);
}

4