Performance DataGridView.Rows.Remove [Résolu]

Question

Posez votre question Signaler

Performance DataGridView.Rows.Remove [Résolu]

blq 98 Messages postés vendredi 22 octobre 1999Date d'inscription 13 juin 2016 Dernière intervention - 17 janv. 2012 à 15:34 - Dernière réponse : blq 98 Messages postés vendredi 22 octobre 1999Date d'inscription 13 juin 2016 Dernière intervention

- 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

Votre réponse

Répondre au sujet

Posez votre question

Accepted Answer

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

Avec quelques mots c'est encore mieux Ajouter un commentaire

Codes Sources a aidé 96 internautes ce mois-ci

Accepted Answer

Meilleure réponse

Signaler Commenter la réponse de blq

blq 98 Messages postés vendredi 22 octobre 1999Date d'inscription 13 juin 2016 Dernière intervention - 22 janv. 2012 à 20:15

3

Merci

Merci pour ce retour ! Je vais tester cela.

Merci blq 3

Avec quelques mots c'est encore mieux Ajouter un commentaire

Codes Sources a aidé 96 internautes ce mois-ci

Accepted Answer

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

Avec quelques mots c'est encore mieux Ajouter un commentaire

Codes Sources a aidé 96 internautes ce mois-ci

Accepted Answer

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

Avec quelques mots c'est encore mieux Ajouter un commentaire

Codes Sources a aidé 96 internautes ce mois-ci

Performance DataGridView.Rows.Remove [Résolu]

Votre réponse

Votre réponse

Votre réponse

Vous n'êtes pas encore membre ? inscrivez-vous, c'est gratuit et ça prend moins d'une minute !

Vous n'êtes pas encore membre ?

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