Discussion :

[l0quito]

Bonjour!

J'ai un grid dans lequel il y a deux colonnes ("Name" et "sel") sel est une checkboxcolonne.
l'utilisateur va cocher dans mon grid les users qu'il ne veut pas et après il les supprime avec ce code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
 
    Private Sub tsbtnDelUser_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles tsbtnDelUser.Click
        Dim j As Integer = dgvUsers.RowCount
        For i As Integer = 0 To j
            If dgvUsers.Rows(i).Cells("sel").Value = True Then
                dgvUsers.Rows.RemoveAt(i)
                i -= 1
            End If
        Next
    End Sub

Mon problème est que quand j'efface un utilisateur, tout le reste perdent une row (d'où mon "i-=1") donc ma variable j n'as plus la bonne valeur, et une fois arriver a la fin de mon grid, il va quand même incrémenter mon i et sa génère une erreur d'index. Comment je fait pour changé ma variable j a la volé?

[Pol63]

fait plutot une collection de la liste des rows checkées
puis tu fais un for each sur cette collection

[l0quito]

fait plutot une collection de la liste des rows checkées
puis tu fais un for each sur cette collection

Comment ça? A la place de le supprimer, je l'ajoute dans une collection et après je fait un for each sur la collection des éléments à supprimer?

ça me parait logique, mais comment fait-tu pour mettre une row dans une collection?


Merci!

[Pol63]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
Dim lignesAsupprimer As New System.Collections.Generic.List(Of DataGridViewRow)
 
For Each ligne As DataGridViewRow In mondgv.rows
    If la cellule du combobox est checké then 
        lignesAsupprimer.Add(ligne)
    End If
Next
 
For Each ligne In lignesAsupprimer
    mondgv.Rows.Remove(ligne)
Next

travailler sur des choses qui bougent c'est pas pratique

[Charaf Eddine]

Bonjour,

ça me parait logique, mais comment fait-tu pour mettre une row dans une collection?

Tu peux utiliser par exemple un ArrayList.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
Dim entries as new ArrayList
 
for i as integer = 0 to j 
  If dgvUsers.Rows(i).Cells("sel").Value = True Then
   entries.add(dgvUsers.Rows(i))
  End If
next
 
for each row as DataGridRow in entries
   dgvUsers.Rows.Remove(row)
next

n'Oublie pas :

[l0quito]

Merci a vous deux pour vos réponse.

Je pense que je vais utiliser la technique de Pol63.

Merci encor.

[Pol63]

qui est plus performante d'au moins 0,0000001 milliseconde par rapport à la solution de Charaf Eddine

(on travaille directement sur les lignes, pas besoin de 3 accès indexés pour retrouver la ligne dans le datagrid)