JavaFX Von Datenbank in ListView

[ffghts]

Hallo,

ich stehe mit meinem Problem ein bisschen auf dem Schlauch und hoffe, dass ihr mir vielleicht helfen könnt. Ich möchte gerne Daten aus der Datenbank in einem ListView anzeigen lassen.

Ich habe eine Klasse für meine MySQL Verbindung und eine Klasse für mein MainWindow(hier ist das ListView).
Aus meiner Datenbank hole ich nun per Button Click mehrere Gruppennamen und Gruppenid's. Jetzt weiss ich aber nicht genau, wie ich die Daten von meiner Sql Klasse in meine MainWindow Klasse bekomme. Wenn ich eine Instanz von meinem MainWindow in der Sql Klasse mache, bekomme ich eine NullPointException, da das ListView nicht existiert. Habe es dann über das Singleton Pattern versucht, was auch nicht funktioniert hat. Habe es daraufhin gleich wieder verworfen, weil ich sowieso nicht genau weiss, ob das eine gute Lösung gewesen wäre.

Habe es dann mit gettern in der Sql Klasse versucht. Das funktioniert auch soweit. Habe also eine ArrayListe erstellt und mir so die Gruppennamen zurückgegeben. Nur fehlen mir jetzt noch die Gruppenid's, wofür ich dann ja eine zweite Liste bräuchte, oder? Wenn ja, wie durchlaufe ich beide gleichzeitig und packe die passende Nummer mit dem passenden Namen in mein ListView?
Oder gibt es eine ganz andere, bessere Lösung?

Hier noch ein bisschen Code:

SqlConnection.class

public static void refreshGroupList() {

        conn = getInstance();

        if(conn != null) {
            Statement query;

            try {

                query = conn.createStatement();

                String sqlQuery = "SELECT group_name, id FROM " + Constants.SQL_TABLE_GROUP;

                ResultSet result = query.executeQuery(sqlQuery);

                while(result.next()) {
                    String groupName = result.getString("group_name");
                    String id = result.getString("id");

                    list.add(groupName);
                }

            } catch (Exception ex) {
                ex.printStackTrace();
            }
        }
    }

    public static List<String> listOfNames() {
        return list;
    }

MainWindowController.class

public void refreshList() {

        SqlConnection sqlConnection = new SqlConnection();
        sqlConnection.refreshGroupList();

        for (String string : sqlConnection.listOfNames()) {
            items.add(new Group(string, "1"));
        }
    }
}

Grüße

 

[Joose]

Du kannst dir auch eine Klasse erstellen, welche diese Daten (Gruppen Id und Gruppenname) kapselt. Beim Auslesen aus der Datenbank erstellst du dann immer Objekte dieser Klasse, füllst diese mit den Daten und legst diese Objekte dann in die Liste.

 

[ffghts]

Hi,

danke erstmal für deine Antwort. Ich bin mir aber nicht sicher, ob ich das richtig verstanden habe.

Ich erstelle eine separate Klasse und kapsle dort meine Id und meinen Namen:

public class SqlData {

    private String groupName;
    private String id;

    public SqlData(final String groupName, final String id) {
        this.setGroupName(groupName);
        this.setId(id);
    }

    public String getGroupName() {
        return groupName;
    }

    public void setGroupName(String groupName) {
        this.groupName = groupName;
    }

    public String getId() {
        return id;
    }

    public void setId(String id) {
        this.id = id;
    }
}

Dann packe ich alles in eine Liste:

private static List<SqlData> list = new ArrayList<SqlData>();

    public static void refreshGroupList() {

        conn = getInstance();

        if(conn != null) {
            Statement query;

            try {

                query = conn.createStatement();

                String sqlQuery = "SELECT group_name, id FROM " + Constants.SQL_TABLE_GROUP;

                ResultSet result = query.executeQuery(sqlQuery);

                while(result.next()) {
                    String groupName = result.getString("group_name");
                    String id= result.getString("id");

                    SqlData sqlData = new SqlData(groupName, id);

                    list.add(sqlData);
                }

            } catch (Exception ex) {
                ex.printStackTrace();
            }
        }
    }

    public List getList() {
        return list;
    }
}

Und die Liste hole ich mir dann in mein MainWindow und packe sie in mein ListView:

public void refreshList() {

        SqlConnection sqlConnection = new SqlConnection();
        sqlConnection.refreshGroupList();

        for(Object object : sqlConnection.getList()) {
            // items.add(new Group("", "")); <--- ??
        }
    }

Nur habe ich dann ja eine Liste von Objekten, wie bekomme ich denn den einzelnen Inhalt der Objekte?

Grüße

 

[Joose]

Ich erstelle eine separate Klasse und kapsle dort meine Id und meinen Namen:

Genau, einzig den Namen dieser Klasse würde ich ändern.
Was ist wenn du noch andere Daten auslesen wirst? Heißt die Klasse dann "SqlData1" usw.? Auch lässt sich nur schwer rauslesen für was die Klasse existiert - ja für Daten aus der Datenbank, aber welche gibt es nicht her.
Nenn diese Klasse doch einfach "Group" oder "GroupData".

private static List<SqlData> list = new ArrayList<SqlData>();

    public static void refreshGroupList() {
......
                String sqlQuery = "SELECT group_name, id FROM " + Constants.SQL_TABLE_GROUP;
......

    public List getList() {
        return list;
    }
}

Warum ist die Liste und die Methode refreshGroupList statisch? Aber getList nicht? --> Vermeide "static"
Statt dir deinen SQL Befehl so zusammenzubauen verwende lieber PreparedStatements, dadurch verhinderst du SQL Injection.
Warum gibt getList nur eine List zurück und keine List<SqlData>?

public void refreshList() {

        SqlConnection sqlConnection = new SqlConnection();
        sqlConnection.refreshGroupList();

        for(Object object : sqlConnection.getList()) {
            // items.add(new Group("", "")); <--- ??
        }
    }

Nur habe ich dann ja eine Liste von Objekten, wie bekomme ich denn den einzelnen Inhalt der Objekte?

Dass deine Klasse die Daten ausliest "SqlConnection" heißt finde ich verwirrend.
getList() gibt dir nur eine List zurück. Wnn du stattdessen ein List<SqlData> zurückgibst, dann kannst du in for-Schleife über deine SqlData Objekte iterieren und normal auf die getter zugreifen.

Frage: Worin unterscheiden sich denn die Klassen "SqlData" und "Group"? Warum erstellst du nicht gleich "Group" Objekte in der Methode "refreshGroupList"?

 

[ffghts]

Danke, jetzt funktioniert es.

Ja, einige Klassennamen sind nicht ganz so gut gewählt. Ich übe gerade bisschen mit der Datenbank und so ist immer mehr Code entstanden und die Namen passten dann irgendwann nicht mehr wirklich.

Habe die Methoden statisch gemacht, da es in einem Tutorial im Internet auch war. Warum genau das gemacht wurde, stand allerdings nicht dabei. Habe es einfach so übernommen.
Ich wusste nicht wirklich, dass es PreparedStatements gibt. Muss mir erstmal angucken, wie die funktionieren, aber auch danke dafür.

Frage: Worin unterscheiden sich denn die Klassen "SqlData" und "Group"? Warum erstellst du nicht gleich "Group" Objekte in der Methode "refreshGroupList"?

Jetzt wo du das so sagst, eigentlich unterscheiden die sich gar nicht. Kann auch einfach die verwenden. Dann brauche ich SqlData gar nicht mehr.

Ich danke dir auf jeden Fall vielmals. Habe mal wieder was dazu gelernt.

Grüße