Modell als XML Speichern / Laden

[Tree]

Hallo,
ich versuche gerade ein Modell per XML abzuspeichern / zu laden, dummerweise funktioniert es anscheinend Fehlerfrei laut Eclipse. Nur bekomme ich beim Aufruf/Laden nicht die Gewünschten Daten / Ergebnisse bzw garnichts.

Das Modell enthällt eine ArrayList mit Objekten die ausgegeben werden.

Hier einmal der XMLEncoder / Decoder ich vermute hier liegt der Fehler

	public void saveModel(String filename) {
		
		
		
		try{
			XMLEncoder encoder = new XMLEncoder(new BufferedOutputStream(new FileOutputStream(filename)));
			encoder.writeObject(this.myModell);
			encoder.close();
		}catch (IOException e){
			System.out.println("Fehler beim Speichern der Datei: " + e);
			}
}

public void loadModel(String filename) {
		
		try{
			XMLDecoder decoder = new XMLDecoder(new BufferedInputStream( new FileInputStream(filename)));
			Modell newModell = (Modell) decoder.readObject();
			
			this.myModell.deleteAllFigures();
			
			for (Iterator iterator = newModell.iterator(); iterator.hasNext();) {
				 Figures loadedFigure = (Figures) iterator.next();
				 this.myModell.addFigure(loadedFigure); 
				 }
			decoder.close();
		
		}catch (IOException e){
			System.out.println("Fehler beim Laden der Datei: " + e);
		
		
	}
		this.modelChanged = false;
		myPaint.repaint();
		
	}

Wär dankbar für jeden Tipp

 

[SlaterB]

hast du noch nähere Informationen? vom Testprogramm will ich vorerst gar nicht sprechen,

wie funktioniert es initial, kannst du in deinem Prgramm auf anderen Wege Figuren hinzufügen?
womit willst du anfangen, mit dem Speichern?
was ist denn dazu das Problem, wird eine Datei erstellt, wie sieht die aus?
wenn es keine Datei gibt: wird die saveModel() überhaupt aufgerufen? durch System.out.println() prüfen

oder geht es dir ums Laden?
zig Fragen ergeben sich auch dort ganz natürlich nach den immer gleichen Prinzipien:
Prozesse in Schritte auftrennen, soweit möglich durch Programmausgaben überprüfen usw.

nur 'geht nicht' hilft als Information wenig..

------

du könntest vorerst übrigens auf eine GUI verzichten,
einfach nur eine main-Methode die paar Daten zusammenstellt, speichert, lädt, mit vorhandenen vergleicht,
wenn das geht wäre das schon ein guter Schritt, danach kann man sich noch mit genug GUI-spezifischen Fehlerquellen herumärgern

 

[Tree]

wie funktioniert es initial, kannst du in deinem Prgramm auf anderen Wege Figuren hinzufügen?

Initial ist es das gleiche im Model gibt es eine ArrayList die figuren mit add Hinzufügt

womit willst du anfangen, mit dem Speichern/Laden?
was ist denn dazu das Problem, wird eine Datei erstellt, wie sieht die aus?

Ich weiß nicht genau wo der Fehler liegt, es wird beim Speichern eine Datei erstellt die beim Öffnen mit XML nur einen Code enthält

Bsp:
[XML]<?xml version="1.0" encoding="UTF-8"?>

<java version="1.7.0" class="java.beans.XMLDecoder">

<object class="modell.Modell"/>

</java>
[/XML]

Und das Problem ist wie wie beschrieben das beim Öffnen der Datei mit der GUI die eigentlich gespeicherten Figures nicht da sind bzw die GUI leer ist

 

[SlaterB]

> Und das Problem ist wie wie beschrieben das beim Öffnen der Datei mit der GUI die eigentlich gespeicherten Figures nicht da sind bzw die GUI leer ist

aber beschrieben war es ohne konkrete Einschränkung etwa 'Speichern funktioniert, Datei sieht gut aus',
so war/ ist letztlich überall der Fehler zu vermuten, nur das Endergebnis falsch

------

> es wird beim Speichern eine Datei erstellt die beim Öffnen mit XML nur einen Code enthält

wie genau meinst du das? wie sieht die erzeugte Datei aus? enthält sie zuwenig Elemente?
dann wäre das schon ein Fehler, ganz egal ob und wie sie je wieder geöffnet wird,

wenn die Datei aber mehrere Elemente enthält, was genau passiert nun beim Laden?
"beim Öffnen mit XML nur einen Code enthält" könnte man so interpretieren, dass bei der Verarbeitung der eigentlich richtigen Datei nur ein Objekt rauskommt,
ist das der Fall? dann wird es (zusammen mit korrekter gefüllter Datei) sicherlich schwierig, denn was 'decoder.readObject();' zurückliefert kannst du ja kaum beeinflussen,
wie genau stellst du fest was gelesen wird, hast du z.B. eine Ausgabe in der iterator()-Schleife?
nur später die GUI anzuschauen ist wieder zu viel auf einmal, bis dahin kann noch viel schiefgehen

da du aber eine komplette XML-Datei mit nur einem Objekt postest, die du dir doch kaum ausgedacht hast,
ist eher anzunehmen, dass dies die Datei von der Festplatte ist, ist das der Fall?
dann wäre es wieder eine Frage des Speicherns, dies scheint nicht korrekt zu funktionieren


du siehst, wie unklar alles ist, für mich zumindest,
solange nicht geklärt ist was funktioniert und was nicht, kann gar nicht angefangen werden

 

[Tree]

Nun das ist die generierte XML Datei von meiner safe Methode und das war der Inhalt wenn ich sie mit dem Editor öffne
Und ich verstehe nicht ganz warum sie nicht richtig speichert

 

[SlaterB]

nun kann also angefangen werden,
da du dir weitere Informationen wie Code der betroffenen Klasse einsparst,
habe ich (glücklicherweise für dich) mir selber ein Testprogramm geschrieben
und es kam auch erst nur der Klassenname raus,

war mir ein bisschen schon bekannt, daher konnte ich dazu schnell was finden,
wobei die Suchbegriffe 'XMLEncoder writeObject empty' auch dir hätten einfallen können

*help* my XMLEncoder.writeObject is not doing all fields

Fourth, don't forget that if your object only has default data, nothing
will be written except an element representing that there is such an
object. That's because the encoder doesn't write any data that the
default constructor can take care of. So a default object is not a good
test case.

Using XMLEncoder

The DefaultPersistenceDelegate begins by calling the no-argument constructor of the appropriate class. It then uses the Introspector class to get a list of properties for the class and writes out each of the property values as a set of statements that are applied to the new instance. When the values of properties are different to their default values the Encoder is called recursively, to write out an encoding for each of the instances it contained as a property value. This typically generates output in the following style (given in Java here for familiarity):

JButton b = new JButton();
b.setText("Press, me");
b.setName("Button1");

es braucht leeren Default-Zustand und vor allem auch schicke setter + getter,
ob eine ArrayList akzeptiert wird darf zunächst nur gehofft werden, bei mir sieht es aber gut aus:

public class Test
{
    public static void main(String[] args)
    {
        Modell m = new Modell();
        m.setName("test");
        List<Entry> a = new ArrayList<Entry>();
        a.add(new Entry());
        a.add(new Entry());
        m.setList(a);
        saveModel(m, "test2.xml");
    }

    public static void saveModel(Modell myModell, String filename)
    {
        try
        {
            XMLEncoder encoder = new XMLEncoder(new BufferedOutputStream(new FileOutputStream(filename)));
            encoder.writeObject(myModell);
            encoder.close();
        }
        catch (IOException e)
        {
            System.out.println("Fehler beim Speichern der Datei: " + e);
        }
    }

    public static class Modell
    {
        private String name;
        private List<Entry> list;

        public String getName()
        {
            return this.name;
        }

        public void setName(String name)
        {
            this.name = name;
        }

        public List<Entry> getList()
        {
            return this.list;
        }

        public void setList(List<Entry> list)
        {
            this.list = list;
        }
    }

    public static class Entry
    {
    }
}

->
[xml]
<?xml version="1.0" encoding="UTF-8"?>
<java version="1.5.0_11" class="java.beans.XMLDecoder">
<object class="test.Test$Modell">
<void property="list">
<object class="java.util.ArrayList">
<void method="add">
<object class="test.Test$Entry"/>
</void>
<void method="add">
<object class="test.Test$Entry"/>
</void>
</object>
</void>
<void property="name">
<string>test</string>
</void>
</object>
</java>
[/xml]

schau bei dir nach ob du geeignete setter + getter usw. verwendest,
oder poste deine Klasse + Befüllung

 

[Tree]

Schonmal vielen dank für deine Bemühungen

Aber so wirklich geht mir noch kein Licht auf aber ich kann ja mal wie du schon sagtest meine Klassen Posten (Ist etwas viel chrm)

public class Modell extends JPanel{

	static Modell modell;

	private List<Figures> figures = new ArrayList<Figures>();

	public void addFigure(Figures figure) {
		figures.add(figure);
	}
	

	public void addFigure(int position, Figures figure) {
		figures.add(position, figure);
	}

	public void deleteFigure(Figures figure) {
		figures.remove(figure);
	}

	public void deleteAllFigures() {
		figures.clear();
	}

	//@Override
	public Iterator<Figures> iterator() {
		return figures.iterator();
	}

	//@Override
	public Figures getFigureByPosition(Point point) {
		
		for (Iterator<Figures> iterator = this.iterator(); iterator
				.hasNext();) {
			Figures myFigure = (Figures) iterator.next();
			
			if (myFigure.getStartPointX() <= point.x
					&& myFigure.getStartPointY() <= point.y
					&& (myFigure.getEndPointX()) >= point.x
					&& (myFigure.getEndPointY()) >= point.y
					|| (myFigure.getClass() == gui.modell.Line.class
							&& myFigure.getStartPointX() <= point.x
							&& myFigure.getStartPointY() >= point.y
							&& myFigure.getEndPointX() >= point.x && myFigure
							.getEndPointY() <= point.y)) {
				return myFigure;
			} 
		}
		
		return null;
	}

}

public abstract class Figures extends JPanel  implements Serializable{
	protected Point start;
	protected Point end;
	protected boolean isSelected;
	protected boolean isMoving;
	
	
	public int getEndPointX() {
		if (this.start.x < this.end.x) {
			return this.end.x;
		} else {
			return this.start.x;
		}
	}

	public int getEndPointY() {
		if (this.start.y < this.end.y) {
			return this.end.y;
		} else {
			return this.start.y;
		}
	}

	public int getStartPointX() {
		if (this.start.x < this.end.x) {
			return this.start.x;
		} else {
			return this.end.x;
		}
	}

	public int getStartPointY() {
		if (this.start.y < this.end.y) {
			return this.start.y;
		} else {
			return this.end.y;
		}
	}
	
	
	
	
	/**
	 * @return getter for <code>{@link #StartX}</code>
	 */
	public int getStartX() {
		return start.x;
	}

	/**
	 * @param setter
	 *            for <code>{@link #StartX}</code>
	 */
	public void setStartX(int startX) {
		start.x = startX;
	}

	/**
	 * @return getter for <code>{@link #StartY}</code>
	 */
	public int getStartY() {
		return start.y;
	}

	/**
	 * @param setter
	 *            for <code>{@link #StartX}</code>
	 */
	public void setStartY(int startY) {
		start.y = startY;
	}

	/**
	 * @return getter for <code>{@link #EndX}</code>
	 */
	public int getEndX() {
		return end.x;
	}

	/**
	 * @param setter
	 *            for <code>{@link #EndX}</code>
	 */
	public void setEndX(int endX) {
		end.x = endX;
	}

	/**
	 * @return getter for <code>{@link #EndY}</code>
	 */
	public int getEndY() {
		return end.y;
	}

	/**
	 * @param setter
	 *            for <code>{@link #EndY}</code>
	 */
	public void setEndY(int endY) {
		end.x = endY;
	}

	public int getWidth() {
		if (this.start.x < this.end.x) {
			return this.end.x - this.start.x;
		} else {
			return this.start.x - this.end.x;
		}
	}

	public int getHeight() {
		if (this.start.y < this.end.y) {
			return this.end.y - this.start.y;
		} else {
			return this.start.y - this.end.y;
		}
	}

	/**
	 * @return getter for <code>{@link #isSelected}</code>
	 */
	public boolean isSelected() {
		return isSelected;
	}

	/**
	 * @param setter
	 *            for <code>{@link #isSelected}</code>
	 */
	public void setSelected(boolean isSelected) {
		this.isSelected = isSelected;
	}

	/**
	 * @return getter for <code>{@link #isMoving}</code>
	 */
	public boolean isMoving() {
		return isMoving;
	}

	/**
	 * @param setter
	 *            for <code>{@link #isMoving}</code>
	 */
	public void setMoving(boolean isMoving) {
		this.isMoving = isMoving;
	}

	



	public void setEnd(Point point) {
		this.end = point;
	}

	public void setStart(Point point) {
		this.start = point;
	}

	public void moveFigure(Point pos, Point tmpPoint) {

		Point tmpNewPoint1 = new Point();
		Point tmpNewPoint2 = new Point();

		if ((this.getStartPointX() <= pos.x && this.getStartPointY() <= pos.y
				&& this.getEndPointX() >= pos.x && this.getEndPointY() >= pos.y)
				|| (this.getClass() == gui.modell.Line.class
						&& this.getStartPointX() <= pos.x
						&& this.getStartPointY() >= pos.y
						&& this.getEndPointX() >= pos.x && this.getEndPointY() <= pos.y)

		) {

			if (tmpPoint.x < pos.x) {
				tmpNewPoint1.x = this.getStartPointX() + (pos.x - tmpPoint.x);
				tmpNewPoint2.x = this.getEndPointX() + (pos.x - tmpPoint.x);
			} else {
				tmpNewPoint1.x = this.getStartPointX() - (tmpPoint.x - pos.x);
				tmpNewPoint2.x = this.getEndPointX() - (tmpPoint.x - pos.x);
			}

			if (tmpPoint.y < pos.y) {
				tmpNewPoint1.y = this.getStartPointY() + (pos.y - tmpPoint.y);
				tmpNewPoint2.y = this.getEndPointY() + (pos.y - tmpPoint.y);
			} else {
				tmpNewPoint1.y = this.getStartPointY() - (tmpPoint.y - pos.y);
				tmpNewPoint2.y = this.getEndPointY() - (tmpPoint.y - pos.y);
			}

			this.setStart(tmpNewPoint1);
			this.setEnd(tmpNewPoint2);

		}
	}
	public double getBoundingBoxSize() {
		// Breite x Höhe
		double size = (this.getEndPointX() - this.getStartPointX())
				* (this.getEndPointY() - this.getStartPointY());

		if (size < 0) {
			size = size * (-1);
		}

		return size;
	}
	public int compareTo(Object o) {
		Figures myFig = (Figures) o;

		if (this.getBoundingBoxSize() < myFig.getBoundingBoxSize()) {
			return -1;
		} else if (this.getBoundingBoxSize() == myFig.getBoundingBoxSize()) {
			return 0;
		}
		;

		return 1;
	}
}

public class Listener implements DrawingListener {

	private Figures currentFigure;
	private Modell myModell;
	private Paint myPaint;
	private Point tmpPoint;
	private boolean modelChanged;

	public Listener(Modell modell, Paint paint) {
		myModell = modell;
		myPaint = paint;
	}

	public void deleteFigures() {

		List<Figures> figures = new ArrayList<Figures>();

		for (Iterator iterator = myModell.iterator(); iterator.hasNext();) {
			Figures myFigure = (Figures) iterator.next();

			if (myFigure.isSelected()) {
				figures.add(myFigure);
			}

		}

		for (Iterator iterator = figures.iterator(); iterator.hasNext();) {
			Figures figure = (Figures) iterator.next();
			myModell.deleteFigure(figure);
		}

		this.modelChanged = true;
		myPaint.repaint();
	}

	public boolean isModelChanged() {
		return this.modelChanged;
	}

	public void loadModel(String fileName) {

		try {
			XMLDecoder decoder = new XMLDecoder(new BufferedInputStream(
					new FileInputStream(fileName)));
			Modell newModell = (Modell) decoder.readObject();

			this.myModell.deleteAllFigures();

			for (Iterator<Figures> iterator = newModell.iterator(); iterator
					.hasNext();) {
				Figures loadedFigure = (Figures) iterator.next();
				this.myModell.addFigure(loadedFigure);
			}

			decoder.close();

		} catch (IOException e) {
			System.out.println("Fehler beim Laden der Datei: " + e);

		}
		this.modelChanged = false;
		myPaint.repaint();

	}

	public void saveModel(String fileName) {

		try {
			XMLEncoder encoder = new XMLEncoder(new BufferedOutputStream(
					new FileOutputStream(fileName)));
			encoder.writeObject(this.myModell);
			encoder.close();
		} catch (IOException e) {
			System.out.println("Fehler beim Speichern der Datei: " + e);

		}

	}

	/**
	 * Startet das Erstellen einer Figur
	 * 
	 * @param figureType
	 *            Typ der Figur
	 * @param pos
	 *            Position der Figur
	 */
	public void startCreateFigure(String figureType, Point pos) {

		if (figureType.equals("circle")) {
			Figures circle = new Circle();
			circle.setStart(pos);
			myModell.addFigure(0, circle);
			currentFigure = circle;

		} else if (figureType.equals("rect")) {
			Figures rectangle = new Rectangle();
			rectangle.setStart(pos);
			myModell.addFigure(0, rectangle);
			currentFigure = rectangle;

		} else if (figureType.equals("line")) {
			Figures line = new Line();
			line.setStart(pos);
			myModell.addFigure(0, line);
			currentFigure = line;

		}

	}

	public void workCreateFigure(Point pos) {
		currentFigure.setEnd(pos);
		myPaint.repaint();
	}

	public void endCreateFigure(Point pos) {
		currentFigure.setEnd(pos);
		myPaint.repaint(currentFigure.getStartPointX(),
				currentFigure.getStartPointY(), currentFigure.getWidth(),
				currentFigure.getHeight());

		this.modelChanged = true;
	}

	public void startMoveFigure(Point pos) {

		Figures myFigure = myModell.getFigureByPosition(pos);

		if (myFigure != null) {
			this.tmpPoint = pos;
			myFigure.setMoving(true);
		}

	}

	public void workMoveFigure(Point pos) {
		for (Iterator<Figures> iterator = myModell.iterator(); iterator
				.hasNext();) {

			Figures myFigure = (Figures) iterator.next();
			if (myFigure.isMoving()) {
				myFigure.moveFigure(pos, this.tmpPoint);
			}
		}

		this.tmpPoint = pos;
		myPaint.repaint();
	}

	public void endMoveFigure(Point pos) {
		for (Iterator<Figures> iterator = myModell.iterator(); iterator
				.hasNext();) {
			Figures myFigure = (Figures) iterator.next();
			if (myFigure.isMoving()) {
				myFigure.setMoving(false);
			}
		}

		this.modelChanged = true;
	}
}

 

[SlaterB]

ein JPanel wäre so oder so ziemlich fraglich zum Speichern, auch wenn in meinem Quote ein JButton als Beispiel genommen wird,

ich könnte jetzt schauen unter welchen Bedingungen die JPanel-Elemente übernommen werden, aber vermute dass man da eh nicht viel ändern kann und möchte das eigentlich auch nicht

verwende zur Kontrolle der Information leichte kleine relativ GUI-unabhängige Objekte,
kennst du JTableModel zu einer JTable? das Model enthält nur die Daten, ist selber keine JTable bzw. hier JPanel,
da gibt es eine sinnvolle Trennung

du kannst meine Modell-Klasse mit der ArrayList nehmen, das ist ein guter Anfang,
Methoden wie addFigure() kannst du in die Klasse aufnehmen, wenn nötig,
damit XML funktioniert ist aber bestimmt wie gesagt getter + setter für die Liste nötig, siehe mein Beispiel

Figures solltest du unbedingt Einzahl Figure nennen (es heißt ja auch JPanel, nicht JPanels),
und ebenfalls nicht von JPanel erben sondern eine einfache Klasse sein lassen, die nur ihre Points + boolean verwaltet,
dafür wiederum unbedingt richtige setter + getter nötig,
getStartPointX() wenn du es magst, aber unbedingt [c]Point getStart() + setStart(Point)[/c], genau passend zum Namen des Attributes,
ah, setStart() hast du ja schon, getter aber sicher auch nötig

------

mit diesen Änderungen müsste dann XML funktionieren, das kannst du wie in meinem Programm schon ganz einfach testen,
ohne jeden Swing-Anteil,

dass deine GUI dann noch funktioniert ist natürlich mit größeren Änderungen verbunden,
du brauchst dann Code der ein Modell durchgeht und dazu JPanel erzeugt,

in der Verarbeitung, in Listener musst du sehr wahrscheinlich unterscheiden ob du gerade mit dem JPanel oder dem zugehörigen Figure-Objekt (wo die Points stehen usw.) arbeiten willst,
wahrscheinlich brauchst du eine direkte Verknüpfung,


-----

viel Arbeit, ich habe bisschen Zweifel ob dir das gelingen kann wenn du schon nachdem was ich bisher gesagt
habe mit den setter + getter wenig verstehst,

ok, da will ich doch nochmal entgegen meiner vorherigen Aussage nachschauen..

JPanel p = new JPanel();
JButton b = new JButton();
p.add(b);

geht sogar zu speichern
[xml]
<object class="javax.swing.JPanel">
<void method="add">
<object class="javax.swing.JButton">
<string></string>
</object>
</void>
</object>
[/xml]
kommt heraus,

bei dir werden die Figuren gar nicht in das JPanel als GUI-Objekt eingefügt, wozu erbst du dann?..
einfach nur korrekte setter + getter (um das zum x-ten Mal zu erwähnen ) für die Liste könnten vielleicht schon reichen!

komplett testen möchte ich es nicht wenn du Figures sogar nur als abstracte Klasse zur Verfügung stellst,
programmiere eine main mit einem Modell, in welches 2-3 Figuren eingefügt werden und rufe Speichern auf,
ganz analog zu meinem Testprogramm,
wenn das nicht geht, dann poste den Code lauffähig, und ich schaue notfalls nach welche setter + getter fehlen

 

[Tree]

Well das ganze ist nur eine Übungsaufgabe und ich sollte es nur in XMLEncoder speichern lustiger weise funktioniert es wunderbar mit normalen ObjectInputStream / ObjectOuputStream
Und an sich ist es schon ein fertiges programm nur ohne Kommentierung es funktionert alles es ging nur um den XMLDecoder der nicht wirlklich tut was er soll, und nun die implementierung / sinn der Getter/ Setters verstehe ich noch nicht so recht

Aber muss dich ja echt loben für deine Hilfe hier hätte ich nicht mit gerechnet tausend Dank :toll:

 

[GUI-Programmer]

Evtl. könntest du auch mit XStream arbeiten. Das erspart dir Einiges.