размерът на файла, изпратен от сокета, е 0 и достъпът е блокиран, освен ако не използвам System.exit(0), как да избегна това?

Имам клиент за таблет с Android, който изпраща и получава няколко файла между него и Java сървър, работещ на Windows 7. Всичко работи перфектно, с изключение на един странен проблем. един от файловете, в този пример, transfer.pdf, който е получен от клиента android tablet, не може да бъде отворен и показва размер 0. когато се опитам да отворя този файл, получавам съобщение за грешка от Adobe Reader „този файл е вече е отворен или се използва от друго приложение."

единственият начин да поправя този проблем е да използвам System.exit(0), за да убия цялото приложение, ако направя това, файлът показва правилния размер и се отваря без проблеми. но предпочитам да не използвам System.exit(0). Опитах всичко и нищо друго не работи.

блокът finally се използва за затваряне на всички входове и изходи на сокета и аз задавам променливите на файла и сокета на нула след затваряне.

някой друг имал ли е този проблем и как мога да накарам това да работи, без да използвам System.exit(0)?

код за сървъра

public class MultiThreader implements Runnable {
Socket socket;
File fileDirectory;
File file;
public int fileSizeFromClient;
FileOutputStream fos = null;
FileInputStream fis = null;
BufferedInputStream bis = null;
BufferedOutputStream bos = null;
DataInputStream dis = null;
DataOutputStream dos = null;
long length;

public MultiThreader(Socket socket){
    System.out.println("print out from multithreader class");
    this.socket = socket;

} // multiThreader

@Override
public void run() {

    System.out.println("multi threader started");

      //================================================================================================
    //==== action #1 read in transfer.pdf file from client ===========================================

    int bufferSize = 0;

    try {

     bis = new BufferedInputStream(socket.getInputStream());
     dis = new DataInputStream(bis);

     fileSizeFromClient = dis.readInt();
     System.out.println("file size for transfer.pdf from client is " + fileSizeFromClient);

      fileDirectory = new File("C:/DOWNLOAD/");
        if (!fileDirectory.exists()) {
            fileDirectory.mkdir();
      }
     file = new File("C:/DOWNLOAD/transfer.pdf");
      file.createNewFile();

       fos = new FileOutputStream(file);
       bos = new BufferedOutputStream(fos);
       dos = new DataOutputStream(bos);

       byte[] buffer = new byte[fileSizeFromClient];

      int totalBytesRead = 0;

        while(totalBytesRead < fileSizeFromClient){
            int bytesRemaining = fileSizeFromClient - totalBytesRead;
            int bytesRead = dis.read(buffer, 0, (int) Math.min(buffer.length, bytesRemaining));
            if(bytesRead == -1) {
                break;
            } else {
                dos.write(buffer, 0, bytesRead); 
                    totalBytesRead += bytesRead;
            }
        } // while

       // ===================================================================================================== 
       // === action #2 send transfer2.pdf file to client =====================================================

      file = new File("C:/DOWNLOAD/transfer2.pdf");
      file.createNewFile();

      fis = new FileInputStream(file);
      bis = new BufferedInputStream(fis);
      dis = new DataInputStream(bis);
      bos = new BufferedOutputStream(socket.getOutputStream());
      dos = new DataOutputStream(bos);

      length = file.length();

       System.out.println("file size for transfer2.pdf sent to client is " + length);

      dos.writeInt((int) length);

      int count = 0;

      buffer = new byte[(int)length];

      while((count = bis.read(buffer)) > 0){
          bos.write(buffer, 0, count);
      }

           //=====================================================================================
      //=== action #3 read in transferImage.jpg from client =================================

     bufferSize = 0;

     bis = new BufferedInputStream(socket.getInputStream());
     dis = new DataInputStream(bis);

     fileSizeFromClient = dis.readInt();
     System.out.println("file size for transferImage.jpg from client is " + fileSizeFromClient);

      file = new File("C:/DOWNLOAD/transferImage.jpg");
      file.createNewFile();

       fos = new FileOutputStream(file);
       bos = new BufferedOutputStream(fos);
       dos = new DataOutputStream(bos);

       buffer = new byte[fileSizeFromClient];

       totalBytesRead = 0;

        while(totalBytesRead < fileSizeFromClient){
            int bytesRemaining = fileSizeFromClient - totalBytesRead;
            int bytesRead = dis.read(buffer, 0, (int) Math.min(buffer.length, bytesRemaining));
            if(bytesRead == -1) {
                break;
            } else {
                dos.write(buffer, 0, bytesRead); 
                    totalBytesRead += bytesRead;
            }
        } // while

            //=====================================================================================

      } catch (IOException ex) {
        Logger.getLogger(MultiThreader.class.getName()).log(Level.SEVERE, null, ex);
      }  finally {

                        try {

                                   bis.close();
                                    fis.close();
                                    dis.close();
                                    fos.flush();
                                    fos.close();
                                    bos.flush();
                                    bos.close();
                                    dos.flush();
                                    dos.close();
                                    socket.close();
                                    file = null;
                                    fileDirectory = null;
                                   // System.exit(0);

                        } catch (IOException e) {
                            // TODO Auto-generated catch block
                            e.printStackTrace();
                        }

        }


 } // run

  } // MultiThreader

 public class Server {
 ServerSocket serverSocket;
 Socket socket;
 boolean runner = true;

 Server() throws IOException{

    serverRunner();
    System.out.println("server constructor started");

 } // Server constructor

 public void serverRunner() throws IOException {

    System.out.println("serverrunner started");

     try {

      serverSocket = new ServerSocket(6789, 100);

      runner = true;

        while (runner) {

            socket = serverSocket.accept();

            MultiThreader multi = new MultiThreader(socket);
            Thread t = new Thread(multi);
            t.start();

        }  // while runner
       } catch (IOException ex) {

    }

 } // serverRunner  

 } // class Server

person Kevik    schedule 07.01.2014    source източник
comment
Защо изграждате буфери върху вашия BufferedOutputStream?   -  person Elliott Frisch    schedule 07.01.2014
comment
по-добре ли е да не правя това?   -  person Kevik    schedule 07.01.2014
comment
Безсмислено е; BufferedOutputStream вече прави буфериране... просто пишете един байт наведнъж и го оставете да се справи с него.   -  person Elliott Frisch    schedule 07.01.2014
comment
@ElliottFrisch Но той не чете байт по байт. Той има масив от байтове, така че трябва да извика write(byte[], int, int). Съгласен съм, че не е задължително да е BufferedOutputStream при това обстоятелство, но записването на байт по байт в нещо никога не е добра идея.   -  person user207421    schedule 07.01.2014
comment
Премахнах всички bufferedOutputSteams и bufferedInputStreams и работи по същия начин. така че предполагам, че ако нямам нужда от буферираните потоци, няма да ги използвам в бъдеще   -  person Kevik    schedule 07.01.2014


Отговори (1)


Не използвайте множество потоци. Създайте единичен набор от потоци и ги използвайте през целия живот на сокета. В момента губите данни, когато замените стар BufferedInputStream с нов.

NB:

  1. Нямате нужда от буфер с размера на целия файл. 8192 ще свърши работа. Вашият код ще продължи да работи с всеки размер на буфера, по-голям от нула.
  2. Не е необходимо да извиквате File.createNewFile() преди new FileOutputStream(). Това е напълно излишно.
person user207421    schedule 07.01.2014