[VB2008] Threads et ListView

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:

---

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:

---

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:

---

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.

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:

---

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

---

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.

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...

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 ?!

Citation:

---

Envoyé par 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...

---

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

Citation:

---

Envoyé par harz62

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...

Citation:

---

Envoyé par harz62

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

Pour compléter:

Il y avait aussi une modification inhérente au type de sortie.

En effet :

Code:

---

1 2 3

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

---

Renvoyait le type d'objet, et non sa valeur.

Solution :

Code:

---

1 2 3

Dim MyItem As ListViewItem MyItem = ItemChecked FileName = MyItem.Text

---

Citation:

---

Envoyé par tomlev

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...

---

OK autant pour moi. J'avais compris qu'il fallait implémenter IList ou IList<T> pour que ce soit une collection.

Citation:

---

Envoyé par tomlev

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

---

Pour les initialisateurs de collections si mais si tu réfléchis à partir du .NET 2.0 alors oui tu as raison ^^