AWT Speichern von Listkomponenten und Öffnen derer

[Sahty]

Halli Hallo!

Ich tüftle schon seit einiger Zeit, komme aber einfach nicht auf den Fehler:
Ich möchte ein paar Werte, die in einer AWT-list stehen, in einem file speichern.
Danach sollen diese auch wieder geöffnet werden können und somit auch wieder in der AWT-list erscheinen.
Jedoch erscheint nichts in der AWT-list, sie bleibt einfach leer :bahnhof:

Hier mein Quelltext:

private void speichern(java.awt.event.ActionEvent evt) {                           
        int k = list1.getItemCount();
        for (int i = 1; i == k; i++){
        int a;
          try{
            a=Integer.parseInt(list1.getItem(i-1));
          }
          catch(NumberFormatException nfe ){
            a=0;
          }
          try {
            RandomAccessFile file = new RandomAccessFile("Test.dat","rw");
            file.writeInt(a);
            file.close();
          }
          catch ( IOException ie ) {
          }
        }
    }


private void oeffnen(java.awt.event.ActionEvent evt) {                         
        int k = 1;
        for(int i = 0; i == k; i++){
        int a=0;
          try {
            RandomAccessFile file = new RandomAccessFile("Test.dat","r");
            k = (int) ((file.length()) / 4);
            a=file.readInt();
            file.close();
          }
          catch ( IOException ie ) {
          }
          list1.add(""+a);
        }
    }

Ich arbeite übrigens mit NetBeans IDE 6.7.1, habe Vista

Liebe Grüße

 

[Sahty]

Hat keiner eine Idee?

 

[Michael...]

Werden die Einträge denn überhaupt in eine Datei geschrieben bzw. wieder korrekt ausgelesen?
Abgefangene Exceptions sollte man auch beachten (Zeile 16 und 32). Lass sie Dir z.B mit printStackTrace() auf der Konsole ausgeben.

 

[Sahty]

Werden die Einträge denn überhaupt in eine Datei geschrieben bzw. wieder korrekt ausgelesen?

Wie überprüfe ich das denn?

Abgefangene Exceptions sollte man auch beachten (Zeile 16 und 32). Lass sie Dir z.B mit printStackTrace() auf der Konsole ausgeben.

Muss ich die beachten, wenn ich zum Probelauf des Programms erstmal nur richtige Werte eingebe?

 

[Oli]

Hallo,

in beiden Funktionen ist der

catch ( IOException ie ) {
}

leer. Das sollte man tunlichst unterlassen, denn dann sieht man nicht, wenn was schiefgeht. Und ich bin sicher, dass das bei dir der Fall sein dürfte.

Versuchs mal so:

catch ( IOException ie ) {
  e.printStackTrace();
}

Dann wird dir die Exception ausgegeben und du kannst auf den Fehler reagieren.


Grüße Oli

 

[Michael...]

Werden die Einträge denn überhaupt in eine Datei geschrieben bzw. wieder korrekt ausgelesen?

Wie überprüfe ich das denn?

In dem Du nach dem Speichern zum einen schaust ob die Datei überhaupt existiert und wenn ja mit einem Texteditor öffnest und schaust was in der Datei steht. Aber vermutlich wird beim Schreiben bereits eine Exception geworfen - deshalb Exception ausgeben lassen (s. Post von Oli)

 

[Sahty]

Danke ersteinmal für eure Hilfe. Ich habe das ausgeben der Exception mit eingebaut, jedoch wird weder beim speichern noch beim öffnen etwas ausgegeben. Also denke ich mal, dass es nicht daran liegen wird.

Wenn ich die for-Schleife in der Klasse "oeffnen" weglasse, dann gibt er mir zwar etwas aus, jedoch nicht die gespeicherten Zahlen, sondern immer die gleiche Zahl (1074266112).
Hier mal der veränderte Quelltext dazu:

private void oeffnen(java.awt.event.ActionEvent evt) {                         
        //int k = 1;
        //for(int i = 0; i == k; i++){
        int a=0;
          try {
            RandomAccessFile file = new RandomAccessFile("Test.dat","r");
            //k = (int) ((file.length()) / 4);
            a=file.readInt();
            file.close();
          }
          catch ( IOException ie ) {
              ie.printStackTrace();
          }
          list1.add(""+a);
       // }
    }

(Ich habe auch mal double anstatt integer verwendet, dann gibt er mir beim Öffnen nicht 1074266112 sondern 3.0 aus ... ???:L)

Liebe Grüße...

 

[Michael...]

Was mir jetzt erst auffällt. Das

RandomAccessFile file = new RandomAccessFile("Test.dat", ...);

muss in beiden Methoden ausserhalb der Schleife passieren und das close() entsprechend nach der Schleife. Denn erst Datei öffnen alle Werte lesen/schreiben, dann Datei schliessen.
Einlesen würde das ganze über eine while Schleife, da man ja in der Regel vorher nicht weiss wieviel in der Datei drin steht.

 

[Michael...]

Die Methode speichern könnte z.B. so aussehen:

private void speichern() {
		try {
			RandomAccessFile file = new RandomAccessFile("Test.dat", "rw");
			for (int i = 0; i < list1.getItemCount(); i++) {
				int a = 0;
				try {
					a = Integer.parseInt(list1.getItem(i - 1));
				} catch (NumberFormatException nfe) {}
				file.writeInt(a);
			}
			file.close();
		} catch (IOException ie) {
			ie.printStackTrace();
		}
}

Das ActionEvent brauchst Du garnicht übergeben, da Du es in der Methode nicht nutzt. Die Abbruchbedingung in einer for Schleife sollten immer mit <(=) oder >(=) gemacht werden und nicht mit == da die Zählvariablen u.U. auch mal innerhalb der Schleife manipuliert werden könnten.

 

[Sahty]

Vielen Dank, Michael :toll:! Jetzt läuft mein Programm so, wie es sein soll.
Das Problem lag wohl wirklich hauptsächlich daran, dass ich das erstellen und schließen des files innerhalb der Schleife gehabt habe.

Hier nochmal mein jetziger Quelltext (man soll doch seine richtige Lösung dann nochmal posten, oder?):

private void speichern(java.awt.event.ActionEvent evt) {                           
        try {
            RandomAccessFile file = new RandomAccessFile("Test.dat", "rw");
            for (int i = 0; i < list1.getItemCount(); i++) {
                int a = 0;
                try {
                    a = Integer.parseInt(list1.getItem(i));
                }
                catch (NumberFormatException nfe) {
                }
                file.writeInt(a);
            }
            file.close();
        }
        catch (IOException ie) {
            ie.printStackTrace();
        }

    }


private void oeffnen(java.awt.event.ActionEvent evt) {                         
        int a=0;
        try {
            RandomAccessFile file = new RandomAccessFile("Test.dat","r");
            for(int i = 0; i < (int) ((file.length()) / 4); i++){
                a=file.readInt();
                list1.add(""+a);
            }
            file.close();
        }
        catch ( IOException ie ) {
            ie.printStackTrace();
        }
                
    }

MfG und noch schöne Feiertage =)!

PS: Das ActionEvent brauche ich doch, damit meine Buttons reagieren, oder etwa nicht? Deswegen habe ich es drinnen gelassen. (Im Übrigen lässt mich NetBeans da nichts ändern)

 

[Michael...]

PS: Das ActionEvent brauche ich doch, damit meine Buttons reagieren, oder etwa nicht? Deswegen habe ich es drinnen gelassen. (Im Übrigen lässt mich NetBeans da nichts ändern)

Die Variable evt wird in der Methode nicht genutzt, daher ist die Übergabe eigentlich überflüssig - aber diese GUI Builder erzeugen manchmal recht graußigen Code.