Discussion :

[ratata]

Bonjour,

voila j'ai un problème au niveau de l'exécution de la fonction en dessous, vous allez remarquez au niveau du code que j'ai bien mis un Result.next(), il existe bien (la ligne en bleu ), le problème c'est qu'il m'affiche l'exception suivante:

Exception in thread "main" org.postgresql.util.PSQLException: Le ResultSet n'est pas positionné correctement, vous devez peut-être appeler next().

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
public User getUserById(Integer id_user) throws ClassNotFoundException, SQLException
    {
            Class.forName("org.postgresql.Driver");
            Connection conn=DriverManager.getConnection("jdbc:postgresql://localhost:5432/Billing2","postgres","reda/23");
            PreparedStatement St=null;

            User user=new User();

            String sqlquery="select id_user,deleted from User where id_user=?";
            St=conn.prepareStatement(sqlquery);
            St.setInt(1,id_user);

            ResultSet Result=St.executeQuery();

            Result.next();
          
            
            user.setDeleted(Result.getBoolean(2));
            user.setIdUser(id_user);
            

            return user;
    }

Merci d'avance et excellente journée.

[Logan Mauzaize]

La méthode next renvoie un booléen qui indique s'il y a un résultat. Ca serait bien de le tester

[ratata]

Merci pour votre réponse Nemek,

effectivement, ca me retourne bien un true et apparemment la fonction est intacte , c'est du au fait que je l'appel différemment deux fois successivement, j'ai fait deux code de test, comme il figure au niveau du code en dessous quand je rajoute un deuxième appel(ligne en bleu) ca crache et ca me retourne l'exception que j'ai déjà posté:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
for(int i=0;i<10;i++)
               {
                   User user= fonctionClient.getUserById(i);
                   User user2=fonctionClient.getUserById(i+1);
                   System.out.println(user.getDeleted());
                   System.out.println(user2.getDeleted());
               }

bonne journée.

[tchize_]

le resultset, le statement et la connexion doivent tous être fermés (méthode close() ) avant de faire le return, de préférence dans un bloc finally.

Ensuite, comme vous travaillez à chaque fois sur une connexion différente, il n'y a pas interaction possible. Je pense plutot que next() retourne false pour le cas id+1.

[adiGuba]

Salut,


Quelques remarques :

• Il est inutile de charger la classe du driver JDBC à chaque fois. Tu peux le faire une seule fois au début du programme.
  (et au passage avec les drivers récents cela devient carrément inutile)
• Il serait préférable d'éviter d'ouvrir une nouvelle connexion à chaque requete.
• Il serait également agréable de respecter les conventions de nommages...
• Enfin il FAUT libérer proprement les ressources (Connection, PreparedStatement et ResultSet)

a++

[Logan Mauzaize]

(et au passage avec les drivers récents cela devient carrément inutile)

Il y a un système de services pour les drivers ?

Sinon on peut utiliser la propriété système "jdbc.drivers"

[adiGuba]

Il y a un système de services pour les drivers ?

Oui depuis Java 6 : http://download.oracle.com/javase/6/...erManager.html



@ratata : c'est résolu mais n'oublie pas de prendre en compte toutes les remarques qu'on t'as donné...


a++

[ratata]

Bonjour,

aveugle que je suis , j'ai laissé trainé un 0 au début de la boucle, et même pour mon autre code(il n'y a pas d'identifiant 0), c'est bon c'est résolu, merci pour vos réponses, et excuses.

bonne journée.