EOFexception verursacht durch leere Datei

Question

Ich habe eine neue Datei erstellt, die leer ist. Jetzt, wenn ich es lese, wirft es mich eine EOFException, die unerwünscht ist. Stattdessen möchte ich, dass, wenn die Datei leer ist, würde es andere zwei Funktionen ausführen, die in if(roomFeed.size() == 0) Zustand sind. Ich könnte diese Anweisung in die EOFException catch-Klausel schreiben; Aber das ist nicht das, was ich tun will, denn jedes Mal, wenn die Datei gelesen wird und das Ende der Datei erreicht, wird sie diese Funktionen ausführen. Wenn die Datei Daten enthält, sollte sie das tun, was in else angegeben ist.

File fileChecker = new File("roomChecker.ser"); 
if(!fileChecker.exists()) { 
    try { 
     fileChecker.createNewFile(); 
    } catch (IOException e) { 
     e.printStackTrace(); 
     System.out.println("Unable to create new File"); 
    } 
} 

try(FileInputStream fis = new FileInputStream("roomChecker.ser"); ObjectInputStream ois = new ObjectInputStream(fis)) { 
    roomFeed = (List<roomChecker>) ois.readObject(); 
    System.out.println("End of read"); 
    if(roomFeed.size() == 0) { 
     System.out.println("your in null if statement"); 
     defaultRoomList(); 
     uploadAvailableRooms(); 
    } else { 
     for(int i=0; i<roomNumber.size(); i++) { 
      for(int j=0; j<roomFeed.size(); i++) { 
       if((roomNumber.get(i)).equals(roomFeed.get(i).getRoomNumSearch())){ 
        System.out.println("Reach Dead End for now"); 
       } else { 
        defaultRoomList(); 
        uploadAvailableRooms(); 
       } 
      } 
     } 
    } 
} catch (IOException ioe) { 
    ioe.printStackTrace(); 
} catch (ClassNotFoundException e) { 
    e.printStackTrace(); 
} 

Answer 1

All dies:

if(!fileChecker.exists()) { 
    try { 
     fileChecker.createNewFile(); 
    } catch (IOException e) { 
     e.printStackTrace(); 
     System.out.println("Unable to create new File"); 
    } 
} 

ist eine Verschwendung von Zeit, und es ist eine von zwei möglichen Ursachen für Ihre leere Datei Problem. Eine Datei nur so zu erstellen, dass Sie sie öffnen können und dann ein anderes Problem bekommen, anstatt das ursprüngliche Problem, dass die Datei nicht da ist, korrekt zu behandeln, ist keine vernünftige Strategie. Stattdessen sollten Sie dies tun:

if (fileChecker.isFile() && fileChecker.length() == 0) { 
    // file is zero length: bail out 
} 

und im folgenden Code, das:

try(FileInputStream fis = new FileInputStream(fileChecker); ObjectInputStream ois = new ObjectInputStream(fis)) { 
    // ... 
} 
catch (FileNotFoundException exc) { 
    // no such file ... 
} 
// other catch blocks as before. 

Natürlich noch Sie EOFException erhalten, wenn Sie die Datei zu Ende lesen, oder wenn die Datei ist unvollständig, und Sie müssen immer noch damit umgehen.