Fehler beim Lesen von Objekten aus Datei (die zweite)

[Anselmus]

Hi Folks,

hab ein kleines Problem, hatte zwar hier schon mal eine Lösung zu einem ähnlichen Problem, aber die Lösung funktioniert leider nicht...

hier das Problem

ich habe Objekte vom Typ "raum", die. solange das program läuft, in einer Linked List untergebracht sind:

static List raum_liste = Collections.synchronizedList(new LinkedList());

während des programs werden die räume folgendermaßen in die liste eingetragen:

    static void neuerRaum()
    {
        try
        {
        raum der_neue_raum = new raum();
        der_neue_raum.neuRaum();
        raum_liste.add(der_neue_raum);
        }
        catch(Exception e)
        {
            System.out.println("Fehler bei neuerRaum");
        }
    }

das geht prima, ich kann mir die liste der räume ansehen und ich kann sie speichern, so:

    static void speichereRaeume()
    {
        raum schreib_raum = new raum();
        try
        {
            System.out.print("...speichere Räume...");
            ObjectOutputStream raum_out = new ObjectOutputStream(new FileOutputStream("e:\\java\\oop\\freaks\\stundenplaneditor\\raeume.dat", true));
            for(int i=0; i<raum_liste.size(); i++)
            {
                schreib_raum = (raum) raum_liste.get(i);
                raum_out.writeObject(schreib_raum);
            }
            raum_out.close();
            System.out.print("gespeichert");
        }
        catch(Exception e)
        {
            System.out.print("Fehler bei raumspeichern");
        }
    }

das geht auch prima...
leider kann ich nur noch das erste objekt von raeume.dat laden
ich würde sie aber gerne alle laden um sie wieder in eine Linked List zu schreiben
hier ist mein (nicht funktionierender) Vorschlag:

    static void ladeRaeume()
    {
        raum ein_raum = new raum();
        try
        {
            System.out.print("\n...lade Räume...");
            ObjectInputStream raum_in = new ObjectInputStream(new FileInputStream   ("e:\\java\\oop\\freaks\\stundenplaneditor\\raeume.dat"));
            FileInputStream fis = new FileInputStream("e:\\java\\oop\\freaks\\stundenplaneditor\\raeume.dat");
            ein_raum = (raum) new ObjectInputStream(fis).readObject();
            while(ein_raum != null)
            {
                raum_liste.add(ein_raum);
                ein_raum = (raum) new ObjectInputStream(fis).readObject();
            }
            System.out.print("geladen1");
        }
        catch(EOFException ende)
        {
            System.out.print("geladen2");
        }
        catch(FileNotFoundException e)
        {
            System.out.println("raeume.dat ist nicht vorhanden\n");
        
        }
        catch(Exception e)
        {
            System.out.println("\nFehler bei Räume laden");
        }
        
    }

aber leider kommt beim zweiten befehl in der whileschleife ein fehler und es ist nur das erste objekt in der liste...

vielleicht kann mir jemand weiterhelfen

danke schonmal im voraus

gruß
stephan

 

[René Link]

Hi,

was du in deiner ladeRaeume-Methode falsch machst ist, dass du in der while
Schleife immer wieder ein neues ObjectInputStream-Objekt erstellst. Erstens führt
soetwas zu einem erhöten Speicherverbauch der nicht sein muss und dadurch, dass
bei jedem Schleifendurchlauf die Referenz zum Objekt verloren geht, hat der Garbage Collector
mehr zu tun.
Zweitens gibt es noch das Header-Problem. Wenn du einen ObjectOutputStream öffnest
schreibt dieser einen Header in den darunterliegenden OutputStream. Also
in die Datei. Öffnest du einen ObjectInputStream so liest dieser den Header wieder
aus und kann jetzt mit Hilfe der Header Daten die Objekte lesen. Öffnest du
jetzt noch ein ObjectInputStream, welcher den gleichen darunterliegenden InputStream
wie der erste hat, dann kann dieser den Header nicht mehr lesen, weil der Dateizeiger schon
darüber gerückt ist. Dem zweiten ObjectInputStream fehlen jetzt wichtige Informationen, weshalb
er eine Exception wirft.

Lose das Problem in dem du den ObjectInputStream öffnest und dann solange benutzt wie du
Objekte aus der Datei liest.

static void ladeRaeume() 
    { 
        raum ein_raum = new raum(); 
        try 
        { 
            System.out.print("\n...lade Räume..."); 
            ObjectInputStream raum_in = new ObjectInputStream(new FileInputStream  "e:\\java\\oop\\freaks\\stundenplaneditor\\raeume.dat")); 
            
            ein_raum = (raum) raum_in.readObject(); 
            while(ein_raum != null) 
            { 
                raum_liste.add(ein_raum); 
                ein_raum = (raum) raum_in.readObject(); 
            } 
            System.out.print("geladen1"); 
        } 
        catch(EOFException ende) 
        { 
            System.out.print("geladen2"); 
        } 
        catch(FileNotFoundException e) 
        { 
            System.out.println("raeume.dat ist nicht vorhanden\n"); 
        
        } 
        catch(Exception e) 
        { 
            System.out.println("\nFehler bei Räume laden"); 
        } 
        
    }

Hinweis
Wenn du Objekte serialisierst müssen diese die Serializeable Schnittstelle implementieren. Bei Kompositionen
auch alle Komponenten des Objekts. Sonst werden diese nicht serialisiert.

Am einfachsten implementierst du die Schnittstelle so:

public class SimpleClassToSerialize implements Serializeable {

  private void writeObject(ObjectOutputStream oos) throws IOException {
    oos.defaultWriteObject();
  }
  private void readObject(ObjectInputStream ois) throws ClassNotFoundException, IOException {
    ois.defaultReadObject();
  }

}

 

[Anselmus]

so klappts...

vielen dank

gruß
stephan