c++ - 使用 Qt 和 QNetworkRequest 恢复失败的 HTTP 下载

Question

我正在尝试为我正在开发的应用程序添加自动更新功能。我将此功能基于 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 范围 header 中。我看过一个如何暂停和恢复下载的示例 at this blog ，但他将流的内容写入文件 at pause，而我在 httpReadyRead 中将它们流式传输到文件中。我不知道这是否会导致问题。

为了测试，我一直在使用 Sysinternals TCPView 在下载期间切断 TCP 连接。我不确定如何进一步调试它，所以请告诉我更多信息是否有用!

Answer 1

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

因此，我使用 VBinDiff 对它们进行了比较(如果您不害怕控制台界面，这是一个很好的实用程序)这是我发现的:

• 文件在地址 0x0154 21F3 处停止匹配。
• 坏文件中的那个地址与好文件中的地址0x0178 1FD3匹配，并且它们继续匹配直到结束。
• 因此，错误文件丢失了 2,358,752 字节 - 这与我在资源管理器中看到的 2MB 近似值相符。

这证实了当我尝试重新开始下载时，我跳过了远程文件的重要部分。由于不确定发生了什么，我决定检查 bytesWritten 的值，我用它来跟踪我写入文件的字节数。这个值是我写入 Range Request Header 的值，所以它的值一定是不正确的。 (请参阅问题中的 httpReadyRead() 函数)。

所以我在设置范围请求 header 之前添加了下面的代码:

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

调试代码，惊奇的发现

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

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

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

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