Lesen von Objekten aus Datei

[Lennyx]

Hallöchen!
Ich hab zur Zeit mal wieder ein Problem. Ich will in eine Datei Objekte schreiben und wieder auslesen.
In diesem Fall eine Collection (bzw die Objekte in der Collection, ist eigtl egal).
An sich kein Problem, aber beim Lesen wird eine Exception gecatchet, sobald ich readObject() aufrufen will, und ich finde nicht raus weshalb! Das Schreiben funktioniert problemlos, Datei existiert, mit Inhalt.

Vielleicht könnt ihr mir ja helfen ... Hier der Quellcode:

public static void writeFiguresToFile(Collection<GeoFigur> c, String filename) throws IOException{
		File f = new File("output/" + filename);
		try{
			f.createNewFile();
		}catch(IOException e){
			System.out.println("Datei existiert bereits.");
		}finally{
			OutputStream fos = null;
			try {
				fos = new FileOutputStream("output/" + filename); 
				ObjectOutputStream o = new ObjectOutputStream( fos ); 
				o.writeObject(c);
				o.flush(); 
			}catch(IOException io){
				System.out.println("In die Datei kann leider nicht geschrieben werden.");
			}finally{
				if(fos!=null){
					fos.close();
				}
			}
		}
	}
	
	public static Collection<GeoFigur> readFiguresFromFile(String filename) throws ClassNotFoundException, IOException{
		InputStream fin = null;
		Collection<GeoFigur> figuren = new HashSet<GeoFigur>();
		try {
			System.out.println("1");
			fin = new FileInputStream("output/" + filename);
			System.out.println("2");
			ObjectInputStream o = new ObjectInputStream(fin);
			System.out.println("3");
			while(o.readObject()!=null){
				System.out.println("4");
				if (o.readObject() instanceof Collection<?>){
					figuren = (Collection<GeoFigur>) o.readObject();
					System.out.println("5");
				}
			}
			System.out.println("6");
			return figuren;
		}catch(IOException io){
			System.out.println("Beim Lesen der Datei ist ein Fehler aufgetreten.");
			return figuren;
		}catch(ClassNotFoundException e){
			System.out.println("Klasse wurde nicht gefunden.");
			return figuren;
		}
		finally{
			if(fin!=null){
				fin.close();
			}
		}
	}

Die Sys.outs sind zur Überprüfung, wo genau der Fehler auftritt.

Die Ausgabe sieht wie folgt aus:

1
2
3
4
Beim Lesen der Datei ist ein Fehler aufgetreten.
[]

 

[eRaaaa]

if (o.readObject() instanceof Collection<?>){
                    figuren = (Collection<GeoFigur>) o.readObject();

In der if-Abfrage holst du dir ja schon die Collection! readObject dann nochmal aufzurufen, würde sich ja dann das nächste Object holen.
Schreibe also
Object obj = o.readObject();

und prüfe danach ob das Objekt von einem bestimmten Typen ist...

 

[Lennyx]

Bist du dir da sicher?

Jdf,egal ob, oder nicht, das ist nicht der Fehler, dadurch ändert sich nichts. Ich hatte auch spaßeshalber mal einfach nur o.readObject() als Zeile vor jeder anderen Ausführung drin, dann hat er da bereits den catchblock ausgeführt ..

 

[Bartleby]

Dann gib im catch doch mal e.printStackTrace() aus, vielleicht sagt das mehr.

 

[Gelöschtes Mitglied 5909]

Hab das hier bei mir auf der Platte gefunden, vielleicht hilfts dir ja (ist ja auch nicht schwer)

import java.io.BufferedInputStream;
import java.io.EOFException;
import java.io.File;
import java.io.FileInputStream;
import java.io.FileNotFoundException;
import java.io.IOException;
import java.io.InputStream;
import java.io.ObjectInputStream;
import java.util.Iterator;
import java.util.NoSuchElementException;

public class GenericInputStream<E> extends ObjectInputStream implements Iterator<E>, Iterable<E> {

	private E	next	= null;

	public GenericInputStream( InputStream in ) throws IOException {
		super(in);
	}

	public GenericInputStream( File file ) throws FileNotFoundException, IOException {
		super(new BufferedInputStream(new FileInputStream(file)));
	}

	@Override
	@SuppressWarnings( "unchecked" )
	public boolean hasNext() {
		try {
			next = (E) readObject();
		} catch ( EOFException e ) {
			next = null;
			return false;
		} catch ( Exception e ) {
			next = null;
			throw new RuntimeException(e);
		}
		return true;
	}

	@Override
	public E next() {
		if ( next == null ) {
			throw new NoSuchElementException();
		}
		return next;
	}

	@Override
	public void remove() {
		throw new UnsupportedOperationException();
	}

	@Override
	public Iterator<E> iterator() {
		return this;
	}

}

kann man dann so benutzen

    public static void main( String[] args ) throws Exception {
        GenericInputStream<Test> stream = new GenericInputStream<Test>(new File("test.ser"));
        for ( Test test : stream ) {
            
        }
    }

 

[Lennyx]

@raiL
danke für das angebot, kann ich nur leider nicht annehmen, da es nicht für ein kommerzielles projekt o. für den eigengebrauch gedacht ist, sondern eine aufgabe vonner uni ...
und da muss ich schon mein eigenes zeug verwenden
außerdem gehts mir ja auch bissl darum, den fehler zu finden ...

trotzdem danke!

achja, stacktrace:

java.io.EOFException
at java.ibjectInputStream$BlockDataInputStream.peekByte(Unknown Source)
at java.ibjectInputStream.readObject0(Unknown Source)
at java.ibjectInputStream.readObject(Unknown Source)
at pr1.a13.Aufgabe13.readFiguresFromFile(Aufgabe13.java:150)
at pr1.a13.Aufgabe13.main(Aufgabe13.java:33)

"Signals that an end of file or end of stream has been reached unexpectedly during input."

ich bilde mir ein, dass der stream korrekt geöffnet und noch nich geschlossen is

 

[eRaaaa]

Und ich bin immer noch der Meinung es liegt daran, was ich oben schon angesprochen habe.
Ersetze mal testweise deine Methode durch diese: ( auf instanceof jetzt mal verzichtet, kann ja immer noch rein später)

    public static Collection<GeoFigur> readFiguresFromFile(String filename) {
	InputStream fin = null;
	Object obj = null;
	try {
	    fin = new FileInputStream("output/" + filename);
	    ObjectInputStream o = new ObjectInputStream(fin);
	    obj = o.readObject();
	    System.out.println(obj);
	} catch (IOException e) {
	    e.printStackTrace();
	} catch (ClassNotFoundException e) {
	    e.printStackTrace();
	} finally {
	    try {
		fin.close();
	    } catch (IOException e) {
		e.printStackTrace();
	    }
	}
	return (Collection<GeoFigur>) obj;
    }

Und poste jetzt nochmal ALLE Fehlermeldungen!

 

[Lennyx]

Okay, ich bin jetzt noch ratloser als vorher.
Dein Quellcode funktioniert einwandfrei ...
Hab daraufhin meinen nochmal abgeändert und alles von dir eingebaut. Und jetzt funktioniert es wieder nich ... mannomann. Ich poste nochmal den Code und die Ausgabe ...

public static Collection<GeoFigur> readFiguresFromFile(String filename) throws ClassNotFoundException, IOException{
	    InputStream fin = null;
	    Object obj = null;
		try {
			fin = new FileInputStream("output/" + filename);
			ObjectInputStream o = new ObjectInputStream(fin);
			while(o.readObject()!=null){
				if (obj instanceof Collection<?>){
					obj = o.readObject();
				}
			}
			return (Collection<GeoFigur>) obj;
			
		}catch (IOException e) {
				e.printStackTrace();
				System.out.println("Beim Lesen der Datei ist ein Fehler aufgetreten.");
		} catch (ClassNotFoundException e) {
			e.printStackTrace();
			System.out.println("Klasse wurde nicht gefunden.");
		} finally {
	        try {
	        	fin.close();
	        }catch (IOException e) {
		        e.printStackTrace();
		    }
	        return (Collection<GeoFigur>) obj;
		}
	}

Konsole:

Beim Lesen der Datei ist ein Fehler aufgetreten.
null
java.io.EOFException
at java.ibjectInputStream$BlockDataInputStream.peekByte(Unknown Source)
at java.ibjectInputStream.readObject0(Unknown Source)
at java.ibjectInputStream.readObject(Unknown Source)
at pr1.a13.Aufgabe13.readFiguresFromFile(Aufgabe13.java:145)
at pr1.a13.Aufgabe13.main(Aufgabe13.java:33)

 

[eRaaaa]

Na du machst doch schon wieder den selben Fehler (2 mal readObject() aufzurufen! (irgendwie ist da auch ein wenig Exception wirr-warr )

    public static Collection<GeoFigur> readFiguresFromFile(String filename) {
	InputStream fin = null;
	Object obj = null;
	Collection<GeoFigur> figuren = null;
	try {
	    fin = new FileInputStream("output/" + filename);
	    ObjectInputStream o = new ObjectInputStream(fin);
	    obj = o.readObject();
	    if (obj instanceof Collection<?>) {
		figuren = (Collection<GeoFigur>) obj;
	    }
	} catch (IOException e) {
	    e.printStackTrace();
	} catch (ClassNotFoundException e) {
	    e.printStackTrace();
	} finally {
	    try {
		fin.close();
	    } catch (IOException e) {
		e.printStackTrace();
	    }
	}
	return figuren;
    }

 

[Lennyx]

ach sry, stimmt.
okay. .. jetzt ist die while-schleife weg.

aber wie um himmels willen les ich jetzt so viele objekte wie darin vorhanden sind aus?

im übrigen gibt er jetzt null zurück, obwohl ich ihm ja eine Collection übergebe (wie gesagt, mit deinem Quellcode gings ja ..)

 

[eRaaaa]

aber wie um himmels willen les ich jetzt so viele objekte wie darin vorhanden sind aus?

Das versteh ich nicht. Was meinst du damit? Du serialisierst doch die komplette Collection, mitsamt den GeoFiguren? (daher ist ja auch GeoFigur serializable! (hoffe ich doch))

im übrigen gibt er jetzt null zurück, obwohl ich ihm ja eine Collection übergebe (wie gesagt, mit deinem Quellcode gings ja ..)

Also ich kann dir nicht mehr folgen, bzw weiß nun nicht mehr was ich schreiben kann, vllt hilft ja jmd anderes weiter (*sry*).

Ich habe den Code jetzt sogar ausprobiert, funktioniert bei mir ohne Probleme. ???:L

 

[Lennyx]

hm das is komisch

und ich meine, wenn ich in der datei mehrere objekte habe, will ich die ja auch alle auslesen können (also zb 2 collections), wie mach ich das ohne meine ursprüngliche while-schleife?

ansonsten läuft der code jetzt immerhin durch, aber gibt halt nur null zurück.

wenn noch iwer ne idee hat ... ich bin ratlos. und muss die aufgabe in 4h abgeben ^^

 

[SlaterB]

erstmal musst du anfangen und überlegen, welchen Sinn ein Programm/ eine Methode einnehmen soll, was ist ihre Aufgabe, welchen Zweck verfolgst du?

wieviele Collections sollen denn in der Datei stehen, was macht dann die read()-Methode?

while(o.readObject()!=null){
    if (obj instanceof Collection<?>){
          obj = o.readObject();
    }
}
return (Collection<GeoFigur>) obj;

was soll dieser Code bedeuten?
wenn mehrere Objekte drin sind, dann überspinge die ersten und gib das letzte zurück, welches eine Collection ist?!

->

Object x = null;
while((x = o.readObject())!=null){
    if (x instanceof Collection<?>){
          obj = x;
    }
}
return (Collection<GeoFigur>) obj;

-------------

perfekt für die Umgehung solcher Probleme, wäre, immer exakt EIN Objekt in die Datei zu schreiben, nicht 2, 3, 2000 und auch kein halbes, sondern genau EINS,
ob das eine Liste ist oder was auch immer,
das Lesen bedeutet dann ohne Schleife einfach nur 'lade das eine Objekt' dessen Typ auch bekannt sein muss,

ob du dann in diesen Haupt-Objekt, welches vielleicht eine Liste beliebiger andere Objekte ist, nach Unterobjekten suchen muss, ist eine andere Frage,
aber das wäre dann Java-Code innerhalb einer Liste, mit get(i), Mehrfachzugriff usw., size() abfragen,
in einer Datei ist das viel schwieriger (Exceptions, einmal gelesenes kommt nie zurück, ..)

 

[Lennyx]

also im programm ist es nur ein Objekt, was in der Datei steht, mir ging es nur darum, dass es doch auch irgendwie möglich sein muss, mehrere auszulesen. dann eben nicht ^^

die while-schleife hatte ich bereits rausgenommen, da dass die einzige änderung war, hatte ich daraufhin nicht nochmal den ganzen quellcode rausgenommen.

Nach meinem jetzigen Quelltext wird aber auch nur ein einziges Objekt gelesen ... das ja leider null ist

 

[SlaterB]

wäre es nicht Zeit diesen zu posten?
dein letzter Code stammt von Heute, 00:25, mit while-Schleife,

ein vollständiges Testprogramm mit main-Methode, welches erst paar Objekte erzeugt und speichert, wäre auch nützlich,
aber das kann ich auch selber machen, habe schon ne Dummy-Klasse GeoFigur erstellt

 

[Lennyx]

gut, dann nochmal das komplette ding.

package pr1.a13;																//Schulte, 766254
import java.io.BufferedReader;
import java.io.File;
import java.io.FileInputStream;
import java.io.FileNotFoundException;
import java.io.FileOutputStream;
import java.io.FileReader;
import java.io.FileWriter;
import java.io.IOException;
import java.io.InputStream;
import java.io.ObjectInputStream;
import java.io.ObjectOutputStream;
import java.io.OutputStream;
import java.util.Collection;
import java.util.HashSet;
import java.util.Set;


public class Aufgabe13 {
	
	public static void main(String[] args) throws IOException{
		Collection<GeoFigur> figuren = new HashSet<GeoFigur>();
		figuren.add(new Point(3,5));
		figuren.add(new Line(new Point(3,5), new Point(9,6)));
		writeFiguresToFile(figuren, "figuren.ser");
		try {
			System.out.println(readFiguresFromFile("figuren.ser"));
		} catch (ClassNotFoundException e) {
			System.out.println("Klasse wurde nicht gefunden.");
		}
	}
	
	public static void writeFiguresToFile(Collection<GeoFigur> c, String filename) throws IOException{
		File f = new File("output/" + filename);
		try{
			f.createNewFile();
		}catch(IOException e){
			System.out.println("Datei existiert bereits.");
		}finally{
			OutputStream fos = null;
			try {
				fos = new FileOutputStream("output/" + filename); 
				ObjectOutputStream o = new ObjectOutputStream( fos ); 
				o.writeObject(c);
				o.flush(); 
			}catch(IOException io){
				System.out.println("In die Datei kann leider nicht geschrieben werden.");
			}finally{
				if(fos!=null){
					fos.close();
				}
			}
		}
	}
	
	public static Collection<GeoFigur> readFiguresFromFile(String filename) throws ClassNotFoundException, IOException{
	    InputStream fin = null;
	    Object obj = null;
		try {
			fin = new FileInputStream("output/" + filename);
			ObjectInputStream o = new ObjectInputStream(fin);
				if (obj instanceof Collection<?>){
					obj = o.readObject();
				}
			return (Collection<GeoFigur>) obj;
			
		}catch (IOException e) {
				e.printStackTrace();
				System.out.println("Beim Lesen der Datei ist ein Fehler aufgetreten.");
		} catch (ClassNotFoundException e) {
			e.printStackTrace();
			System.out.println("Klasse wurde nicht gefunden.");
		} finally {
	        try {
	        	fin.close();
	        }catch (IOException e) {
		        e.printStackTrace();
		    }
	        return (Collection<GeoFigur>) obj;
		}
	}
}

und dann noch GeoFigur, falls ihr das auch wollt ^^

package pr1.a13;
import java.io.Serializable;

import j.schimkat.lernhilfe.simpleDrawingFramework.userPort.Drawable;

public abstract class GeoFigur implements Drawable, Moveable, Serializable{

	private static final long serialVersionUID = 1L; //versichert, dass beim deserialisieren auch das korrekte Objekt rauskommt

	public abstract void zoom(double factor);
	
}

Ausgabe: null

 

[SlaterB]

 if (obj instanceof Collection<?>){
                    obj = o.readObject();
                }

was denn das, erst der Test?, obj ist doch noch null,
ob null instanceof irgendwas nicht true ergibt, ist gar nicht mal so klar,
in jedem Fall macht das read() darin keinen Sinn, welchen Typ das neu gelesene Objekt hat, wird gar nicht geprüft

schreibe z.B.

            Object x = o.readObject();
            if (x instanceof Collection<?>)
            {
                obj = x;
            }

oder einfach nur

                obj = o.readObject();

du weißt ja dass nur eine Collection drin sein sollte, gegen böse äußere Einflüsse ist dein Programm eh kaum sicher

> throws ClassNotFoundException, IOException
kannst du dir sparen, wenn du das mit try/ cach abfängst

 

[Lennyx]

oh mann. danke!!! klar, dass der nich in die if-schleife springt .... muss ich wohl übersehen haben.

jetzt läufts fehlerfrei!!


vielen vielen dank =)