Swing JTable: Löschen vom Daten im Model

[hdi]

Hallo,
also ich hab mir das JTable Tutorial in der FAQ angeschaut, aber da wird nur auf Verändern des TableModel über das Editieren von Zellen eingegangen.

Bei meinem JTable sind alle Zellen nicht editierbar, allerdings kann man auf andere Art und Weise was im Model ändern. zB löschen.

Und ich hab hier grad ein paar Probleme damit.
Also meine Daten liegen in einer Liste, ich hab ein TableModel implementiert, vllt liegt hier schon der Fehler, aber dazu später.

Gut also das löschen geschieht eben über einen Button und das sieht so aus:
(MovieTable ist mein JTable, getAll() liefert eine List<Movie>)

		// get index of selected row within the table
		int selRow = MovieTable.getInstance().getSelectedRow();

		// retrieve movie object. ATTENTION: If table is sorted, the selection
		// index does not equal with the real index of the movie object within
		// the database, so we must convert that!
		int realIndex = MovieTable.getInstance().convertRowIndexToModel(selRow);
		Movie selectedMovie = Database.getInstance().getAll().get(realIndex);

		// delete it!
		Database.getInstance().getAll().remove(selectedMovie);
                // must call this so that the table updates itself:
		MovieTable.getInstance().revalidate();
		MovieTable.getInstance().repaint();

Also der obige Code ist getestet und das ist auch richtig so, auch das Konvertieren des Index.

Konkret habe ich jetzt aber 2 verschiedene Fehler:

1) Ich selektiere im Table die letzte Reihe, und klick auf meinen Delete-Button. In diesem Fall verschwindet sie aus dem Table, und es kommt keine Exception. Soweit also alles prima, in der getValueAt sieht man an einem sysout, dass im Datenmodel auch alles passt und das Objekt aus der Liste raus ist. Außerdem passen alle Zeilen-Indices im Table zu den Daten. Wenn ich dann aber wieder eine Zeile markiere, kommt folgende Exception:

Exception in thread "AWT-EventQueue-0" java.lang.IndexOutOfBoundsException: Invalid index
at javax.swing.DefaultRowSorter.convertRowIndexToModel(Unknown Source)
at javax.swing.JTable.convertRowIndexToModel(Unknown Source)
at javax.swing.JTable.setValueAt(Unknown Source)
at javax.swing.JTable.editingStopped(Unknown Source)
at javax.swing.AbstractCellEditor.fireEditingStopped(Unknown Source)
at gui.table.SpecialCellRenderer.fireEditingStopped(SpecialCellRenderer.java:50)
at javax.swing.DefaultCellEditor$EditorDelegate.stopCellEditing(Unknown Source)
at javax.swing.DefaultCellEditor.stopCellEditing(Unknown Source)
at gui.table.SpecialCellRenderer.stopCellEditing(SpecialCellRenderer.java:46)

Hier muss ich wohl sagen: die setValueAt()-Methode habe ich nicht implementiert! Ich hab es nicht getan weil man meine Daten ja eh nicht über den Table editieren kann, und ich wüsste auch nicht warum der Table hier meine Daten verändern können sollte. Aber immerhin kommt diese Methode in der Exception vor. Also kA, verstehen tu ich's aber nicht, denn der Table soll ja nur meine Daten anzeigen, wofür braucht er da bitte eine setValueAt() ?

Der zweite Fehler kommt vor, wenn ich die Table sortiert habe (es liegt der Default-RowSorter drauf, also setAutoCreateRowSorter(true)), und dann irgendwas anklicke und löschen will:

Exception in thread "AWT-EventQueue-0" java.lang.IndexOutOfBoundsException: Index: 2, Size: 2
at java.util.ArrayList.RangeCheck(Unknown Source)
at java.util.ArrayList.get(Unknown Source)
at gui.table.MyTableModel.getValueAt(MyTableModel.java:35)
at javax.swing.JTable.getValueAt(Unknown Source)

Hier ist etwas ganz strange. Zum Verständnis erstmal: Diese Auszüge kamen aus einem Test wo ich 3 Elemente in der Liste hatte. Die Size wird als 2 angezeigt, weil diese Expcetion wohl geworfen wird, nachdem ich in der Datenliste das Element gelöscht habe, also ist die Size dann halt nur noch 2.
Aber was ich gar nicht kapiere: Der Index ist 2, und zwar immer. Total egal, welche Zeile ich vor dem Sortieren angeklickt hatte, oder nach welcher Spalte ich sortiere. Mein ListSelectionListener des JTable sieht das ganz anders, und zwar richtig! Wieso geht er hier von nem Index = 2 aus? Woher hat er den, wie gesagt der ListSelectionListener zeigt mir korrekte Werte!

Hilfe ;(

 

[Marco13]

Is schon spät, darum nur kurz:

Gut also das löschen geschieht eben über einen Button und das sieht so aus:
(MovieTable ist mein JTable, getAll() liefert eine List<Movie>)

...
  // delete it!
  Database.getInstance().getAll().remove(selectedMovie);
  // must call this so that the table updates itself:
  MovieTable.getInstance().revalidate();
  MovieTable.getInstance().repaint();

Wenn ich das richtig verstehe, veränderst du dort eine Liste, die die eigentliche Datenhaltung deines TableModels erledigt? Und dann revalidate um die JTable zu aktualisieren?

Falls ja: Eigentlich sollten bei Änderungen am TableModel sofort (und vom TableModel selbst) die passenden Events geworfen werden, die jeden Listener (also auch die JTable) über diese Änderungen benachrichtigen. Am einfachsten ist das, wenn man von AbstractTableModel (Java 2 Platform SE v1.4.2) erbt, und bei allen Änderungen die passenden "fire"-Methoden aufruft. Wenn man die Daten einer List verändert, und die JTable dann z.B. beim Neuzeichnen indirekt über's TableModel auf diese List zugreifen will, kracht's.

EDIT: Hach ja ZU spät: Das delete steht ja beim Database.getInstance ... sorry.

 

[hdi]

Also ich weiss jetzt nicht ob das jetzt stimmt was du sagst? Weil du am Ende irgendwas von edit geschreiben hast?

Also zur Klarstellung: Dieses Database.getInstance().getAll() liefert eine Liste, und die stellt die Daten meines Table dar.
D.h. in meinem TableModel greife ich darauf zu, zB die getValueAt:

@Override
	public Object getValueAt(int rowIndex, int columnIndex) {
		// rowIndex here is already converted to model index!
		Movie m = Database.getInstance().getAll().get(rowIndex);

		switch (columnIndex) {
		// je nachdem wird jetzt ein Getter von "m" aufgerufen und returned
		}
	}

Bisher hatte ich komplett TableModel implementiert, habs jetzt geändert auf extends AbstractTableModel, die beiden Fehler bestehen aber weiterhin unverändert

 

[ModellbahnerTT]

Schau mal hier: http://www.java-forum.org/java-faq-beitraege/7035-jtable-teil-2-wie-kommen-daten-die-tabelle.html

 

[hdi]

@Modellbahner:

Ich habe jetzt mein Model von DefaultTableModel abgeleitet, es sieht nun so aus:

public class MyTableModel extends DefaultTableModel {

	private static final MyTableModel instance = new MyTableModel();

	public static MyTableModel getInstance() {
		return instance;
	}

	private MyTableModel() {
	}

	@Override
	public Class<?> getColumnClass(int columnIndex) {
		return TableColumns.getColumnClass(columnIndex);
	}

	@Override
	public int getColumnCount() {
		return 8;
	}

	@Override
	public String getColumnName(int columnIndex) {
		return TableColumns.getTitle(columnIndex);
	}

	@Override
	public int getRowCount() {
		return Database.getInstance().getAll().size();
	}

	@Override
	public Object getValueAt(int rowIndex, int columnIndex) {
		// rowIndex here is already converted to model index!
		Movie m = Database.getInstance().getAll().get(rowIndex);

		switch (columnIndex) {
		case TableColumns.TITLE:
			return m.getTitleGerman();
		case TableColumns.YEAR:
			return m.getYear();
		case TableColumns.GENRE:
			return m.getGenre();
		case TableColumns.FORMAT:
			return m.getFormat();
		case TableColumns.DURATION:
			return m.getDuration();
		case TableColumns.FSK:
			return m.getFsk();
		case TableColumns.RATING:
			return m.getRating();
		default:
			return null;
		}
	}

	@Override
	public boolean isCellEditable(int rowIndex, int columnIndex) {
		return columnIndex == TableColumns.SPECIAL;
	}

	@Override
	public void setValueAt(final Object aValue, final int rowIndex,
			final int columnIndex) {
		fireTableCellUpdated(rowIndex, columnIndex);
	}

}

Und mein Delete-Button macht dies hier:

                // get index of selected row within the table
		int selRow = MovieTable.getInstance().getSelectedRow();
                // delete it
		MyTableModel.getInstance().removeRow(selRow);

Allerdings passiert hier nix. Meine Daten sind noch immer die selben danach.

 

[Marco13]

Wenn das so gemacht ist, wie du jetzt geschrieben hast, dann sollte das, was ich gesagt habe, trotz des edits stimmen.

                // get index of selected row within the table
		int selRow = MovieTable.getInstance().getSelectedRow();
                // delete it
		MyTableModel.getInstance().removeRow(selRow);

Allerdings passiert hier nix. Meine Daten sind noch immer die selben danach.

In dem letzten TableModel steht ist auch die Methode removeRow nicht überschrieben?!

FALLS ich das richtig verstanden habe, solltest du aber nicht von DefaultTableModel ableiten. Das DefaultTableModel hat intern eine Liste, die die Tabellendaten verwaltet. Wenn du davon ableitest, verwendest du ggf. eine Mischung aus der Liste des DefaultTableModels, und der Liste, die von deiner DataBase geliefert wird.

Eigentlich sollte es das AbstractTableModel schon tun. Dort würde dann sowas stehen wie

public void removeRow(int index)
{
    Database.getInstance().getAll().remove(index);
    fireTableDataChanged() ;
}

Aber noch ganz "Off Topic" und nebenbei: Das mit dem getAll sieht etwas seltsam aus: Die Datenbank-Instanz speichert eine Liste, die man von außen verändern kann, und die die Liste ist, die ein TableModel füttert? Und das ganze als Singletons - in der Hoffnung, dass du NIE zwei TableModels haben willst Man wird mit dieser Struktur IMMER auf Probleme stoßen, weil mit dem "Database.getInstance().getAll()" jeder (Idiot) sich die Liste holen, und nach belieben drin rumpfuschen kann. Die Table kreigt davon nichts mit, weil ja nur irgendwo eine Liste der Datenbank verändert wird, und niemand weiß, dass das "zufällig" auch die Liste des TableModels ist.
Wichtig zu wissen wäre: Wer erstellt die Liste, die mit Database.getInstance().getAll() zurückgelifert wird, und wie ist die Verbindung dieser Liste zur eigentlichen Datenbank?

 

[ModellbahnerTT]

Also ich weiß jetzt nich warum du von DefaultTableModel abgeleitet hast, dass bei deinem removeRow aufruf nichts passiert ist auch nicht verwunderlich, du änderst ja auch nie was an deinem datenmodel :noe:
Marco13 hat auch eigentlich schon alles wichtige gesagt, in der FAQ steht das alles nochmal etwas ausführlicher, einfach mal implementieren...
Und entfern am besten erstmal den ganzen static-Müll, warum soll ein TableModel ein singleton sein? :shock:

 

[hdi]

EDIT: Ich hoffe keiner ist gerade dabei den 8-seiten Text zu lesen Ich hab jetzt eine Idee: Ich fange das Programm neu an, from scratch, und ich würde dann immer posten, wenn ich etwas bestimmtes getan habe (natürlich nur was den Table und die Daten betrifft, nicht jeden Schmarrn!!), damit wir das gemeinsam "absegnen" können.
Würdet ihr euch da bereit erklären für?
Weil ich glaube mein Programm ist jetzt schon soviel Choas... ;( Ich möchte halt auch einfach eine tolle GUI machen, hab ne tolle Vorstellung aber das erfordert eben lauter Custom Renderers und blabla, und das hab ich wohl scheinbar nicht so drauf..

Also ich würde jetzt neu beginnen, und wenn ich zB das TableModel implementiert habe, würde ich es hier gleich posten und fragen, ob das so okay ist. Wenn ich dann zB die Sortierung implementiert habe, würd ich das auch posten, und wenn ich einen speziellen Renderer geschrieben habe, auch.

So kommt ihr halt auch mit, es ist völlig klar dass es höllisch schwer für euch ist zu helfen wenn ich schon 4000 Zeilen Code habe und überall lauter Fehler..

Ich mache dafür jetzt einen neuen Thread auf, ihr findet ihn unter "AWT, Swing & SWT" mit dem Titel "Gemeinsame JTable-Implementierung".

Hoffe das ist okay, ansonsten löschen. Aber ich glaube nur auf diese Art und Weise krieg ich etwas vernünftiges zusammen, meine Ansprüche an das Programm sprengen halt leider auch das, was es in der FAQ zum Thema Table zu lesen gibt...

Merci euch, see you im anderen Thread hoffentlich