Discussion :

[francky74]

Bonjour,

Je rencontre un problème avec un comboBox. Je n'arrive pas à écrire la commande "addItem" que l'on écrit normalement après le point : exemple : maComboBox.addItem(strVilleClient);

La liste de cette (ce ?) comboBox doit être remplie à partir d'une base de données que j'ai crée en PreparedStatement et qui (normalement) fonctionne.

Dans le code, j'ai inséré plusieur system.out qui renvoie du texte, ce qui me prouve que les lignes sont bien lues. Et justement, je vois que ma comboBox1 n'est pas lue car le system.out ne s'affiche pas. Pourtant, cette comboBox est dans une simple boucle conditionnelle "WHILE" ....
La comboBox est déclarée dans l'autre classe d'où j'apelle la classe TestBdd.

voici le code :

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
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
//import com.mysql.jdbc.PreparedStatement;
import java.sql.PreparedStatement;
import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.ResultSet;
import java.sql.SQLException;
 
public class TestBdd {
    TestBdd(Object comboClients1)
    {
 
        // Variables relatives à la connexion à la base de données
        String url = "jdbc:mysql://localhost:3306/BDD";
        String utilisateur = "strUser";
        String motDePasse = "pwd";
        //Connection connexion = null;
 
        //chargement du pilote
        try
        {
            Class.forName("com.mysql.jdbc.Driver");
            //Class.forName("mysql-connector-java-5.1.29-bin.jar");
            System.out.println("chargement pilote testBdd");
        }
        catch (ClassNotFoundException e)
        {
            System.out.println("erreur ClassNotFoundException e dans testBdd");
        }
 
        //Connection à la base de données
        try
        {
            System.out.println("connexion testBdd");
            Connection connexion = DriverManager.getConnection( url, utilisateur, motDePasse );
        }
        catch (SQLException e)
        {
            System.out.println("SQLException erreur testBdd");
        }
 
        try
        {
            Connection connexion = DriverManager.getConnection( url, utilisateur, motDePasse );
            String sql = "SELECT * FROM client WHERE ville = '?'";
            PreparedStatement prep1 = connexion.prepareStatement(sql);
            ResultSet rs = prep1.executeQuery(sql);
            while (rs.next())
            {
                String villeClient = rs.getString("ville");
                comboClients1.addItem(villeClient);
            }
            System.out.println(prep1 + "dans testBdd");
        }
        catch (SQLException e)
        {
            e.printStackTrace();
            System.out.println("erreur SQLException requete dans testBDD");
        }
 
        /*finally
        {
            if ( connexion != null )
                try 
                {
                    // Fermeture de la connexion
                    System.out.println("fermeture connexion dans RecBdd !?!");
                    connexion.close();
                }
                catch ( SQLException ignore )
                {
                    // Si une erreur survient lors de la fermeture, il suffit de l'ignorer.
                }
        }*/
 
    }
 
}

J'apelle la classe TestBdd depuis la commande suivante :

Code :

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

1
2
3
4
5
6
7
8
9
 
import javax.swing.JComboBox;
....
JComboBox comboClients1;
.....
comboClients1 = new JComboBox();
.....
 
TestBdd testBdd = new TestBdd(comboClients1);

Voici le texte qui s'affiche dans la fenêtre de sortie de NetBeans :

Code :

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

1
2
3
4
5
chargement pilote testBdd
connexion testBdd
com.mysql.jdbc.JDBC4PreparedStatement@15fff9c: SELECT * FROM client WHERE ville = '?'dans testBdd
chargement pilote commande afficher
connexion BDD commande afficher

Merci pour vos aides !!!

[Alkhan]

bonjour,

Le problème est simple ta requête n'est pas correct, il ne faut pas les cotes autour du "?" !

Code :

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

String sql = "SELECT * FROM client WHERE ville = ?";

Et si tu executes ça directement tu auras une erreur car tu ne renseignes pas le critère "ville".
Tu dois donc ajouter :

Code :

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

1
2
PreparedStatement prep1 = connexion.prepareStatement(sql);
prep1.setString(1, "maVille");

D'autre part, vue que la requête est déclaré dans le PreparedStatement, il est aucunement utile de la mettre dans le executeQuery :

Code :

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

1
2
 
ResultSet rs = prep1.executeQuery();

[francky74]

Bonjour,

Merci beaucoup pour cette réponse très argumentée !

Il est vrai qu'en mettant le ? sans les quotes j'avais une erreur et c'est ce qui m'a fait mettre les quotes ..... Je n'avais plus d'erreurs mais c'était quand même une erreur de ne plus avoir d'erreur ... donc merci de m'avoir bien éclairé

J'ai suivi tous vos conseils, et NetBeans ne me remonte plus d'erreur, toutefois, la commande WHILE n'est toujours pas exécutée.

Voici ce qu'est devenu la partie modifiée du programme :

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
        try
        {
            Connection connexion = DriverManager.getConnection( url, utilisateur, motDePasse );
            String sql = "SELECT * FROM client WHERE ville = ?";
            PreparedStatement prep1 = connexion.prepareStatement(sql);
            prep1.setString(1, "ville");
            ResultSet rs = prep1.executeQuery();
            while (rs.next())
            {
                String villeClient = rs.getString("ville");
                comboClients1.addItem(villeClient);
                System.out.println(villeClient + " / ");
            }
            System.out.println(prep1 + "dans testBdd");
        }
        catch (SQLException e)
        {
            e.printStackTrace();
            System.out.println("erreur SQLException requete dans testBDD");
        }

Et voici le texte de la fenêtre de sortie de NetBeans :

Code :

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

1
2
3
4
chargement pilote testBdd
connexion testBdd
com.mysql.jdbc.JDBC4PreparedStatement@11730ce: SELECT * FROM client WHERE ville = 'ville'dans testBdd
BUILD SUCCESSFUL (total time: 42 seconds)

Merci beaucoup !

[Alkhan]

Ca ne passe pas dans le while tout simplement parce qu'il n'existe pas en base de ville qui s'appelle "ville";
dans le setString il faut mettre la ville recherché, c'est le paramètre de ta requête.
Par exemple :

Code :

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

prep1.setString(1, "paris");

ou si la ville se trouve dans une variable :

Code :

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

prep1.setString(1, maVariableQuiContientLaVille);

[francky74]

Bonsoir,

Merci pour ces infos.

En fait, je souhaite faire lister dans le comboBox tous les noms de villes présents dans le tableau (sans répétitions : DISTINCT). Est-ce que le fait d'avoir apeller une classe nouvelle (testBdd) pour faire ce traitement ne gênerait pas le bon déroulement du code ?

De plus, on peut voir dans le code que j'ai écris ceci :

Code :

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

comboClients1.addItem(ville);

Mais en fait, NetBeans n'accepte même la mention "addItem" ! et je ne comprends pas non plus.

{:o/

[francky74]

Bonsoir,

J'ai modifié la requête comme ceci mais le code me donne encore du fil retordre ...

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
        try
        {
            Connection connexion = DriverManager.getConnection( url, utilisateur, motDePasse );
            String sql = "SELECT DISTINCT ? FROM client";
            PreparedStatement prep1 = connexion.prepareStatement(sql);
            prep1.setString(1, "ville");
            ResultSet rs = prep1.executeQuery();
            while (rs.next())
            {
                String villeClient = rs.getString("ville");
                //comboClients1.addItem(villeClient);
                System.out.println(villeClient + " / ");
            }
            System.out.println(prep1 + "dans testBdd");
        }

[francky74]

Re bonsoir,

J'ai encore avancé dans mes recherches de la bonne requête et j'ai trouvé ce qu'il me faut comme indiqué ci-dessous, mais il me reste mon problème du comboBox1 qui n'accepte pas dans NetBeans que j'écrive "addItem" mais il me propose "equals(object)", "getClass()", "hashCode()" ..etc :

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
 
        try
        {
            Connection connexion = DriverManager.getConnection( url, utilisateur, motDePasse );
            String sql = "SELECT DISTINCT ville FROM client";
            PreparedStatement prep1 = connexion.prepareStatement(sql);
            ResultSet rs = prep1.executeQuery();
            while (rs.next())
            {
                String villeClient = rs.getString("ville");
                comboClients1 // ici NETBeans n'accepte pas l'écriture de "addItem" mais il me propose "equals(object)", "getClass()", "hashCode()"
                System.out.println(villeClient + " / ");
            }
            System.out.println(prep1 + "dans testBdd");
        }

Et voici le texte dans la fenêtre de sortie :

Code :

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

1
2
3
4
5
6
7
8
9
chargement pilote testBdd
connexion testBdd
ville1 / 
ville2 / 
ville3 / 
ville4 / 
ville5 / 
com.mysql.jdbc.JDBC4PreparedStatement@4e8204: SELECT DISTINCT ville FROM clientdans testBdd
BUILD SUCCESSFUL (total time: 18 seconds)

Merci !

[jeffray03]

Salut
ton Constructeur a comme Argument un Objet

Code :

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

TestBdd(Object comboClients1)

donc quand tu lui fais:

Code :

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

comboClients1.addItemm()

c´est normal qu´il ne l´accepte pas.
ce que je proposerais, c´est que tu lui fasses savoir que l´objet en question est du type:

Code :

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

JComboBox

en faisant:

Code :

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

1
2
3
4
5
6
 
  if(comboClients1 instanceOf(JComboBox)
  {
     // donc
     comboClients1.addItem(villeClient);     
  }

Eric

[joel.drigo]

Salut,

Pourquoi ne pas plutôt changer le type de l'argument du constructeur, au lieu de tester la classe et de caster (parce qu'il faudrait caster, @Jeffray03, pour que ton code fonctionne, et virer la parenthèse ouvrante en trop) ?

Code :

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

TestBdd(JComboBox comboClients1)

[francky74]

Bonjour à tous,

C'était bien le type de l'argument du constructeur qui faisait défaut !
Tout fonctionne à merveille à présent et je vous en remercie tous.

Grâce à vous on peux évoluer dans notre compréhension des languages informatiques