Exception ResultSet - avant le démarrage

changez pStatement.executeQuery (sql) par pStatement.executeQuery ()

userID est également int code > donc pas besoin d'utiliser le guillemet simple ' dans la requête (mais ce n'est pas la racine de l' Exception)

Je suggère d'utiliser un if instruction sur Rset.next () (qui incrémente le curseur, pas besoin d'utiliser beforeFirst () ) cela peut provoquer des exceptions lors de l'obtention des valeurs si la requête ne renvoie rien.

Utilisez le code comme suit

public static User getUser (User user) throws Exception {
        String sql = "SELECT * FROM login WHERE userID = " + user.getuserID();
        //is FROM Correct?
        PreparedStatement pStatement = conn.prepareStatement(sql);
        ResultSet Rset = pStatement.executeQuery();
        if(Rset.next())
        {
             int userID = Rset.getInt("userID");
             String email = Rset.getString("email");
             User userReturn = new User (userID, email);
             return userReturn;
        }
        return null;

    }

Eh bien, votre JDBC / SQL pourrait utiliser certaines mises à jour.

Mais tout d'abord. Voici ce qui se passe:

public static User getUser (User user) throws Exception {
    String sql = "SELECT * FROM login WHERE userID = ?;
    try(PreparedStatement pStatement = conn.prepareStatement(sql)) {
      pStatement.setInt(1, user.getuserID());
      try(ResultSet Rset = pStatement.executeQuery()) {  
        if(Rset.next()) {
          int userID = Rset.getInt("userID");
          String email = Rset.getString("email");
          User userReturn = new User (userID, email);
          return userReturn;
        }
      }
    } catch(SQLException e) {
      //Deal with the exception
      e.printStackTrace();
    }
    //Error or no user found
    return null;
}

La deuxième ligne lance une exception car le curseur est FORWARD_ONLY. Plus: Vous n'avez pas du tout besoin de cet appel!

Cela devrait résoudre votre problème immédiat:

    ResultSet Rset = pStatement.executeQuery(sql);
    if(Rset.next()) {
      int userID = Rset.getInt("userID");
      String email = Rset.getString("email");
      User userReturn = new User (userID, email);
      return userReturn;
    } else {
      //Deal with no user found
    }

Maintenant, pour le truc général:

• il est bon que vous prépariez la déclaration. Mais comme Bashir l'a déjà souligné: si vous utilisez executeQuery (sql), qui est membre de la classe Statement, vous jetez l'instruction précompilée.
• lors de l'utilisation de PreparedStatement - utilisez également les paramètres de liaison. Cela rend votre code moins fréquent pour l'injection SQL et
• utilisez try-with-resource pour vous assurer de tout fermer correctement. Traquer les fuites de connexion ou les curseurs ouverts est un problème facilement évité.

À la fin, vous vous retrouverez probablement avec quelque chose comme ceci:

    ResultSet Rset = pStatement.executeQuery(sql); //Executes the query
    Rset.beforeFirst(); //Moves the cursor backwards before the first row (!)
    Rset.next(); //Goes to the next row

Lorsque vous créez un ResultSet standard, vous ne pouvez utiliser que la méthode rs.next () pour itérer les lignes résultantes. Donc rs.beforeFirst () n'est pas autorisé. Et ce n'est pas non plus nécessaire dans votre code. Lorsque le ResultSet est créé, le curseur est automatiquement placé avant la première ligne. Donc, utilisez simplement if (rs.next ()) {...} . Lorsque rs.next () renvoie false après le premier appel, cela signifie que le ResultSet est vide.