Discussion :

[tomenbois]

Bonjour, voila j'ai un combobox qui liste tout les pays du monde a partir d'une base de donnée, voici mon bout de code :

Code :

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

1
2
3
4
5
6
7
8
9
private void comboBoxPays_SelectedIndexChanged(object sender, EventArgs e)
        {
            SQLiteCommand requete = SQL_Conn.CreateCommand();
            requete.CommandText = " Select pays From Pays; ";
            SQLiteDataReader nompays = requete.ExecuteReader();
            while (nompays.Read())
            {
                comboBoxPays.Items.Add(nompays["pays"]);
            }

Mais sa marche pas. Je pense que la requete ne s'execute pas, si c'est le cas où dois-je la mettre pour qu'elle s'execute automatiquement a l'ouverture du winform ?

[Bluedeep]

Pour commencer pourquoi le chargement de la combo est sur l'event SelectedIndexChanged ?
Désolé, mais ça n'a aucun sens. Pour que le SelectedIndexChanged soit appelé il fait que la Combo ait un contenu; quand bien même serait ce le cas, j'imagine mal pourquoi tu devrais recharger le contenu quand l'utilisateur sélectionne une entrée.

Sinon, a priori,une table pays ne change pas tous les quatre matins, et je ne vois aucune utilité à la recharger à chaque fois.


Ensuite, déjà mettre

Code :

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

nompays["pays"].ToString()

Puis, à quoi sert le ; en fin de select ????

Quand tu dis que cela ne marche pas, c'est un peu vague. Ca pête ? ça ne retourne rien ? tu ne rentres pas dans ta boucle ? sois un peu plus précis, SVP.

[neilbgr]

Renseigne toi sur les propriétés DataSource, DisplayMember, ValueMember et le DataBinding en général...

[ced600]

Heu si je ne me trompe pas le SelectedIndexChanged c'est pour executer du code à chaque fois que "l'index selectionné a été changé"

Mais la ta combo est vide donc tu ne peux pas la changer et donc ta requête ne s'exécute pas.
De toute façon c'est du code d'initialisation de combobox.

La combobox fonctionne comme une liste, si tu veux lui ajouté des items dès le départ, bah le mieux c'est juste après la création de l'objet en question.

C'est une phase d'initialisation, tu as bien une méthode d'initialisation de tes objets winform quelque part non ?

Edit : Me suis fait devancer par deux personnes

Edit2 : Dites, avec tant de personne qui viennent critiquer son code (à juste titre), lui conseiller de revoir le fonctionnement de base d'une combo, il ne risque pas d'avoir peur ? En même temps on ne peut pas le laisser utiliser un code qui ne fonctionne pas

[tomenbois]

oua j'en prend plein en peu de temps !!!

Désolé, je commence le c#.

Mais j'ai resolu le probléme. Il falait faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
private void Nouveau_Load(object sender, EventArgs e)
        {
            SQLiteCommand requete = SQL_Conn.CreateCommand();
            requete.CommandText = " Select pays From Pays; ";
            SQLiteDataReader nompays = requete.ExecuteReader();
            while (nompays.Read())
            {
                comboBoxPays.Items.Add(nompays["pays"]);
            }
        }

merci, de votre aide.

et pour repondre a Bluedeep, le ";" n'est pas obligatoire donc que je le mette ou pas sa change rien.

[ced600]

je te conseille de suivre la recommandation de BlueDeep :

Ensuite, déjà mettre

Code :

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

nompays["pays"].ToString()

[tomenbois]

Qu'est ce que sa change ?

je te conseille de suivre la recommandation de BlueDeep :

Ensuite, déjà mettre

Code :

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

nompays["pays"].ToString()

[ced600]

Code :

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

nompays["pays"]

Cela ne te renvois pas forcement un string. Si cela fonctionne comme les ListDictionnary, cela te renvois une valeur de type Object.

C'est mieux de la retypé en string des l'ajout dans la combobox. Car même si elle peut lister des objets, lorsque tu voudra manipuler un item de la combo, ce sera un objet et tu seras peut être obligé de te faire un cast en string.

[tomenbois]

OK, en gros c'est du confort. Sympa, merci pour l'astuce.

[meddy]

Bonjour,
J'ai eu le même problème c'est à dire que j'essaye de remplir ma combobox, j'ai donc utilisé ton bout de code.

Pourrait-tu me dire comment déclarer SQLiteCommand, SQLiteDataReader et surtout à quoi correspond ton SQL_Conn.

Merci

[ced600]

A mon avis SQL_Conn est l'objet SqlConnexion (pas sur du nom je fait pas d'accés au BD)

Un objet de ce genre est présent dans tous les langages pour permettre la connexion à une BD.

[Bluedeep]

Un objet de ce genre est présent dans tous les langages pour permettre la connexion à une BD.

Ce n'est pas lié au langage, mais au framework.

[meddy]

Merci