Возобновление неудачной загрузки HTTP с помощью Qt & amp; QNetworkRequest

Я пытаюсь добавить возможности автообновления в приложение, которое я разрабатываю. Я основал эту функциональность от Пример Qt HTTP (и на основании я имею в виду, что я скопировал этот пример точно, а затем пошел оттуда). Он загружает ZIP-файл и затем извлекает его содержимое для исправления приложения.

Иногда при загрузке происходит сбой соединения, и загрузка останавливается. Чтобы быть немного более удобным для пользователя, я решил добавить возможности автоматического перезапуска в загрузчик, где он попытается перезапустить загрузку один раз, если она потерпит неудачу.

Вот основные моменты моего кода — имена методов совпадают с именами методов в примере:

void Autopatcher::httpReadyRead()
{
//file is a QFile that is opened when the download starts
if (file) {
QByteArray qba = reply->readAll();
//keep track of how many bytes have been written to the file
bytesWritten += qba.size();
file->write(qba);
}
}

void Autopatcher::startRequest(QUrl url)
{
//doResume is set in httpFinished() if an error occurred
if (doResume) {
QNetworkRequest req(url);
//bytesWritten is incremented in httpReadyRead()
QByteArray rangeHeaderValue = "bytes=" + QByteArray::number(bytesWritten) + "-";
req.setRawHeader("Range",rangeHeaderValue);
reply = qnam.get(req);
} else {
reply = qnam.get(QNetworkRequest(url));
}
//slot connections omitted for brevity
}

//connect(reply, SIGNAL(error(QNetworkReply::NetworkError)), this, SLOT(fileGetError(QNetworkReply::NetworkError)));
void Autopatcher::fileGetError(QNetworkReply::NetworkError error) {
httpRequestAborted = true;
}

void Autopatcher::httpFinished() {
//If an error occurred
if (reply->error()) {
//If we haven't retried yet
if (!retried) {
//Try to resume the download
doResume=true;
//downloadFile() is a method that handles some administrative tasks
//like opening the file if doResume=false
//and calling startRequest() with the appropriate URL
QTimer::singleShot(5000,this,SLOT(downloadFile()));
}
//If we have retried already
else {
//Give up :(
if (file) {
file->close();
file->remove();
delete file;
file = 0;
}
}
//If no error, then we were successful!
} else {
if (file) {
file->close();
delete file;
file = 0;
}
//Apply the patch
doPatch();
}
reply->deleteLater();
reply = 0;
}

Теперь, если загрузка завершается нормально, без перерывов, она работает просто отлично. ZIP извлекает отлично. Однако, если соединение не установлено и приложение перезапускает загрузку, оно делает закончите загрузку, и я вижу все содержимое ZIP-файла в 7-zip, но не могу их извлечь (7-zip сказал что-то вроде «попытался переместить указатель перед началом файла).

Я предполагаю, что где-то допустил простую ошибку, как в заголовке HTTP Range. Я видел пример того, как сделать паузу & возобновить загрузку в этом блоге, но он записывает содержимое потока в файл в паузе, тогда как я передаю их в файл в httpReadyRead. Я не знаю, вызывает ли это проблему.

Для тестирования я использовал Sysinternals TCPView для разрыва TCP-соединения во время загрузки. Я не уверен, как отлаживать это дальше, так что дайте мне знать, если будет полезно больше информации!

4

Решение

Поэтому сегодня я исследовал глубже. Первоначально я думал, что размеры файлов непрерывной и прерванной версий были одинаковыми + — несколько байт, но я ошибся. Я скачал две версии файла, и размеры были примерно 2 мегабайта.

Итак, я сравнил их, используя VBinDiff (хорошая утилита, если вы не боитесь консольного интерфейса) и вот что я нашел:

  • Файлы перестали совпадать по адресу 0x0154 21F3.
  • Этот адрес в плохом файле соответствует адресу 0x0178 1FD3 в хорошем файле, и они продолжали совпадать до конца.
  • Поэтому плохой файл отсутствовал 2358752 байт — что соответствует приближению 2 МБ, которое я видел в Проводнике.

Это подтвердило, что когда я попытался перезапустить загрузку, Я пропускал значительную часть удаленного файла. Не зная, что происходит, я решил проверить ценность bytesWritten, который я использовал, чтобы отслеживать, сколько байтов я записал в файл. Это значение было тем, что я записывал в заголовок запроса диапазона, поэтому его значение должно быть неверным. (См. httpReadyRead() функция в вопросе).

Поэтому я добавил код ниже, прямо перед настройкой заголовка запроса диапазона:

file->flush();
bytesWritten = file->size();

Отлаживая код, я с удивлением обнаружил, что

bytesWritten = 28,947,923
file->size() = 26,589,171

Подтверждая, что bytesWritten значение было неверным. На самом деле, когда я использовал размер файла, а не bytesWritten значение, загрузка смогла перезагрузите и закончите успешно!

Я не буду углубляться, так как это работает для меня. Фактически, это позволило бы возобновить загрузку между экземплярами приложения, поэтому, на мой взгляд, это лучший метод.

tl; dr Не отслеживать байты, записанные в файл. Просто проверьте размер файла при перезапуске неудачной загрузки.

3

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


По вопросам рекламы [email protected]