JTable. Beim löschen der Zeilen Fehlermeldung

[Moonlight1234]

Ich habe eine JTable mit eignem CellRenderer und eigenem Editor programmiert.
Die Daten sind im TableModel in einem Vector<String> gespeichert.
Das Problem ist folgendes:
Wenn ich mehrere Zeilen selektiere und lösche, und sofort danach wieder Zeilen selektiere kann es zu einer ArrayIndexOutOfBounds-Exception kommen.
Wenn ich nicht sofort nach dem löschen etwas selektiere, tritt das Problem nicht auf.
Meine Vermutung ist, da bei einer Table mit Events gearbeitet wird und Events parallel ablaufende Prozesse sind, das wenn Zeilen gelöscht werden andere Events auf nicht mehr existierende Elemente das Vectors zugreifen.
Ich weiß im Moment nicht wie ich dieses Problem beseitigen kann.
Ein Vector ist threadsicher und die Methoden die Werte im Vector lesen, verändern wurden von mir als synchronized deklariert.

Kennt jemand das Problem?
Wie kann ich das beseitigen?

 

[Guest]

Die Tabelle hat deine Löschaktion noch nicht mitgekriegt
und versucht auf Zeilen zuzugreifen, die nicht mehr existieren.

Siehe
AbstractTableModel#fireTableRowsDeleted(int firstRow, int lastRow)

 

[Moonlight1234]

Ich mache genau das was auch das AbstractTableModel macht:

    public synchronized void removeRow(int row){
        dataVector.removeElementAt(row);
        fireTableRowsDeleted(row, row);
    }

    public void fireTableRowsDeleted(int firstRow, int lastRow) {
        fireTableChanged(new TableModelEvent(this, firstRow, lastRow,
                TableModelEvent.ALL_COLUMNS, TableModelEvent.DELETE));
    }

 

[Guest]

Hmm... mir fällt dazu nicht viel ein.
Ich kann mir vorstellen, dass so etwas passieren könnte, wenn man direkt
nach dem Löschen eine andere Zeile selektiert (im Code, nicht mit der Maus)
Der ChangeEvent könnte noch in der EventQueue sein, solange der
aktuelle Event (ActionEvent o.ä.), aus dem heraus du die Zeilen löschst, nicht
abgearbeitet ist. ???:L

Wie sieht die Exception bzw. das Stacktrace aus? An welcher Stelle genau
kommt es zu der Exception?
Sicher, dass du in deinem Renderer nicht irgendwo einen Fehler übersehen hast?

 

[Moonlight1234]

Ich kann mir vorstellen, dass so etwas passieren könnte, wenn man direkt
nach dem Löschen eine andere Zeile selektiert (im Code, nicht mit der Maus)
Der ChangeEvent könnte noch in der EventQueue sein, solange der
aktuelle Event (ActionEvent o.ä.), aus dem heraus du die Zeilen löschst, nicht
abgearbeitet ist. ???:L

Wie sieht die Exception bzw. das Stacktrace aus? An welcher Stelle genau
kommt es zu der Exception?
Sicher, dass du in deinem Renderer nicht irgendwo einen Fehler übersehen hast?

Das wäre möglich, der Löschvorgang wird durch ein "Remove"-Button ausgelöst.
Ich bekomme die selektierten Zeilen durch table.getSelectedRows();.

In der actionPerformed()-Methode rufe ich die og. removeRow()-Methode mit eine Schleife auf.

Du meinst also der ActionListener steht noch in der Event-Queue drin und blockiert damit erstmal die ganzen TableEvents?
Also wird der ActionListener erst aus der EventQueue genommen wenn ich die actionPerformed()-Methode verlasse?
Wie soll ich das umgehen?
Irgendwie muß ich ja den Löschvorgang auslösen.

Der Stack-Trace ist:
java.lang.ArrayIndexOutOfBoundsException: 13 >= 3
at java.util.Vector.elementAt(Vector.java:432)
at editor.spriteeditor.SpriteTableModel.setValueAt(SpriteTableModel.java:91)
at javax.swing.JTable.setValueAt(JTable.java:1875)
at javax.swing.JTable.editingStopped(JTable.java:3333)
at editor.spriteeditor.SpriteCellEditor.fireEditingStopped(SpriteCellEditor.java:143)
at editor.spriteeditor.SpriteCellEditor.stopCellEditing(SpriteCellEditor.java:123)
at javax.swing.JTable.editCellAt(JTable.java:2616)
at javax.swing.plaf.basic.BasicTableUI$Handler.adjustFocusAndSelection(BasicTableUI.java:922)
at javax.swing.plaf.basic.BasicTableUI$Handler.mousePressed(BasicTableUI.java:889)
at java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:222)
at java.awt.Component.processMouseEvent(Component.java:5485)
at javax.swing.JComponent.processMouseEvent(JComponent.java:3126)
at java.awt.Component.processEvent(Component.java:5253)
at java.awt.Container.processEvent(Container.java:1966)
at java.awt.Component.dispatchEventImpl(Component.java:3955)
at java.awt.Container.dispatchEventImpl(Container.java:2024)
at java.awt.Component.dispatchEvent(Component.java:3803)
at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4212)
at java.awt.LightweightDispatcher.processMouseEvent(Container.java:3889)
at java.awt.LightweightDispatcher.dispatchEvent(Container.java:3822)
at java.awt.Container.dispatchEventImpl(Container.java:2010)
at java.awt.Window.dispatchEventImpl(Window.java:1774)
at java.awt.Component.dispatchEvent(Component.java:3803)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:463)
at java.awt.EventDispatchThread.pumpOneEventForHierarchy(EventDispatchThread.java:242)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:163)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:157)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:149)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:110)

 

[Guest]

Kann es sein, dass Du wie folgt vorgehst?

...
rows = table.getSelectedRows();
for(int r : rows) {
  model.removeRow(r);
}
...

Nur, um diesen Fehler auszuschliessen.
Nach dem ersten Löschen sind die restlichen Indizies nicht mehr gültig.

???:L

 

[Guest]

Noch was. Versuche das hier in Deinem Model.

public void removeRows(int rows[])
{
  // Sicherstellen, dass die Indizies aufsteigend sortiert sind
  Arrays.sort(rows); 
  // Aktuelle Größe holen
  int size = dataVector.size();
  // Und alle entsprechenden Zeilen rückwärts löschen
  for(int i=rows.length-1; i>=0; i--)
  {
    if(rows[i] < size)
    {
      dataVector.removeElementAt(rows[i]);
      size--;
    }
  }
  // Tabelle aktualisieren
  fireTableDataChanged();
}

 

[Guest]

Ätsch... :autsch: Ist noch nicht Idiotensicher. :wink:

public void removeRows(int rows[]) 
{ 
  int size;
  // Wenn nix zu tun, dann raus hier
  if(rows == null || (size = dataVector.size()) == 0)
    return;
  // Sicherstellen, dass die Indizies aufsteigend sortiert sind 
  Arrays.sort(rows); 
  // Und alle entsprechenden Zeilen rückwärts löschen 
  for(int i=rows.length-1; i>=0; i--) 
  { 
    if(rows[i] < size) 
    { 
      dataVector.removeElementAt(rows[i]); 
      size--; 
    } 
  } 
  // Tabelle aktualisieren, wenn etwas gelöscht wurde
  if(size != dataVector.size()) 
    fireTableDataChanged(); 
}

 

[Moonlight1234]

Das Array mit den selectedRows wird von mir in aufsteigender Reihenfolge sortiert und anschließend "rückwärts" gelöscht, da die Indices sich sonst verschieben.
Genau wie du das machst.
Du hattest unten firedDataChanged() geschrieben.
Ich feure für jede gelöschte Zeile ein:

    public void fireTableRowsDeleted(int firstRow, int lastRow) {
        fireTableChanged(new TableModelEvent(this, firstRow, lastRow,
                TableModelEvent.ALL_COLUMNS, TableModelEvent.DELETE));
    }

    public void fireTableChanged(TableModelEvent e) {
        Object[] listeners = listenerList.getListenerList();
        for (int i = listeners.length-2; i>=0; i-=2) {
            if (listeners[i]==TableModelListener.class) {
                ((TableModelListener)listeners[i+1]).tableChanged(e);
            }
        }
    }

Ist das korrekt?

Da der Fehler zeitabhängig ist nehme ich an es hat was mit den Events zu tun.

 

[Guest]

Wie sieht es in der Methode addTableModelListener aus?
Es ist ziemlich doof, ohne den ganzen Code zu sehen und ohne
die Möglichkeit das grosse Verrecken live zu erleben. ???:L

Wir kriegen den Fehler aber noch heraus.

Übrigens, in dem Code von mir wird nur ein Event ausgelöst, wodurch
die Tabelle nur einmal aktualisiert wird. Wenn Du bei jeder Zeile ein
Update auslöst, dann macht die Tabelle eine Sliedeshow daraus
(wird unnötig oft aktualisiert).

Woher kommt eigentlich das editCellAt(...)?

 

[Guest]

Noch interessanter wäre der Code aus

editor.spriteeditor.SpriteTableModel.setValueAt(...)

 

[Moonlight1234]

    public synchronized void setValueAt(Object value, int row, int col) {
            String[] rowStr=dataVector.elementAt(row);
            rowStr[col]=(String)value;
            dataVector.setElementAt(rowStr, row);
            e.printStackTrace();
        fireTableCellUpdated(row, col);
    }

Die rows sind als Vector<String[]> angelegt.
Da die Anzahl der Columns fest ist, (es werden nur ganze Zeilen hinzugefügt oder gelöscht) habe ich diese als String-Array angelegt, dh. jede row enthält die columns als String-Array.

 

[Guest]

Irgendwie blicke ich da nicht ganz durch. ???:L
Knallt es direkt beim Löschen oder wenn Du anschliessend auf die
Tabelle klickst? Wenn das zweite zutrifft, ist die Tabelle zu diesem
Zeitpunkt bereits neugezeichnet?

Es ist schwierig einen Fehler zu finden, wenn man es nicht vor sich hat.
Versuche mal das hier vor dem Löschen aufzurufen.

if(table.isEditing())
  table.getCellEditor().cancelCellEditing(); // oder stopCellEditing()

(Eigentlich passiert dies automatisch, wenn die Tabelle den Focus verliert; Buttonklick)

Ansonsten auch das Löschen auf einen Schlag + 1 Event, wie oben schon gezeigt.

Du kannst den Fehler dadurch umgehen, indem Du in der Methode setValueAt(...)
den Zeilenindex prüfst.

public synchronized void setValueAt(Object value, int row, int col)
{ 
  if(row < dataVector.size())
  {
    String[] rowStr=dataVector.elementAt(row); 
    rowStr[col]=(String)value; 
    fireTableCellUpdated(row, col); 
  }
}

Es ist aber Pfuscherei, ändert nichts am Fehler selbst. Verhindert nur, dass
Dir das ganze um die Ohren fliegt.

Übrigens, die Zeile
dataVector.setElementAt(rowStr, row);
kannst Du Dir sparen, da Du eine Zeile davor in das String-Array direkt
reinschreibst. Ist aber nicht so wichtig...

 

[Moonlight1234]

Es liegt am Editor. Wenn ich den DefaultEditor verwende tritt das Problem nicht mehr auf.
Mittlerweile habe ich fast alles aus dem DefaultEditor herauskopiert, der Fehler tritt immer noch auf.

Ich fange den Fehler in der setValueAt-Methode jetzt ab. Ist vielleicht nicht besoders elegant, aber was solls.


Danke, für die Hilfe.

 

[Mag1c]

Hi,

möglicherweise ist der Editor noch im Edit-Modus, wenn du die Zeilen löschst. Prüfe
das mal mit isEditing() bzw. stopCellEditing().

Es ist übrigens schwierig, mit deinen Angaben und den paar Code-Häppchen einen
Fehler zu finden. Und den Fehler mit einem catch() und dem Kommentar "was solls"
zu ignorieren, ist ganz sicher keine gute Idee.

Gruß
Mag1c

 

[Moonlight1234]

Hallo!

Du hast ja recht.
Ich war nur etwas gefrustet. Ich habe in den letzten Wochen die Grundlage für ein Spiel programmiert, das funktioniert problemlos.
An der JTable beisse ich mir die Zähne aus, die sollte eigentlich einfacher sein. :?

Es ist schwierig Code zu posten die Table ist Bestandteil eines Editors der mittlerweile sehr umfangreich ist.

Die Sache mit dem fireEditingStopped() vor dem löschen der Zelle hatte ich gemacht.

Ich habe jetzt das "Hauptprogramm" geändert. Dadurch kann ich die DefaultCellEditoren verwenden. So funktionierts.
Die Lösung mit einem eigenen CellEditor wäre allerdings eleganter gewesen.

Danke für dein Engagement.

PS: Ich hatte in einem anderen Thread gelesen das nicht für jede Zelle ein eigener Renderer verwendet wird, ich bin immer davon ausgegangen das jede Celle einen eigenen Editor/Renderer hat. Das war wohl mein Fehler.
Ich glaube mein Problem ist das zwischen Renderer/Model/Editor keine Verbindung besteht, bzw. die Verbindung sind wohl die Events. Ich finde es schwer zu durchschauen wer welche Werte von wem bekommt.

 

[Mag1c]

Hi,

hehe, die Info aus dem anderen Thread kommt mir bekannt vor

Im Prinzip ist das ganz einfach. Angenommen du benutzt nur einen Typ CellRenderer.
Dann geht die JTable beim Darstellen jede einzelne Zelle durch, holt sich
mit getTableCellRendererComponent(...) den fertigen CellRenderer und benutzt diesen,
um die Zelle zu zeichnen. Es wird aber nur gezeichnet, die Komponente wird
NICHT als GUI-Komponente integriert. Das funktioniert in etwa so wie ein
Stempel, bei dem man einen Text einstellen und anschließend aufs
Papier drücken kann.

Der CellEditor wird erst aktiv, wenn eine Zelle editiert werden soll. Er wird
von der JTable an der Stelle der Zelle platziert. Wird das Editieren beendet,
wird der Wert des Editors per setValueAt(...) in das TableModel geschrieben.
Anschließend wird die Zelle, da sich der Wert ja geändert hat, mit dem
Renderer neu gezeichnet.

Gruß
Mag1c

 

[Moonlight1234]

Der CellEditor wird erst aktiv, wenn eine Zelle editiert werden soll. Er wird
von der JTable an der Stelle der Zelle platziert. Wird das Editieren beendet,
wird der Wert des Editors per setValueAt(...) in das TableModel geschrieben.
Anschließend wird die Zelle, da sich der Wert ja geändert hat, mit dem
Renderer neu gezeichnet.

Mit den Events im TableModel setze ich auch die JTable darüber in Kenntnis das Zeilen gelöscht, hinzugefügt wurden?

Also wenn ich im TableModel (abgeleitet vom AbstractTableModel) die im AbstractTableModel enthaltene Methode
fireTableRowsDeleted(row, row) aufrufe (und damit den Listener aufrufe), nimmt dann auch die JTable zur Kenntniss das Zeilen gelöscht wurden?
Ich rufe ja keine Methode von JTable auf mit der ich JTable direkt darüber unterrichte das sich etwas geändert hat. Ich habe leider keine Ahnung wie die Listener untereinander kommunizieren.

 

[Mag1c]

n Abend,

ja genau, so läuft das. Du kannst dir das in den Sourcen von JTable
in der Listener-Methode tableChanged(TableModelEvent e) anschauen.

Aber wenn du keine speziellen Wünsche hast und alles korrekt anwendest,
musst du über die Interna nicht viel wissen.

Gruß
Mag1c

 

[Moonlight1234]

Hallo Magi1c

Ich werde diese Tage nochmal eine einfachereTable mit eigenem Editor programmieren und schauen ob das funktioniert.

Wenn es dir recht ist komme ich dann auf dich zurrück. Evtl kann ich dann etwas Code posten.

Gruß

Moonlight

 

[Mag1c]

Moin,

na gerne doch

Gruß
Mag1c

 

[Moonlight1234]

Ich habe mittlerweile eine andere Table programmiert, bei dieser funktioniert das löschen der Zeilen problemlos.
Ich habe bei der anderen Table nicht beachtet das nicht für jede Zelle ein Editor angelegt wird, außerdem habe ich wohl ein Event vergessen.
Ich habe allerdings den Eindruck das es bei sehr ungünstigen Bedingungen, z.B mehrere Fenster offen, mehrere Listener laufend, passieren kann das der Event welcher der JTable mitteilt das die Zeilen gelöscht wurden "hinterherhängt".
Es kommt dann zu Fehlermeldungen die sich aber meistens nicht auf das Programm auswirken, dh. das Programm läuft weiter.

Nochmals danke für die Unterstützung.

 

[Mag1c]

Moin,

wie meinst du das ? Was für Fehlermeldungen kommen denn ?

Gruß
Mag1c