resultSet.next() возвращает false, даже если таблица заполнена

Question

У меня есть несколько функций, которые помогают с извлечением объектов из базы данных.

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(); 
    } 
} 

Странная вещь, я знаю, что данные действительно существует по двум причинам:

• Моя программа пытается создать запись, если она не существует, но попытка, что дает что единственное ограничение не соблюдается.

• Выполнение запроса в моей БД SQLite браузере работает просто отлично:

Я очень сомневаюсь, что это проблема с неподтвержденными данными, так как это база данных на основе файлов, и открытие этого файла текстовым редактором показывает экземпляры имен пользователей в данных.

Answer 1

Посмотрите внимательно на то, что вы здесь делаете:

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(); 
} 

Я считаю, что это договор примерки с-ресурсами, чтобы гарантировать закрытие ресурса, указанного в пункте try, после того, как выражение заканчивает выполнение. Я считаю, что набор результатов также закрывается в конце блока try, поэтому вызов next() возвращает false, потому что ничего не существует.

Как я написал бы ваш код для заполнения User POJO внутри try блока, и возвращает User объекта вместо возвращения результирующего набора:

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; 
} 

Теперь ваше разделение задач лучше, чем до. Если что-то пойдет не так с соединением, набором результатов и т. Д., Оно обрабатывается в фактическом коде, который имеет дело с этими вещами. В случае исключения или другой ошибки пользовательский объект null будет возвращен, и вы должны обновить свой код, чтобы обработать эту возможность.