variable kopieren bzw. woanders benutzen

[Gregorio]

Hallo Freunde,

ich sehe momentan den Wald vor lauter Bäume nicht und ich glaube ich denke zu kompliziert.
Habe ein kleines Programm daß Werte aus einer PostgreSQL DB ausliest und in einer JTable anzeigt.
Möchte aber eine Variable davon woanders noch benutzen ohne daß ich in der neuen Methode die SELECT Abfrage
nochmal erstelle aber ich kriege es einfach nicht hin, wo denke ich falsch?, wo ist der Fehler? Ist es überhaupt machbar?

Hier der Code:

import java.sql.Connection;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.util.logging.Level;
import java.util.logging.Logger;
import javax.swing.table.DefaultTableModel;

public class populateJTableJavaPostgresqlDatabase extends javax.swing.JFrame {
    PreparedStatement pst = null;
    ResultSet rst = null;
    Connection connDbc = null;
    databaseConnection dbc = new databaseConnection();
    public populateJTableJavaPostgresqlDatabase() {
        initComponents();
        connDbc = dbc.databaseConn();
        populateJtable();
    }
    public void populateJtable() {

        try {
            String  sqlSelectDataFromDatabase = "select * from dbtest where dbfirm=? and dbpern=? order by dbpern";
            pst = connDbc.prepareStatement(sqlSelectDataFromDatabase);
            int firm = 1;
            int pern = 1;
            pst.setInt(1, firm);
            pst.setInt(2, pern);
            rst = pst.executeQuery();

            while(rst.next()) {
                String CompanyName = rst.getString("dbname");
                String Nummer = rst.getString("dbpern");
                String ContactTitle = rst.getString("dbtitl");
                String Address = rst.getString("dbaddr");

                DefaultTableModel dftable = (DefaultTableModel) jTablePopulateData.getModel();
                Object[] obj = {CompanyName,Nummer,ContactTitle,Address};
                dftable.addRow(obj);
            }

        } catch (SQLException ex) {
            Logger.getLogger(populateJTableJavaPostgresqlDatabase.class.getName()).log(Level.SEVERE, null, ex);
        }

    }
    @SuppressWarnings("unchecked")
    private void initComponents() {

        jScrollPane1 = new javax.swing.JScrollPane();
        jTablePopulateData = new javax.swing.JTable();

        setDefaultCloseOperation(javax.swing.WindowConstants.EXIT_ON_CLOSE);

        jTablePopulateData.setModel(new javax.swing.table.DefaultTableModel(
                new Object [][] {

                },
                new String [] {"Firma", "Nummer", "Anrede", "Adresse"
                }
                ));
        jScrollPane1.setViewportView(jTablePopulateData);

        javax.swing.GroupLayout layout = new javax.swing.GroupLayout(getContentPane());
        getContentPane().setLayout(layout);
        layout.setHorizontalGroup(
                layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                .addGroup(layout.createSequentialGroup()
                        .addContainerGap()
                        .addComponent(jScrollPane1, javax.swing.GroupLayout.PREFERRED_SIZE, 375, javax.swing.GroupLayout.PREFERRED_SIZE)
                        .addContainerGap(19, Short.MAX_VALUE))
                );
        layout.setVerticalGroup(
                layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                .addGroup(layout.createSequentialGroup()
                        .addComponent(jScrollPane1, javax.swing.GroupLayout.PREFERRED_SIZE, 275, javax.swing.GroupLayout.PREFERRED_SIZE)
                        .addGap(0, 25, Short.MAX_VALUE))
                );

        pack();
    }
    
    
//    NEU
    public void testVariableAnzeigen() {
        try {
            System.out.println("Variable - Test 1: " + rst.getString("dbfirm"));
            System.out.println("Variable - Test 2: ");
        } catch (SQLException e) {
            // TODO Automatisch generierter Erfassungsblock
            e.printStackTrace();
        }
        
    }
//    NEU

    public static void main(String args[]) {
        try {
            for (javax.swing.UIManager.LookAndFeelInfo info : javax.swing.UIManager.getInstalledLookAndFeels()) {
                if ("Nimbus".equals(info.getName())) {
                    javax.swing.UIManager.setLookAndFeel(info.getClassName());
                    break;
                }
            }
        } catch (ClassNotFoundException ex) {
            java.util.logging.Logger.getLogger(populateJTableJavaPostgresqlDatabase.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        } catch (InstantiationException ex) {
            java.util.logging.Logger.getLogger(populateJTableJavaPostgresqlDatabase.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        } catch (IllegalAccessException ex) {
            java.util.logging.Logger.getLogger(populateJTableJavaPostgresqlDatabase.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        } catch (javax.swing.UnsupportedLookAndFeelException ex) {
            java.util.logging.Logger.getLogger(populateJTableJavaPostgresqlDatabase.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        }

        java.awt.EventQueue.invokeLater(new Runnable() {
            public void run() {
                new populateJTableJavaPostgresqlDatabase().setVisible(true);
            }
        });
    }

    private javax.swing.JScrollPane jScrollPane1;
    private javax.swing.JTable jTablePopulateData;

}

Vielen Dank

 

[kneitzel]

Sowas ist problemlos machbar. Du musst die Daten nur in einer Datenstruktur speichern und dann aus anderen Klassen darauf zugreifen.

Das ist auch das generelle Vorgehen. Du packst alles wild in eine Klasse und das ist extrem unübersichtlich und macht auch alles deutlich schwerer.

Also bei so einer Applikation hätte man dann aus Sicht der UI:
- eine Klasse, die die UI steuert.
- ein Model, das die Daten hält.
Hier hat man dann oft Pattern wie MVC oder MVVM.

Aus Applikationssicht hat man dann halt im Model:
- die Datenklassen, die die Daten halten
- Klassen für den Zugriff (also in Deinem Fall für den Zugriff auf die Datenbank).

Das ist aber nur die jeweilige Strukturierung. Wie man welche Daten genau vorhält, das hängt natürlich damit zusammen, was Man genau machen will / muss. "eine Variable davon woanders noch benutzen" besagt nicht wirklich etwas aus.

 

[Gregorio]

Vielen Dank schon mal für die Info,
hast du einen klitzekleinen Minicode wo man das sich bildlich vorstellen könnte?, falls überhaupt machbar..

Was ich damit möchte ist folgendes.
Ich lasse die Daten der DB in einer JTable anzeigen, ich möchte aber eine bestimmte Spalte in einer neuen Methode TableCellRenderer farblich verändern und später noch in dieser Spalte die maximale Länge der eingegeben Zeichen überprüfen, dafür brauche ich nur eine Variable aus der DB und da die ganze Tabelle sowieso in der anderen Methode eingelesen wird, dachte ich mir... kopieren oder übergeben..

Ich wollte mir halt die Arbeit ersparen die Abfrage neu einzugeben, aber das was du mir da empfohlen hast übersteigt noch meine Java Kenntnisse 🙂

 

[kneitzel]

Der übliche Anfang ist immer ohne Oberfläche:
a) Du willst die sogenannten Entities erstellen. Die Tabelle heisst dbtest - der Name besagt irgendwie nichts, aber ok: Klasse DBTest.
Also etwas wie:

public class DBTest {
    private String dbName;
    private String dbPern;
    private String dbTitl;
    private String dbAddr;
    private String dbFirn
    
    // Getter, Setter, Konstruktor, ....
}

b) Dann willst Du die Elemente aus der Datenbank lesen können oder schreiben, löschen, was auch immer. Das wird dann also sowas wie DBTestDAO (DAO = Data Access Object) oder DBTestRepository.
Da hast Du dann halt die Zugriffe auf die Datenbank und dabei halt auch sowas wie

List<DBTest> getDBTestByPernAndFirn(String pern, String firn)

Dann kannst Du Dir jetzt überlegen, was denn die Applikation machen soll. Das kann man dann z.B. in der Art aufbauen:

Klasse MainWindowController
- Da ist die eigentliche Logik drin, die irgendwas machen. Der Controller hat dann ein Repository und lädt die Daten.
- Und da finden sich dann auch alle Aktionen, die der Nutzer machen kann.

Klasse MainWindow
- Da Swing keine deklarative UI unterstützt, hast Du hier den Aufbau der Oberfläche.
- Die Daten, die Angezeigt werden sollen, speicherst Du auch. Das kann also eine List<DBTest> sein, die Du vom Controller bekommst. Sollte da einiges an Daten zusammen kommen, dann macht es Sinn, das ggf. in einer Klasse MainWindowModel zu kapseln.
- Hier werden Daten aufbereitet, z.B. für die Tabelle. Bei so einer Sache arbeite ich gerne mit entsprechender Unterstützung direkt für das Model. Das gibt es bei Swing aber nicht so, wie es das bei anderen Frameworks schon so gibt. Daher wird es dann eine weitere Klasse geben:

public class DBTestTableModel extends AbstractTableModel {
    // ...
}

Siehe z.B. https://stackoverflow.com/questions...om-table-model-to-provide-data-from-arraylist
- Wenn Du auf Aktionen reagierst, also z.B. Nutzer drückt einen Knopf, dann übersetzt Du das nur in Aufrufe zu dem Controller. Das ist oft nur ein direkter Aufruf, was also gut als Lambda Ausdruck geht: Also Action vom Delete Button ist dann sowas wie
e -> controller.deleteDBTest(getSelectedItem())

Das wäre eine Struktur, die denkbar wäre.

 

[Oneixee5]

"select * from dbtest where dbfirm=? and dbpern=? order by dbpern"

Das order by dbpern ist auch ganz schön sinnlos.

 

[Gregorio]

Das order by dbpern ist auch ganz schön sinnlos.

das ist nur ein Teil der Abfrage, sinnlos ist das sortieren nicht da es sich um mehrere 100 Datensätze handelt.

 

[thecain]

da du das select auf genau einen möglichen dbpern einschränkst schon...