JTable aktualisieren (nach Datenbank updade)

[Spyder]

Hallo,
ich weiß, das Problem ist hier bekannt, leider finde ich aber keine konkrete Lösung zu meinem Problem, versuche das jetzt schon Stunden.

Also habe eine Tabelle und daneben ein Eingabebereich. Nach Ausfüllen der Textfelder werden die Daten in die Datenbank geschrieben. Leider wird aber nicht die Tabelle aktualiisert, bekomme dies auch nicht hin.

Der Code sieht wie folgt aus:

AbstractTableModel model = new MyTableModel();

//Im Konstruktor wird setTable() aufgerufen
public void setTable(){
        table = new JTable(model);
        table.setPreferredScrollableViewportSize(new Dimension(250, 270));
        table.setFillsViewportHeight(true);
        leftpanel.add(new JScrollPane(table));
    }

class MyTableModel extends AbstractTableModel {
        private String[] columnNames = {"Team", "Ort"};

        private Object[][] data = getTeams();
        
        public int getColumnCount() {
            return columnNames.length;
        }
        
        public int getRowCount() {
            return data.length;
        }
        
        public String getColumnName(int col) {
            return columnNames[col];
        }
        
        public Object getValueAt(int row, int col) {
            return data[row][col];
        }
        
        public Class getColumnClass(int c) {
            return getValueAt(0, c).getClass();
        }
        
        public boolean isCellEditable(int row, int col) {
            return false;
        }

        public void setValueAt(Object value, int row, int col) {
            data[row][col] = value;
            fireTableCellUpdated(row, col);
        }
        
    }
//Nach klick auf Hinzufügen Button wird dann per ActionListener die Methode insertTeam() aufgerufen,
//welche den Eintrag zur Datenbank hinzufügt.

Habe schon verschiedene Varianten mit TableListenern versucht, aber irgendwie hat nichts geklappt. Denke ich mache irgendwelche Fehler im Code. Ist das erste mal das ich mit TableModels arbeite.

Gruß und schon mal Danke

 

[SlaterB]

> Nach Ausfüllen der Textfelder werden die Daten in die Datenbank geschrieben. Leider wird aber nicht die Tabelle aktualiisert, bekomme dies auch nicht hin.

was für Textfelder? wie soll die JTable aktualisiert werden, welche Operation rufst du dazu bisher auf (oder gar nix?)

> Habe schon verschiedene Varianten mit TableListenern versucht, aber irgendwie hat nichts geklappt.

was sollen TableListenern mit dem Problem 'die JTable ändern' zu tun haben?

> Denke ich mache irgendwelche Fehler im Code.

welchen Code hast du denn bisher, der sich mit 'die JTable ändern' beschäftigt?
ich kann noch keinen entdecken

 

[Spyder]

Ok, dachte die Infos sind eher unwichtig fürs aktualisieren der JTable.

Aber gut hier die Methode die den Eintrag in die Datenbank schreibt

 public void insertTeam(){
        String d_tname = text_team.getText();
        String d_plz = text_plz.getText();
        String d_ort = text_ort.getText();
        String d_name = text_name.getText();
        String d_vname = text_vname.getText();
        String d_tel = text_telefon.getText();
        
        try{
            Class.forName("sun.jdbc.odbc.JdbcOdbcDriver");
            Connection con =
                    java.sql.DriverManager.getConnection("jdbc:odbc:turnier");
            Statement s = con.createStatement();
            try {
                s.executeUpdate("INSERT INTO Teilnehmer VALUES ('"+d_tname+"', '"+d_plz+"', '"+d_ort+"', '"+d_name+"', '"+d_vname+"', '"+d_tel+"')");} catch (SQLException e) {}
        } catch (Exception e) {
            System.out.println("insertTeam - Fehler: " + e.getMessage());
        }
        
        text_team.setText("");
        text_plz.setText("");
        text_ort.setText("");
        text_name.setText("");
        text_vname.setText("");
        text_telefon.setText("");
        
    }

Die verschiedenen Versuche die ich unternommen habe, habe ich wieder rausgelöscht wenn sie nicht funktioniert haben, allein schon wegen der Übersichtlichkeit.

Wie im oberen Code schon zu sehen wird im abstracttablemodel mit

private Object[][] data = getTeams();

das data Array für das TableModel gefüllt.

Hier noch die Methoden die ich dazu benutze

    /*
     *Füllt Tabellenarray mit Werten aus Datenbank
     */
    public Object[][] getTeams(){
        int rowcnt = staticmethods.getRowcnt("Teilnehmer", "Teamname");
        Object[][] result = new Object[rowcnt][2];
        Object[] teams = getColumn("Teamname", rowcnt);
        Object[] orte = getColumn("Ort", rowcnt);
        
        for (int i=0; i<rowcnt; i++){
            result[i][0] = teams[i];
            result[i][1] = orte[i];
        }
        
        return result;
    }
    
    /*
     *Holt Spalte "Column" aus Tabelle "table"
     */
    public Object[] getColumn(String column, int rowcnt){
        int i=0;
        String[] combofeld = new String[rowcnt];
        
        try {
            Class.forName("sun.jdbc.odbc.JdbcOdbcDriver");
            Connection con = java.sql.DriverManager.getConnection("jdbc:odbc:turnier");
            Statement statement = con.createStatement();
            
            ResultSet result = statement.executeQuery("SELECT "+column+" FROM Teilnehmer");
            
            while (result.next()) {
                String team = result.getString(column);
                combofeld[i]=team;
                i++;
            }
            con.close();
        } catch (Exception e) {System.out.println("getColumn Fehler: " + e.getMessage());}
        return combofeld;
    }

 

[SlaterB]

aber diese Operation wird ja nur am Anfang ausgeführt, nicht nach dem Ausführen von
insertTeam()


am Ende von insertTeam() musst du das Model informieren,
dieses muss erneut
data = getTeams();
ausführen sowie mit
fireTableDataChanged();
die GUI (JTable) informieren

 

[Spyder]

So was hab ich mir schon gedacht, nur wie komme ich wieder an data ran? Muss ich ein ganz neues TableModel erzeugen, wenn ja wie?

 

[SlaterB]

nicht unbedingt,
du kannst auch das vorhandene Model dazu veranlassen, sich upzudaten

von außen sagst du nur
updateDich();
der Rest passiert intern im Model

 

[Spyder]

OK, nur wie sage ich dem Model das es sich updaten soll?
Meine Idee wäre alle Zeilen zu löschen und die neu hinzuzufügen?
Aber auch dafür hat AbstractTableModel keine Methoden.
Bin da ratlos? Wäre schön wenn es da eine einfache update Methode geben würde.

 

[SlaterB]

du hast doch class MyTableModel, das ist deine Klasse, da kannst du reinschreiben was du willst:

public void myUpdate() {
this.data = getTeams();
fireTableDataChanged();
}

fertig, nur noch aufrufen

 

[Spyder]

Jetzt hab ichs. Hatte model ja als AbstractTableModel und nicht direkt als MyTableModel definiert, daher kam ich an die selbstgeschriebenen Methoden in der Klasse MyTableModel nicht ran.
Jetzt klappts.

Danke dir.

Das nächste Ziel ist für mich jetzt das Ändern und Löschen. Sollte aber jetzt kein Problem mehr sein

 

[Meru]

fireTableDataChanged();

Das bringt doch aber nichts wenn die Daten in der Datenbank sind und diese vorher neu geladen werden sollen, oder?


Also ich mache das immer so:

public class MyTableModel extends DefaultTableModel {
	
    public MyTableModel() {
		super();
	}

	public MyTableModel(int rowCount, int columnCount) {
		super(rowCount, columnCount);
	}

	public MyTableModel(Object[] columnNames, int rowCount) {
		super(columnNames, rowCount);
	}

	public MyTableModel(Object[][] data, Object[] columnNames) {
		super(data, columnNames);
	}

	public MyTableModel(Vector columnNames, int rowCount) {
		super(columnNames, rowCount);
	}

	public MyTableModel(Vector data, Vector columnNames) {
		super(data, columnNames);
	}

	/*
     * JTable uses this method to determine the default renderer/
     * editor for each cell.  If we didn't implement this method,
     * then the last column would contain text ("true"/"false"),
     * rather than a check box.
     */
    public Class getColumnClass(int c) {
        return getValueAt(0, c).getClass();
    }
}

// Vereinfacht dargestellt
ClassXYZ{
  JTable _table;
  MyTableModel _tableModel;

  public void initComponents(){
    // Datenbankabfrage -> Speichern der Daten in 2 Vectoren (header und data)
    _table = new JTable();
    _tableModel = new MyTableModel(_v_tableData, _v_tableHeader);
    table.setModel(tableModel);
  )

  public void refreshTableData(){
     // Datenbankabfrage -> Speichern der Daten in 2 Vectoren (header und data)
    _tableModel.setDataVector(_v_tableData, _v_tableHeader);
  }
}

 

[SlaterB]

> Das bringt doch aber nichts wenn die Daten in der Datenbank sind und diese vorher neu geladen werden sollen, oder?


deswegen steht ja auch
this.data = getTeams();
davor, denn das sind die Daten, die bei getValueAt() zurückgeliefert werden,
entspricht _tableModel.setDataVector() beim DefaultTableModel

 

[Meru]

Na da hab ich den Quelltext in diesem Thread zu schnell überflogen. Hast dann natürlich recht ^^

 

[Spyder]

Hab nun ein neues (kleineres) Problem.
Habe jetzt weiter implementiert das bei klick auf eine Zeile, die Daten zur Zeile wieder in die Textfelder geschrieben wird.
Wenn man jetzt den Löschen-Button drück soll dieser Datensatz entfernt werden. Dies passiert auch, nur der Aufruf der Update Methode fürs TableModel funktioniert nicht mehr richtig.

Ich habe rausgefunden das

fireTableDataChanged();

einen ArrayOutOfBounds Error -1 zurückliefert. Aber warum?
Das selber passiert auch wenn ich die Felder per Buttonklick alle zurücksetze (also mit "" befülle) und dann
wieder einen Datensatz hinzufügen oder entfernen will. Denke das hat irgendwas damit zu tun das eine Zeile selektiert ist, es gibt aber keine Methode die eine Zeile deselektiert. Oder liegts wo anders dran?

Also wenn ich ohne vorher eine Zeile zu selektieren einen Datensatz einfüge klappts.

nochmal der geänderte Code:


RowListener

private class RowListener implements ListSelectionListener {
        public void valueChanged(ListSelectionEvent event) {
            
            /*if (event.getValueIsAdjusting()) {
                return;
            }*/
            
            int rowcnt = staticmethods.getRowcnt("Teilnehmer", "Teamname");
            
            String tname = (String) table.getValueAt(table.getSelectedRow(),0);
            
            try {
                Class.forName("sun.jdbc.odbc.JdbcOdbcDriver");
                Connection con = java.sql.DriverManager.getConnection("jdbc:odbc:turnier");
                Statement statement = con.createStatement();
                
                ResultSet result = statement.executeQuery("SELECT * FROM Teilnehmer WHERE Teamname='"+tname+"'");
                
                while (result.next()) {
                    text_team.setText(result.getString(1));
                    text_plz.setText(result.getString(2));
                    text_ort.setText(result.getString(3));
                    text_name.setText(result.getString(4));
                    text_vname.setText(result.getString(5));
                    text_telefon.setText(result.getString(6));
                }
                con.close();
            } catch (Exception e) {System.out.println("getColumn Fehler: " + e.getMessage());}
        }
    }

Hier die funktionierende delete Methode

public void deleteTeam(){
        String d_tname = text_team.getText();
        
        try{
            Class.forName("sun.jdbc.odbc.JdbcOdbcDriver");
            Connection con =
                    java.sql.DriverManager.getConnection("jdbc:odbc:turnier");
            Statement s = con.createStatement();
            try {
                s.executeUpdate("DELETE FROM Teilnehmer WHERE Teamname='"+d_tname+"'");} catch (SQLException e) {}
            con.close();
        } catch (Exception e) {
            System.out.println("deleteTeam - Fehler: " + e.getMessage());
        }

        text_team.setText("");
        text_plz.setText("");
        text_ort.setText("");
        text_name.setText("");
        text_vname.setText("");
        text_telefon.setText("");
    }

Wäre super wenn jemand da noch ne Lösung hat. Ich sehe noch nicht mal richtig das Problem.

Gruß

 

[SlaterB]

> es gibt aber keine Methode die eine Zeile deselektiert

fängt mit remove an, siehe API

zum Rest kann ich nicht viel sagen ohne ein Gesamtprogramm und Ablaufbeschreibung zum Fehler hin,
und selbstverständlich ohne DB-Operationen, kannst du auch alles (testweise) im Arbeitsspeicher vorhalten



---------

warum hast du eigentlich
try {
s.executeUpdate("DELETE FROM Teilnehmer WHERE Teamname='"+d_tname+"'");} catch (SQLException e) {}

innerhalb des
try {
} catch (Exception e) {
Ausgabe
}
Blocks? lass das innere try/ catch weg

 

[Spyder]

Hi,
ich habs durch zufall gefunden. Nach dem löschen eines Datensatzes
rufe ich ja deleteTeam(); und dann model.myUpdate(); auf. Es fehlt aber aus irgendeinem Grund noch ein repaint(); damit der die Tabelle korrekt darstellt. fireTableDataChanged schmeißt zwar immer noch ein Fehler, den hab ich aber abgefangen. Also beim Einfügen von Datensätzen benötigt der fireTableDataChanged (funktoniert in dem Fall auch) und beim löschen benötigt der noch ein repaint();

Danke nochmal für deine Hilfe!