Discussion :

[boss_gama]

Salut, je dispose d'une form composé de deux panels. Le panel de gauche contient une arborescence d'objets. Le panel de droite contient un affichage des caractéristiques de ces objets. Pour l'instant je charge les caractéristiques dans des labels. A chaque click de l'utilisateur sur un node de mon treeview, de nouvelles caractéristiques sont chargées depuis mon datatable. Mon problème est le temps d'affichage de ces informations. J'aimerais bien pouvoir afficher ces infos de manière instantanée, alors que pour l'instant il y a un battement le temps que les données se réinitialisent.
C'est un problème d'optimisation de code ou j'utilise le mauvais composant?

[Mose]

Optimisation.

Ton panel de gauche, ca serait pas un Treeview par hasard ? Je vois pas bien comment il peut contenir une arborescence d'objet.

[boss_gama]

Salut Mose.
C'est effectivement un treeview qui contient les dénominations d'objets (ex: reférence d'une médicament, nom d'un client).

Si c'est un problème d'optimisation, qu'est-ce que je pourrais améliorer dans mon code?

Ce code permet de déterminer une ligne du datatable TableMedicaments. Cette ligne est indiquée dans le tag du node sur lequel l'utilisateur a cliqué. Les labels du panel de droite sont ensuite renseignés avec les champs correspondant.

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
public void TreeviewMedicament_MouseUp(object sender, MouseEventArgs e)
        {
            //On récupère le node sur lequel l'utilisateur a cliqué
            Point ClickPoint = new Point(e.X, e.Y);
            TreeNode ClickNode = TreeviewMedicament.GetNodeAt(ClickPoint);
 
            if (ClickNode != null)
            {
                //Si l'utilisateur a cliquer sur un médicament
                if (ClickNode.Parent != null)
                {
                    String Tag = ClickNode.Tag.ToString();
                    String[] s = Tag.Split(new Char[] { ' ' });
                    Tag = s[0];
 
                    int Index = Convert.ToInt32(Tag);
                    DataRow row = TableMedicaments.Rows[Index - 1];
 
                    LblNomMedicament.Text = row["Médicament"].ToString();
                    LblCompositionMedicament.Text = row["Composition"].ToString();
                    LblContreIndicationsMedicament.Text = row["ContreIndications"].ToString();
                    LblEffetsSecondairesMedicament.Text = row["EffetsSecondaires"].ToString();
                    LblIndicationsMedicament.Text = row["Indications"].ToString();
                    LblStocksMedicament.Text = row["Stocks"].ToString();
                    if (row["Ordonnance"].ToString() == "0")
                    {
                        LblOrdonnanceMedicament.Text = "Non";
                    }
                    else
                    {
                        LblOrdonnanceMedicament.Text = "Oui";
                    }
                    lblPrixMedicament.Text = row["Prix"].ToString() + "€";
                    LblLaboratoireMedicament.Text = row["Laboratoire"].ToString();
                    LblRemboursementMedicament.Text = row["TauxRemboursement"].ToString() + "%";
                    lblCategorieMedicament.Text = row["Catégorie"].ToString();
                    LblTypeMedicament.Text = row["TypeMedicament"].ToString();
                }
            }
        }

[Mose]

J'affirme : c'est un problème d'optimisation.
Les contrôles n'ont rien à voir là dedans, sauf si tu joues avec leur paramètres et que tu as des effets de bord sur les Click ou MouseDown.

Je ne comprend pas ce que tu as mis dans ton Tag.
Pourquoi tu n'y colles pas directement ton DataRow ?

Sinon perso je jouerai sur le SelectedNode pour récupérer le Node clické, plutôt que sur GetNodeAt.

Enfin, je commenterai mon code beaucoup plus que ça
C'est con mais de le voir écrit littéralement ça permet de mieux voir quand on fait de boulettes ou quand on se prend le chou pour rien.

[boss_gama]

J'affirme : c'est un problème d'optimisation.
Les contrôles n'ont rien à voir là dedans, sauf si tu joues avec leur paramètres et que tu as des effets de bord sur les Click ou MouseDown.

Ce sont de simples labels, mais je les recentre dans un TableLayoutPanel après avoir modifier leurs textes.

Je ne comprend pas ce que tu as mis dans ton Tag.
Pourquoi tu n'y colles pas directement ton DataRow ?

En fait j'utilise le Tag pour conserver en mémoire une référence unique de l'objet (Id_Client, Id_Médicament...) et d'autres champs que j'affiche dans un tooltip.

Sinon perso je jouerai sur le SelectedNode pour récupérer le Node clické, plutôt que sur GetNodeAt.

Enfin, je commenterai mon code beaucoup plus que ça
C'est con mais de le voir écrit littéralement ça permet de mieux voir quand on fait de boulettes ou quand on se prend le chou pour rien.

C'est modifié: j'ai transformé l'évênement en un 'NodeMouseClick' et j'ai ajouté des commentaires. En espérant que j'ai été plus clair sur mon code.

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
        //Gestion des clicks sur le treeview
        public void TreeviewMedicament_NodeMouseClick(object sender, TreeNodeMouseClickEventArgs e)
        {
            //Si le Tag n'est pas nul (si on a bien cliqué sur un médicament
            if (e.Node.Tag.ToString() != null)
            {
 
                //On récupère le Tag du node sélectionné
                String Tag = e.Node.Tag.ToString();
                //On récupère l'index du row contenu dans Tag (Tag = Index + " " + (Txt Infobulle))
                String[] s = Tag.Split(new Char[] { ' ' });
                Tag = s[0];
 
                //On convertit le tag en numérique
                int Index = Convert.ToInt32(Tag);
 
                //On récupère la ligne correspondante du datatable TableMedicaments
                DataRow row = TableMedicaments.Rows[Index - 1];
 
                //On met àjour les labels
                LblNomMedicament.Text = row["Médicament"].ToString();
                LblCompositionMedicament.Text = row["Composition"].ToString();
                LblContreIndicationsMedicament.Text = row["ContreIndications"].ToString();
                LblEffetsSecondairesMedicament.Text = row["EffetsSecondaires"].ToString();
                LblIndicationsMedicament.Text = row["Indications"].ToString();
                LblStocksMedicament.Text = row["Stocks"].ToString();
                if (row["Ordonnance"].ToString() == "0")
                {
                    LblOrdonnanceMedicament.Text = "Non";
                }
                else
                {
                    LblOrdonnanceMedicament.Text = "Oui";
                }
                lblPrixMedicament.Text = row["Prix"].ToString() + "€";
                LblLaboratoireMedicament.Text = row["Laboratoire"].ToString();
                LblRemboursementMedicament.Text = row["TauxRemboursement"].ToString() + "%";
                lblCategorieMedicament.Text = row["Catégorie"].ToString();
                LblTypeMedicament.Text = row["TypeMedicament"].ToString();
            }
        }

Je trouve toujours l'affichage lent. Mais peut être suis-je trop exigeant?