SQLite ResultSet closed -- Wiederholende DB-Abfragen

[Jochim]

Guten Abend,

ich arbeite mit einem RFID-Kartenleser und immer wenn eine Karte gelesen wird, wird eine DB-Abfrage nach der ID des Produktes gemacht.

public String getID(String rfid){
        conn=JavaConnection.ConnecrDB();
        String ID;
        System.out.println("Methode: searchRFID");
        String sql="";
        try{
            sql="SELECT ID,RFID FROM products WHERE RFID=?";
            pst=conn.prepareStatement(sql);
            pst.setString(1,rfid);
            rs=pst.executeQuery();
            ID=rs.getString("ID");
            return ID;
        }catch(Exception e){
            JOptionPane.showMessageDialog(null, e, sql+" --> leads to exception", 1);
            return null;
        }finally{
            try{
                conn.close();
                pst.close();
                rs.close();
            }catch(Exception e){
                JOptionPane.showMessageDialog(null, e, "finally exception", 1);
            }
        }
    }

Da die Karten öfters gelesen werden kommt es zu wiederholenden Abfragen, doch obwohl ich die Verbindung immer neu aufbaue bekomme ich eine Exception: ResultSet ist closed;

Wie vermeide ich diesen Fehler?

Freundliche Grüße Jochim

 

[Neumi5694]

Du schließt das Resultset nach der Connection, die Wahrscheinlichkeit ist hoch, dass das Resultset dann schon zu ist.
Dreh die Reihenfolge der close()-Befehle um, fang immer mit dem Kleinsten an.

Oder aber, du machst gleich Nägel mit Köpfen und verwendest try-with-resources, da sparst du dir das Schließen der Objekte.

    public String getID(String rfid){
        System.out.println("Methode: searchRFID");
        String sql="SELECT ID,RFID FROM products WHERE RFID=?";
        String ID;
        try (Connection conn=JavaConnection.ConnecrDB()) {
            try(PreparedStatement pst = conn.prepareStatement(sql)) {
                pst.setString(1,rfid);
                try (ResultSet rs=pst.executeQuery()) {
                    ID = rs.getString(1);
                }
            }
        } catch (SQLException e) {
            JOptionPane.showMessageDialog(null, e, sql+" --> leads to exception", 1);
            ID = null;
        }
        return ID;
    }

 

[klauskarambulut]

Lerne lieber mal schnellstens den Unterschied zwischen Lokalen Variablen und Instanz Variablen und statischen Variablen kennen. Das Zeug was den wenigsten Ärger macht sind lokale Variablen.

Danach auch mal einen Blick zu Try-with-Resource werfen, wie Neumi das schon so schön vorschlägt, dann spart man sich das ganze geclose.

try(Connection conn=JavaConnection.ConnecrDB()){
           try(PreparedStatement pst = conn.prepareStatement(sql)){

Kann man hier allerdings auch in einen Block verfrachten.

try(Connection conn=JavaConnection.ConnecrDB();
           PreparedStatement pst = conn.prepareStatement(sql)){

Da beides conn und pst Autoclosable ist.

Darüber hinaus sollte man den Mist lassen Swing in Form von JOptionPane und JDBC in einer Methode zu verwenden.
Das ganze auftrennen und ordentlich strukturieren. Swing und JDBC sollten nicht mal im selben Package importiert werden.

Und ausserdem muss nach einem executeQuery() zwingend ein next() folgen, der den Cursor des Ergebnisses auf die erste Ergebniszeile setzt aufgerufen werden.

try(ResultSet rs=pst.executeQuery()){
  if(rs.next()) {
    String ID = rs.getString(1); //Lokale Variable mit möglichst kleinem Scope
    return ID;
  } else {
    return null; //weil nix gefunden, oder fachliche Exception schmeißen.
  }
}
...Fehlerbehandlung, null zurückgeben oder fachliche Exception schmeißen.

 

[Neumi5694]

In meinem Beispiel fehlt natürlich

if (rs.next())

, was vor dem Auslesen stehen muss. (Im Falle mehrer Ergebnisse ist

while (rs.next)

zu verwenden)
Das kommt davon, wenn man davon ausgeht, dass es im ersten Beispiel richtig war 🙂

 

[Jochim]

Guten Abend,
danke für die Antworten. Ich habe nun die Try-with-Resources eingebaut und es funktioniert.
Ich habe einfach das Problem, dass in der Schule noch nach alten Methoden gelehrt wird.

Noch einmal meine fertige Methode:

public String getID(String rfid){
        System.out.println("Methode: searchRFID: "+rfid);
        String sql="SELECT ID,RFID FROM products WHERE RFID=?";
        try(Connection conn=JavaConnection.ConnecrDB();
           PreparedStatement pst = conn.prepareStatement(sql)){
                pst.setString(1,rfid);
                rs=pst.executeQuery();
                if(rs.next()) {
                    String ID = rs.getString(1); //Lokale Variable mit möglichst kleinem Scope
                    System.out.println("id: "+ID);
                    return ID;
                } else {
                    System.out.println("id: null");
                    return ""; //weil nix gefunden, oder fachliche Exception schmeißen.
                }
        } catch (SQLException e) {
            e.printStackTrace();
            return null;
        }
    }

Freundliche Grüße

Damien Jochim.

 

[Neumi5694]

Die Methode aus der Schule funktioniert genau so gut, ist zum Lernen vielleicht noch besser.
Die try-with-Methode erspart dir nur etwas Tipparbeit, der compilierte Code funktioniert genau so, als ob du nacheinander die Objekte manuell schließen würdest. Bei dir war letztendlich nur die Reihenfolge falsch (arbeite nach dem LIFO-Prinzip, Zugriffe in der verkehrten Reihenfolge schließen, wie du sie geöffnet hast.), try-with ist dann halt noch etwas eleganter.

ps: Deine catch-Klausel kannst du noch etwas sinnvoller gestalten.
1. Nutze als Ausgabe System.err und nicht den Default (System.out)
2. Gib vor dem Stacktrace auch getMessage() der Exception aus

Besser noch: Verwende einen Logger anstatt der manuellen Ausgabe nach System.err

 

[Joose]

Wie oben schon gesagt erspart try-with Schreibarbeit.
Ich finde es gut ihr die normale try/catch/finally Variante gelehrt bekommt. Dabei lernt man eben auch das Ressourcen geschlossen werden sollten und es kommt war das man nicht mit try-with arbeiten kann.