XML Datei speichern und laden

[clash]

Hallo, ich möchte gerne, dass was ich programmiert habe in eine XML-Datei speichern. Dazu habe ich zwei Buttons (Save und Load). das Speichern klappt auch soweit ganz gut, aber ich bekomme es nicht hin diese Datei dann zu laden, weil sie mir nicht angezeigt wird. Ich hoffe ihr habt einen Tipp. Vielen Dank schon mal

package view;

import java.beans.XMLEncoder;
import java.io.File;
import java.io.FileInputStream;
import java.io.FileOutputStream;
import java.io.IOException;
import java.io.ObjectInputStream;
import java.io.ObjectOutputStream;
import model.*;

import javafx.event.ActionEvent;
import javafx.event.Event;
import javafx.event.EventHandler;
import javafx.geometry.Insets;
import javafx.scene.control.Button;
import javafx.scene.layout.GridPane;
import javafx.stage.FileChooser;

import viewmodel.MainViewModel;

public class FilePane extends GridPane {
    final Button btnSave = new Button ("Save");
    final Button btnLoad = new Button ("Load");
    final FileChooser fileChooser = new FileChooser ();
    
    private EventHandler<ActionEvent> handler = new EventHandler<ActionEvent>() {
        public void handle (ActionEvent me) {
            if(me.getSource()==btnLoad) btnLoadHandle(me);
            else if(me.getSource()==btnSave) btnSaveHandle(me);
        
        }
    };
    
    
    public FilePane() {
        this.setPadding(new Insets(20, 10,10, 10));
        this.setVgap(15);
        this.setHgap(20);
//        this.setGridLinesVisible(true);
        this.add(btnLoad, 22, 0);
        this.add(btnSave, 32, 0);
        btnSave.addEventFilter(ActionEvent.ACTION, handler);
        btnLoad.addEventFilter(ActionEvent.ACTION, handler);
//        btnSave.setOnAction(e -> {
//            File selectedFile = fileChooser.showOpenDialog(null);
//        });
        fileChooser.getExtensionFilters().addAll( new FileChooser.ExtensionFilter("Serialize", "*.ser")   
                ,new FileChooser.ExtensionFilter("XMLEncoder Files", "*.xml")   
                ,new FileChooser.ExtensionFilter("XMLDOM Files", "*.xml2") );
        fileChooser.setInitialDirectory(new File("./"));
    }
    
    @SuppressWarnings("unchecked")
    public void btnLoadHandle(Event e) {
        
        
        Figure f = null;
    File selectedFile = fileChooser.showOpenDialog(null); // es ruft das Fenster auf, aber ich sehe dort keine Dateien sondern nur Ordner
        try {
            java.beans.XMLDecoder in = new java.beans.XMLDecoder(new FileInputStream(selectedFile));
//            f = (Figure) readXMLDecoder(in);
            Tupel<Double> t = null;
            TupelColor tc = null;
            f = (Figure) in.readObject();
            t = (Tupel<Double>) in.readObject();
            f.setPosition(t.x, t.y);
            t = (Tupel<Double>) in.readObject();
            f.setSize(t.x, t.y);
            tc = (TupelColor) in.readObject();
            f.setColors(tc.x, tc.y);
            in.close();
        } catch (IOException ex) {
            System.err.println(ex.getMessage());
        }
    }


    public void btnSaveHandle (Event e) {
        File selectedFile = fileChooser.showSaveDialog(null);
        MainViewModel vm = new MainViewModel();
        Drawing d = vm.getDrawing();
        // Write Encoder
        try {
            java.beans.XMLEncoder out = new java.beans.XMLEncoder(new FileOutputStream("Test"));
            writeXMLEncoder(d, out);
            out.flush();
            out.close();
        } catch (IOException ex) {
            System.err.println(ex.getMessage());
        }
    }

    private void writeXMLEncoder(Drawing d, XMLEncoder out) {
        // TODO Auto-generated method stub
        out.writeObject(d);
        Figure f;
        for (int i =0; i < d.getShapes().size(); i++) {
        out.writeObject(d.getShapes().get(i));
        f = (Figure)d.getShapes().get(i);
        out.writeObject(f.getPosition());
        out.writeObject(f.getSize());
        out.writeObject(f.getColors());
        }
    }
}

 

[kneitzel]

Du schreibst - so ich das richtig sehe, eine Datei Test ohne Endung. Und du lässt dir aber nur Dateien mit Endung anzeigen. Daher siehst du die Datei nicht.

Lass die Filter weg oder schreib eine Datei mit entsprechender Endung.

 

[clash]

Okay, vielen dank! das hat schon mal geholfen.
jetzt hab ich noch ein Problem. Ich hab mittels

  out.writeObject(d);

die Klasse Drawing in die XML geschrieben und diese hatte beim Speichern 3 weitere Klassen drin. Das wird mir auch alles korrekt anzeigt, aber wenn ich dann mittels

d = (Drawing) in.readObject();

die "Oberklasse" auslesen will um danach dann die anderen Figuren aufzurufen und zu speichern, ist "d" immer leer.

 

[Jw456]

hallo frage
wo und wann machst du das lesen? Hoffe nach dem schliesen des out Stream.

 

[kneitzel]

Kannst du noch einmal das Speichern und Laden zeigen. Der Code des ersten Posts dürfte ja nicht mehr aktuell sein.

Und bekommst du eine Exception? Oder läuft es ohne Exception durch?

 

[clash]

Es läuft ohne Exception durch.

Laden:

@SuppressWarnings("unchecked")
    public void btnLoadHandle(Event e) {
       
        Figure f = null;
        Drawing d = null;
    File selectedFile = fileChooser.showOpenDialog(null);
        try {
            java.beans.XMLDecoder in = new java.beans.XMLDecoder(new FileInputStream(selectedFile));
//            f = (Figure) readXMLDecoder(in);
            Tupel<Double> t = null;
            TupelColor tc = null;
            d = (Drawing) in.readObject();
            System.out.println(d);
 //           f = (Figure) in.readObject();
//            System.out.println(f);
             for (int i =0; i < d.getCnt(); i++) {
               
            f = (Figure)d.getShapes().get(i);
            t = (Tupel<Double>) in.readObject();
            ((Figure) f).setPosition(t.x, t.y);
            t = (Tupel<Double>) in.readObject();
            ((Figure) f).setSize(t.x, t.y);
            tc = (TupelColor) in.readObject();
            ((Figure) f).setColors(tc.x, tc.y);
            System.out.println(f);
            d.add(f);
            in.close();
            vm = new MainViewModel();
            vm.addFigure(f);
            System.out.println(d);
           
             }
        } catch (IOException ex) {
            System.err.println(ex.getMessage());
        }
    }

Speichern klappt ohne Probleme und auch die XML-Datei sieht gut aus.

public void btnSaveHandle (Event e) {
        File selectedFile = fileChooser.showSaveDialog(null);
        MainViewModel vm = new MainViewModel();
        Drawing d = vm.getDrawing();
        // Write Encoder
        try {
            java.beans.XMLEncoder out = new java.beans.XMLEncoder(new FileOutputStream("Test.xml"));
            writeXMLEncoder(d, out);
            out.flush();
            out.close();
        } catch (IOException ex) {
            System.err.println(ex.getMessage());
        }
    }private void writeXMLEncoder(Drawing d, XMLEncoder out) {
        // TODO Auto-generated method stub
        out.writeObject(d);
        Figure f;
        for (int i =0; i < d.getShapes().size(); i++) {
        out.writeObject(d.getShapes().get(i));
        f = (Figure)d.getShapes().get(i);
        out.writeObject(f.getPosition());
        out.writeObject(f.getSize());
        out.writeObject(f.getColors());
        }
    }

 

[Jw456]

zeigt "selectedFile" wirklich auf die Datei "Text.xml" ?
Ist es für den xml Entcoder auch möglich ein Drawing Objekt zurück zugeben ?

 

[clash]

Also wenn ich am Anfang statt

d = (Drawing) in.readObject();

gleich schreibe

f = (Figure) in.readObject();

dann gibt er mir direkt die Figure aus. Also zeigt er auf die Datei. Ähm na ich hoffe, dass er das kann. Keine Ahnung, ich dachte, wenn der Decoder es schreiben kann, dann kann der Encoder es auch lesen/zurückgeben.

 

[Jw456]

also wenn er dir gleich deine Figure als erstes zurück gibt scheint er es nicht geschieben zu haben.
Wenn auf dem Förderband Stream das als erstes ligt.

 

[clash]

Also das hier ist der Anfang der XML-Datei:

<?xml version="1.0" encoding="UTF-8"?>
<java version="16.0.1" class="java.beans.XMLDecoder">
 <object class="model.Drawing">
  <void property="cnt">
   <int>3</int>
  </void>
 </object>
 <object class="model.Ellipse">
  <void property="form">
   <string>Ellipse</string>
  </void>
 </object>

 

[Jw456]

weiss ich es jetzt nicht.

was wird den auf der console ausgegeben.? bei System.out.println(d);

 

[clash]

Drawing [cnt=0, shapes=[]

 

[Jw456]

ist doch genau das was du auch gespeichert hast.

 

[clash]

? Cnt ist mit 3 gespeichert, aber als Ausgabe bekomme ich 0. oder verstehe ich was falsch?

 

[kneitzel]

Also bekommst Du das Drawing zurück, nur eben ist cnt=0 und das müsste Deiner Meinung nach 3 sein? Da wäre die Klasse Drawing einmal interessant um zu sehen, wie die aussieht.

Ich habe Dein Model ja nicht wirklich durchschaut und die Abkürzungen sind wenig hilfreich. Derzeit sieht es aber so aus:

Du hast ein Drawing und darin sind 3 Figures. cnt ist der Count, also die Anzahl der Figures? Dann ist im Code evtl. etwas, das dies überschreibt, aber das ist komplett geraten.

Was mich jetzt wundert ist: Wenn ein Drawing 3 figures hat, dann wäre meine Erwartung, dass dies mit gespeichert wird. Also die Fugures sind innerhalb des Drawings enthalten und alles was die Serialisierung machen würde, wäre das Drawing schreiben bzw lesen.
Das wäre dann auch so z.B. auf https://sol.cs.hm.edu/4129/html/103-xmlencoderundxmldecoder.xhtml gezeigt. Wenn Du da das Beispiel anschaust, dann hat das das Objekt eine Property, die ein Array von einer Klasse ist und so ...

Ich frage mich auch gerade, wieso Du nicht jaxb oder jackson oder so nutzt. java.beans.* würde ich bei der Serialisierung eher als veraltet bezeichnen. jaxb ist eher das, was als Standard durchgehen würde in meinen Augen....

 

[Jw456]

ja seltsam

 

[clash]

Ja, genau, dass dachte ich.

Genau Drawing ist eine Liste, in der 3 Figuren gespeichert sind und cnt ist der Zähler, wie viele Figuren in der Liste enthalten sind. Also genau wie du sagtest.
Hm ja das sieht deutlich anders aus als bei mir :/

Warum ich java.beans verwende ist relativ simpel. Der Dozent hat es uns damit gezeigt bzw. erklärt und gesagt wir sollen das nehmen, für andere Aufgaben davor. Jetzt ist es aber egal, hauptsache das Ergebnis stimmt. Also werde ich mir mal jaxb angucken. Danke!

Das ist der Code zu dem Drawing:

package model;

import java.util.ArrayList;
import java.util.Comparator;
import java.util.List;
import java.util.Collections;



public class Drawing {
    private int cnt;
    private List<IFigure> shapes;

    
    /*
     * Konstruktoren
     */
    
    public Drawing() {
        shapes = new ArrayList<IFigure>();
        
    }
//    public Drawing(int size) {
//        shapes = new ArrayList <IFigure>();
//        this.cnt= shapes.size();
//    }
    /*
     * Setter
     */
    
    public void setShapes(List<IFigure> shapes, int cnt) {
        this.shapes = shapes;
        if  (cnt >=0 && cnt <= shapes.size())  {
            this.cnt =cnt;
        }
        else
            this.cnt= shapes.size();
    }
    
    public void setCnt(int cnt) {
        if (cnt >= 0 && cnt <= shapes.size())
        this.cnt= cnt;
    }
    
    public void set (IFigure shape, int k) {
        if (k>=0 && k< cnt) {
        shapes.set(k, shape);}
        }
    
    /*
     * Getter
     */
    public List<IFigure> getShapes() {
        return  shapes;
    }
    public int getCnt() {
        return cnt;
        
    }
    
    public IFigure get(int k) {
        if (k>=0 && k< cnt)
        return shapes.get(k);
        else return null;
    
    }
    /*
     * Hinzufügen einer Figur
     */
    public int add(IFigure shape) {
        shapes.add(shape);
        cnt = shapes.size();
        return cnt;
    }
    /*
     * Löschen einer Figur   
     */
    public boolean remove (int k) {
        if (k>=0 && k<shapes.size()-1) {
        shapes.remove(k);
            cnt--;
            return true;
        }
        else return false;
    }
    /*
     * Löschen aller Figuren
     */
    public void remove () {
        cnt =0;
    }
    /*
     * Sortieren
     */
    public void sort (Comparator <? super IFigure> c) {
        Collections.sort(shapes);
    }
    
    @Override
    public int hashCode() {
        final int prime = 31;
        int result = 1;
        result = prime * result + cnt;
        result = prime * result + ((shapes == null) ? 0 : shapes.hashCode());
        return result;
    }
    @Override
    public boolean equals(Object obj) {
        if (this == obj)
            return true;
        if (obj == null)
            return false;
        if (getClass() != obj.getClass())
            return false;
        Drawing other = (Drawing) obj;
        if (cnt != other.cnt)
            return false;
        if (shapes == null) {
            if (other.shapes != null)
                return false;
        } else if (!shapes.equals(other.shapes))
            return false;
        return true;
    }
    
    
    @Override
    public String toString() {
        return "Drawing [cnt=" + cnt + ", shapes=" + shapes + "]";
    }
    @Override
    public Drawing clone(){
        Drawing d = new Drawing ();
           for (int i = 0; i < shapes.size(); i++) {
               if (shapes.get(i)!= null) {
                   if (shapes.get(i) instanceof Rectangle)
                       d.shapes.add(i, ((Rectangle) shapes.get(i)).clone());
                   else if (shapes.get(i) instanceof Ellipse)
                       d.shapes.add(i, ((Ellipse) shapes.get(i)).clone());
               }
               else d.shapes.add(shapes.get(i));
           }
           d.setCnt(getCnt());
               return d;
    }

 

[kneitzel]

Ok, der Setter setzt cnt nur, wenn es <= shapes.siue() ist. Da shapes.size aber 0 ist, wird die 3 nicht gespeichert. Ich bin jetzt nicht ganz sicher, die der Serialisierungsprozess vor geht, aber wenn der Setter benutzt wird, dann hätten wir die Erklärung für das Verhalten.

Aber die Instanzvariable cnt solltest Du ganz entfernen und den getter einfach shapes.size() zurück geben lassen. Oder ist beabsichtigt, dass diese Werte sich unterscheiden? Was macht es für einen Sinn. cnt auf 3 zu setzen, wenn 5 Elemente in shapes sind? (Derzeit wäre dies ja möglich.)

Wenn ihr das nutzen sollt, dann ist das ok. Aber da sollte man dann schauen, wie man die shapes mit serialisiert bekommt. Ich kenne den von die benutzten Weg jetzt nicht so dass ich da so auf Anhieb nicht helfen kann. Aber ich denke, dass da auf der Seite https://sol.cs.hm.edu/4129/html/108-neueserialisierer.xhtml diesbezüglich etwas zu finden ist.

 

[clash]

Danke für die Hilfe, er ließt jetzt zumindest etwas ein, aber die XML sieht immer noch nicht wie ein Baum aus. Ich muss da noch einen Fehler haben, den ich aber nicht finde. Ich hab mir den LINK und den Inhalt ganz genau durchgelesen und es so gemacht wie es dort steht. Ich gucke mal, dass ich das irgendwie anders hin bekomme