Ошибка отправки файла через java-сокет

Question

У меня проблема с отправкой файлов через сокет java. Иногда код работает, иногда нет. Я тестирую блок while в обоих, кажется, что код отправляет все байты, но сервер не получает (но даже в этом тесте файл отправляется правильно). В этом случае сервер прекратил принимать данные. Все файлы составляют около 150 КБ. Я использую порт 9191.

Сервер:

while (true) { 
     try { 
      Socket socket = ss.accept(); 
      ObjectInputStream in = new ObjectInputStream(socket.getInputStream()); 
      String fileName = in.readUTF(); 

      FileOutputStream fos = new FileOutputStream(destinationPath + fileName); 
      byte[] buf = new byte[1024]; 

      int len; 
      while ((len = in.read(buf)) >= 0) { 
       fos.write(buf, 0, len); 
      } 
      fos.flush(); 

     } catch (Exception ex) { 
      ex.printStackTrace(); 
     } 
} 

Клиент:

try { 
     Socket socket = new Socket(host, port); 
     ObjectOutputStream out = new ObjectOutputStream(socket.getOutputStream()); 

     out.writeUTF(file.getName()); 
     out.writeLong(file.length()); 
     FileInputStream in = new FileInputStream(file); 
     byte[] buf = new byte[1024]; 

     int len; 
     while ((len = in.read(buf)) >= 0) { 
      out.write(buf, 0, len); 
     } 

     out.close(); 
     socket.close(); 

    } catch (Exception e) { 
     throw e; 
    } 

Answer 1

Сначала вы должны заменить ObjectInputStream и ObjectOutputStream с DataInputStream и DataOutputStream соответственно. Вы не сериализуете Java-объекты, и нет смысла использовать классы потоков, предназначенные для этой цели.

Во-вторых, вы отправляете длину файла в клиенте, но не специально читаете его на сервере. Вместо этого он добавляется в начало файла.

Answer 2

На клиенте вы говорите,

out.writeUTF(file.getName()); 
out.writeLong(file.length()); 

Но на сервере вы говорите,

String fileName = in.readUTF(); 
FileOutputStream fos = new FileOutputStream(destinationPath + fileName); 
byte[] buf = new byte[1024]; 
int len; 
while ((len = in.read(buf)) >= 0) { 
    fos.write(buf, 0, len); 
} 
fos.flush(); 

Вы не читали длину файла. И вам нужно убедиться, что вы прочитали все отправленные байты (или ваш файл будет поврежден). Также не забудьте указать close()FileOutputStream (или используйте try-with-resources, чтобы закрыть его для вас). Нечто подобное,

String fileName = in.readUTF(); 
try (FileOutputStream fos = new FileOutputStream(destinationPath 
     + fileName)) { 
    long size = in.readLong(); 
    byte[] buf = new byte[1024]; 
    long total = 0; 
    int len; 
    while ((len = in.read(buf)) >= 0) { 
     fos.write(buf, 0, len); 
     total += len; 
    } 
    if (total != size) { 
     System.err.printf("Expected %d bytes, but received %d bytes%n", 
       size, total); 
    } 
}