Discussion :

[Memeche]

Bonjour,

Travaillant principalement autour des bases de données, je n'ai pas fait de programmation depuis quelques années et je voudrais me remettre à Java pour réaliser des prototypes (et me remettre à jour sur ces technos).
J'ai donc commencé par créer une classe toute simple permettant de créer une connexion JDBC vers un DB2 et d’exécuter une requête toute simple.
Tout marche bien mais je suis curieux du warning que m'indique Eclipse et qui dit la chose suivante : "The static method seConnecter(String, String, String) from the type ConnexionJDBC should be accessed in a static way"
Dans mes souvenirs (qui ont peut être étés altérés), il fallait créer des classes et gérer des instances. J'ai donc créé une classe ConnexionJDBC dans laquelle j'ai créé des méthodes seConnecter, seDeconnecter, executerSQL et commiter. Ensuite dans mon autre classe (qui génère les requêtes et gère l'affichage) je fais donc (en simplifiant):

Code :

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

1
2
3
4
5
6
ConnexionJDBC maConnexionJDBC = null;
ResultSet rs;
 
maConnexionJDBC.seConnecter(url, user, password);
rs = maConnexionJDBC.executerSQL(ordreSQL); 
maConnexionJDBC.seDeconnecter();

J'ai donc des Warnings dés que j'essaie d'utiliser mon instance de ConnexionJDBC et Eclipse me propose d'utiliser directement la classe (donc du genre ConnexionJDBC.seConnecter(...)). En effet ça marche également mais j'ai du mal à saisir le pourquoi de ce warning. Si je respecte ce que me dit Eclipse, comment suis-je censé gérer plusieurs connexions ? C'est juste de la curiosité mais si quelqu'un pouvait m'expliquer rapidement ce serait sympa.

merci d'avance

[plawyx]

Cela vient du fait que tu appelles une méthode statique via une instance de la classe.

Tu as déclaré seConnecter(url, user, password) comme une méthode static de la classe ConnexionJDBC. Pour ne plus avoir de warnings, tu as deux solutions:

*) soit utiliser l'appel (méthode de classe)

Code :

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

ConnexionJDBC.seConnecter(url, user, password);

*) soit ne plus déclarer seConnecter(url, user, password) comme static est alors, tu peux l'appliquer sur ton objet maConnexionJDBC :

Code :

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

1
2
ConnexionJDBC maConnexionJDBC = null;
maConnexionJDBC.seConnecter(url, user, password);

[OButterlin]

Code :

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

1
2
3
4
5
6
ConnexionJDBC maConnexionJDBC = null;
ResultSet rs;
 
maConnexionJDBC.seConnecter(url, user, password);
rs = maConnexionJDBC.executerSQL(ordreSQL); 
maConnexionJDBC.seDeconnecter();

D'après ton exemple, il semblerait que ta classe ConnexionJDBC encapsule un objet Connection, du coup, je ne vois pas trop l'intérêt d'une méthode static, dans tous les cas tu auras besoin d'une variable d'instance.

Que rajoutes-tu dans cette classe pour justifier ce mécanisme ? De mon expérience, il est plus judicieux d'avoir une classe utilitaire pour récupérer un objet Connection et laisser la fonction appelante gérer son existence... mais bon, à toi de voir