Unvollständige Daten in der TableView

[Tacoma]

Hallo Community,


ich habe eine SQLite DB mit 5 Columns und 3 Rows Testdaten. Ich versuche, diese bestehende Daten in der DB in einer TableView darzustellen.


Folgenden Code habe ich getestet.

        data = FXCollections.observableArrayList();
        try {
            //c = SqliteConnect.SQLiteConnection();
            Connection connection = DatabaseConnection.getConnection();
            String SQL = "SELECT * from spiritlog";
            ResultSet rs = connection.createStatement().executeQuery(SQL);

            for (int i = 0; i < rs.getMetaData().getColumnCount(); i++) {
                final int j = 1;
                TableColumn col = new TableColumn(rs.getMetaData().getColumnName(i + 1));
                col.setCellValueFactory(new Callback<CellDataFeatures<ObservableList, String>, ObservableValue<String>>() {
                    public ObservableValue<String> call(CellDataFeatures<ObservableList, String> param) {
                        return new SimpleStringProperty(param.getValue().get(j).toString());
                    }
                });
                tableView.getColumns().addAll(col);
                System.out.println("Column [" + i + "]");
            }
            while (rs.next()) {
                ObservableList<String> row = FXCollections.observableArrayList();
                for (int i = 1; i <= rs.getMetaData().getColumnCount(); i++) {
                    row.add(rs.getString(i));
                }
                System.out.println("Row [1] added " + row);
                data.add(row);
            }
            tableView.setItems(data);
        } catch (Exception e) {
            e.printStackTrace();
            System.out.println("Error on Building data");
        }

Wenn ich dies laufen lasse, erhalte ich in der TableView die korrekte Darstellung der Felderbeschreibung / Überschrift. Es wird mir im Inhalt jedoch für alle 5 Columns der Wert der 2. Column angezeigt. Siehe Screeshot.


In der IntelliJ IDE sehe ich im Ausgabefenster die korrekten Daten für jede Row.

Row [1] added [testdate1, testtime1, testvallue1, testtag1, testcomment1]
Row [1] added [testdate2, testtime2, testvalue2, testtag2, testcomment2]
Row [1] added [testdate3, testtime3, testvalue3, testtag4, testcomment4]

Was habe ich falsch gemacht?

 

[LimDul]

Du musst in der Schleife while (rs.next()) auch ein neues Row-Objekt erzeugen. Du hast nur ein Row Objekt, das du immer änderst.

 

[Tacoma]

Du musst in der Schleife while (rs.next()) auch ein neues Row-Objekt erzeugen. Du hast nur ein Row Objekt, das du immer änderst.

Besten Dank für die Hilfe.

 

[Tacoma]

Eine Frage hierzu habe ich noch.

Gibt es eine Dokumentation, die das mit dem neuen Row-Object beschreibt, sodass man das auch als Anfänger nachvollziehen kann. In den Code Beispielen, wo man sich das herholt, gibt es keinen Hinweis auf ein neues Row-Object. Trotzdem wird dann so ein Code Beispiel als passend markiert.

 

[KonradN]

Also Oracle hat da eigentlich brauchbare Dokumentation. Ich selbst würde auch nicht so mit den ArrayListen für die Werte arbeiten. Der Weg, den ich für üblich halte, ist immer: Daten sauber strukturieren. Dann hast Du eine ObservableList von den Entities von Dir. Und diese Daten werden dann angezeigt, wie es z.B. bei:
Using JavaFX UI Controls: Table View | JavaFX 2 Tutorials and Documentation (oracle.com)
gezeigt wird.

Dann hast Du das Problem auch sauber in Teilprobleme zerlegt. Bei Dir wäre es z.B. das Lesen der Daten aus der Datenbank und dann separat davon die Anzeige in einer Table.

Und wenn ich Deinen Code ansehe, dann ist das Problem doch nicht wie von @LimDul beschrieben. Du erzeugst doch in der Schleife immer das row Objekt neu.

Wenn ich Dich aber richtig verstanden habe, ist Dein Problem, dass Du in allen Spalten immer die testtime angezeigt bekommst. Aber das ist doch auch klar, denn in der CellValueFactory nimmst immer das Element 1 (final int j = 1) aus dem row Array.

 

[Tacoma]

@KonradN
Besten Dank für den Link und die Info.

 

[KonradN]

Jetzt beim erneuten Hineingehen verstehe ich auch den Fehler. Ich hatte mich über das j = 1 gewundert - Da hast Du evtl. einen Kopier oder Abtipp Fehler:
Zeile 9 sollte vermutlich ein final int j = i; sein.

Hintergrund: Du gehst die Spalten durch. dazu hast Du die Schleife mit dem i. Aber das i kannst Du nicht in der anonymen inneren Klasse verwenden. Da muss es ein (pseudo) final Wert sein. Dazu wird das i in der lokalen final Variable j gespeichert. Und schon stimmt auch der Zugriff auf den Wert des Arrays.

Aber so Fehler sind schwer zu sehen, wenn der Code so unstrukturiert ist. Daher wäre es auf jeden Fall sinnvoll, den Code besser zu strukturieren und die Werte einer Zeile nicht einfach in einer String List zu speichern.

 

[Tacoma]

Jetzt beim erneuten Hineingehen verstehe ich auch den Fehler. Ich hatte mich über das j = 1 gewundert - Da hast Du evtl. einen Kopier oder Abtipp Fehler:
Zeile 9 sollte vermutlich ein final int j = i; sein.

Hintergrund: Du gehst die Spalten durch. dazu hast Du die Schleife mit dem i. Aber das i kannst Du nicht in der anonymen inneren Klasse verwenden. Da muss es ein (pseudo) final Wert sein. Dazu wird das i in der lokalen final Variable j gespeichert. Und schon stimmt auch der Zugriff auf den Wert des Arrays.

Aber so Fehler sind schwer zu sehen, wenn der Code so unstrukturiert ist. Daher wäre es auf jeden Fall sinnvoll, den Code besser zu strukturieren und die Werte einer Zeile nicht einfach in einer String List zu speichern.

Vielen Dank für die Lösung aber auch für die Erklärung. Als Anfänger habe ich mich mit der TableView schwer getan und war froh, überhaupt einen Lösungsansatz zu haben.

final int j = i;

Damit funktioniert es 😇

Ist der Code aus der o.a. Oracle Dokumentation eher geeignet oder gibt es hier noch eine "bessere" Lösung?

 

[KonradN]

Hier wäre meine Frage: Was verstehst Du unter besser geeignet?

Aus meiner Sicht ist es auf jeden Fall besser geeignet, da Du eine bessere Trennung der einzelnen Teile hättest. Die Idee ist, dass Du diverse Aufgaben jeweils etwas gekapselt hättest:

• Die Daten und was da direkt dazu gehört (Also z.B. das Lesen der Daten aus der Datenbank) Dabei ist wichtig: Die Daten liegen in einem natürlichen Datenformat vor. Du hast also nicht alles Strings.
• Die reine Anzeige ist davon losgelöst.
• In der Anzeige hast Du dann separat auch die Formatierungen. Also wie soll z.B. das Datum formatiert werden?

Wenn Du diese Aufteilung hast, dann ist in der Regel jede Teil für sich testbar. Änderungen sind dann auch einfach durchzuführen, denn Du kannst das dann zentral haben. (Also z.B. definierst Du nur einmal, die das Format sein soll und nutzt das dann an allen Stellen, Wo Du ein Datum anzeigst.)


ABER: Ich weiss jetzt im Augenblick nicht sicher, ob es für Dich tauglich sein kann, denn ich kenne Deine Anforderungen nicht.

Im Augenblick hast Du Code, der das Ergebnis einer beliebigen Query, anzeigen kann. Wenn Du nicht weisst, was für Daten Du anzeigen wirst, dann kannst Du natürlich auch keine Datenklassen dafür haben. Und dann fällt das, was in dem Link von mir gezeigt wird, natürlich flach. Je nach Anforderungen würde ich es dort aber dann auch etwas mehr kapseln, aber das ist dann nicht mehr eine 1:1 Übernahme von dem Code.

Aber normalerweise hast Du ja ein konkretes Datenbankschema in Deiner Anwendung und dann hast Du das Datenmodell in Java. Also hier hättest Du evtl. eine Klasse LogEntry mit den entsprechenden Feldern. Und dann hättest du eine andere Klasse, die dafür zuständig ist, die LogEntry Daten aus der Datenbank zu laden (oder zu speichern). Das wäre dann sozusagen das Model. Dann ist die Frage: Was für ein Pattern will man für die Anzeige verwenden? MVC und MVVM sind da mit am verbreitetsten. Da könntest Du Dir das MVC Pattern evtl. einmal näher ansehen.

Hier ist aber auch wichtig: Wie tief willst Du Dich hinein knien? Was willst Du da bauen? Wie komplex wird das? Es kommen halt hier sehr schnell sehr viele Themen ins Spiel...