Swing JTable: Problem beim zeile löschen

[Felix]

Hallo, ich möchte mit der Entfernen-Taste eine oder mehrere Zeilen aus einer JTable löschen. Das TableModel habe ich selbst geschrieben.
Das Problem ist, dass die markierte Zelle (die Tabelle hat zwei Spalten) den Wert der Zelle darunter annimmt, während die nicht markierte Zelle der Zeile erhalten bleibt. Erst wenn ich eine andere Zeile markiere erscheint das erwünschte Bild (Zeile gelöscht, die anderen Zeile rücken auf). Ich hab die Tabelle einfach mit einem Keylistener ausgestattet. Der Code sieeht so aus:

table.addKeyListener(new KeyAdapter() {

            @Override
            public void keyPressed(KeyEvent ke) {
                if (ke.getKeyCode() == KeyEvent.VK_DELETE) {
                    final TableCellEditor editor = table.getCellEditor();
                    if (editor == null || editor.stopCellEditing()) {
                        manipulationlistener.deleteRows(table.getSelectedRows());
                   }
                }
            }
        });

Das Problem ist, dass editor == null immer zutrifft, obwohl die Tabelle bzw. das Model editierbar sind.
Ich hab mich hier im Forum schon umgesehen (so bin ich ja auch den Code mit dem TableCell Editor gekommen), aber das Problem hatte glauch ich bis jetzt noch keiner.

Der ManipulationListener:

    /*
     * ManipulationListener
     */
    public void deleteRows(int [] row){
        for (int i = 0; i < row.length; i++){
            Dataset d = (Dataset) list.get(row[i]);
            dbconnection.deleteVocable(d.getIndex());
            model.removeRow(row[i]);
        }
    }

 

[Ebenius]

Das Problem ist, dass editor == null immer zutrifft, obwohl die Tabelle bzw. das Model editierbar sind.

[c]getCellEditor()[/c] gibt immer [c]null[/c] zurück, außer während eine Zelle editiert wird.

Dein eigentliches Problem ist, dass Dein Tabellenmodell in [c]removeRow(…)[/c] keinen Event (oder einen falschen) feuert. Wie sieht denn diese Methode aus?

Ebenius

 

[Felix]

Stimmt, ja. Die Methode sieht so aus:

public void removeRow(int i){
    data.remove(i);
}

"data" ist hierbei ein Vector, der die Daten der Tabelle enthält.

EDIT:
Jetzt habe ich die Methode so geschrieben:

    public void removeRow(int i){
        data.remove(i);
        
        // Zuerst ein Event erstellen
        TableModelEvent e = new TableModelEvent(this, i, i,
                TableModelEvent.ALL_COLUMNS, TableModelEvent.DELETE);

        // Nun das Event verschicken
        for (int j = 0, n = listeners.size(); j < n; j++) {
            ((TableModelListener) listeners.get(j)).tableChanged(e);
        }

    }

Funktioniert auch super, wenn ich nur eine Zeile markiert habe und die löschen will. Wenn ich mehrere markiert habe, dann wird seltsamerweise nur jede zweite gelöscht...

 

[Felix]

Das lag natürlich nicht an dem Code, der oben steht, der ist vollkommen richtig. Hatte was in einer for-Schleife vermasselt.

Vielen Dank für den Tipp!