Discussion :

[Tagazog]

Bonjour,

Voici le contexte:
J'ai un Thread principal qui est ma Form dans laquel est créée une ListView.
En cliquant sur un bouton, j'exécute un Thread (pas un BackGroundWorker je précise) qui doit lire certaines données de la ListView

Exemple :

Code :

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

1
2
3
            For Each ItemChecked In ListView.CheckedItems
                FileName = ItemChecked.ToString
            Next

Mais c'est pas franchement Thread Safe. J'ai donc réalisé les Function suivantes :

Code :

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

1
2
3
4
5
    Private Function GetListViewChecked(ByRef MyControl As ListView)
        Return MyControl.CheckedItems
    End Function
 
    Delegate Function GetSafeListViewChecked(ByRef MyControl As ListView)

Et j'ai modifié mon code en conséquence:

Code :

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

1
2
3
            For Each ItemChecked In Invoke(New GetSafeListViewChecked(AddressOf GetListViewChecked), ListView) 
                FileName = ItemChecked.ToString
            Next

Mais j'ai toujours l'erreur d'invocation multi-thread sur la lecture de ma ListView.

Quelqu'un aurait-il une idée? merci d'avance.

[tomlev]

En fait, je pense que la collection CheckedItems que tu récupères est encore liée au ListView, donc quand tu y accèdes ça revient à accéder à l'interface graphique. Il faudrait que ta méthode GetListViewChecked renvoie une liste des éléments cochés indépendante de la ListView

D'autre part, évite de déclarer des fonctions sans préciser le type de retour, sinon tu risques d'avoir de mauvaises surprises...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
    Private Function GetListViewChecked(ByRef MyControl As ListView) As List(Of ListViewItem)
        Dim lst as New List(Of ListViewItem)
        lst.AddRange(MyControl.CheckedItems)
        Return lst
    End Function
 
    Delegate Function GetSafeListViewChecked(ByRef MyControl As ListView) As List(Of ListViewItem)
 
...
 
            For Each ItemChecked In Invoke(New GetSafeListViewChecked(AddressOf GetListViewChecked), ListView) 
                FileName = ItemChecked.ToString
            Next

[Tagazog]

Si j'ai bien compris, au lieu de renvoyer les valeurs Checked de la ListView qui sont toujours rattachées au Thread Principal, le but est de créer une List Mémoire qui contient les valeurs de la ListView pour travailler directement avec celle-ci.

Par contre, je recontre un autre type de problème en essayant de récupérer les ItemChecked de la nouvelle List renvoyée par la fonction:

Impossible d'effectuer un cast d'un objet de type 'CheckedListViewItemCollection' en type 'System.Collections.Generic.IEnumerable`1[System.Windows.Forms.ListViewItem]'.

Par contre, merci pour ton conseil pour le typage des valeurs de retour. Etant débutant en VB et venant de Windev, c'est pas encore un automatisme.

[tomlev]

Ah ok, au temps pour moi... ListView.CheckedListViewItemCollection n'implémente pas IList(Of ListViewItem) comme je l'avais supposé. Tu n'as qu'à remplacer List(Of ListViewItem) par ArrayList, ça devrait le faire...

[Tagazog]

Ok merci, ça passe effectivement mieux mais je ne peux pas te blâmer pour ton erreur, sans toi, je n'aurais pas avancé. Et l'erreur est humaine hein ?!

[harz62]

Ah ok, au temps pour moi... ListView.CheckedListViewItemCollection n'implémente pas IList(Of ListViewItem) comme je l'avais supposé. Tu n'as qu'à remplacer List(Of ListViewItem) par ArrayList, ça devrait le faire...

En fait, pour qu'une collection puisse être énumérée il faut juste que la méthode GetEnumerator<T>() ou GetEnumerator() soit implémentée dans la classe (pas besoin d'implémenter l'interface IEnumerable ou IEnumerable<T>.

D'après un article écrit par Microsoft, il suffirait d'une méthode Add et de la présence d'un énumérateur pour prétendre qu'une classe est une collection

[tomlev]

En fait, pour qu'une collection puisse être énumérée il faut juste que la méthode GetEnumerator<T>() ou GetEnumerator() soit implémentée dans la classe (pas besoin d'implémenter l'interface IEnumerable ou IEnumerable<T>.

En fait ma remarque était par rapport au type d'argument accepté par la méthode AddRange (d'ailleurs c'est IEnumerable<T> et non IList<T>), pas par rapport à l'énumération avec foreach...

D'après un article écrit par Microsoft, il suffirait d'une méthode Add et de la présence d'un énumérateur pour prétendre qu'une classe est une collection

Je ne pense pas que la méthode Add soit requise, vu que certaines collections sont en lecture seule... à mon avis l'énumerateur suffit