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

Question

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

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();
}

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

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

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

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

Answer 1

Невзаимодействие потому что 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);
}