Remplir une ComboBox depuis Sql Server

Version imprimable

Voir 40 message(s) de cette discussion en une page

Bonjour,

Je cherche à remplir une ComboBox à partir de données récupérées depuis Sql Server à l'aide du code ci-dessous:

Code:

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
 private void BindClients()
        {
            string cnString = ConfigurationManager.ConnectionStrings["SqlServer"].ConnectionString;
            SqlConnection myConnection;
            myConnection = new SqlConnection(cnString);
            myConnection.Open();
 
            /* Efface le contenu de notre comboBox */
            cbxClient.Items.Clear();
 
            //Tu créés une liste générique
            List<ClientBO> liste = new List<ClientBO>();
 
            try
            {
                SqlCommand myCommand = new SqlCommand("SELECT * FROM CLIENT", myConnection);
                SqlDataReader Reader = myCommand.ExecuteReader();
 
                while (Reader.Read())
                {
                    ClientBO c = new ClientBO();
 
                    c.ClientId = (int)Reader["NumClient"];
                    c.ClientName = Reader["NomClient"].ToString();
 
                    //Ici tu remplis ta liste des objets que tu récupères
                    liste.Add(c);
                }
 
                Reader.Close();
 
                //Ici tu remplis ton combobox
                //tu indiques la valeur à récupérer (Nom de la propriété dans le BusinessObject)
                cbxClient.ValueMember = "Id";
 
                // tu indiques la valeur à afficher (Nom de la propriété dans le BusinessObject)
                cbxClient.DisplayMember = "NomClient";
 
                cbxClient.DataSource = liste;
 
                MessageBox.Show("Listing fini", "OK");
            }
            catch (SqlException exp)
            {
                MessageBox.Show(exp.Message);
            }
 
            finally
            {
                myConnection.Close();
            }
 
        }

mais aucune donnée ne s'affiche dans la ComboBox.

Merci,

05/06/2013, 14h08
DotNetMatt
C'est probablement à cause des mauvaises valeurs des propriétés ValueMember et DisplayMember. Essaie avec :
Code:

1 2 cbxClient.ValueMember = "ClientId"; cbxClient.DisplayMember = "ClientName";
Ca doit toujours correspondre aux noms des propriétés de l'objet que tu bind.
05/06/2013, 14h14
Jinkas

Merci pour votre réponse mais c'est pareil.
05/06/2013, 14h31
DotNetMatt

C'est bizarre il n'y a pas d'autre erreur ? Remplace SqlException par Exception dans le catch pour être sûr.

Es-tu également sûr d'avoir des données dans la table CLIENT ?
05/06/2013, 16h12
Jinkas
Toujours pareil.

Oui, il y a des données dans la table.

J'appelle la procédure comme suit dans le Load de Form1
Code:

1 2 3 4 private void Form1_Load(object sender, EventArgs e) { BindClients(); }

Bonjour,

Comme l'a indiqué DotNetMatt, essayes de revoir la valeur de la propriété ValueMember car dans ton code, tu as mis ceci:
Code:

1 2 3 4 5 6 7 8 9 10 11 while (Reader.Read()) { ClientBO c = new ClientBO(); c.ClientId = (int)Reader["NumClient"]; // ICI TU AS MIS NumClient c.ClientName = Reader["NomClient"].ToString(); //Ici tu remplis ta liste des objets que tu récupères liste.Add(c); }
Cepedant au moment de remplir la combobox, tu as mis:
Code:

1 2 3 4 //tu indiques la valeur à récupérer // TU DOIS METTRE NumClient. pas non plus cbxClient.ValueMember = "ClientId" comme suggéré par DotNetMatt cbxClient.ValueMember = "Id";
ce qui est incorrect. Fallait mettre:
Code:

1 2 cbxClient.ValueMember = "NumClient";
Si tu as déjà effectué toutes ces corrections et la combobox est toujours vide, pose un point d'arrêt sur la ligne while (Reader.Read()) et regarde si la boucle while est exécutée au moins un fois.

05/06/2013, 19h36
DotNetMatt
Citation:
Envoyé par callo

Code:

1 2 3 4 //tu indiques la valeur à récupérer // TU DOIS METTRE NumClient. pas non plus cbxClient.ValueMember = "ClientId" comme suggéré par DotNetMatt cbxClient.ValueMember = "Id";

ce qui est incorrect. Fallait mettre:

Code:

1 2 cbxClient.ValueMember = "NumClient";
Ben non, il faut mettre le nom de la propriété de l'objet qu'on bind. Ici, on bind un objet ClientBO, donc tu dois mettre le nom de la propriété liée à cet objet, soit ClientId ou ClientName.

Je récapitule:

J'ai une classe ClientBO qui contient les informations suivantes:

Code:

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
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
 
namespace BO
{
    public class ClientBO
    {
        #region Constructor
 
        public ClientBO()
        {
 
        }
 
        #endregion
 
        #region Variable Declaration
        private Int32 iClientId = 0;
        private string sClientName = string.Empty;
        #endregion
 
        #region Properties
        public int ClientId
        {
            get
            {
                return iClientId;
            }
            set
            {
                iClientId = value;
            }
        }
 
        public string ClientName
        {
            get
            {
                return sClientName;
            }
            set
            {
                sClientName = value;
            }
        }
        #endregion
    }
}

Le code qui permet de remplir le ComboBox:

Code:

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
        private void BindClients()
        {
            string cnString = ConfigurationManager.ConnectionStrings["SqlServer"].ConnectionString;
            SqlConnection myConnection;
            myConnection = new SqlConnection(cnString);
            myConnection.Open();
 
            /* Efface le contenu de notre comboBox */
            cbxClient.Items.Clear();
 
            //Tu créés une liste générique
            List<ClientBO> liste = new List<ClientBO>();
 
            try
            {
                SqlCommand myCommand = new SqlCommand("SELECT * FROM CLIENT", myConnection);
                SqlDataReader Reader = myCommand.ExecuteReader();
 
                while (Reader.Read())
                {
                    ClientBO c = new ClientBO();
 
                    c.ClientId = (int)Reader["NumClient"];
                    c.ClientName = Reader["NomClient"].ToString();
 
                    //Ici tu remplis ta liste des objets que tu récupères
                    liste.Add(c);
                }
 
                Reader.Close();
 
                //Ici tu remplis ton combobox
                //tu indiques la valeur à récupérer (Nom de la propriété dans le BusinessObject)
                cbxClient.ValueMember = "NumClient";
 
                // tu indiques la valeur à afficher (Nom de la propriété dans le BusinessObject)
                cbxClient.DisplayMember = "NomClient";
 
                cbxClient.DataSource = liste;
 
                MessageBox.Show("Listing fini", "OK");
            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.Message);
            }
 
            finally
            {
                myConnection.Close();
            }
 
        }
 
        private void Form1_Load(object sender, EventArgs e)
        {
            BindClients();
        }

J'ai fait tout ce qui a été dit et ça ne fonctionne toujours pas.

Merci,

05/06/2013, 20h35
DotNetMatt

Il y a forcément un truc qui foire :weird:

En tout cas j'insiste, et c'est même marqué dans les commentaires de ton code : "(Nom de la propriété dans le BusinessObject)". Il faut utiliser le nom des propriétés de ton objet ClientBO.

:arrow: Donc ton dernier code est toujours erroné.

Ensuite, comme te l'a suggéré callo, met un point d'arrêt dans le while pour voir si tu passes dedans en debug (touche F5). Si l'exécution ne s'arrête pas au point d'arrêt, c'est qu'il n'y a aucune donnée renvoyée par la base.
06/06/2013, 11h45
Jinkas
Bonjour à tous,

Le problème venait de ma connection string de mon fichier App.config.
J'ai mis \\ au lieu de \ dans Data Source.

Comme ceci:

Code:

connectionString="Data Source=myServerName\\myInstanceName

au lieu de

Code:

connectionString="Data Source=myServerName\myInstanceName

Pour remplir la ComboBox, il faut mettre les valeurs ci-dessous:
Code:

1 2 3 cbxClient.ValueMember = "ClientId "; cbxClient.DisplayMember = "ClientName";
Merci à tous,
06/06/2013, 15h10
callo

Citation:

Envoyé par DotNetMatt

Ben non, il faut mettre le nom de la propriété de l'objet qu'on bind. Ici, on bind un objet ClientBO, donc tu dois mettre le nom de la propriété liée à cet objet, soit ClientId ou ClientName.

Effectivement, tu avais raison DotNetMatt. J'avais carrément oublié qu'il utilisait un BusinessObject pour le binding.
18/06/2013, 14h11
Jinkas

J'ai oublié de demander s'il était possible d'initialiser le ComboxBox à partir d'une variable déclarée. C'est à dire en plus d'avoir les données récupérées depuis la base de données dans le ComboxBox, j'ai aussi la valeur de la variable déclarée.

Merci,
18/06/2013, 14h36
DotNetMatt
C'est-à-dire ? Tu as une variable de type ClientBO et tu veux l'intégrer dans le résultat de la requête (donc dans la variable liste) ?

Si c'est bien ça, tu peux faire :
Code:

1 2 var taVariable = leClientBO; liste.Add(taVariable);
Si tu veux l'insérer à un endroit précis de la liste, tu peux utiliser la méthode Insert :

Code:

liste.Insert(0, taVariable);

Dans ce cas, on insère la variable à l'index 0.

Comment le faire ici:

Code:

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
  private void BindClients()
        {
            string cnString = ConfigurationManager.ConnectionStrings["SqlServer"].ConnectionString;
            SqlConnection myConnection;
            myConnection = new SqlConnection(cnString);
            myConnection.Open();
 
            cbxClient.Items.Clear();
 
            List<ClientBO> liste = new List<ClientBO>();
 
            try
            {
                SqlCommand myCommand = new SqlCommand("SELECT * FROM CLIENT", myConnection);
                SqlDataReader Reader = myCommand.ExecuteReader();
 
                while (Reader.Read())
                {
                    ClientBO c = new ClientBO();
 
                    c.ClientId = (int)Reader["NumClient"];
                    c.ClientName = Reader["NomClient"].ToString();
 
                    liste.Add(c);
                }
 
                Reader.Close();
 
                cbxClient.ValueMember = "NumClient";
 
                cbxClient.DisplayMember = "NomClient";
 
                cbxClient.DataSource = liste;
 
            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.Message);
            }
 
            finally
            {
                myConnection.Close();
            }
 
        }

Comme ça par exemple :

Code:

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
private void BindClients()
{
    string cnString = ConfigurationManager.ConnectionStrings["SqlServer"].ConnectionString;
    SqlConnection myConnection;
    myConnection = new SqlConnection(cnString);
    myConnection.Open();
 
    // Ta variable, déclarée ici en variable locale, ou en tant que membre de la classe...
    var taVariable = new ClientBO
    {
        ClientId = 123456,
        ClientName = "Jean Dupond"
    };
 
    cbxClient.Items.Clear();
 
    List<ClientBO> liste = new List<ClientBO>();
 
    try
    {
        SqlCommand myCommand = new SqlCommand("SELECT * FROM CLIENT", myConnection);
        SqlDataReader Reader = myCommand.ExecuteReader();
 
        while (Reader.Read())
        {
            ClientBO c = new ClientBO();
 
            c.ClientId = (int)Reader["NumClient"];
            c.ClientName = Reader["NomClient"].ToString();
 
            liste.Add(c);
        }
 
        Reader.Close();
 
        // Dans le cas où tu veux l'insérer à la fin (après les résultats de la requête) :
        liste.Add(taVariable);
        // Sinon, faire ceci pour l'insérer à un endroit précis (au début de la liste par exemple) :
        liste.Insert(0, taVariable);
 
        cbxClient.ValueMember = "NumClient";
 
        cbxClient.DisplayMember = "NomClient";
 
        cbxClient.DataSource = liste;
 
    }
    catch (Exception ex)
    {
        MessageBox.Show(ex.Message);
    }
 
    finally
    {
        myConnection.Close();
    }
 
}

18/06/2013, 16h39
Jinkas

ça ne fonctionne pas
18/06/2013, 16h44
Jinkas

et quand je change le style de mon ComboBox comme ceci:

Code:

cbxClient.DropDownStyle = ComboBoxStyle.DropDownList;

je perds mes données
18/06/2013, 16h46
DotNetMatt

Euh, je ne sais pas trop ce que tu es en train de faire là :weird: Tout à l'heure tu voulais insérer une variable dans ta liste (si j'ai bien compris), et maintenant tu changes le style du ComboBox...

Que veux-tu faire exactement ?
18/06/2013, 17h27
Jinkas
En fait c'est pour vous confirmer que rien ne se passe sur le ComboBox.

Ce que je veux, c'est pouvoir ajouter une valeur dans mon ComboBox à partir d'une variable initialisée comme par exemple:
Code:

1 2 string maVariable = "Hello"; cbxClient.Items.Add(maVariable);
qui va se rajouter à la liste de ma ComboBox.

Si possible pouvoir changer le style pour empêcher la saisie.
19/06/2013, 10h43
callo
Citation:
Envoyé par Jinkas

Ce que je veux, c'est pouvoir ajouter une valeur dans mon ComboBox à partir d'une variable initialisée comme par exemple:

Code:

1 2 string maVariable = "Hello"; cbxClient.Items.Add(maVariable);

qui va se rajouter à la liste de ma ComboBox.
Précédemment, tu remplissais ta combobox avec un BusinessObject en l'occurence ClientBO. Mais dans le code ci-dessus, tu utilises carrément un variable de type string. C'est pas correct. Tu peux ajouter un variable certes, mais il doit être de type ClientBO.

Citation:

Envoyé par Jinkas

Si possible pouvoir changer le style pour empêcher la saisie.

Le style de la combobox n'a rien à voir avec les data à utiliser pour remplir la combobox.
En outre, quand tu dis

Citation:

ça ne fonctionne pas

. As-tu un message d'erreur?

Voir 40 message(s) de cette discussion en une page