我正在尝试为我正在开发的应用程序添加自动更新功能。我已经基于这个功能的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范围标头中。我看过一个如何暂停&但是他在暂停时将流的内容写入文件,而我将它们流到 httpReadyRead
中的文件中。我不知道这是不是造成问题的原因。
为了测试,我一直在使用Sysinternals TCPView在下载期间切断TCP连接。我不确定如何进一步调试这个,所以让我知道如果更多的信息将是有用的!
所以今天我做了更深入的调查。我最初认为不间断和中断版本的文件大小是相同的+几个字节,但我错了。我下载了两个版本的文件,大小相差大约2兆。
所以,我用VBinDiff(一个很好的工具,如果你不害怕控制台界面)比较了它们,下面是我的发现:
- 文件在地址0x0154 21F3.
- 坏文件中的地址与好文件中的地址0x0178 1FD3匹配,直到最后。 因此,坏文件丢失了2,358,752字节-这与我在资源管理器中看到的2MB近似匹配。
这证实了当我尝试重新启动下载时,我跳过了远程文件的重要部分。不确定发生了什么,我决定检查bytesWritten
的值,我用它来跟踪我向文件中写入了多少字节。这个值是我写到范围请求头,所以它的值必须是不正确的。(见题中httpReadyRead()
函数)
所以我添加了下面的代码,在设置范围请求头之前:
file->flush();
bytesWritten = file->size();
调试代码,我惊奇地发现
bytesWritten = 28,947,923
file->size() = 26,589,171
确认bytesWritten
值不正确。事实上,当我使用文件大小而不是bytesWritten
值时,下载能够重新启动并成功完成!
我不打算再深入了,因为这对我来说很有效。事实上,这将允许在应用程序的实例之间重新启动下载,所以在我看来,这是一个更好的方法。
tl;dr不要跟踪写入文件的字节数。重新启动失败的下载时检查文件大小