Remplir un combobox avec une base de donnée

Version imprimable

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:

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 ?

11/12/2007, 10h31
Bluedeep

Pour commencer pourquoi le chargement de la combo est sur l'event SelectedIndexChanged ? 8O
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:

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.
11/12/2007, 10h35
neilbgr

Renseigne toi sur les propriétés DataSource, DisplayMember, ValueMember et le DataBinding en général...
11/12/2007, 10h35
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 :D

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 :mrgreen:

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:

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.

11/12/2007, 11h12
ced600

je te conseille de suivre la recommandation de BlueDeep :

Citation:

Ensuite, déjà mettre

Code:

nompays["pays"].ToString()
11/12/2007, 11h18
tomenbois

Qu'est ce que sa change ?

Citation:

je te conseille de suivre la recommandation de BlueDeep :

Citation:

Ensuite, déjà mettre

Code:

nompays["pays"].ToString()
11/12/2007, 11h38
ced600

Code:

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.
11/12/2007, 11h43
tomenbois

OK, en gros c'est du confort. Sympa, merci pour l'astuce.
11/12/2007, 20h10
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
12/12/2007, 09h18
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.
12/12/2007, 10h08
Bluedeep

Citation:

Envoyé par ced600

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.
12/12/2007, 20h37
meddy

Merci