resultSet.next() gibt false zurück, obwohl die Tabelle ausgefüllt ist

Question

Ich habe ein paar Funktionen, die beim Abrufen der Objekte aus der Datenbank helfen.

public User getUser(int beamID) throws NoSuchUserException { 
    return userFromResultSet(getUserResultSet(beamID)); 
} 


private ResultSet getUserResultSet(int beamID) { 
    try(Connection conn = dataSource.getConnection()) { 

     // queries.getUserByBeamID() returns "SELECT * FROM user WHERE beamID=?" 
     PreparedStatement stmt = conn.prepareStatement(queries.getUserByBeamID()); 

     stmt.setInt(1, beamID); 
     System.out.println(stmt.toString()); 
     return stmt.executeQuery(); 

    } catch (SQLException e) { 
     e.printStackTrace(); 
     throw new IllegalStateException(); 
    } 
} 

private User userFromResultSet(ResultSet resultSet) { 
    try { 
     boolean next = resultSet.next(); // Debugger tells me this is false. 
     if (!next) 
      throw new NoSuchUserException(); 

     User user = new User(this, 
      resultSet.getInt("beamID"), 
      resultSet.getString("name"), 
      resultSet.getInt("points"), 
      resultSet.getInt("time") 
     ); 

     if (resultSet.next()) 
      throw new IllegalStateException("Duplicate user entries exist - database integrity compromised!"); 

     return user; 
    } catch (SQLException e) { 
     e.printStackTrace(); 
     throw new IllegalStateException(); 
    } 
} 

Das Merkwürdige ist, dass ich die Daten wissen nicht bestehen aus zwei Gründen:

• Mein Programm versucht, den Eintrag zu erstellen, wenn es nicht existiert, aber versucht, das gibt ein Fehler, dass die eindeutige Bedingung nicht befolgt wird.

• Ausführen der Abfrage in meinem SQLite DB-Browser funktioniert gut:

ich sehr bezweifle, dass dies ein Problem mit unbestätigten Daten, da dies eine dateibasierte ist Datenbank, und das Öffnen dieser Datei mit einem Texteditor zeigt Instanzen der Benutzernamen in den Daten.

Answer 1

Schauen Sie genau auf, was Sie tun hier:

try (Connection conn = dataSource.getConnection()) { 
    PreparedStatement stmt = conn.prepareStatement(queries.getUserByBeamID()); 

    stmt.setInt(1, beamID); 
    System.out.println(stmt.toString()); 
    return stmt.executeQuery(); 
} catch (SQLException e) { 
    e.printStackTrace(); 
    throw new IllegalStateException(); 
} 

Ich glaube, es der Vertrag von Try-mit-Ressourcen ist in der try Klausel zu gewährleisten angegebene Ressource zu schließen, nachdem der Ausdruck Ausführung beendet. Ich glaube, dass die Ergebnismenge auch am Ende des try Blocks geschlossen wird, daher ruft next() false zurück, weil nichts da ist.

So wie ich den Code geschrieben haben würde, ist die User POJO innerhalb des try Blockes zu füllen, und gibt ein User Objekt stattdessen eine Ergebnismenge zurückzugeben:

private User getUserResultSet(int beamID) { 
    User user = null; 
    try (Connection conn = dataSource.getConnection()) { 
     PreparedStatement stmt = conn.prepareStatement(queries.getUserByBeamID()); 

     stmt.setInt(1, beamID); 

     ResultSet rs = stmt.executeQuery(); 
     user = new User(this, 
      rs.getInt("beamID"), 
      rs.getString("name"), 
      rs.getInt("points"), 
      rs.getInt("time") 
     ); 

    } catch (SQLException e) { 
     e.printStackTrace(); 
     throw new IllegalStateException(); 
    } 

    return user; 
} 

jetzt Ihre Trennung von Bedenken ist besser als Vor. Wenn etwas mit der Verbindung, der Ergebnismenge usw. schief geht, wird es im eigentlichen Code behandelt, der sich mit diesen Dingen beschäftigt. Im Falle einer Ausnahme oder eines anderen Fehlers wird ein null Benutzerobjekt zurückgegeben, und Sie sollten Ihren Code aktualisieren, um diese Möglichkeit zu umgehen.