Java und MySQL MariaDB - WHERE-Klausel

[Juelin]

Hallo,
ich schreibe gerade eine Anwenung in javaFX mit einer MySQL MariaDB Datenbank.
Der Query zum aus lesen der Datenbank funktioniert einwandfrei.
Aber wenn ich mit WHERE-Klausel auslesen will tritt die Exception " catch(SQLException err)" in kraft. (siehe bild1).

Das ist die Routine die ich aufrufe:

    public void dbTabellenEintraegeArray(int fanz, String feld1, String feld2, String feld3, String feld4, String feld5)
        {
        int a = 0;
        int f = 0;
        String x;
        dbRet = 6;
        String[] dbgelesen = new String[999999];
        VitalShowController.anzeintraege = 0;
        if (dbOpen == 1)
            {
            boolean l = true;
            boolean g;
            dbRet = 0;
            String nachname;
            String vorname;
            String geburtsdatum;
            String datum;
            String uhrzeit;
            String systole;
            String diastole;
            String puls;
            String temperatur;
            String groesse;
            String gewicht;
            String bemerkung;
            String spo2;
            String mnachname = "";
            String mvorname = "";
            String mgeburtsdatum = "";
            try
                {
                Statement befehl = verbindung.createStatement(ResultSet.TYPE_SCROLL_SENSITIVE, ResultSet.CONCUR_UPDATABLE);
                ResultSet rs = befehl.executeQuery(dbBefehl);
                while (l)
                    {
                    rs.next();
                    g = rs.isAfterLast();
                    if (g)
                        {
                        l = false;   
                        }
                    else
                        {
                        if (VitalShowController.dbLesArt == 1)
                            {
                            nachname =rs.getString("Nachname");
                            vorname =rs.getString("Vorname");
                            geburtsdatum =rs.getString("Geburtsdatum");
                            if (!(nachname.equals(mnachname)) || !(vorname.equals(mvorname)) || !(geburtsdatum.equals(mgeburtsdatum)))
                                {
                                x = vorname+" "+nachname+" "+geburtsdatum;
                                dbgelesen[VitalShowController.anzeintraege] = x;
                                VitalShowController.anzeintraege++;
                                mnachname = nachname;
                                mvorname = vorname;
                                mgeburtsdatum = geburtsdatum;
                                }
                            }
                        if (VitalShowController.dbLesArt == 2)
                            {
                            nachname =rs.getString("Nachname");
                            vorname =rs.getString("Vorname");
                            geburtsdatum =rs.getString("Geburtsdatum");
                            datum =rs.getString("Datum");
                            uhrzeit =rs.getString("Uhrzeit");
                            systole =rs.getString("Systole");
                            diastole =rs.getString("Diastole");
                            puls =rs.getString("puls");
                            temperatur =rs.getString("temperatur");
                            groesse =rs.getString("groesse");
                            gewicht =rs.getString("gewicht");
                            bemerkung =rs.getString("bemerkung");
                            spo2 =rs.getString("SpO2");
                            if (fanz > 0)
                                {
                                f = 0;   
                                if (nachname.equals(feld2))
                                    {
                                    if (fanz == 1)
                                        {
                                        f = 1;   
                                        }
                                    }
                                if (fanz > 1)
                                    {
                                    if (vorname.equals(feld1))
                                        {
                                        if (fanz == 2)
                                            {
                                            f = 1;   
                                            }
                                        }
                                    }
                                if (fanz > 2)
                                    {
                                    if (geburtsdatum.equals(feld3))
                                        {
                                        if (fanz == 3)
                                            {
                                            f = 1;   
                                            }
                                        }
                                    }
                                if (fanz > 3)
                                    {
                                    if (datum.equals(feld4))
                                        {
                                        if (fanz == 4)
                                            {
                                            f = 1;   
                                            }
                                        }
                                    }
                                if (fanz > 4)
                                    {
                                    if (uhrzeit.equals(feld5))
                                        {
                                        f = 1;   
                                        }
                                    }
                                if (f == 1)
                                    {
                                    x = vorname+"/"+nachname+"/"+geburtsdatum+"/"+datum+"/"+uhrzeit+"/"+systole+"/"+diastole+"/"+puls+"/"+temperatur+"/"+groesse+"/"+gewicht+"/"+bemerkung+"/"+spo2+"/";
                                    dbgelesen[VitalShowController.anzeintraege] = x;
                                    VitalShowController.anzeintraege++;
                                    }
                                }
                            else
                                {
                            x = vorname+"/"+nachname+"/"+geburtsdatum+"/"+datum+"/"+uhrzeit+"/"+systole+"/"+diastole+"/"+puls+"/"+temperatur+"/"+groesse+"/"+gewicht+"/"+bemerkung+"/"+spo2+"/";
                            dbgelesen[VitalShowController.anzeintraege] = x;
                            VitalShowController.anzeintraege++;
                                }
                            }
                        a++;   
                        }
                    }
                rs.close();
                }
            catch(SQLException err)
                {
                dbRet = 5;   
                }
            }
        else
            {
            dbRet = 6;   
            }
        if (VitalShowController.anzeintraege > 0)
            {
            VitalShowController.eintraege = new String[VitalShowController.anzeintraege];
            for (a=0; a<VitalShowController.anzeintraege; a++)
                {
                x = dbgelesen[a];
                VitalShowController.eintraege[a] = x;
                }
            }
        }

Habe im Internet geschaut und mein SQL-Befehl

Vital.datenbank.dbBefehl = "SELECT * FROM "+Vital.datenbank.dbTabelle+" WHERE Vorname = '"+dvorname+"' ORDER BY Nachname, Vorname, Geburtsdatum, Datum DESC, Uhrzeit DESC";

müsste ok sein.

Keine Ahnung was da falsch ist.
Kann da jemand helfen?

Danke und Gruß
Jürgen

 

[LimDul]

Ein paar Anmerkungen.
So einen SQL Befehl zusammenzubauen, macht man nicht - man nutzt Prepared Statements. Weil sobald im Namen Sonderzeichen vorkommen geht das komplett auf Bretter bis hin zur kompletten SQL-Injection.

            catch(SQLException err)
                {
                dbRet = 5;   
                }

Auch ganz ganz schlecht. Du verschluckst die Exception. Mindestens ein err.printStacktrace gehört rein.
Auf den ersten Blick sieht der SQL Befehl ok aus - das heißt vermutlich passen Spalten oder Tabellennamen nicht. Lass dir den Stacktrace mit der Nachricht ausgeben, da sollten mehr Infos drinstehen

 

[M.L.]

MariaDB hat auch einen HeidiSQL-Manager, der beim korrekten Formulieren und Testen von Abfragen helfen könnte.

 

[KonradN]

Bitte gewöhne Dir direkt an, Exceptions nicht einfach zu ignorieren! Lass Dir zumindest die Details der Exception ausgeben, also die Message als auch den Stacktrace!

Und die Schleife kann einfach ein while (rs.next()) sein - dann musst Du da keine Variable nutzen und nichts prüfen a.la. rs.isAfterLast().

Und wenn man dann Details weiss, dann kann man auch eher sagen, was schief gelaufen ist. Nur ohne exakte Fehlermeldung ist es schwer, etwas zu sagen. Evtl. stimmt der Tabellenname nicht oder es gibt eines der Felder nicht?

 

[Manul]

Aber wenn ich mit WHERE-Klausel auslesen will tritt die Exception " catch(SQLException err)" in kraft. (siehe bild1).
...
Keine Ahnung was da falsch ist.

Um genau das zu vermeiden sollte man Exceptions im Code nie einfach verschlucken, sondern zumindest loggen, wenn man sie nicht nach oben delegiert.

Ich würde dafür bei der Gelegenheit auch sinnvollerweise gleich einen richtigen Logger verwenden.
System.err.println halte ich maximal für lokale Debuggingzwecke während der Entwicklung akzeptabel (sollte aber möglichst nicht committed werden), und hat spätestens im Produktivcode nichts mehr verloren.

 

[Juelin]

Danke an alle.
Ihr habt recht, man sollte sich die Fehlermeldung von SQLError an sehen.
Die Fehlermeldung war wrong row.
Ich habe debuged und fest gestellt, das die Abfrage von "isAfterLast()" nicht richtig funktioniert.
Ich will einen Satz lesen, den es NICHT gibt.
Aber isAfterLast() gibt immer FALSE zurück, obwohl gar kein Satz gelesen wurde.
Seltsam, seötsam.
Hat da einer eine Idee?
Danke und Gruß
Jürgen
PS: habe auch isLats() probiert, genau so wie isAfterLast().

 

[Oneixee5]

Danke an alle.
Ihr habt recht, man sollte sich die Fehlermeldung von SQLError an sehen.
Die Fehlermeldung war wrong row.
Ich habe debuged und fest gestellt, das die Abfrage von "isAfterLast()" nicht richtig funktioniert.
Ich will einen Satz lesen, den es NICHT gibt.
Aber isAfterLast() gibt immer FALSE zurück, obwohl gar kein Satz gelesen wurde.
Seltsam, seötsam.
Hat da einer eine Idee?
Danke und Gruß
Jürgen
PS: habe auch isLats() probiert, genau so wie isAfterLast().

Wie ich schon in einem anderen deiner Beiträge geschrieben habe: https://www.java-forum.org/thema/if-abfrage-funktioniert-nicht-richtig.204183/#post-1368916

Ich bin etwas erstaunt über die "Milde" mit welcher auf diese Beiträge geantwortet wird. Alles was du in deinem Code oben machst, sollte in 7-10 Zeilen Code erledigt sein - nicht in 158. Viele Probleme, dieser Art vorzugehen, wurden auch noch gar nicht angesprochen.

 

[KonradN]

Und die Schleife kann einfach ein while (rs.next()) sein - dann musst Du da keine Variable nutzen und nichts prüfen a.la. rs.isAfterLast().

Und nur um es noch einmal mit Dokumentation zu belegen:
ResultSet (Java SE 17 & JDK 17) (oracle.com)

A ResultSet object maintains a cursor pointing to its current row of data. Initially the cursor is positioned before the first row. The next method moves the cursor to the next row, and because it returns false when there are no more rows in the ResultSet object, it can be used in a while loop to iterate through the result set.

Bezüglich isAfterLast:

Note:Support for the isAfterLast method is optional for ResultSets with a result set type of TYPE_FORWARD_ONLY

Daher ist das keine gute Idee, das hier zu verwenden.

Die Fehlermeldung war wrong row.

Bitte immer die ganze Meldung der Exception melden und im idealen Fall auch bitte den Stacktrace. Gerade letzteres ist z.B. wichtig um zu sehen, wo die Exception geworfen wird. Dann würde man direkt sehen, ob es in isAfterLast oder in executeQuery oder sonst irgendwo ausgelöst wurde.

Daher bitte immer ein ex.printStackTrace() oder so mit im Catch-Block haben (oder wie schon in #5 genannt: korrektes Logging / Tracing).

 

[M.L.]

Milde

(offtopic)Mit wenig Erfahrung z.B. mit Java ist es aber verständlich erstmal lesbaren Code zu schreiben (statt hochoptimierten, dessen Sinn man adhoc nicht erkennt)

 

[Manul]

Mit wenig Erfahrung z.B. mit Java ist es aber verständlich erstmal lesbaren Code zu schreiben

Genau das ist der Code in seiner jetzigen Form eben leider (noch) nicht.

(statt hochoptimierten, dessen Sinn man adhoc nicht erkennt)

Von (hoch)optimiert war bis jetzt nirgendwo die Rede hier (und wäre auch gar nicht das Ziel, das kann der Compiler eh meist besser).

Soweit ich das sehe, gingen bis jetzt alle Kommentare eher in die Richtung, sich mit den Idiomen und Konventionen/Best-Practices der Sprache und ihrer APIs vertraut zu machen und diese anzuwenden. Was zusammen mit einer halbwegs gängigen Formatierung den Code auch deutlich besser verständlich machen würde.
Statt lediglich ein Pascal-Programm in Java-Syntax zu schreiben, bzw. wie bei der while-Schleife das Rad komplett neu zu erfinden. Wodurch eben leicht schlecht lesbarer und fehleranfälliger Code entsteht.

 

[M.L.]

falsch

Eventuell könnte der geeignete Einsatz von Hochkommata doch weiterhelfen, zumindest unter dem HeidiSQL-Manager und einer Beispielquery (nebenbei kann man z.B. die Vital-DB mit dem Manager auch exportieren inkl. z.B. Queries zur Tabellendefinition und Datenbefüllung):

 

[Juelin]

Also, ich habe das Problem gelöst.
Nur muß ich sagen, in Java ist alles ganz schön kompliziert. In Delphi ist das einfacher.
Es gibt in Java keine Eigenschaft Anzahl Rows.
Ich musste die Anzahl Rows mit den Befehlen:
Last()
getRow()
First()
ermitteln.
Und wenn keine Daten in der Datenbank gefunden worden sind geht der Befehl
isAfterLast nicht. Ist immer FALSE.
Wenn keine Rows vorhanden darf man nicht in Leseroutine.
Also Ihr Spezialisten, es ist eben nicht alles Gold was glänzt.
Gruß
Jürgen

 

[KonradN]

Also Ihr Spezialisten, es ist eben nicht alles Gold was glänzt.

Das Problem ist, dass Du vermutlich irgend welche Lösungsansätze, die unter Delphi üblich sind, versuchst 1:1 zu übernehmen. Damit rennst Du in viele x y Probleme. Du willst eigentlich x machen, hast dafür aber den falschen Ansatz, so dass Du in lauter komische y Probleme rennst.

Das ist schon mehrfach aufgefallen und da wurde Dir schon öfters der eine oder andere Hinweis zu gegeben.

Wozu brauchst Du die Anzahl der rows? Ist das, weil Du ein Array fester Größe für das Ergebnis haben willst?
==> Nimm einfach eine List. Dazu gibt es die Collections im Framework.

Dir muss halt klar sein, dass es in Java Abweichungen gibt bezüglich der zu Grunde liegenden Bibliotheken gibt. Und Dir sollte bewusst sein, dass es für viele Deiner Probleme bereits einfach umzusetzende Lösungen gibt.

 

[Manul]

Nur muß ich sagen, in Java ist alles ganz schön kompliziert. In Delphi ist das einfacher.

Klar ist es manchmal etwas frustig, wenn man nach langjähriger Erfahrung mit Sprache X plötzlich auf Y umlernt. Erfahrungsgemäß sind aber auch in Java die Dinge nur so kompliziert, wie man sie sich macht. Java als Sprache an sich ist sogar recht einfach gestrickt, aber die APIs und das ganze Ökosystem aus Tools, Libraries und Frameworks brauchen schon etwas Zeit um reinzukommen und ein gutes Gespür dafür zu entwickeln, wann man was nutzt um daraus guten Code entstehen zu lassen. Gerade die Wahl der richtigen Libraries ist bei Java fast schon obligatorisch, um sich das Leben nicht unnötig schwer zu machen. Zum Glück gibt es da aber für (fast) alles was, sodass du nicht jedesmal das Rad neu erfinden oder quer durch alle Abstraktionsschichten schießen musst.

 

[M.L.]

Java keine Eigenschaft Anzahl Rows

Mit JDBC : getInt(1)-Methode auf dem ResultSet anwenden

 

[KonradN]

Mit JDBC : getInt(1)-Methode auf dem ResultSet anwenden

Wie kommst Du auf diese Idee? Das liefert den Wert der ersten Spalte der aktuellen Row als int und macht daher nur dann Sinn, wenn man eine Abfrage auf Count(*) oder so gemacht hat (als eigenständige Abfrage vorab).

Das Verhalten von ResultSet bezüglich getInt ist klar: Das liefert den int aus der ersten Spalten der aktuellen Zeile.
Wenn Du noch kein next() aufgerufen hast, dann bist Du noch auf keiner row und Du bekommst eine SQLException.
(Das ist also kein Hack bezüglich "vor dem ersten next() aufrufen oder so)

Einfach ein schneller Test mit einer InMemory Datenbank. HSQLDB bietet sich da an und man kann einfach ein Beispiel aus dem Netz kopieren:
Javabeginners - HyperSQLDB

Da kann man z.B. vor der while Schleife einmal versuchen, das getInt(1) hinzu zu fügen:

java.sql.SQLException: invalid cursor state: identifier cursor not positioned on row in UPDATE, DELETE, SET, or GET statement: ; ResultSet is positioned before first row

 

[LimDul]

Achtung - längerer Beitrag
Wie @KonradN geschrieben hat, sind einige deiner Probleme darauf zurückzuführen, dass du nicht Java programmierst - sondern Delphi/Pascal mit Java Syntax. Das ist wie, wenn man vom Tischtennis zum Tennis wechselt und versucht die gleichen Schlagbewegungen durchzuführen, macht auch nur Probleme und Schmerzen

Ich mach jetzt drei Dinge:
a) Mal ein komplettes Code-Review auf die formalen Aspekte
b) Review bzgl. der Struktur
c) Wie kann man Dinge besser Java-Like machen.

Das wird in a) und b) recht viel sein - da bitte nicht falsch verstehen, dass man Anfang halt viele Dinge nicht richtig macht, ist normal.

    public void dbTabellenEintraegeArray(int fanz, String feld1, String feld2, String feld3, String feld4, String feld5)
        {

Hier fallen direkt mehrere Dinge auf.
Formatierung: In Java schreibt man gemäß den Java Coding Conventions die öffenende Klammer in die gleiche Zeile. Ist natürlich am Ende jedem selber überlassen - aber 99,9% der Java Welt machen es anders und wenn man mal mit anderen Java Entwicklern zusammenarbeiten will, sollte man es sich direkt angewöhnen.
Benennung: Die Benennung der Paramater ist schlecht. feld1 bis feld5? Was soll das sein? fanz? Irgendeine Anzahl? Aber was? Klare Bezeichnungen sind wichtig. (Aus dem Code geht es nachher hervor, aber es muss schon aus dem Methodenrumpf hervorgehen)
Anzahl/Art der Parameter: feld1 bis feld5 und ein Anzahl, wie viele Felder gefüllt sind? Das hat nix mehr mit Java zu tun Für sowas gibt es Listen. Beispiel:

public void dbTabellenEintraegeArray(List<String> zuLesendeFelder) {

Rückgabe? Die Methode liest es aus der Datenbank hat aber keine Rückgabe. Das riecht direkt nach Seiteneffekten, dass sie irgendwas irgendwo anders ändert - ganz großes Antipattern, macht man nicht. Wenn eine Methode, was liest, dann hat sie es entweder zurückzugeben oder man übergibt ein Objekt mit, dass die gelesenen Daten übergeben bekommt.

        int a = 0;
        int f = 0;
        String x;

Bennenung: a, f und x sind keine Variablen Namen, die man verwendet.
Deklaration: In Java werden Variablen nicht am Anfang deklariert, sondern da wo sie verwendet werden.

        dbRet = 6;

Return-Codes: Return Codes sollte man man zu 90% vermeiden - Für eine Steuerung von Fehlern gibt es Exceptions.
Scope Return Code ist ein Feld? Ganz ganz schlechter Stil, die Variable ist nur für den Kontext der Methode relevant, also gehört sie auch nur in dieser Methode - wenn überhaupt - definiert.
Magic-Numbers: 6? Was bedeutet das? Insbesondere - wenn man schon Return Codes verwendet - dann müssen die mit statischen Konstanten definiert sein. Was 6 bedeutet, kann man nicht erkennen.

        String[] dbgelesen = new String[999999];

Ein Array mit der Anzahl Einträgen? Weg damit - für sowas gibt es Listen.

        VitalShowController.anzeintraege = 0;

Statische Felder als Übergabe von Daten zu nutzen - gewaltiges Antipattern und hat absolut nichts mehr mit Java zu tun.

            boolean l = true;
            boolean g;
            dbRet = 0;
            String nachname;
            String vorname;
            String geburtsdatum;
            String datum;
            String uhrzeit;
            String systole;
            String diastole;
            String puls;
            String temperatur;
            String groesse;
            String gewicht;
            String bemerkung;
            String spo2;
            String mnachname = "";
            String mvorname = "";
            String mgeburtsdatum = "";

Gleiches wie schon oben:

• Variablen erst definieren wenn man sie braucht
• Variablen sinnvoll benennen
• Was unterscheited geburtsdatum vom mgeburtsdatum

Zusätzlich
Datentypen:Warum ist Geburtsdatum, Uhrzeit ein String und kein Datum/Uhrzeit Datentyp? Warum ist puls, gewicht, etc kein Integer?
Viel zu viel: Bei einer Methode, die deartig viele lokale Variablen braucht, stimmt was nicht. Die Methode macht demnach vermutlich viel zu viele Dinge und gehört in einzelnen Methoden und Klassen aufgeteilt

                while (l)
                    {
                    rs.next();
                    g = rs.isAfterLast();
                    if (g)
                        {
                        l = false;   
                        }
                    else					
                        {

Aus der API-Doc

The next method moves the cursor to the next row, and because it returns false when there are no more rows in the ResultSet object, it can be used in a while loop to iterate through the result set.

Das ganze kann man schreiben als

while(rs.next()) {

So werden aus 10 Zeilen eine Zeile.

                                x = vorname+" "+nachname+" "+geburtsdatum;
// ...
                                    x = vorname+"/"+nachname+"/"+geburtsdatum+"/"+datum+"/"+uhrzeit+"/"+systole+"/"+diastole+"/"+puls+"/"+temperatur+"/"+groesse+"/"+gewicht+"/"+bemerkung+"/"+spo2+"/";

Das ganze hat nichts in der Methode zu suchen, sie soll Daten einlesen und keine fachlichen Operationen machen und Ausgabe Strings bauen. Das gehört wo anders hin.

                            if (fanz > 0)
                                {
                                f = 0;   
                                if (nachname.equals(feld2))
                                    {
                                    if (fanz == 1)
                                        {
                                        f = 1;   
                                        }
                                    }
                                if (fanz > 1)
                                    {
                                    if (vorname.equals(feld1))
                                        {
                                        if (fanz == 2)
                                            {
                                            f = 1;   
                                            }
                                        }
                                    }
                                if (fanz > 2)
                                    {
                                    if (geburtsdatum.equals(feld3))
                                        {
                                        if (fanz == 3)
                                            {
                                            f = 1;   
                                            }
                                        }
                                    }
                                if (fanz > 3)
                                    {
                                    if (datum.equals(feld4))
                                        {
                                        if (fanz == 4)
                                            {
                                            f = 1;   
                                            }
                                        }
                                    }
                                if (fanz > 4)
                                    {
                                    if (uhrzeit.equals(feld5))
                                        {
                                        f = 1;   
                                        }
                                    }
                                if (f == 1)
                                    {
                                    x = vorname+"/"+nachname+"/"+geburtsdatum+"/"+datum+"/"+uhrzeit+"/"+systole+"/"+diastole+"/"+puls+"/"+temperatur+"/"+groesse+"/"+gewicht+"/"+bemerkung+"/"+spo2+"/";
                                    dbgelesen[VitalShowController.anzeintraege] = x;
                                    VitalShowController.anzeintraege++;
                                    }
                                }
                            else
                                {
                            x = vorname+"/"+nachname+"/"+geburtsdatum+"/"+datum+"/"+uhrzeit+"/"+systole+"/"+diastole+"/"+puls+"/"+temperatur+"/"+groesse+"/"+gewicht+"/"+bemerkung+"/"+spo2+"/";
                            dbgelesen[VitalShowController.anzeintraege] = x;
                            VitalShowController.anzeintraege++;
                                }
                            }

Kurzfassung: Hä? So eine if-Kaskade kann nie richtig sein. Das geht entweder einfacher oder muss in Methoden unterteilt werden. Mir ist da auch nicht klar, was da passieren soll. Es scheint ein Filter zu sein - da gehört dann aber auf SQL Ebene gemacht und nicht über ein deartiges Konstrukt (Bei dem ich nicht glaube, dass es überhaupt funktioniert)

            catch(SQLException err)
                {
                dbRet = 5;   
                }

Exceptions sollte man nach Möglichkeit zur Programmflusssteuerung nutzen und nicht Return Codes. Zudem sollte man nie Exceptions wie hier verschlucken, sondern mindestens ausgeben oder protokollieren.


Das erstaml direkt zum Code. Dann zum generellen Stil. Da muss man leider sagen, dass ist zwar synaktisch Java Code, aber das wars auch schon. Ansonsten hat das nichts mit Objektorienterter Programmierung in Java zu tun und ist in der Form unwartbar. Grundsätzlich muss folgendes gelten:

• Eine Methode macht genau eine Sache. Entweder Daten lesen oder Strings zusammenzubauen oder ähnliches - aber nicht beides
• Daten werden nicht in zig Variablen gehalten (Vorname, nachname etc.) sondern in Objekten
• Eine Methode darf keine Objekte/Daten verändern, die ihr nicht übergeben werden. Sie darf nur auf Objekten arbeiten, die ihr übergeben werden. Ihre Ergebnisse gibt sie am besten als Rückgabe zurück.

Wie kann man es besser machen?
Viele mehr aufteilen.
Für die Daten eine eigene Klasse:

package de.limdul.javaforum;

import java.math.BigDecimal;
import java.time.LocalDate;
import java.time.LocalTime;

public class MedizinDaten {

    private String vorname;
    private String nachname;
    private LocalDate geburtsdatum;
    private LocalDate datum;
    private LocalTime uhrzeit;
    private Integer systole;
    private Integer diastole;
    private Integer puls;
    private Integer groesseCm;
    private String gewichtKg;
    private String bemerkung;
    private String spo2;
    private BigDecimal temperatur;

    public String getVorname() {
        return vorname;
    }

    public void setVorname(String vorname) {
        this.vorname = vorname;
    }

    public String getNachname() {
        return nachname;
    }

    public void setNachname(String nachname) {
        this.nachname = nachname;
    }

    public LocalDate getGeburtsdatum() {
        return geburtsdatum;
    }

    public void setGeburtsdatum(LocalDate geburtsdatum) {
        this.geburtsdatum = geburtsdatum;
    }

    public LocalDate getDatum() {
        return datum;
    }

    public void setDatum(LocalDate datum) {
        this.datum = datum;
    }

    public LocalTime getUhrzeit() {
        return uhrzeit;
    }

    public void setUhrzeit(LocalTime uhrzeit) {
        this.uhrzeit = uhrzeit;
    }

    public Integer getSystole() {
        return systole;
    }

    public void setSystole(Integer systole) {
        this.systole = systole;
    }

    public Integer getDiastole() {
        return diastole;
    }

    public void setDiastole(Integer diastole) {
        this.diastole = diastole;
    }

    public Integer getPuls() {
        return puls;
    }

    public void setPuls(Integer puls) {
        this.puls = puls;
    }

    public Integer getGroesseCm() {
        return groesseCm;
    }

    public void setGroesseCm(Integer groesseCm) {
        this.groesseCm = groesseCm;
    }

    public String getGewichtKg() {
        return gewichtKg;
    }

    public void setGewichtKg(String gewichtKg) {
        this.gewichtKg = gewichtKg;
    }

    public String getBemerkung() {
        return bemerkung;
    }

    public void setBemerkung(String bemerkung) {
        this.bemerkung = bemerkung;
    }

    public String getSpo2() {
        return spo2;
    }

    public void setSpo2(String spo2) {
        this.spo2 = spo2;
    }

    public BigDecimal getTemperatur() {
        return temperatur;
    }

    public void setTemperatur(BigDecimal temperatur) {
        this.temperatur = temperatur;
    }

    public String getKurzform() {
        return vorname + " " + nachname + " " + geburtsdatum;
    }

    public String getLangform() {
        return vorname + "/" + nachname + "/" + geburtsdatum + "/" + datum + "/" + uhrzeit + "/" + systole + "/" + diastole + "/" + puls + "/" + temperatur + "/" + groesseCm + "/" + gewichtKg + "/" + bemerkung + "/" + spo2 + "/";

    }
}

Diese Klasse hat zwei Methoden - getKurzform & getLangform. Die sind dafür zuständig, eine Kurz- oder Langform darzustellen. (Evtl. gehört das nicht mal hier hin, sondern in eine UI-Klasse)

Die Methode, die die Daten ausliest - tut auch nur noch das, die Daten auslesen & zurückzugeben. Da ich die if-Kaskade nicht verstanden habe im Code, hab ich mal vorgesehen, dass man von außen noch zusätzliche Filter übergeben kann, die entscheiden, ob ein Datensatz ins Ergebnis kommt.

Die Filter:

package de.limdul.javaforum;

/** Filtert Medizindaten, dass sie nicht in der Ergebnissliste stehen sollen
 */
@FunctionalInterface
public interface MedizinDatenFilter {

    public boolean isRelevant(MedizinDaten medizinDaten);
}

Das Lesen aus der Datenbank:

package de.limdul.javaforum;

import java.sql.Connection;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.time.LocalDate;
import java.util.ArrayList;
import java.util.List;

public class DbConnector {

    private static final String SELECT_MEDIZIN_DATEN = "SELECT * FROM TABELLE WHERE Vorname = ? ORDER BY Nachname, Vorname, Geburtsdatum, Datum DESC, Uhrzeit DESC";

    public List<MedizinDaten> leseDaten(String vornameFilter, List<MedizinDatenFilter> zusaetzlicheFilter) {
        try {
            Connection connection = getDbConnection();
            PreparedStatement preparedStatement = connection.prepareStatement(SELECT_MEDIZIN_DATEN);
            preparedStatement.setString(1, vornameFilter);
            ResultSet resultSet = preparedStatement.executeQuery();
            List<MedizinDaten> results = new ArrayList<>();
            while (resultSet.next()) {
                MedizinDaten medizinDaten = createMedizinDaten(resultSet);
                if (zusaetzlicheFilter.stream().allMatch(filter->filter.isRelevant(medizinDaten))) {
                    results.add(medizinDaten);
                }
            }
            return results;
        } catch (SQLException e) {
            e.printStackTrace();
            throw new RuntimeException(e);
        }
    }

    private MedizinDaten createMedizinDaten(ResultSet resultSet) throws SQLException {
        MedizinDaten medizinDaten = new MedizinDaten();
        medizinDaten.setNachname(resultSet.getString("nachname"));
        medizinDaten.setVorname(resultSet.getString("vorname"));
        medizinDaten.setDatum(resultSet.getDate("datum").toLocalDate());
        medizinDaten.setPuls(resultSet.getInt("puls"));
        // usw.

        return medizinDaten;
    }

    private Connection getDbConnection() {
        return null;
    }
}

Das ganze ist mit Sicherheit auch noch nicht komplett - aber schon mal deutlich strukturierter.

 

[Oneixee5]

Was man auch nicht tun sollte, ist es java.sql.Connection über Methoden hinweg zu übergeben bzw. während der ganzen Programmlaufzeit zu halten. Der Compiler sollte das Vorgehen auch mit entsprechenden Warnungen quittieren. Da das Erzeugen von Connection's sehr teuer ist und lange dauern kann, empfiehlt es sich deshalb DataSource's zu verwenden.
Falsch:

private Connection getDbConnection() {
    return ...;
}

Besser:

        private static final String DB_URL = "jdbc:h2:mem:test;DB_CLOSE_DELAY=-1";
        private static final String DB_USER = "sa";
        private static final String DB_PASSWORD = "";
 
        private JdbcDataSource dataSource;
 
        public DataSource getDataSource() {
            if (dataSource == null) {
                dataSource = new JdbcDataSource();
                dataSource.setURL(DB_URL);
                dataSource.setUser(DB_USER);
                dataSource.setPassword(DB_PASSWORD);
            }
            return dataSource;
        }

In dem Beispiel geht es nicht um MySQL sondern H2, das müsste entsprechend angepasst werde.

Verwendung:

Collection<MedizinDaten> queryMedizinDaten(final String vornameFilter, ...) throws SQLException {
        final Set<MedizinDaten> records = new LinkedHashSet<>();
        final String sql = "SELECT ...";
        try (
            Connection connection = getDataSource().getConnection();
            PreparedStatement statement = connection.prepareStatement(sql);
        ) {
            statement.setString(1, vornameFilter);
            ...
        
            try (ResultSet resultSet = statement.executeQuery()) {
                while (resultSet.next()) {
                    
                    records.add(new MedizinDaten(...));

                 }      
            }
        }
        return records;
    }

In dem Beispiel von @LimDul ist leider ein grundsätzlicher Fehler enthalten - was vermutlich der Vereinfachung als Beispiel geschuldet ist. Alles was mit JDBC/Datenbankverbindungen zu tun hat, sind native Ressourcen, welche nach der Nutzung freigegeben werden müssen. Das wurde dort komplett weggelassen. Es sollten try-with-ressources Statements verwendet werden: https://docs.oracle.com/javase/tutorial/essential/exceptions/tryResourceClose.html