4

我正在尝试向我正在开发的应用程序添加自动更新功能。我将此功能基于我已经基于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 完美提取。但是，如果连接失败并且应用程序重新开始下载，它会完成了下载，我可以看到 7-zip 中 ZIP 文件的所有内容，但我无法提取它们（7-zip 说了类似“试图在文件开始之前移动指针）。

我假设我在某处犯了一个简单的错误，例如在 HTTP Range 标头中。我在这个博客上看到了一个如何暂停和恢复下载的示例，但是他在暂停时将流的内容写入文件，而我将它们流式传输到httpReadyRead. 我不知道这是否会导致问题。

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

4

1 回答 1

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 不要跟踪写入文件的字节。重新启动失败的下载时只需检查文件大小。

于 2014-07-01T20:43:40.623 回答