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

[ostenhard]

Bonjour,

As-tu "épuré" le code car (et je rejoins Pol63) dans ce que tu as fourni, il n'y a aucun paramètre à ta procédure stockée, donc elle te retourne toujours la même chose donc pourquoi le refaire plusieurs fois ?

[devvanjier]

Salut,
ostenhard, effectivement il n'y a aucun paramètre, la liste est susceptible de changer entre deux chargements. Je prends le cas où un utilisateur charge la liste des fournitures et s'aperçoit qu'une fourniture n'y figure pas. Il vas dans ce cas dans les paramètres pour ajouter l'élément qui manque. Mais quand il va revenir cliquer dans la zone de texte il faut qu'il retrouve l'élément ajouté plutôt c'est donc pour ca.

[ostenhard]

Ok donc je rejoins à 100% Pol63, tu ne charges d'une seule fois, stocke le résultat en "session" et travaille dessus si nécessaire.

Par contre je ne connais pas le contexte de ton application mais si un autre utilisateur modifie les données tu n'auras pas nécessairement la dernière version de celles-ci.

[Pol63]

si tu es sur une application multi utilisateur, rafraichir à chaque fois peut etre utile, m'enfin dans ce cas il faut analyser ta procédure stockée et ton modèle de base, car 22 secondes c'est le temps qu'il faut pour avoir 100 000 lignes et 30 colonnes ^^

si tu es sur du mono utilisateur tu peux quand meme charger les données une fois au début, si l'utilisateur ajoute un element, tu le rajoute dans le datatable en meme temps que dans la base ...

[devvanjier]

Salut,
en suivant vos conseils j'ai refais la méthode de chargement en mettant une condition pour que le chargement ne se fasse que quand il y a ajout et tout marche bien le temps d'affichage qui était dans l'ordre de 22s est passé à moins d'une seconde. C'est lorsqu'il y a ajout que le temps de chargement devient long. C'est peut être pas la meilleure solution mais pour l'instant cela fais gagner du temps aux utilisateurs.
Merci et a bientôt !!

[Pol63]

tu peux utiliser le multithreading pour ne pas figer l'interface
m'enfin le temps d'accès à ta base de données dénote un problème de conception