HttpURLConnection 出现意外结果 - 读取远程二进制文件

Question

我正在尝试从互联网上读取远程二进制文件（例如图像），如下所示：

HttpURLConnection connection = (HttpURLConnection) myUrl.openConnection(); //myUrl - URL object pointing for some location
if(connection.getResponseCode() == 200){
    File temp = File.createTempFile("blabla", fileName); //fileName - string name of file
    FileOutputStream out = new FileOutputStream(temp);
    int fileSize = Integer.parseInt(connection.getHeaderField("content-length"));
    int counter = 0;
    DataInputStream in = new DataInputStream(connection.getInputStream());
    byte ch[] = new byte[1024];
    System.out.println(counter);
    while((counter += in.read(ch)) > 0){
        out.write(ch);
        if(counter == fileSize){
            out.close();
            break;
        }
    }
}

在本地使用本地网络服务器（本地主机），它工作得很好。

但。 然后 myUrl 是某个远程 Web 服务器上文件的 URL - 它返回意外结果。 例如，从给定文件的来源来看，它似乎重复了一些包（我认为是因为以前的包或某些包被损坏），并且由于这种重复，生成的文件通常比原始文件大 10% 左右。 因此文件已损坏，无法使用图像查看器正确打开。

我该如何解决这个问题？

Answer 1

read 不一定读取整个缓冲区（特别是如果它位于流的末尾）。

因此，改变你的循环：

for (;;) {
    int len = in.read(ch);
    if (len == -1) {
        break;
    }
    out.write(ch, 0, len);
}

也许将该代码放在某个方法中。

另请注意：

• 此处使用 DataInputStream 没有意义（尽管 readFully 通常很有用）。

• 始终使用通常的习惯用法关闭资源（例如流）：

  最终资源资源 = acquire(); 
    尝试 { 
        使用（资源）； 
    } 最后 { 
        资源.close(); 
    } 
    

• 可能不会有太大区别，但 1024 的缓冲区大小有点小。 我倾向于任意默认为 8192。