Swing JTable: Fehler beim Nutzen von RowSorter#setRowFilter

[hdi]

Hi, mir fliegen hier grad NullPointer-Exceptions um die Ohren, genau genommen eine einzige. Abgesehen davon hab ich noch ein weiteres Problem, was zwar keine Exception wirft, aber trotzdem in meinen Augen fehlerhaft ist.

Also ich hab ne Methode updateTable, die aufgerufen wird nachdem die Daten im Model verändert wurden (ich mach das von außen, also nicht durch die Editier-Funktion des Tables).
Die schaut so aus:

	protected void updateTable() {
		/*
		 * We just re-validate the affected row, and then update the sorter,
		 * which is less expensive than firing a complete data change
		 * notification to the table. Besides, we won't lose the selection, so
		 * we don't have to re-set the selection.
		 */
		Database model = (Database) table.getModel();
		int modelIndex = model.getCopy().indexOf(movie);
		model.fireTableRowsUpdated(modelIndex, modelIndex);
		table.getRowSorter().allRowsChanged();

		/*
		 * the entry might have gotten out of the view area, so we snap the
		 * scroll bar to make it visible
		 */
		int viewIndex = table.convertRowIndexToView(modelIndex);
		Rectangle cellRect = table.getCellRect(viewIndex, 0, false);
		table.scrollRectToVisible(cellRect);
	}

Das funzt soweit auch. allRowsChanged() muss ich aufrufen damit neu sortiert wird, denn es kann ja sein dass durch die Änderung die aktuell angewählte Zeile neu einsortiert werden muss.

Jetzt hab ich mir noch einen Filter geschrieben. Durch nen Toggle button kann man den ein- und ausschalten. Beim Einschalten wird ein RowFilter erzeugt, und gesetzt:

sorter.setRowFilter(myFilter); // sorter = table.getRowSorter...

Beim deaktivieren wird einfach null übergeben. Das funktioniert soweit auch noch alles.

Beim Einschalten filtert er mir also alles raus, was nicht mit meinem Suchbegriff matcht. Aber hier tritt die erste Merkwürdigkeit auf: Wenn ich einen Eintrag dann anklicke, wird er zwar ausgewählt (der ListSelectionListener reagiert, und der Index stimmt), aber irgendwie wird die Zeile nicht markiert. D.h. der Renderer springt wohl nicht an?!

Okay aber das ist jetzt gar nicht mein Hauptproblem. Mein Hauptproblem ist: Wenn ich den Filter wieder deaktiviere (was dazu führt dass wieder alle Einträge sichtbar sind), und ich klicke dann auf einen Eintrag, kommt ne Null-Pointer-Exception:

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at javax.swing.JTable$SortManager.restoreSelection(Unknown Source)
at javax.swing.JTable$SortManager.processChange(Unknown Source)
at javax.swing.JTable.sortedTableChanged(Unknown Source)
at javax.swing.JTable.sorterChanged(Unknown Source)
at javax.swing.RowSorter.fireRowSorterChanged(Unknown Source)
at javax.swing.RowSorter.fireRowSorterChanged(Unknown Source)
at javax.swing.DefaultRowSorter.sort(Unknown Source)
at javax.swing.DefaultRowSorter.allRowsChanged(Unknown Source)
at gui.details.DetailsPanel.updateTable(DetailsPanel.java:96)

das allRowsChanged() ist das Problem. Nimm ich es raus, kommt kein Fehler. Aber ich brauch das ja, wie ich weiter oben erläutert hab. Ich versteh das nicht, wieso NPE bei restoreSelection()? Was hat das mit allRowsChanged() zu tun.

... falls das irgendwie alles unklar ist, bitte fordert weiteren Code ein. Hat jemand eine Idee, was hier abgeht?

 

[Spacerat]

Der fehler scheint eindeutig in [c]allRowsChanged()[/c] zu liegen.
Mal sehen, ob ich das richtig verstanden habe...
1. Beim deaktiviertem Filter fliegt diese Exception
2. Bei aktiviertem Filter werden die Einträge zwar angezeigt, können aber nicht selektiert werden.

Ich vermute mal, dass für Ausgabe und Sortierung ungewollt zwei verschiedene Listen verwendet werden, bzw. entstehen.
Die NPE bekommst du wohl, weil in einer unsynchonisierten Methode, der Filter plötzlich Null werden kann, selbst dann, wenn vorher gegen Null geprüft wurde.
Naja, das ist jetzt alles rein hypothetisch... Die Methoden für Ausgabe und [c]allRowsChanged()[/c] wären für mich mal interessant.

 

[hdi]

Beim deaktiviertem Filter fliegt diese Exception

Nur, wenn man ihn vorher aktiviert hatte, und nach dem Abschalten wieder einen Eintrag anklickt

Bei aktiviertem Filter werden die Einträge zwar angezeigt, können aber nicht selektiert werden.

Selektiert werden können sie schon, wie gesagt der SelectionListener reagiert und das Event liefert auch den richtigen Index. Aber ich hab nen CellRenderer der bei isSelected() den Hintergrund der Zeile färbt. Das passiert beim erstmaligen Anklicken eines Eintrags nach dem Aktivieren des filters nicht...Da kommt auch kein Fehler oder so, aber die Zeile färbt sich nicht ein, so, als wäre sie nicht selected.

Die Methoden für Ausgabe und allRowsChanged() wären für mich mal interessant.

allRowsChanged() ist nicht meine Methode, schon klar oder? Hört sich so an als denkst du das wäre eine eigene. Das ist aber eine Standardmethode der API.
Und was meinst du mit Ausgabe?

Danke für die Hilfe!

 

[Spacerat]

Äh... [c]allRowsChanged()[/c] ist 'ne abstrakte Methode in [c]RowSorter[/c]. Nun kenne ich die tieferen Analen von Swing nicht weit, um zu Wissen, was man mit der Klasse überhaupt machen kann, wie z.B. eigene Sortiermethoden basteln um Filter darauf anzuwenden. Eines istmir jedoch Klar: Zwischen [c]allRowsChanged()[/c] und [c]myFilter[/c] besteht ein dichter Zusammenhang. Ich komm nicht umhin, anzunehmen, dass die Methode vor dem Rendern eine Methode in deinem Filter aufruft bzw. dein Filter vorher schon etwas mit dem SelectionModel des Tables gemacht (bzw. nicht gemacht) hat, z.B. beim entfernen "null" gesetzt. Hab' da mal den verdächtigen Abschitt der Fehlermeldung rausgefischt:[JAVA=4016]// Aus JTable$SortManager
private void restoreSelection(ModelChange change) {
syncingSelection = true;
if (lastModelSelection != null) {
restoreSortingSelection(lastModelSelection,
modelLeadIndex, change);
lastModelSelection = null;
} else if (modelSelection != null) {
ListSelectionModel viewSelection = getSelectionModel();
viewSelection.setValueIsAdjusting(true);
viewSelection.clearSelection();
int min = modelSelection.getMinSelectionIndex();
int max = modelSelection.getMaxSelectionIndex();
int viewIndex;
for (int modelIndex = min; modelIndex <= max; modelIndex++) {
if (modelSelection.isSelectedIndex(modelIndex)) {
viewIndex = convertRowIndexToView(modelIndex);
if (viewIndex != -1) {
viewSelection.addSelectionInterval(viewIndex,
viewIndex);
}
}
}
// Restore the lead
int viewLeadIndex = modelSelection.getLeadSelectionIndex();
if (viewLeadIndex != -1) {
viewLeadIndex = convertRowIndexToView(viewLeadIndex);
}
SwingUtilities2.setLeadAnchorWithoutSelection(
viewSelection, viewLeadIndex, viewLeadIndex);
viewSelection.setValueIsAdjusting(false);
}
syncingSelection = false;
}[/code]Ich nehme mal an, das Zeile 4025 die Exception wirft, weil das Entfernen des Filters auch das SelectionModel ungültig gemacht hat. Aber wie gesagt... Bin nicht wirklich fit in Swing...
Naja... Mit der Ausgabe habe ich wohl den CellRenderer gemeint.

 

[hdi]

Keine Ahnung wie du jetzt auf Zeile 4025 gekommen bist, aber du hast Recht! Ich hatte in meinen ListSelectionListenern keine Überprüfung auf e.getValueIsAdjusting. Eig. auch so geplant, da ich es cooler fand wenn man mit gezogener Maustaste durch die Table gehen kann und sich die Anzeige updatet...

Aber das bringt einige Fehler mit sich und ist mit Vorsicht zu genießen wie es aussieht. Ich reagiere jetzt nur wenn es nicht adjusting ist. Damit
- hab ich, soweit ich bisher sehen konnte, alle Probleme mit dem Filter gelöst
- mein Performance-Problem gelöst (siehe anderer Thread).

Dafür, dass du dich mit Swing ja sowas von nicht auskennst, war Zeile 4025 ja ganz gut geraten... :autsch:

Merci !!