套接字：JPEG 文件过早结束

Question

我正在尝试通过套接字将图像文件从服务器发送到客户端。 套接字以前用于将一些字符串从服务器发送到客户端（使用缓冲的输入/输出流）。

问题是无法正确接收图像文件，出现“JPEG 文件过早结束”错误。

服务器首先将文件大小发送给客户端，然后客户端创建该大小的 byte[]，并开始接收文件。

以下是代码：

服务器：

        DataOutputStream dos = new DataOutputStream(socket.getOutputStream());
        //Send file size
        dos.writeInt((int) file.length());

        BufferedInputStream bis = new BufferedInputStream(new FileInputStream(file));
        byte[] fileBytes = new byte[bis.available()];
        bis.read(fileBytes);
        bis.close();

        BufferedOutputStream bos = new BufferedOutputStream(socket.getOutputStream());
        bos.write(fileBytes);
        bos.flush();

客户端：

        DataInputStream dis = new DataInputStream(socket.getInputStream());
        //Receive file size
        int size = dis.readInt();

        BufferedInputStream bis = new BufferedInputStream(socket.getInputStream());
        byte[] fileBytes = new byte[size];

        bis.read(fileBytes, 0, fileBytes.length);

更有趣的是，如果我让服务器在发送文件大小和写入字节[]之间休眠大约 2 秒，则可以正确接收图像。我想知道服务器和客户端之间是否存在某种竞争条件

Answer 1

错误最有可能出现在此处：

byte[] fileBytes = new byte[bis.available()];

方法 available 不返回文件的大小。它可能只返回输入缓冲区的大小，该大小小于文件的大小。请参阅 BufferedInputStream 中该方法的 API 文档。

另外，下面一行中的 read 不能保证一次性读取整个文件。它返回实际读取的字节数，该数可能小于您要求的字节数。在客户端代码中，您以相同的方式使用read，而无需实际检查它是否读取了所有数据。

Answer 2

请使用 FileUtils 和 IOUtils 检查 commons-io。这应该会让工作变得更加容易。

http://commons.apache.org/io/

Answer 3

在 Java 中复制流的正确方法如下：

int count;
byte[] buffer = new byte[8192]; // more if you like, but over a network it won't make much difference
while ((count = in.read(buffer)) > 0)
{
  out.write(buffer, 0, count);
}

您的代码在多个点上无法在逻辑上匹配。

此外，available() 也不是确定文件大小或传入网络传输大小的有效方法 - 请参阅 Javadoc。它几乎没有正确的用途，而且这不是其中的两个。