Aus XML Datei Objekte in Arraylist laden - Fehler!

[xiix]

Hi,
Ich schreibe einen Vokabeltrainer für das Schuljahr und möchte nun das er alle Vokabeln die hinzugefügt werden automatisch speichert. Dazu rufe ich die save() und read() Methoden auf.

Doch warum lädt er mir die Daten aus der XML Datei nicht in die Arraylist?

    public static void read() {
        try {
            XMLDecoder de = new XMLDecoder(new BufferedInputStream(new FileInputStream("Datenbank.xml")));
            Object result1, result2;
            for(int i = 0; i < d.size(); i++) {
                result1 = de.readObject();
                result2 = de.readObject();
                Db.add(result1.toString(), result2.toString());
            }
            de.close();
        }
        catch(Exception e) { }
    }

 

[Murray]

Ohne den Rest anzusehen: das hier

catch(Exception e) { }

solltest Du schnellstens abstellen, wenn Du wissen willst, warum Dein Programm nicht macht, was es soll.

Wenn Du Excpetions abfängst, solltest Du sie auch behandeln. Das Minimum wäre

catch(Exception e) { 
  e.printStackTrace();
}

 

[xiix]

ok, danke, werde

e.printStackTrace();

ab jetzt für Tests immer hinzufügen (Faulheit lässt grüßen ) . ;-) - doch es gab auch keine Fehlermeldung aus --> somit besteht das Problem weiterhin. :/

 

[Ariol]

Haste mal ein kleines Beispielprogramm? (samt XML)

 

[xiix]

Das Programm:
2 Arraylists wo Vokabeln darin gespeichert werden
Eingelesen wird von einer anderen Klasse...
speichern funktioniert ohne Probleme...

import java.util.ArrayList;
import java.io.File;
import java.io.FileWriter;
import java.io.IOException;
import java.io.PrintWriter;
import java.beans.*;
import java.io.*;
 
public class Db {
    static ArrayList<String> d = new ArrayList<String>();
    static ArrayList<String> e = new ArrayList<String>();
    
    public static void add(String deutsch, String englisch) {
        d.add(deutsch);
        e.add(englisch);
    }

    public static void remove(int index) {
            d.remove(index);
            e.remove(index);
    }
    
    public static String getIndexD(int index) {
        return d.get(index);
    }
    
     public static String getIndexE(int index) {
        return e.get(index);
    }
    
    public static int size() {
        return d.size();
    }
    
    public static void save() {
        try {
            XMLEncoder en = new XMLEncoder(new BufferedOutputStream(new FileOutputStream("Datenbank.xml")));
            for(int i = 0; i < d.size(); i++) {
                en.writeObject(new String(d.get(i)));
                
            }
            for(int i = 0; i < e.size(); i++) {
               en.writeObject(new String(e.get(i))); 
            }
            en.close();
        }
        catch(Exception e) { e.printStackTrace();  }
    }
    
    public static void read() {
        try {
            XMLDecoder de = new XMLDecoder(new BufferedInputStream(new FileInputStream("Datenbank.xml")));
            Object result1, result2;
            for(int i = 0; i < d.size(); i++) {
                result1 = de.readObject();
                result2 = de.readObject();
                Db.add(result1.toString(), result2.toString());
            }
            de.close();
        }
        catch(Exception e) { e.printStackTrace();  }
    }
    
    public void print() {
        for(int i = 0; i < d.size() / 2; i++) {
            System.out.println(d.get(i) + " - " + e.get(i));
        }
    }
}

XML mit 3 Vokabeln:

 <?xml version="1.0" encoding="UTF-8" ?> 
- <java version="1.6.0_07" class="java.beans.XMLDecoder">
  <string>Flasche</string> 
  <string>Tastatur</string> 
  <string>Fenster</string> 
  <string>bottle</string> 
  <string>keyboard</string> 
  <string>window</string> 
  </java>

 

[Murray]

Was auffällt: Du schreibt erst alle deutschen, dann alle englischen Wörter. Beim Lesen interpretierst Du dann aber die einzelnen Strings immer abwechselnd als deutsch und englisch.

 

[xiix]

Ja, stimmt, ich hab noch die alte Methode mit der Neuen vermischt, doch auch wenn er sie mir dann falsch einliest, bin ich schon glücklich.
Doch was ist an read() falsch? (bis auf das er es falsch einliest)

 

[Ariol]

Wieso schreibst du nicht die ArrayListen raus?

import java.util.ArrayList;
import java.io.File;
import java.io.FileWriter;
import java.io.IOException;
import java.io.PrintWriter;
import java.beans.*;
import java.io.*;

public class Db {
    static ArrayList<String> d = new ArrayList<String>();
    static ArrayList<String> e = new ArrayList<String>();
   
    public static void add(String deutsch, String englisch) {
        d.add(deutsch);
        e.add(englisch);
    }

    public static void remove(int index) {
            d.remove(index);
            e.remove(index);
    }
   
    public static String getIndexD(int index) {
        return d.get(index);
    }
   
     public static String getIndexE(int index) {
        return e.get(index);
    }
   
    public static int size() {
        return d.size();
    }
   
    public static void save() {
        try {
            XMLEncoder en = new XMLEncoder(new BufferedOutputStream(new FileOutputStream("Datenbank.xml")));
            en.writeObject(d);
            en.writeObject(e);
            en.close();
        }
        catch(Exception e) { e.printStackTrace();  }
    }
   
    public static void read() {
        try {
            XMLDecoder de = new XMLDecoder(new BufferedInputStream(new FileInputStream("Datenbank.xml")));
           d = (ArrayList<String>) de.readObject();
           e = (ArrayList<String>) de.readObject();
           de.close();
        }
        catch(Exception e) { e.printStackTrace();  }
    }
   
    public void print() {
        for(int i = 0; i < d.size(); i++) {
            System.out.println(d.get(i) + " - " + e.get(i));
        }
    }
    
    /**Test**/
    public static void main(String[] args)
	{
		Db db = new Db();
		db.add("Flasche", "bottle");
		db.add("Bier", "beer");
		db.add("Tastatur", "keyboard");
		db.save();
		
		d = new ArrayList<String>();
		e = new ArrayList<String>();
		
		Db db2 = new Db();
		db2.read();
		
		db2.print();
	}
}

Aber insgesamt wäre es besser, wenn du eine eigene Klasse Vokabel anlegen würdest! Dann hättest du genau eine Liste und keine Probleme durch falsche Zuordnungen.

Und falls du mehrer Vokabel-Listen anlegen willst wäre es verdammt sinnvoll, die Variablen und Methoden nicht statisch zu machen.

import java.util.ArrayList;
import java.io.File;
import java.io.FileWriter;
import java.io.IOException;
import java.io.PrintWriter;
import java.beans.*;
import java.io.*;

public class Db {
    ArrayList<String> d = new ArrayList<String>();
    ArrayList<String> e = new ArrayList<String>();
   
    public void add(String deutsch, String englisch) {
        d.add(deutsch);
        e.add(englisch);
    }

    public void remove(int index) {
            d.remove(index);
            e.remove(index);
    }
   
    public String getIndexD(int index) {
        return d.get(index);
    }
   
     public String getIndexE(int index) {
        return e.get(index);
    }
   
    public int size() {
        return d.size();
    }
   
    public void save() {
        try {
            XMLEncoder en = new XMLEncoder(new BufferedOutputStream(new FileOutputStream("Datenbank.xml")));
            en.writeObject(d);
            en.writeObject(e);
            en.close();
        }
        catch(Exception e) { e.printStackTrace();  }
    }
   
    public void read() {
        try {
            XMLDecoder de = new XMLDecoder(new BufferedInputStream(new FileInputStream("Datenbank.xml")));
           d = (ArrayList<String>) de.readObject();
           e = (ArrayList<String>) de.readObject();
           de.close();
        }
        catch(Exception e) { e.printStackTrace();  }
    }
   
    public void print() {
        for(int i = 0; i < d.size(); i++) {
            System.out.println(d.get(i) + " - " + e.get(i));
        }
    }
    
    /**Test**/
    public static void main(String[] args)
	{
		Db db = new Db();
		db.add("Flasche", "bottle");
		db.add("Bier", "beer");
		db.add("Tastatur", "keyboard");
		db.save();
		
		Db db2 = new Db();
		db2.read();
		
		db2.print();
	}
}

...überleg dir das mit der Vokabel-Klasse !


EDIT: Ach ja:
1. deine print()-Methode war falsch!
2. d.size() ist 0,wenn du das erste mal einliest (noch nix drin) - deshalb list du auch 0 Objekte ein. Alternativ:

public static void read()
	{
		try
		{
			XMLDecoder de = new XMLDecoder(new BufferedInputStream(new FileInputStream("Datenbank.xml")));
			Object result1;
			Object result2;
			try
			{
				result1 = de.readObject();
				result2 = de.readObject();
			}
			catch (ArrayIndexOutOfBoundsException e)
			{
				result1 = null;
				result2 = null;
			}
			while (result1 != null && result2 != null)
			{
				try
				{
					Db.add(result1.toString(), result2.toString());
					result1 = de.readObject();
					result2 = de.readObject();
				}
				catch (ArrayIndexOutOfBoundsException e)
				{
					result1 = null;
					result2 = null;
				}

			}
			de.close();
		}
		catch (Exception e)
		{
			e.printStackTrace();
		}
	}

 

[xiix]

So, danke waren ein paar dumme Fehler...
hier der funktionierende Code:

import java.beans.XMLDecoder;
import java.beans.XMLEncoder;
import java.io.BufferedInputStream;
import java.io.BufferedOutputStream;
import java.io.FileInputStream;
import java.io.FileOutputStream;
import java.util.ArrayList;

public class Db {
	static ArrayList<String> d = new ArrayList<String>();
	static ArrayList<String> e = new ArrayList<String>();

	public static void add(String deutsch, String englisch) {
		d.add(deutsch);
		e.add(englisch);
	}

	public static void remove(int index) {
		d.remove(index);
		e.remove(index);
	}

	public static String getIndexD(int index) {
		return d.get(index);
	}

	public static String getIndexE(int index) {
		return e.get(index);
	}

	public static int size() {
		return d.size();
	}

	public static void save() {
		try {
			XMLEncoder en = new XMLEncoder(new BufferedOutputStream(
					new FileOutputStream("Datenbank.xml")));
			en.writeObject((Object) d.size());
			for (int i = 0; i < d.size(); i++) {
				en.writeObject(new String(d.get(i)));

			}
			for (int i = 0; i < e.size(); i++) {
				en.writeObject(new String(e.get(i)));
			}
			en.close();
		} catch (Exception e) {
			e.printStackTrace();
		}
	}

	public static void read() {
		try {
			XMLDecoder de = new XMLDecoder(new BufferedInputStream(
					new FileInputStream("Datenbank.xml")));
			int size = Integer.parseInt(de.readObject().toString());
			for (int i = 0; i < size; i++) {
				Object r = de.readObject();
				d.add(r.toString());
			}
			for (int i = 0; i < size; i++) {
				Object r = de.readObject();
				e.add(r.toString());
			}
			de.close();
		} catch (Exception e) {
			e.printStackTrace();
		}
	}
}

Danke

 

[Murray]

Vielleicht solltest Du die Länge auch noch in der Datei speichern, dann kannst Du auch Dateien einlesen, ohne dass die Liste vorher schon da in der passenden Größe existieren muss.