Posez votre question Signaler

Performance DataGridView.Rows.Remove [Résolu]

Messages postés
98
Date d'inscription
vendredi 22 octobre 1999
Dernière intervention
13 juin 2016
 - - Dernière réponse : blq
Messages postés
98
Date d'inscription
vendredi 22 octobre 1999
Dernière intervention
13 juin 2016
 - 31 janv. 2012 à 22:45
VB.NET Posez votre question Signaler
Bonjour,

Depuis VB.net 2010, je supprime des lignes d'un DataGridView, rien d'extraordinaire, sauf que c'est très lent... Je n'arrive pas à comprendre la raison de cette lenteur, surtout que j'ai trouvé une solution plus rapide, mais qui me semble pas en accord avec une suppression directe.

Voici le code original : 

' Boucle sur les lignes du DataGridView
For ctRow As Integer = 0 To DataGridView1.RowCount - 1
    ' Si le premier caractère de la première cellule ne commence pas par C
    If DataGridView1.Rows(ctRow).Cells(0).Value.ToString.StartsWith("C") = False Then
        ' Alors suppression de la ligne
        Call DataGridView1.Rows.Remove(DataGridView1.Rows(ctRow))
    End If
Next


Ce code fonctionne, mais sur un tableau de 500 lignes, chauqe suppression dure beaucoup trop longtemps...
J'ai ajouté un Call Call DataGridView1.Refresh() juste pour bien vérifier que chaque ligne en partant de la fin était supprimé.

A défautr d'avoir trouvé l'explication de cette lenteur et d'accélerer cette suppression, j'ai opté pour une solution différente, mais qui se déroule très bien, donc sans lenteur !


J'ajoute chaque ligne trouvée dans un tableau à une Collection déclarée : 
Dim theList As New Collection()
' ...
' Dans la boucle sur les Rows du DataGridView, comme au-dessus, je teste le premier caractère
If DataGridView1.Rows(ctRow).Cells(0).Value.ToString.StartsWith("C") = True Then
    Dim theLine(DataGridView1.ColumnCount - 1) as String

     Conservation de chaque Cellule
    For ctCell As Integer = 0 To DataGridView1.Rows(ctRow).Cells.Count - 1
        theLine(ctCell) = DataGridView1.Rows(ctRow).Cells(ctCell).Value.ToString
    Next

    Call theList.Add(theLine)
End If

' Ensuite je vide le tableau
Call DataGridView1.Rows.Clear()

' Je remplie le tableau avec le résultat du filtre
For Each theLine() As String In theList 
    Call DataGridView1.Rows.Add(theLine)
Next



Bref, j'ai répondu à mon problème, mais peut-être que quelqu'un pourra m'indiquer la raison de la lenteur de la première écriture via Rows.Remove() qui est plus courte en écriture de code.

Coté test de perf, j'ai même supprimé le filtre. 

' Boucle sur les lignes du DataGridView
For ctRow As Integer = 0 To DataGridView1.RowCount - 1
        ' suppression de la ligne
        Call DataGridView1.Rows.Remove(DataGridView1.Rows(ctRow))
Next


Même en bouclant tout simplement sur les lignes et en supprimant chaque ligne, cela reste lent tout de même...

Merci d'avance pour votre aide
Afficher la suite 

Votre réponse

Répondre au sujet

4 réponses

Meilleure réponse
Signaler Commenter la réponse de Renfield
Messages postés
17308
Date d'inscription
mercredi 2 janvier 2002
Dernière intervention
22 août 2018
3
Merci
tentes un SuspendLayout / ResumeLayout avant/apres ton traitement

ça evitera au controle d'etre redessiné a chaque suppression

Renfield - Admin CodeS-SourceS - MVP Visual Basic & Spécialiste des RegExp

Merci Renfield 3

Quelques mots de remerciements seront grandement appréciés. Ajouter un commentaire

Codes Sources a aidé 98 internautes ce mois-ci

Commenter la réponse de Renfield
Signaler Commenter la réponse de blq
Messages postés
98
Date d'inscription
vendredi 22 octobre 1999
Dernière intervention
13 juin 2016
3
Merci
Merci pour ce retour ! Je vais tester cela.

Merci blq 3

Quelques mots de remerciements seront grandement appréciés. Ajouter un commentaire

Codes Sources a aidé 98 internautes ce mois-ci

Commenter la réponse de blq
Signaler Commenter la réponse de blq
Messages postés
98
Date d'inscription
vendredi 22 octobre 1999
Dernière intervention
13 juin 2016
3
Merci
Après divers tests SuspendLayout() et ResumeLayout() n'apporte pas plus de performance sur le temps de traitement, meme en utilisant RemoveAt(). J'ai essayer en utilisant Refresh() avant et après, en mettant Visible = False avant et Visible = True après la suppression massive : Rien à faire. Et sur des milliers de lignes c'est pire.

Le plus rapide reste d'ajouter toutes les lignes à conserver dans un tableau temporaire, de vider complétement le DataGridView et de le remplir avec le contenu du tableau temporaire.

Merci blq 3

Quelques mots de remerciements seront grandement appréciés. Ajouter un commentaire

Codes Sources a aidé 98 internautes ce mois-ci

Commenter la réponse de blq
Signaler Commenter la réponse de blq
Messages postés
98
Date d'inscription
vendredi 22 octobre 1999
Dernière intervention
13 juin 2016
3
Merci
Finalement, je crois que j'ai trouvé la réponse... il suffisait de chercher dans l'aide !

Meilleures pratiques pour la mise à l'échelle du contrôle DataGridView Windows Forms

[...]
Les lignes, colonnes et en-tête peuvent être automatiquement redimensionnées, à mesure que le contenu de la cellule change, pour que le contenu des cellules soit visible, sans être tronqué.
Changer de mode de redimensionnement permet également de redimensionner les lignes, les colonnes et les en-têtes.
Pour déterminer la taille appropriée, le contrôle DataGridView doit examiner la valeur de chaque cellule qu'il doit prendre en compte.
Lorsque vous travaillez avec de grandes quantités de données, cette analyse peut affecter négativement les performances du contrôle lors du redimensionnement automatique.
[...]


La création des colonnes était faite dynamiquement dans mon code par une fonction générique, qui positionnait la valeur AutoSizeMode de la colonne à DataGridViewAutoSizeColumnMode.AllCells.

Bref à chaque suppression (mais aussi ajout), le tableau cherche à redimensionner chaque colonne en fonction du contenu. Quand il s'agit de quelques colonnes et quelques dizaines de lignes, cela passait inaperçu, mais dés qu'il s'agit de traiter plusieurs milliers de lignes, le temps s'écoulait en minutes...

J'espère que ma mésaventure servira à d'autres

Merci blq 3

Quelques mots de remerciements seront grandement appréciés. Ajouter un commentaire

Codes Sources a aidé 98 internautes ce mois-ci

Commenter la réponse de blq

Votre réponse

Répondre au sujet
Posez votre question
S'inscrire maintenant

Vous n'êtes pas encore membre ?

inscrivez-vous, c'est gratuit et ça prend moins d'une minute !

Les membres obtiennent plus de réponses que les utilisateurs anonymes.

Le fait d'être membre vous permet d'avoir un suivi détaillé de vos demandes et codes sources.

Le fait d'être membre vous permet d'avoir des options supplémentaires.