Objekte werden nicht aus Datei eingelesen

[iCeFrEsH]

Hallo zusammen,

bin derzeit daran eine Highscoreliste für ein Spiel zu schreiben, welche sich automatisch in einer Datei speichert und lädt. Speichern tut sie, öffnen geht aber nicht. Hier der Code:

 File datei = new File("C:\\highscore.dat");
    try {
      FileInputStream fis = new FileInputStream(datei);
      ObjectInputStream ois = new ObjectInputStream(fis);
      // Objekte einlesen
      TableModelHighscoreList.getInstance().setHighscoreliste((ArrayList<HighscoreEintrag>) ois.readObject());
      // Stream schließen
      fis.close();
      System.out.println("GELESEN");  // nur zum debuggen da
    }
    catch (Exception e1) {
      // TODO Auto-generated catch block
      e1.printStackTrace();
    }

Das GELESEN aus dem System.out erscheint auch, sonst kommt auch keine Fehlermeldung =(


Danke schonmal!

 

[SlaterB]

nichts spricht dafür, dass in C:\\highscore.dat irgendwas drinsteht,
es fehlt die zugehörige write-Operation sowie überhaupt ein komplettes Programm drumherum,

und das ganze möglichst kurz,
HighscoreEintrag sollte eine Dummy-Serializable-Klasse sein, nur mit einem String "test" oder so darin,

TableModelHighscoreList usw. wird nicht benötigt,
nur eine main mit 'Liste erstellen', 'Liste schreiben', 'Liste lesen' ausgeben und fertig

 

[iCeFrEsH]

Also in C:\highscore.dat steht etwas drinnen, das hat es vorher hineingeschrieben. Dort steht drinnen (falls es wen interessiert):

¬í sr java.util.ArrayListx￾Ò™Ça￾ 
I sizexp   
w   
sr spiderschwein.HighscoreEintrag×£ˆ’};ó I punkteL namet Ljava/lang/String;xp  ît Testx

Hier mal die komplette Datei (Hinweis: die beiden Buttons dienen derzeit nur zum debuggen):

package spiderschwein;

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 java.util.ArrayList;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.JToggleButton;
import javax.swing.JButton;

public class HighscoreListGUI extends JFrame {

  private JPanel jContentPane = null;
  private JScrollPane scpHighscoreList = null;
  private JTable tabHighscoreList = null;
//  private ArrayList einträge = new ArrayList();
  private JToggleButton btnRestart = null;
  private JButton btnBeenden = null;
  
  /**
   * This is the default constructor
   */
  public HighscoreListGUI() {
    super();
    initialize();
  }

  /**
   * This method initializes this
   * 
   * @return void
   */
  private void initialize() {
    this.setSize(284, 242);
    this.setDefaultCloseOperation(javax.swing.JFrame.EXIT_ON_CLOSE);
    this.setContentPane(getJContentPane());
    this.setTitle("Highscore Liste");
    this.addWindowListener(new java.awt.event.WindowAdapter() {   
    	public void windowClosed(java.awt.event.WindowEvent e) {    
       speichern();
    	}
      public void windowOpened(java.awt.event.WindowEvent e) {
       oeffnen();
      }
      
});
  }

  /**
   * This method initializes jContentPane
   * 
   * @return javax.swing.JPanel
   */
  private JPanel getJContentPane() {
    if (jContentPane == null) {
      jContentPane = new JPanel();
      jContentPane.setLayout(null);
      jContentPane.add(getScpHighscoreList(), null);
      jContentPane.add(getBtnRestart(), null);
      jContentPane.add(getBtnBeenden(), null);
    }
    return jContentPane;
  }

  /**
   * This method initializes jScrollPane  
   *  
   * @return javax.swing.JScrollPane  
   */
  private JScrollPane getScpHighscoreList() {
    if (scpHighscoreList == null) {
      scpHighscoreList = new JScrollPane();
      scpHighscoreList.setBounds(new java.awt.Rectangle(0,-2,280,179));
      scpHighscoreList.setViewportView(getTabHighscoreList());
    }
    return scpHighscoreList;
  }

  /**
   * This method initializes jTable 
   *  
   * @return javax.swing.JTable 
   */
  private JTable getTabHighscoreList() {
    if (tabHighscoreList == null) {
      tabHighscoreList = new JTable(TableModelHighscoreList.getInstance());
    }
    return tabHighscoreList;
  }

//  public boolean add(Object neuerUser) {
//    return einträge.add(neuerUser);
//  }

  /**
   * This method initializes btnRestart	
   * 	
   * @return javax.swing.JToggleButton	
   */
  private JToggleButton getBtnRestart() {
    if (btnRestart == null) {
      btnRestart = new JToggleButton();
      btnRestart.setText("Neustart");
      btnRestart.setSize(new java.awt.Dimension(130,30));
      btnRestart.setLocation(new java.awt.Point(0,175));
      btnRestart.addActionListener(new java.awt.event.ActionListener() {
        public void actionPerformed(java.awt.event.ActionEvent e) {
          oeffnen();
        }
      });
    }
    return btnRestart;
  }

  /**
   * This method initializes btnBeenden	
   * 	
   * @return javax.swing.JButton	
   */
  private JButton getBtnBeenden() {
    if (btnBeenden == null) {
      btnBeenden = new JButton();
      btnBeenden.setLocation(new java.awt.Point(138,175));
      btnBeenden.setText("Beenden");
      btnBeenden.setSize(new java.awt.Dimension(130,30));
      btnBeenden.addActionListener(new java.awt.event.ActionListener() {
        public void actionPerformed(java.awt.event.ActionEvent e) {
          speichern();
        }
      });
    }
    return btnBeenden;
  }

  
  public void speichern(){
    File datei = new File("C:\\highscore.dat");
    try {
      datei.createNewFile();
      FileOutputStream fos = new FileOutputStream(datei);
      ObjectOutputStream oos = new ObjectOutputStream(fos);
      // Kundenliste serialisieren
      oos.writeObject(TableModelHighscoreList.getInstance().getHighscoreliste());
      fos.close();
      System.out.println("OK, saved");
    }
    catch (IOException e1) {
      // TODO Auto-generated catch block
      e1.printStackTrace();
      System.out.println("Fehler");
    }
  }
  
  public void oeffnen(){
    File datei = new File("C:\\highscore.dat");
    try {
      FileInputStream fis = new FileInputStream(datei);
      ObjectInputStream ois = new ObjectInputStream(fis);
      // Objekte einlesen
      TableModelHighscoreList.getInstance().setHighscoreliste((ArrayList<HighscoreEintrag>) ois.readObject());
      // Stream schließen
      fis.close();
      System.out.println("GELESEN");
    }
    catch (Exception e1) {
      // TODO Auto-generated catch block
      e1.printStackTrace();
    }
  }
  
}  //  @jve:decl-index=0:visual-constraint="10,10"

Und die Objektklasse:

package spiderschwein;
import java.io.Serializable;
public class HighscoreEintrag implements Serializable{

  private String name;
  private int punkte;
  
    
  public HighscoreEintrag(String name, int punkte) {
    this.name = name;
    this.punkte = punkte;
  }
  
  public String getName() {
    return name;
  }
  public int getPunkte() {
    return punkte;
  }
  
  
}

und zu guter letzt das Tablemodel:

package spiderschwein;
import java.util.ArrayList;
import java.util.Observable;
import java.util.Observer;

import javax.swing.table.AbstractTableModel;

public class TableModelHighscoreList extends AbstractTableModel implements Observer {

  private String[] colHeads={"Name", "Punkte"};
  private ArrayList<HighscoreEintrag> einträge = new ArrayList<HighscoreEintrag>();
  private static TableModelHighscoreList container = null;
  
    private TableModelHighscoreList() {
    }
    
    public static TableModelHighscoreList getInstance(){
      if(container==null){
        container = new TableModelHighscoreList();
      }
      return container;
    }

    public boolean add(HighscoreEintrag o) {
    return einträge.add(o);
  }

  public Object get(int index) {
    return einträge.get(index);
  }

  public Object remove(int index) {
    return einträge.remove(index);
  }

  public Object set(int index, HighscoreEintrag element) {
    return einträge.set(index, element);
  }

  public int size() {
    return einträge.size();
  }
  
  public ArrayList<HighscoreEintrag> getHighscoreliste() {
    return einträge;
  }
  
  public void setHighscoreliste(ArrayList<HighscoreEintrag> einträge) {
    einträge = this.einträge;
  }



  //gibt die Anzahl der Tabellenzeilen an
    public int getRowCount() {
      return einträge.size();
    }

    //gibt die Anzahl der Spalten an
    public int getColumnCount() {
      return colHeads.length;
    }

    public Object getValueAt(int rowIndex, int columnIndex) {
      HighscoreEintrag einEintrag = einträge.get(rowIndex);
      switch(columnIndex){
        case 0: return einEintrag.getName();
        case 1: return einEintrag.getPunkte();
        }
      return null;
    }
    
    //gibt die Spaltenüberschrift zurück
    public String getColumnName(int spalte){
      return colHeads[spalte];
    }

    public void update(Observable o, Object arg) {
      fireTableDataChanged();
    }

  }

Vielleicht hilft das jemanden weiter - ohne euch verwirren zu wollen

 

[NTB]

Bitte nochmal den Beitrag von SlaterB lesen und verstehen

 

[iCeFrEsH]

Bitte nochmal den Beitrag von SlaterB lesen und verstehen

Hmmm, okay, dann arbeiten wir uns mal Stück für Stück durch:

nichts spricht dafür, dass in C:\\highscore.dat irgendwas drinsteht,

--> doch, habe ich ja gepostet

es fehlt die zugehörige write-Operation sowie überhaupt ein komplettes Programm drumherum,

--> ist auch drinne, siehe Programmcode - funktioniert auch, sonst würde ja nichts in der Datei stehen

und das ganze möglichst kurz, HighscoreEintrag sollte eine Dummy-Serializable-Klasse sein, nur mit einem String "test" oder so darin,

--> Objekt von HighscoreEintrag wird nach Spiel (dessen Quelltext ich nicht gepostet habe, da es funktioniert und mit der Highscore wenig zu tun hat) angelegt. Name + Punktzahl wird in der Tabelle des HighscoreListGUI auch angezeigt.

TableModelHighscoreList usw. wird nicht benötigt, nur eine main mit 'Liste erstellen', 'Liste schreiben', 'Liste lesen' ausgeben und fertig

Wird imo doch benötigt, da ich eine JTable on JScrollPane verwende. Aber daran sollte es ja nicht liegen

Sofern, ich blick irgendwie gerade nicht mehr so ganz durch, wieso es nicht geht

 

[SlaterB]

public class Test
{

    public static void main(String[] args)
        throws Exception
    {
        speichern();
        oeffnen();
    }

    public static void speichern()
        throws Exception
    {
        List<HighscoreEintrag> list = new ArrayList<HighscoreEintrag>();
        list.add(new HighscoreEintrag());

        File datei = new File("highscore.dat");
        FileOutputStream fos = new FileOutputStream(datei);
        ObjectOutputStream oos = new ObjectOutputStream(fos);
        oos.writeObject(list);
        fos.close();
        System.out.println("OK, saved");
    }

    public static void oeffnen()
        throws Exception
    {
        File datei = new File("highscore.dat");
        FileInputStream fis = new FileInputStream(datei);
        ObjectInputStream ois = new ObjectInputStream(fis);
        List<HighscoreEintrag> list = (List<HighscoreEintrag>)ois.readObject();
        fis.close();
        System.out.println("GELESEN: " + list);
    }

}


class HighscoreEintrag
    implements Serializable
{
    public String toString()
    {
        return "huhu";
    }
}

funktioniert,
was in deinem Code dagegen nicht funktioniert ist ohne ein lauffähiges kurzes Programm nicht nachvollziehbar,

vielleicht klappt ja sogar schon das Lesen (gib die Liste nach dem Lesen aus)
und dein Problem hat gar nichts mit Dateien zu tun?

> Wird imo doch benötigt, da ich eine JTable on JScrollPane verwende

es ist allein interessant, ob die Liste nach dem Lesen da ist oder nicht, ein TableModel kann dabei nicht helfen

 

[HoaX]

  public void setHighscoreliste(ArrayList<HighscoreEintrag> einträge) {
    einträge = this.einträge;
    fireTableDataChanged(); // der Table will wissen dass sich die Daten geändert haben
  }

ich vermute jetz einfach mal das es geht

 

[iCeFrEsH]

  public void setHighscoreliste(ArrayList<HighscoreEintrag> einträge) {
    einträge = this.einträge;
    fireTableDataChanged(); // der Table will wissen dass sich die Daten geändert haben
  }

ich vermute jetz einfach mal das es geht

Sehr gute Idee, aber es funktioniert dennoch nicht =(

 

[SlaterB]

stell doch bitte endlich mal fest, ob das Datei-einlesen geht oder nicht
und der Rest nur noch ein Problem der Anzeige ist,

dann kann man von vorne anfangen und den unnützen Datei-Code rausschmeißen und ein anderes Test-Programm nur zum aktuellen Problem schreiben

immer Schritt für Schritt vorgehen..

 

[iCeFrEsH]

Okay, habe ich jetzt gemacht.
Er liest die Datei wirklich ein, aber die Tabelle aktualisiert sich nicht...

Btw: noch ein Fehler... Wenn ich die Highscoreliste speicher, klappt alles. Lade ich sie danach (im noch offenen GUI) - funktioniert es auch ohne Fehlermeldung. Schließe ich aber das GUI (die Liste ist jetzt immer noch in einer Datei gespeichert), starte das GUI neu und lese sie ein bekomme ich für meine Debug-System.out. ausgabe (gibt den Namen auf der Konsole aus, zum debuggen --> so habe ich ja rausgefunden dass die Datei eig. korrekt eingelesen wird):

java.lang.IndexOutOfBoundsException: Index: 0, Size: 0

Obwohl der Eintrag in der Datei drinnen steht =(

 

[SlaterB]

an welcher Stelle?


----------

zum ersten Fehler:
hat sich hartnäckig versteckt, aber mit viel Debuggen bei sich nicht ändern wollender Anzeige kommt man schließlich zu

this.einträge = einträge;
statt
einträge = this.einträge;

in setHighscoreliste()

fireTableDateChanged() trotzdem noch dazu

 

[iCeFrEsH]

Danke!!! Da hatte ich wohl voll denen Dreher drinnen...
So, das Einlesen funktioniert jetzt. Jedoch nicht korrekt:

1. Erster User spielt Spiel, kommt in Highscoreliste - steht auch drinnen, wird auch gespeichert, kann auch geöffnet werden,
2. Zweiter User spielt Spiel, kommt auch in Highscoreliste. Trägt seinen Namen ein, Name erscheint aber nicht. Speichert, aber nur erster User steht in der Liste... =(