Discussion :

[Knightrider]

Bonjour,

J'ai une ComboBox avec 30000 items, l'affichage de cette ComboBox est extrêment lente. Pour limiter l'impact, j'utilise une structure de donnée que je remplit au chargement de l'application et que je garde en mémoire, les données n'étant pas modifiées au cours de la journée.

Malgré le fait que ma structure de donnée soit en mémoire, je trouve le binding extrêmement long, j'utilise la méthode AddRange du ComboxBox.

Est ce qu'il y a une méthode pour charger la ComboBox de manière optimale ?

Pour contourner ce problème, j'ai essayé de garder une référence vers ma fenêtre qui contient cette ComboBox, au lieu de détruire la fenêtre, je fait un Hide(). Mais j'ai toujours un problème de performance avec la ComboBox lorsque j'affiche de nouveau la fenêtre (en fait, d'après ma recherche, la ComboBox est détruite lorsque l'on fait un Hide()).

N'y a-t-il pas un moyen de garder une référence vers une fenêtre sans que les contrôles contenus dans celle-ci soit détruits ?

Merci d'avance,

Michael

Cordialement,

Michael Chevalier

[theMonz31]

1 - Suffit que ta fenetre ne soit pas détruire (donc, que la variable "fenetre" qui contient ta combo soit un membre de ta classe )

2 - Tu peux utiliser le SuspendLayout et le ResumeLayout pour améliorer les données


Après, petite remarque "gratuite", même si il y a surement une explication logique à la présence de 30000 Objets dans ta combo, celà me semble tellement aberrant que les bras m'en tombent .

En effet, en tant qu'utilisateur, aller chercher une "référence" dans une combo de 30000 référence me semble vraiment incongru et peu pratique !!!

Maintenant, ya surement comme je le dis une logique qui m'échappe liée au contexte d'utilisation.

En tout cas, avec les pistes données, tu devrais pouvoir avancer rapidement !

[Louis-Guillaume Morand]

comme theMonz31, je pense qu'il y a clairement un problème dans l'utilisation d'une combobox. Si tu as 30 000 éléments, tu n'utilises surtout PAS une combobox.

une simple textbox avec autocompletion ferait parfaitement l'affaire. ca prend dix lignes, et ca évite de faire des ruses d'ouzbek pour essayer d'amenuire la perte de performance suite à un mauvais choix ergonomique. Et au niveau performance, c'est ultra light l'autocompletion.

pour la version simple, ca donne ceci: http://lgmorand.developpez.com/dotnet/autocompletion/
mais le mieux c'est encore de faire une requête "à la ajax" pour éviter de retourner plus d'items que ce que l'on aura besoin.

[Knightrider]

Ma variable "Fenetre" est bien membre de ma classe Fenetre(pattern Singleton). J'ai créé une classe CustomComboBox pour surcharger la méthode WndProc, lorsque je fais un Hide() sur la fenêtre, j'intercepte un WM_DESTROY pour la ComboBox, et je me prends des message WM_ADDSTRING et d'autres messages lorsque j'affiche la fenêtre de nouveau.

Du coup, le fait de garder l'instance de ma fenêtre perd de son intérêt du point de vue performance.
Je n'ai plus trop de solutions pour améliorer les performances pour le moment.

[Louis-Guillaume Morand]

Je n'ai plus trop de solutions pour améliorer les performances pour le moment.

on vient de te le dire.
ton problème de performance, TU l'as créés en ajoutant une combobox là où il ne faut surtout pas l'utiliser. t'aurais le même problème avec beaucoup d'autres contrôles. Il n'y a que les listview en mode virtual qui peuvent gérer un grand nombre d'items.

Utilise autre chose, ca serait performant et plus facile à utiliser pour l'utilisateur. comment veux-tu qu'un utilisateur choisisse parmi 30 000 items?²

[Knightrider]

Dans mon cas, la combobox est plutot fonctionnel une fois les données chargées. Les données de ma ComboBox viennent de ma couche Business, et j'utilise l'évènement OnKeyPress comme suit :

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
 
// Custom behaviour for the custom combo box
private void CustomComboBox_KeyPress(object sender, KeyPressEventArgs e)
        {
            switch ((Keys)e.KeyChar)
            {
                case Keys.Down:
                    break;
                case Keys.Up:
                    break;
                case Keys.Enter:
                    // Close the drop down list
                    this.DroppedDown = false;
 
                    // Set the current selected index to the item selected
                    this.SelectedIndex = this.FindString(this.Text);
                    break;
                default:
                    // Set the cursor at the end of the input string
                    if (string.IsNullOrEmpty(this.Text) == false)
                    {
                        this.Select(this.Text.Length, 0);
                    }
 
                    // Open the drop down list
                    this.DroppedDown = true;
                    break;
            }
        }

Je trouve que la sélection est plutot simple, j'avais pas besoin de faire des requêtes. Je pense rester comme cela pour le moment.

Merci