Remplir une ComboBox depuis Sql Server

**Jinkas** · 05/06/2013, 13h52

Bonjour,

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

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
 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,

**DotNetMatt** · 05/06/2013, 14h08

C'est probablement à cause des mauvaises valeurs des propriétés ValueMember et DisplayMember. Essaie avec :

Code C# :

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

1
2
cbxClient.ValueMember = "ClientId";
cbxClient.DisplayMember = "ClientName";

Ca doit toujours correspondre aux noms des propriétés de l'objet que tu bind.

**Jinkas** · 05/06/2013, 14h14

Merci pour votre réponse mais c'est pareil.

**DotNetMatt** · 05/06/2013, 14h31

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 ?

**Jinkas** · 05/06/2013, 16h12

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 :

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

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

**callo** · 05/06/2013, 18h07

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 :

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

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 :

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

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 :

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

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.

**DotNetMatt** · 05/06/2013, 19h36

Envoyé par callo

Code :

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

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 :

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

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.

**Jinkas** · 05/06/2013, 20h03

Je récapitule:

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

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

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
        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,

**DotNetMatt** · 05/06/2013, 20h35

Il y a forcément un truc qui foire

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.

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.

**Jinkas** · 06/06/2013, 11h45

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 :

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

connectionString="Data Source=myServerName\\myInstanceName

au lieu de

Code :

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

connectionString="Data Source=myServerName\myInstanceName

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

Code :

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

1
2
3
 
cbxClient.ValueMember = "ClientId ";
cbxClient.DisplayMember = "ClientName";

Merci à tous,

**callo** · 06/06/2013, 15h10

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.

**Jinkas** · 18/06/2013, 14h11

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,

**DotNetMatt** · 18/06/2013, 14h36

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 C# :

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

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 C# :

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

liste.Insert(0, taVariable);

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

**Jinkas** · 18/06/2013, 16h20

Comment le faire ici:

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
  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();
            }
 
        }

**DotNetMatt** · 18/06/2013, 16h32

Comme ça par exemple :

Code C# :

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
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();
    }
 
}

**Jinkas** · 18/06/2013, 16h39

ça ne fonctionne pas

**Jinkas** · 18/06/2013, 16h44

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

Code :

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

cbxClient.DropDownStyle = ComboBoxStyle.DropDownList;

je perds mes données

**DotNetMatt** · 18/06/2013, 16h46

Euh, je ne sais pas trop ce que tu es en train de faire là

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 ?

**Jinkas** · 18/06/2013, 17h27

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 :

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

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.

**callo** · 19/06/2013, 10h43

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 :

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

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.

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

ça ne fonctionne pas

. As-tu un message d'erreur?