Problem mit Serialisierung von jTable

[wottpal]

Hi,
ich wollte meine 2 jTables serialisieren und speichern, da sich auch z.B. Spalten-/Zeilenbreite ändern, muss ich neben dem DefaultTableModel auch das TableColumnModel serialisieren, nur damit keine Fragen auftreten.
Wenn ich den unten aufgeführten Quelltext ausführe funktioniert alles wunderbar, kann auch alles wieder deserialisieren ohne Fehler. Doch wenn ich zum 2. mal speichern will (ohne etwas gemacht zu haben) kommt die Fehlermeldung:

java.io.NotSerializableEception: com.sun.java.swing.plaf.windows.WindowsTableHeaderUI

Wie kann ichs jetzt verhindern, dass der diesen WindowsTableHeaderUI-Dünsch mit serialisiert, brauch ich doch bestimmt sowieso nicht..

Quelltext:

    public void save() throws IOException {
	if (jTable1.getCellEditor() != null && !jTable1.getCellEditor().stopCellEditing()) {
            jTable1.getCellEditor().cancelCellEditing();
        }
        if (jTable2.getCellEditor() != null && !jTable2.getCellEditor().stopCellEditing()) {
            jTable2.getCellEditor().cancelCellEditing();
        }
        jTable1.requestFocusInWindow();
        jTable2.requestFocusInWindow();
		
        try {
            String filename = "filename";
            if (jFileChooser1.showSaveDialog(jMenu1) == JFileChooser.APPROVE_OPTION) {
                filename = jFileChooser1.getSelectedFile().toString();
                if (!filename.endsWith(".turing")) {
                    filename += ".turing";
                }

                FileOutputStream file = new FileOutputStream(filename);
                ObjectOutputStream o = new ObjectOutputStream(file);

                // jTable1
                DefaultTableModel model1 = cloneModelWithoutListeners((DefaultTableModel) jTable1.getModel());
                TableColumnModel cModel1 = jTable1.getColumnModel();
                o.writeObject(model1);
                o.writeObject(cModel1);

                // jTable2
                DefaultTableModel model2 = cloneModelWithoutListeners((DefaultTableModel) jTable2.getModel());
                TableColumnModel cModel2 = jTable2.getColumnModel();
                o.writeObject(model2);
                o.writeObject(cModel2);
                
                o.close();
            }
        } catch (IOException e) {
            System.err.println(e);
        }
    }
	
    // Entfernt unötige Listener, die nicht mit serialisiert werden müssen
    public DefaultTableModel cloneModelWithoutListeners(
            DefaultTableModel orgModel) {
        final Vector<?> clonedData = (Vector<?>) orgModel.getDataVector().clone();
        final int columnCount = orgModel.getColumnCount();
        final Vector<String> columnNames = new Vector<String>(columnCount);
        for (int i = 0; i < columnCount; i++) {
            columnNames.add(orgModel.getColumnName(i));
        }
        return new DefaultTableModel(clonedData, columnNames);
    }

Vielen Dank im Voraus, wottpal

 

[bERt0r]

Probier mal jTable1.removeEditor()

 

[SlaterB]

ansonsten überlege, die Spaltenbreiten und was immer noch fehlt auch im TableModel abzulegen,
du braust ja eh individuellen Code um das ColumnModel auszulesen und zuzuweisen,
an dieser Stelle könnte auch saubere Initialisierung stehen

 

[bERt0r]

Also bei mir funktioniert

o.writeObject(jTable1.getModel());
o.writeObject(jTable1.getColumnModel());

einwandfrei. Über den Sinn der ganzen clonerei wegen seine Listeners lässt sich vielleicht streiten.
Zum ColumnModel gehören aber neben Spaltenbreiten auch die Reihenfolge der Spalten afaik.

Edit: Anscheinend doch keine optimale Lösung. Sauberer ist es, einfach beim TableModel writeObject und readObject zu überschreiben, damit nicht alle abhängigen GUI elemente wie Listeners etc mitserialisiert werden. z.B:

table.setModel(new DefaultTableModel(
			new Object[][] {
				{"Sepp",new Integer(1)},
				{ "Fritz", 2},
				{ "Karl", 3},
				{ "Georg",4},
				{ "Franz", 5},
			},
			new String[] {
				"Name", "Bewertung"
			}
		) {
			/**
			 * 
			 */
			private static final long serialVersionUID = 1L;
			Class[] columnTypes = new Class[] {
				Object.class, String.class
			};
			public Class getColumnClass(int columnIndex) {
				return columnTypes[columnIndex];
			}
			private void writeObject(ObjectOutputStream out) throws IOException
			{
				out.writeObject(this.columnIdentifiers);
				out.writeObject(columnTypes);
				out.writeObject(this.dataVector);
			}
			
			private void readObject(ObjectInputStream in) throws IOException
			{
				try{
					Object o=in.readObject();
					if(o instanceof Vector)
					{
						this.columnIdentifiers=(Vector)o;
					}
					o=in.readObject();
					if(o instanceof Class[])
					{
						this.columnTypes=(Class[])o;
					}
					o=in.readObject();
					if(o instanceof Vector)
					{
						this.dataVector=(Vector)o;
					}
				}catch(ClassNotFoundException e)
				{
					e.printStackTrace();
				}

			}
		});

 

[wottpal]

Okay, danke erstmal, aber es funktioniert immer noch nur 1x
Aber jetzt gibt es eine andere Exception:

java.io.NotSerializableException: com.sun.java.swing.plaf.windows.XPStyle$Skin

Außerdem gab es bei:

 public Class getColumnClass(int columnIndex) {
                return columnTypes[columnIndex];
            }

immer eine NullpointerException, ohne liefs.

Und wie bekomme ich da jetzt noch Spaltenbreite hinein?

Schönen Sonntag, wottpal.

 

[bERt0r]

In der Hoffnung dass du dir vielleicht doch meine Posts auch durchliest, nochmal eine komplette ausführbare Klasse. Ansonsten, wer code postet, dem kann geholfen werden, wer nicht dem nicht.

import java.awt.BorderLayout;
import java.awt.Component;
import java.awt.EventQueue;
import java.io.File;
import java.io.FileInputStream;
import java.io.FileNotFoundException;
import java.io.FileOutputStream;
import java.io.IOException;
import java.io.ObjectInputStream;
import java.io.ObjectOutputStream;
import java.util.Vector;

import javax.swing.JFrame;
import javax.swing.JOptionPane;
import javax.swing.JPanel;
import javax.swing.border.EmptyBorder;
import javax.swing.table.DefaultTableModel;
import javax.swing.table.TableModel;
import javax.swing.JTable;
import javax.swing.JButton;
import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;


public class Serialize2 extends JFrame {

	private JPanel contentPane;
	private JTable table;
	private File tempFile;

	/**
	 * Launch the application.
	 */
	public static void main(String[] args) {
		EventQueue.invokeLater(new Runnable() {
			public void run() {
				try {
					Serialize2 frame = new Serialize2();
					frame.setVisible(true);
				} catch (Exception e) {
					e.printStackTrace();
				}
			}
		});
	}

	/**
	 * Create the frame.
	 */
	public Serialize2() {
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setBounds(100, 100, 450, 300);
		contentPane = new JPanel();
		contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
		contentPane.setLayout(new BorderLayout(0, 0));
		setContentPane(contentPane);
		
		table = new JTable();
		table.setModel(new DefaultTableModel(
	            new Object[][] {
	                {"Sepp",new Integer(1)},
	                { "Fritz", 2},
	                { "Karl", 3},
	                { "Georg",4},
	                { "Franz", 5},
	            },
	            new String[] {
	                "Name", "Bewertung"
	            }
	        ) {
	            /**
	             * 
	             */
	            private static final long serialVersionUID = 1L;
	            Class[] columnTypes = new Class[] {
	                Object.class, String.class
	            };
	            public Class getColumnClass(int columnIndex) {
	                return columnTypes[columnIndex];
	            }
	            private void writeObject(ObjectOutputStream out) throws IOException
	            {
	                out.writeObject(this.columnIdentifiers);
	                out.writeObject(columnTypes);
	                out.writeObject(this.dataVector);
	            }
	            
	            private void readObject(ObjectInputStream in) throws IOException
	            {
	                try{
	                    Object o=in.readObject();
	                    if(o instanceof Vector)
	                    {
	                        this.columnIdentifiers=(Vector<?>)o;
	                    }
	                    o=in.readObject();
	                    if(o instanceof Class[])
	                    {
	                        this.columnTypes=(Class[])o;
	                    }
	                    o=in.readObject();
	                    if(o instanceof Vector)
	                    {
	                        this.dataVector=(Vector<?>)o;
	                    }
	                }catch(ClassNotFoundException e)
	                {
	                    e.printStackTrace();
	                }
	 
	            }
	        });
		contentPane.add(table, BorderLayout.CENTER);
		
		JPanel panel = new JPanel();
		contentPane.add(panel, BorderLayout.SOUTH);
		
		JButton btnSerialize = new JButton("Serialize");
		btnSerialize.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent arg0) 
			{
				try {
					tempFile=File.createTempFile("Table", "tmp");
					ObjectOutputStream o=new ObjectOutputStream(new FileOutputStream(tempFile));
					table.removeEditor();
					o.writeObject(table.getModel());
					o.close();
				} catch (FileNotFoundException e) {
					// TODO Auto-generated catch block
					e.printStackTrace();
				} catch (IOException e) {
					// TODO Auto-generated catch block
					e.printStackTrace();
				}
			}
		});
		panel.add(btnSerialize);
		
		JButton btnDeserialize = new JButton("Deserialize");
		btnDeserialize.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) 
			{
				if(tempFile==null)
				{
					JOptionPane.showMessageDialog((Component) e.getSource(), "Fehler","Erst serialisieren!",JOptionPane.ERROR_MESSAGE);
				}
				else
				{
				try {
					ObjectInputStream in=new ObjectInputStream(new FileInputStream(tempFile));
					TableModel tm=(TableModel)in.readObject();
					table.setModel(tm);
					table.revalidate();
				} catch (FileNotFoundException e1) {
					// TODO Auto-generated catch block
					e1.printStackTrace();
				} catch (IOException e1) {
					// TODO Auto-generated catch block
					e1.printStackTrace();
				} catch (ClassNotFoundException e1) {
					// TODO Auto-generated catch block
					e1.printStackTrace();
				}
				}
			}
		});
		panel.add(btnDeserialize);
	}

}

 

[wottpal]

Krass, vielen dank dafür aber mein Problem bleibt bestehen, write- und read- Methoden überschrieben, klappt trotzdem nicht.

Angehängt ist mal die .jar damit du dich selbst davon überzeugen kannst, dass mehr als 1x speichern nicht drin ist und die .java, also die mit dem "Hauptquelltext", ist ein Netbeans-Projekt. Wenn du noch irgendwas brauchst sags einfach.

wottpal

 

[bERt0r]

Versuch mal:

 public void beansprucheTabellen() {
        jTable1.removeEditor();
        jTable1.removeEditor();         //jTable2.removeEditor();
    }

Ausserdem: "turt" dein Programm noch weiter wenn du Serialisierst? Wenn du am Model herumwerkst während du serialisierst könnte das dein Problem erklären.

 

[wottpal]

Versuch mal:

 public void beansprucheTabellen() {
        jTable1.removeEditor();
        jTable1.removeEditor();         //jTable2.removeEditor();
    }

Ausserdem: "turt" dein Programm noch weiter wenn du Serialisierst? Wenn du am Model herumwerkst während du serialisierst könnte das dein Problem erklären.

Schreibfehler -.- aber die Korrektur bringt trotzdem noch den selben Fehler hervor..
Also ich fummele während dessen eigentlich nicht am Model herum, mich verwundert ja so stark, dass einmal speichern-laden funktioniert, nur beim 2. mal nicht.

Grüße

 

[Pauler]

Hallo,

ich habe genau das gleiche Problem.

Ich versuche meine TableModel zu serialisieren.

Es funktioniert beim ersten mal einwandfrei, aber immer wenn man es ein 2. mal versucht gibt es eine Exception:
java.io.NotSerializableException: com.sun.java.swing.plaf.windows.XPStyle$Skin
.
.
.
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException


Ich verstehe nicht warum es beim ersten mal immer funktioniert...aber egal was man danach macht (etwas ändern...nichts ändern...ne Stunde warten...) funktioniert es beim 2. Versuch nicht.
Jedesmal nach dem speichern das Programm neustarten ist für mich keine akzeptable Lösung...

Ich würde mich sehr freuen wenn mir jemand das Problem dabei erklären könnte.

 

[SlaterB]

ist das erste Speichern zufällig ganz am Anfang, noch bevor die GUI sichtbar ist?
dann wäre ja verständlich dass manche Komponenten wir irgendein Style noch nicht gesetzt sind

 

[wottpal]

Nope, man sieht alles schon - alles eigentlich genauso wie beim zweiten mal speichern.
Hier mal das ganze Netbeans-Projekt^^

 

[Pauler]

Nein, die GUI wird angezeigt...dann mache ich irgendwas (trage ein paar Felder ein, lass ein wenig in anderen Teilen des Programms arbeiten...oder auch nicht...und dann speicher ich) und es funktioniert. Nur aber eben allein beim 1. mal. Egal was ich alles in der GUI rumklicke...schiebe, veränder oder sonstwas.

Ich habe herausgefunden das der beim schreiben des Objekts beim ersten mal die "com.sun.java.swing.plaf.windows.XPStyle$Skin" garnicht schreibt...
Das diese als zugehörig angesehen wird, scheint mit dem ersten schreiben des TableModel zusammenzuhängen.

Es hängt aber mit dem LookAndFeel zusammen, denn mit Nimbus LAF funktioniert das speichern ohne Probleme.

 

[bERt0r]

Das funktioniert genau so wie ich oben geschrieben haben. Mehrmals. Nach beliebigem herumdrücken.

 

[Pauler]

Leider nicht.
Ich habe deine Lösungsvorschläge bereits versucht...angefangen vom überschreiben von writeObject() bis hin zu jTable.removeEditor()

möglicherweise habe ich etwas bei deinen Lösungsvorschlägen übersehen (was ich ja nicht komplett ausschließen würde), aber ansonsten besteht das Problem leider immernoch

 

[bERt0r]

In dem Fall, Bedienungsanleitung fürs Forum:
1. Problem beschreiben. Konkret, exakt, am besten den gesamten StackTrace deiner Exception.
2. KSKB posten: Kurzes Beispiel, das dein Fehlerverhalten reproduziert. Wenn ich dein KSKB in ein leeres File kopiere, sollte es nachher lauffähig sein.