Swing JTable - Editierung von Zellen abfangen

[Strece]

Hallo,

ich arbeite grade an einer JTable die Daten aus einer MySQL DB anzeigt.
Nun möchte ich gerne, das man diese Daten bearbeiten kann und die Änderungen dann in die DB geschrieben wird.

So nach ein wenig googeln bin ich auf den TableModelListener getroffen, dieser scheint ja meine Anforderungen zu erfüllen.

Nun aber das scheint nicht zu funktionieren, mein Problem ist das sich meine Tabelle beim Start des Programm mit den Werten aus der DB füllt, dadurch wird direkt das Listener Event ausgelöst und ich bekomme eine ArrayIndexOutOfBounds Exception und zwar bei der Zeile "Object data = model.getValueAt(row, column);" (siehe unten).

Ich habe dann versucht den Listener bei jedem Refresh der Tabelle zu entfernen und dann wieder aktivieren, aber das klappt leider nicht wie gewollt.

Also derzeit sieht mein Code so aus:

grdSearch.getModel().addTableModelListener(new javax.swing.event.TableModelListener() {

            public void tableChanged(javax.swing.event.TableModelEvent evt) {
                grdSearchTableChanged(evt);
            }
        });

private void grdSearchTableChanged(javax.swing.event.TableModelEvent evt) {
        int row = evt.getFirstRow();
        int column = evt.getColumn();
        TableModel model = (TableModel) evt.getSource();
        String columnName = model.getColumnName(column);
        Object data = model.getValueAt(row, column);

        try {
            st.executeUpdate("Update verbindungen Set " + columnName + "= '" + data + "' Where ID = " + grdSearch.getValueAt(row, 0) + ";");
        } catch (SQLException ex) {
            JOptionPane.showMessageDialog(null, ex.getMessage(), "Fehler", JOptionPane.CANCEL_OPTION);
        }
        refresh();
    }

Mit folgendem Code wollte ich den Listener entfernen:

grdSearch.getModel().removeTableModelListener(grdSearch);

Vielleicht hab ich irgendwo einen Denkfehler oder gehe die Sache falsch an. Hatte auch schon an einen CellEditorListener gedacht, aber ich denke da ist der TableModelListener besser.
Habe auch gelesen, das man den Listener so anpassen kann das er nur bei Änderungen in einer Zelle reagiert, aber so wie ich das gelesen habe, muss man dabei die Zeile und Spalte immer genau angeben. Dies wäre aber nicht schön, da es schon mit jeder Spalte und Zeile funktionieren sollte.

Mfg, Strece

 

[Michael...]

Mit folgendem Code wollte ich den Listener entfernen:

grdSearch.getModel().removeTableModelListener(grdSearch);

Im Codeausschnitt weiter oben wird ein anonymer Listener am TableModel registriert, den kriegt man so nicht weg.

Grundsätzlich kann man das TableEvent aber auch "fragen" was für eine Änderung vorgenommen wurde. So wie ich das verstanden habe sind ja nur Updates interessant und keine Inserts.

 

[Strece]

Ja genau, es gut nur um Updates. Alle Änderungen werden per Update Statement in die DB gschrieben und es soll auch immer nur eine Zelle gleichzeitig bearbeitet werden, nicht ganze Zeilen.

Das würde ja heißen ich müsste überprüfe ob eine Zelle geändert wurde, auf der Seite vom TableModelEvent wäre da ja entweder
TableModelEvent(source, 1); // Row 1 changed oder
TableModelEvent(source, 2, 2, 6); // Cell at (2, 6) changed

Würde er aber nicht beim ersten auch den Listener beim Start aufrufen da ich ja erstmal aus der DB alle Daten auslese und somit alle Zeilen einfüge?

Beim zweiten sehe ich als Problem die Begrenzung, so wie ich das verstanden habe würde er z.B. darauf achten wenn sich eine Zelle zwischen den Zeilen 2 und 6 ändert oder liege ich da falsch? Weil die Anzahl an Zeilen variiert ja sehr stark in einer DB.

Mfg, Strece

EDIT: Habe mal wieder zu schnell geschrieben als gedacht, habe grade weiter unten etwas über Update gelesen.

public static final int UPDATE

Ist es damit möglich herauszufiltern ob ein Update stattgefunden hat?

EDIT2:
So habe nun den Code folgendermaßen erweitert:

private void grdSearchTableChanged(javax.swing.event.TableModelEvent evt) {
        if(evt.getType() == 0)
        {
            int row = evt.getFirstRow();
            int column = evt.getColumn();
            TableModel model = (TableModel) evt.getSource();
            String columnName = model.getColumnName(column);
            Object data = model.getValueAt(row, column);

            try {
                st.executeUpdate("Update verbindungen Set " + columnName + "= '" + data + "' Where ID = " + grdSearch.getValueAt(row, 0) + ";");
            } catch (SQLException ex) {
                JOptionPane.showMessageDialog(null, ex.getMessage(), "Fehler", JOptionPane.CANCEL_OPTION);
            }
            refresh();
        }
    }

Aber dennoch bekomme ich beim Start eine ArrayIndexOutOfBounds Exception bei der gleichen Zeile.

 

[Strece]

Ok Problem hat sich erledigt, habe nach ein wenig rumtesten die Lösung gefunden.
Mir fehlte nur eine weitere Abfrage ob alle Columns geändert wurden oder nicht, also meine Lösung sieht so aus:

if (evt.getType() == TableModelEvent.UPDATE) {
            int row = evt.getFirstRow();
            int column = evt.getColumn();
            if (column != TableModelEvent.ALL_COLUMNS)
            {
                TableModel model = (TableModel)evt.getSource();
                String columnName = model.getColumnName(column);
                Object data = model.getValueAt(row, column);

                System.out.println("Zeile: " + row);
                System.out.println("Spalte: " + column + " " + columnName);
                System.out.println("Wert: " + data);
           }
}