Swing JTable aktualisieren

[ralfb1105]

Hallo zusammen,

ich habe eine Frage zur Aktualisierung eines JTable Objektes. Hier kurz die Beschreibung meines Problems.
Ich habe ein Programm (Swing) mit dem ich mich nach Eingabe von Oracle DB Zugangsdaten (Host, Port, SID) an die Datenbank anbinde um verschiedene Aktionen auszuführen, wie z.B. Stress Tests oder das auslesen von Patch Informationen. Für letzeres rufe ich über ein JMenu ein JDialog auf in dem ich ein JTable erstelle. Dem JTable übergebe ich beim erstellen ein 2D Array:

private JTable getJTablePatchInfo() {
        if(jTablePatchInfo == null) {
            jTablePatchInfo = new JTable(getArray2dPatchInfo(), getTableArrayTitlePatchInfo("PATCH_ID","ACTION","STATUS","DESCRIPTION"));
            jTablePatchInfo.setBounds(0, 0, 400, 400);
            jTablePatchInfo.getTableHeader().setFont(new java.awt.Font("Dialog",1,14));
            jTablePatchInfo.getTableHeader().setForeground(new java.awt.Color(206,91,39));
            jTablePatchInfo.getColumnModel().getColumn(0).setPreferredWidth(10);
            jTablePatchInfo.getColumnModel().getColumn(1).setPreferredWidth(10);
            jTablePatchInfo.getColumnModel().getColumn(2).setPreferredWidth(10);
            jTablePatchInfo.getColumnModel().getColumn(3).setPreferredWidth(500);
            TableCellRenderer rendererFromHeader = jTablePatchInfo.getTableHeader().getDefaultRenderer();
            JLabel headerLabelPatchInfo = (JLabel) rendererFromHeader;
            headerLabelPatchInfo.setHorizontalAlignment(JLabel.CENTER);   
        }
        return jTablePatchInfo;
    }

In der Methode "getTablePatchInfoArrayData()" wird der Connect zur Datenbank durchgeführt und das Query zum ermitteln der Patch Infos. Die Ergebnisse packe ich dann in ein 2D Array und gebe das dann zurück.
Wenn ich das Programm aufrufe, den Connect zur Datenbank teste und anschließend den Button "Patch Info" drücke erhalte ich auch den Dialog mit dem JTable Objekt und den entsprechenden Daten. Soweit alles gut.

Wenn ich allerdings jetzt z.B. andere DB Zugangsdaten eingebe, ohne das Programm zu beenden, und dann erneut "Patch Info" aufrufe, hat das JTable Objekt weiterhin die alten Daten 🙁

Frage:

Was muss ich programmieren damit beim erneuten Aufruf des JDialog das JTable Objekt erneut die Daten von "getTablePatchInfoArrayData()" abholt und darstellt?

Workaround:

Als Workaround lösche ich JDialog incl. JPanel und JTable beim Aufruf wenn es diese Objekte schon gibt. Das führt dazu das alles noch einmal erstellt wird, was natürlich dauert und meiner Meinung nach nicht sehr elegant ist 😵 Ich gehe davon aus das es weitaus bessere Lösungen gibt.

Danke schon mal für Eure Tipps.

Gruß

Ralf

 

[Thallius]

Die sauberste Art wäre wenn du dir ein eigenes tablemodell programmierst und in diesem dan. Auch erst die Daten aus der DB freched

 

[ralfb1105]

Hallo Thallius,

Dabke für Deine Antwort - ich hatte das schon fast "befürchtet" weil ich beim googlen zu diesem Thema immer wieder auf das Thema Table Modell gestoßen bin. Da ich bei diesem Thema leider Neuling bin ist mir bei den diversen Artikeln aber nie wirklich klar geworden wie ich das zu programmieren habe. Hättest Du vielleicht ein einfaches Beispiel was so etwas beschreibt und wie man dort die DB Daten aktualisiert? Würde ich dort dann auch die Methode, die ich ja schon habe, einbauen?
Gruß
Ralf

 

[ralfb1105]

Hallo zusammen,

ich habe den Rat von Thallius befolgt und mich mit dem TableModel beschäftigt, sprich ich habe ein eigenes Table Model implementiert. Hier einmal als Referenz der Code dazu.

Spoiler: Class PatchInfoTableModel

package oracle;

import javax.swing.table.AbstractTableModel;

@SuppressWarnings("serial")
public class PatchInfoTableModel extends AbstractTableModel {

    // Fields
    String dbHost;
    String tnsPort;
    String dbSID;
    String dbSysUser;
    String dbSysPwd;
    PatchInfoCollectData patchInfoData;
    String[] columnName = null;

 
    // Constructor
 
    public PatchInfoTableModel(String dbHost, String tnsPort, String dbSID, String dbSysUser, String dbSysPwd) {
        super();
        this.dbHost = dbHost;
        this.tnsPort = tnsPort;
        this.dbSID = dbSID;
        this.dbSysUser = dbSysUser;
        this.dbSysPwd = dbSysPwd;
        patchInfoData = new PatchInfoCollectData(dbHost, tnsPort, dbSID, dbSysUser, dbSysPwd);
    }

    public PatchInfoTableModel() {
        super();
    }
 
    // Methods

    @Override
    public int getRowCount() {
         return patchInfoData.getRowCount();
    }

    @Override
    public int getColumnCount() {
        return patchInfoData.getColumnCount();
    }
 
    @Override
    public String getColumnName(int column) {
        return patchInfoData.getColumnName()[column];
    }
 
    @Override
    public Object getValueAt(int row, int col) {
        return patchInfoData.getPatchInfoValue()[row][col];
    }

}

Dann die Klasse welche die Daten von der Datenbank holt:

Spoiler: Class PatchInfoCollectData

package oracle;

import java.sql.Connection;
import java.sql.ResultSet;
import java.sql.ResultSetMetaData;
import java.sql.SQLException;
import java.sql.Statement;
import java.util.ArrayList;

public class PatchInfoCollectData {

    // Fields
    String dbHost;
    String tnsPort;
    String dbSID;
    String dbSysUser;
    String dbSysPwd;
    String dbConnectString;
    Connection connSys;
    String queryPatchInfo = "SELECT PATCH_ID,ACTION,STATUS,DESCRIPTION FROM DBA_REGISTRY_SQLPATCH";
    Statement stmtPatchInfo;
    ResultSet rs;
    ResultSetMetaData rsmd;
    int columnCount=0;
    int rowCount=0;
    String[] columnName = null;
    DB database = new DB();
    ArrayList<String> arrayListAllPatchInfo = new ArrayList<String>();
    String array2dPatchInfo[][];
 
    // Constructor
 
    public PatchInfoCollectData(String dbHost, String tnsPort, String dbSID, String dbSysUser, String dbSysPwd) {
        super();
        this.dbHost = dbHost;
        this.tnsPort = tnsPort;
        this.dbSID = dbSID;
        this.dbSysUser = dbSysUser;
        this.dbSysPwd = dbSysPwd;
        dbConnectString = "jdbc:oracle:thin:@" + dbHost + ":" + tnsPort + "/" + dbSID;
        connSys = database.dbConnect(dbConnectString, dbSysUser, dbSysPwd);
        try {
            stmtPatchInfo = connSys.createStatement();
            rs = stmtPatchInfo.executeQuery(queryPatchInfo);
            rsmd = rs.getMetaData();
        } catch (SQLException e) {
            e.printStackTrace();
        }
    }

    // Methods
 
    public int getColumnCount() {
        try {
            columnCount = rsmd.getColumnCount();
        
        } catch (SQLException e) {
            e.printStackTrace();
        }
        return columnCount;
    }
 
    public int getRowCount() {
        try {
            while (rs.next()) {
                for (int col = 1; col <= columnCount; col++) {
                    arrayListAllPatchInfo.add(rs.getString(col));
                }
            }
            array2dPatchInfo = new String[arrayListAllPatchInfo.size()/columnCount][columnCount];
            int offset = 0;
        
            // 2D Array mit Daten befuellen aus ArrayList
            for (int zeile=0; zeile < array2dPatchInfo.length; zeile++) {
                for (int spalte=0; spalte <columnCount; spalte ++) {
                    array2dPatchInfo[zeile][spalte] = arrayListAllPatchInfo.get((spalte + offset));
                }
                offset +=4;
            }
            rowCount = array2dPatchInfo.length;
        } catch (SQLException e) {
            e.printStackTrace();
        }
        return rowCount;
    }
 
    public String[] getColumnName() {
        try {
            columnName = new String[rsmd.getColumnCount() + 1];
            int i = 1;
            while (i<= rsmd.getColumnCount()) {
                columnName[i-1] = rsmd.getColumnName(i);
                i++;
            }
        } catch (SQLException e) {
            e.printStackTrace();
        }
        return columnName;
    }
 
    public String [][] getPatchInfoValue() {
        try {
            while (rs.next()) {
                for (int col = 1; col <= columnCount; col++) {
                    arrayListAllPatchInfo.add(rs.getString(col));
                }
            }
            array2dPatchInfo = new String[arrayListAllPatchInfo.size()/columnCount][columnCount];
            int offset = 0;
        
            // 2D Array mit Daten befuellen aus ArrayList
            for (int zeile=0; zeile < array2dPatchInfo.length; zeile++) {
                for (int spalte=0; spalte <columnCount; spalte ++) {
                    array2dPatchInfo[zeile][spalte] = arrayListAllPatchInfo.get((spalte + offset));
                }
                offset +=4;
            }
        } catch (SQLException e) {
            e.printStackTrace();
        }
        return array2dPatchInfo;
    }

}

Wenn ich über das JMenu den JDialg zum anzeigen der Infos aufrufe wird aber leider die Tabelle mit den PatchInfos *NICHT* aktualisiert - also gleiches Verhalten wie vor dem eigenen Table Model 😕

Ich habe zum testen dann mal asl Action auf den JMenu Button nicht den JDialog aufgerufen, sondern einfach mal ein neues JFrame mit der JTable erstellt. Hier der Code dazu:

private AbstractAction getPatchInfoAction() {
        if(patchInfoAction == null) {
            patchInfoAction = new AbstractAction("Patch Info", null) {
                public void actionPerformed(ActionEvent evt) {
                    showPatchInfo();
                    //getJDialogPatchInfos().pack();
                    //getJDialogPatchInfos().setLocationRelativeTo(null);
                    //getJDialogPatchInfos().setVisible(true);
                }
            };
        }
        return patchInfoAction;
    }

Hier die einfache Methode zum erzeugen und darstellen des JFrame mit Table:

Spoiler: Methode showPatchInfo()

private void showPatchInfo() {
    
        String dbHost = jTFDBhost.getText();
        String tnsPort = jTFListenerPort.getText();
        String dbSID = jTFDBsid.getText();
        String dbSysUser = "sys as sysdba";
        char[] inputSysPwd = jPWDbSysPwd.getPassword();
        String dbSysPwd = String.valueOf(inputSysPwd);
        JFrame f = new JFrame();
        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    
        // Eigenes JTable Model erstellen
                PatchInfoTableModel sqtm = new PatchInfoTableModel(dbHost, tnsPort, dbSID, dbSysUser, dbSysPwd);
                JTable patchInfoTable = new JTable(sqtm);
                patchInfoTable.setAutoResizeMode( JTable.AUTO_RESIZE_OFF );
                // Auto resizing the JTable column widths
                final TableColumnModel columnModel = patchInfoTable.getColumnModel();
                for (int column = 0; column < patchInfoTable.getColumnCount(); column++) {
                    int width = 20; // Min width
                    for (int row = 0; row < patchInfoTable.getRowCount(); row++) {
                        TableCellRenderer renderer = patchInfoTable.getCellRenderer(row, column);
                        Component comp = patchInfoTable.prepareRenderer(renderer, row, column);
                        width = (Math.max(comp.getPreferredSize().width +1 , width)) + 1;
                    }
                    columnModel.getColumn(column).setPreferredWidth(width);
                }
            
                // JScrollPane Objekt erstellen welches neues JTable Objekt basierend auf neuem
                // Model aufnimmt und darstellt
                f.getContentPane().add(new JScrollPane(patchInfoTable));

                // JFrame darstellen ...
                f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
                f.pack();
                f.setVisible(true);
    
    }

Wenn ich den Button drücke wird ein JFrame mit den aktuellen JTable Daten erstellt. Ändere ich dann die DB Zugangsdaten und verbinde mich mit einer anderen DB, drücke den Button erneut, wird ein neues JFrame mit aktualisierten JFrame Daten angezeigt 😉

Das Problem scheint also mit der Implementation des JDialog zusammen zu hängen .. ??

Kann mir hier jemand weiterhelfen? Ich komme hier nicht wirklich weiter, habe aber das Gefühl das ich kurz vorm Ziel bin und würde ungerne jetzt aufgeben. Danke schon mal!

Gruß

Ralf

 

[mihe7]

package oracle;

import javax.swing.table.AbstractTableModel;

@SuppressWarnings("serial")
public class PatchInfoTableModel extends AbstractTableModel {

    PatchInfoCollectData patchInfoData;
    String[] columnName = null;


    // Constructor

    public PatchInfoTableModel(PatchInfoCollectData data) {
        patchInfoData = data;
    }

    public void setPatchInfoData(PatchInfoCollectData data) {
        patchInfoData = data;
        fireTableStructureChanged();
    }

    // Methods

    @Override
    public int getRowCount() {
         return patchInfoData.getRowCount();
    }

    @Override
    public int getColumnCount() {
        return patchInfoData.getColumnCount();
    }

    @Override
    public String getColumnName(int column) {
        return patchInfoData.getColumnName()[column];
    }

    @Override
    public Object getValueAt(int row, int col) {
        return patchInfoData.getPatchInfoValue()[row][col];
    }

}

Nachdem Du die neuen Verbindungsdaten vom Benutzer abgefragt hast, erzeugst Du einfach ein neues PatchInfoCollectData und rufst PatchInfoTableModel#setPatchInfoData auf. Dabei musst Du natürlich die PatchInfoTableModel-Instanz verwenden, die von JTable angezeigt wird.

 

[ralfb1105]

Hallo mihe7,

vielen Dank für Deine Unterstützung - es kann so einfach sein wenn man mit der "Nase drauf gestoßen wird" 🙄

Hier nochmal mein Code im Detail, für den Fall des es mal jemand anders ggf. gebrauchen kann 😉
In der Klasse "PatchInfoTableModel" habe ich die Methode "setPatchInfoData" hinzugefügt:

public void setPatchInfoData(String dbHost, String tnsPort, String dbSID, String dbSysUser, String dbSysPwd) {
        patchInfoData = new PatchInfoCollectData(dbHost, tnsPort, dbSID, dbSysUser, dbSysPwd);
        fireTableStructureChanged();
    }

Im Main Programm rufe ich diese Methode über die vorher erstellte Instanz "sqtm" von "PatchInfoTableModel" auf nachdem der User eine DB Connection mit neuen Werten erfolgreich getestet hat:

sqtm.setPatchInfoData(dbHost, tnsPort, dbSID, dbSysUser, dbSysPwd);

Damit funktioniert der Update der PatchInfo Tabelle.

Gruß

Ralf