使用Qt&amp ;;恢复失败的HTTP下载QNetworkRequest

时间:2014-06-28 18:56:02

标签: c++ qt http qt5 qthttp

我正在尝试为我正在开发的应用添加自动更新功能。我已将此功能基于Qt HTTP Example(基于我的意思是我完全复制了此示例,然后从那里开始)。它下载了一个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完美提取。但是,如果连接失败并且应用程序重新启动下载,它 完成下载,我可以看到7-zip的ZIP文件的所有内容,但我无法提取它们(7-zip说了一些"试图在文件开始之前移动指针的东西。)

我假设我在某处发生了一个简单的逐个错误,就像在HTTP Range标题中一样。我已经看到了如何暂停&恢复下载at this blog,但他将流的内容写入文件暂停,而我将其流式传输到 httpReadyRead 中的文件中。我不知道这是否会导致问题。

为了测试,我一直在使用Sysinternals TCPView在下载过程中切断TCP连接。我不确定如何进一步调试,所以如果有更多信息有用,请告诉我!

1 个答案:

答案 0 :(得分:3)

所以今天我进行了更深入的调查。我原本以为不间断和中断版本的文件大小是相同的+ - 几个字节,但我错了。我下载了两个版本的文件,大小已经减少了大约2兆字节。

所以,我使用VBinDiff比较了它们(如果你不害怕控制台界面,这是一个不错的实用工具),这就是我发现的:

  • 文件已在地址 0x0154 21F3 停止匹配。
  • 错误文件中的地址与好文件中的地址 0x0178 1FD3 相匹配,并且它们会一直匹配到最后。
  • 因此,坏文件缺少 2,358,752 字节 - 这与我在资源管理器中看到的2MB近似值相符。

这证实当我尝试重新启动下载时,我正在跳过远程文件的重要部分。不确定发生了什么,我决定检查bytesWritten的值,我用它来跟踪我写入文件的字节数。这个值是我写入Range Request Header的值,所以它的值必须是不正确的。 (参见问题中的httpReadyRead()功能)。

所以我在设置Range Request Header之前添加了下面的代码:

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

调试代码,我很惊讶地发现

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

确认bytesWritten值不正确。事实上,当我使用文件大小而不是bytesWritten值时,下载能够重新启动并成功完成!

我不会再深入了,因为这对我有用。实际上,这将允许在应用程序的实例之间重新启动下载,因此在我看来这是一种更好的方法。

tl; Dr Don不跟踪写入文件的字节数。只需在重新启动失败的下载时检查文件大小。