[Débutant] Remplir une ComboBox à partir d'une table SQL (avis sur mon code)

Remplir une ComboBox à partir d'une table SQL (avis sur mon code)

Bonjour à tous,
Débutant dans le langage C# je viens solliciter votre expérience car je n'arrive pas à remplir ma ComboBox avec le champs "typologie" de ma table "HDC_connaissance" (BDD : "G4_HDC").

* code C# de la ComboBox *
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 private void comboBox1_SelectedIndexChanged(object sender, EventArgs e) { SqlConnection cnx2; cnx2 = new SqlConnection("Data Source=PC601;Initial Catalog=G4_HDC;User Id=sa;Password=******;"); SqlCommand requete = new SqlCommand(); requete.CommandType = CommandType.Text; requete.CommandText = "Select distinct typologie from HDC_connaissance"; requete.Connection = cnx2; cnx2.Open(); SqlDataReader typo = requete.ExecuteReader(); while (typo.Read()) { comboBox1.Items.Add(typo["typologie"]); } typo.Close(); }
Merci d'avance pour votre aide et vos remarques sur mon code car pour le moment ce bout de code ne donne rien mais ne renvoie pas d'erreur non plus.

Tu ne dois pas remplir une combo box en plein dans un event selection_changed.

Tu dois le faire lors du chargement de ton application.

L'event selection changed sera là pour savoir, quel élément de ta combo tu auras sélectionné.

Je ne peut pas t'aider plus avec le peu d'informations que tu donnes.

@Dynamès: merci de prendre du temps pour me répondre.
Donc si j'ai bien compris mon code n'est pas placé au bon endroit.
Mon projet est une petite application permettant de faire des recherches dans une base de données stockant des connaissances.
La recherche se lance pour le moment dans l’événement clic du bouton "button1"
qui lance une procédure stockée dont le résultat s'affiche dans un DataGridView.
Les différentes ComboBox que j'essaye d'initialiser avec les valeurs des tables associées vont permettre à l'utilisateur de sélectionner dans les ComboBox les paramètres d'entrée de la procédure stockée puis d'afficher le résultat dans le DataGridView.

* code complet *

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
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
using System.Data.SqlClient;
 
 
namespace WindowsFormsApplication1
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
 
        }
 
        private void Form1_Load(object sender, EventArgs e)
        {
 
 
 
        }
 
        private void button1_Click(object sender, EventArgs e)
        {
            System.Data.DataTable workTable = new DataTable("Connaissance_G4");
 
            DataColumn column;
            DataColumn column1;
            DataColumn column2;
            DataColumn column3;
            DataColumn column4;
            DataColumn column5;
            DataColumn column6;
            DataColumn column7;
 
            column = new DataColumn();
            column.DataType = System.Type.GetType("System.Int32");
            column.ColumnName = "Id";
            column.ReadOnly = true;
            column.Unique = true;
            workTable.Columns.Add(column);
 
            column1 = new DataColumn();
            column1.DataType = System.Type.GetType("System.String");
            column1.ColumnName = "Typologie";
            column1.ReadOnly = true;
            column1.Unique = false;
            workTable.Columns.Add(column1);
 
            column2 = new DataColumn();
            column2.DataType = System.Type.GetType("System.String");
            column2.ColumnName = "Categorie";
            column2.ReadOnly = true;
            column2.Unique = false;
            workTable.Columns.Add(column2);
 
            column3 = new DataColumn();
            column3.DataType = System.Type.GetType("System.String");
            column3.ColumnName = "Libelle";
            column3.ReadOnly = true;
            column3.Unique = false;
            workTable.Columns.Add(column3);
 
            column4 = new DataColumn();
            column4.DataType = System.Type.GetType("System.String");
            column4.ColumnName = "Description";
            column4.ReadOnly = true;
            column4.Unique = false;
            workTable.Columns.Add(column4);
 
            column5 = new DataColumn();
            column5.DataType = System.Type.GetType("System.String");
            column5.ColumnName = "Createur";
            column5.ReadOnly = true;
            column5.Unique = false;
            workTable.Columns.Add(column5);
 
            column6 = new DataColumn();
            column6.DataType = System.Type.GetType("System.DateTime");
            column6.ColumnName = "Date_Creation";
            column6.ReadOnly = true;
            column6.Unique = false;
            workTable.Columns.Add(column6);
 
            column7 = new DataColumn();
            column7.DataType = System.Type.GetType("System.String");
            column7.ColumnName = "Emplct_Fichier";
            column7.ReadOnly = true;
            column7.Unique = false;
            workTable.Columns.Add(column7);
 
            SqlConnection cnx;
            cnx = new SqlConnection("Data Source=PC601;Initial Catalog=G4_HDC;User Id=sa;Password=******;");
 
            SqlCommand maCommande;
            maCommande = new SqlCommand();
            maCommande.CommandType = CommandType.StoredProcedure;
            maCommande.CommandText = "G4HDC_ConsultationSp";
            maCommande.Connection = cnx;
 
            SqlParameter parm1 = new SqlParameter("@typologie", SqlDbType.NVarChar, 50);
            parm1.Value = null;
            parm1.Direction = ParameterDirection.Input;
            maCommande.Parameters.Add(parm1);
 
            SqlParameter parm2 = new SqlParameter("@categorie", SqlDbType.NVarChar, 50);
            parm2.Value = null;
            parm2.Direction = ParameterDirection.Input;
            maCommande.Parameters.Add(parm2);
 
            SqlParameter parm3 = new SqlParameter("@libelle", SqlDbType.NVarChar, 200);
            parm3.Value = null;
            parm3.Direction = ParameterDirection.Input;
            maCommande.Parameters.Add(parm3);
 
            SqlParameter parm4 = new SqlParameter("@description", SqlDbType.NVarChar, 1000);
            parm4.Value = null;
            parm4.Direction = ParameterDirection.Input;
            maCommande.Parameters.Add(parm4);
 
            SqlParameter parm5 = new SqlParameter("@createur", SqlDbType.NVarChar, 50);
            parm5.Value = null;
            parm5.Direction = ParameterDirection.Input;
            maCommande.Parameters.Add(parm5);
 
            cnx.Open();
 
            SqlDataReader Rdata;
            Rdata = maCommande.ExecuteReader();
            Rdata.Close();
            SqlDataAdapter adapter = new SqlDataAdapter();
            adapter.SelectCommand = maCommande;
            adapter.Fill(workTable);
            dgvResultat.DataSource = workTable;
 
        }
 
        private void label1_Click(object sender, EventArgs e)
        {
 
        }
 
        private void textBox1_TextChanged(object sender, EventArgs e)
        {
 
        }
 
        private void comboBox1_SelectedIndexChanged(object sender, EventArgs e)
        {
            SqlConnection cnx2;
            cnx2 = new SqlConnection("Data Source=PC601;Initial Catalog=G4_HDC;User Id=sa;Password=******;");
            SqlCommand requete = new SqlCommand();
            requete.CommandType = CommandType.Text;
            requete.CommandText = "Select distinct typologie from HDC_connaissance";
            requete.Connection = cnx2;
            cnx2.Open();
            SqlDataReader typo = requete.ExecuteReader();
            while (typo.Read())
            {
                comboBox1.Items.Add(typo["typologie"]);
            }
            typo.Close();
        }
 
        private void comboBox2_SelectedIndexChanged(object sender, EventArgs e)
        {
 
        }
 
        private void comboBox3_SelectedIndexChanged(object sender, EventArgs e)
        {
 
        }
 
        private void textBox1_TextChanged_1(object sender, EventArgs e)
        {
 
        }
 
        private void textBox7_TextChanged(object sender, EventArgs e)
        {
 
        }
 
 
    }
}

A savoir que ma procédure stockée dispose de 5 paramètres d'entrée (@typologie, @categorie, @libelle, @description, @createur) dont chaque valeur de paramètre pourra être choisi dans le ComboBox associé (5 paramètres d'entrée = 3 ComboBox + 2 zones de saisie) si un paramètre n'est pas renseigné il est "null".
Merci d'avance pour vos remarques.

Problème résolu, il suffisait juste de placer le code au bon endroit comme le signalait Dynamès, c'est à dire à l'intérieur de :

Code:

private void Form1_Load(object sender, EventArgs e)

Merci pour tes remarques Dynamès.