Discussion :

[devvanjier]

Salut à tous,
J'ai un soucis avec une combo dans une application. En effet, j'ai une combo que je remplis avec une datatable à chaque fois que l'utilisateur clik dans une zone de texte. Le problème est qu'au premier chargement, la combo se remplie en moins d'une second mais au après le premier remplissage c'est le calvaire le remplissage de la combo peut mettre jusqu'a 22 secondes en moyenne avant d'afficher les données ce qui fait perdre au moins une minute à un utilisateur en trois clicks. Que faire ??
Merci!

[Jerede]

Montre ton code pour commencer , c'pas vraiment normal que ça mette plus de temps pas la suite

[oussi]

c'pas vraiment normal que ça mette plus de temps pas la suite

Effectivement. Normalement sa doit être le contraire.

[Pol63]

+1

m'enfin il nous en effet du code, ou au moins le nombre de colonnes/lignes du datatable et la manière de transférer du datatable au combobox

[devvanjier]

C'est vrai vous avez raison j'ai oublié de mettre mon code. Le voici:
Sur un clik dans la zone de texte "txt_Libelle" je rends visible un groupbox dans lequel se trouve ma combo avec la propriétés DropDownStyle à simple.

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
 
        private void txt_Libelle_Click(object sender, EventArgs e)
        {
            try
            {
                this.grp4.Visible = true;
                this.cmb_fournitures.Focus();
                tm.remplirCombo(this.cmb_fournitures, "PS_SEL_ListFourniture");
            }
            catch(Exception Ex)
            {
                tm.messageErreur(Ex.Message.ToString());
            }
 
        }

la methode "remplirCombo" se trouve dans une classe appelée toutesMethodes que j'instancie comme suit :

Code :

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

1
2
 
 toutMethodes tm = New toutesMethodes();

ensuite je lui passe le nom de la combo et la procédure stockée qui permet la sélection. Une procédure stockée avec une requête de sélection simple.

voici le code de la méthode :

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
 
 
public void remplirCombo(ComboBox Cmb, string Procedure) 
        { 
            try
            {
                Cnx.SqlCnx.Open(); //Une classe qui contient les informations de connexion à ma base sql server
                Cmd.Connection = Cnx.SqlCnx;
                Cmd.Parameters.Clear();
 
                Cmd.CommandType = CommandType.StoredProcedure;
                Cmd.CommandText = Procedure;
                Dtable.Columns.Clear();
                Dtable.Clear(); //DataTable
                SqlDataReader SqlDr = Cmd.ExecuteReader();
 
                Dtable.Columns.Add("C1", typeof(decimal));
                Dtable.Columns.Add("C2", typeof(string));
                Dtable.Rows.Add(0, "");
 
                while (SqlDr.Read())
                {
                    Dtable.Rows.Add(SqlDr.GetDecimal(0), SqlDr.GetString(1));
                } SqlDr.Close();
                Cmb.DataSource = Dtable;
                Cmb.DisplayMember = "C2";
                Cmb.ValueMember = "C1";
            }
            catch (Exception Ex)
            {
                MessageBox.Show(Ex.Message.ToString(), Version(), MessageBoxButtons.OK, MessageBoxIcon.Information);
            }
            finally
            {
                Cnx.SqlCnx.Close();
            }
        }

Voila! c'est le code qui s'exécute à chaque click dans la zone de texte.

[Pol63]

c'est vrai que c'est pas du code très optimisé déjà
il vaudrait mieux charger toutes les données possibles au début, et sur le click travailler sur des collections (ou un datatable si tu préfères) pour remplir le combobox sur l'event click

mais il faudrait aussi voir le code de la procédure stockée, car exécuter une requete qui remonte quelques lignes c'est pas censé être long