parrallel.foreach avec datagridview

Version imprimable

Bonsoir à tous,

J'ai dans une interface winform un datagridview. Lorsque je clique sur un bouton, je voudrais vérifier les éléments sélectionnés Mon code est :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
 
 
private List<Mark> Test = new List<Mark>();
 
private void button2_Click(object sender, EventArgs e)
		{
			List<Mark> lst = new List<Mark>();
			Mark m = new Mark();
			foreach (DataGridViewRow row in dataGridView1.SelectedRows)
			{
				m = row.DataBoundItem as Mark;			
				lst.Add(m);
			}
 
 
			Parallel.ForEach(lst, new ParallelOptions { MaxDegreeOfParallelism = 4 }, cm => 
			                 {
			                 	cm.Titre = MarkNet.Tools.HtmlTools.gettitle(m.Href);
			                 	if (cm.Titre != "")
			                 	{
			                 		lock (Test)
			                 		{
			                 			Mark obj = Test.FirstOrDefault( x=> x.UId == m.UId);
			                 			if (obj != null) obj.Titre = m.Titre;
			                 			dataGridView1.Update();
			                 		}
			                 	}
			                 }
			                );
}

Pourtant le seul message que j'obtient à l’exécution est : Une ou plusieurs erreurs se sont produites.
Rien dans le compilateur ni dans le déboguage.

Pourriez-vous m'éclairer? Merci

20/12/2016, 22h42
François DORIN
Bonjour,

Le problème vient très certainement du fait que seul le thread graphique est autorisé à accéder aux éléments graphiques (par exemple, en faisant dataGridView1.Update();).

Maintenant, Parallel.ForEach utilises plusieurs thread. Dans ton code, jusqu'à 4. Et le code dans la boucle accède à des éléments graphiques. Résultat : InvalidOperationException.

Solutions :
- ou bien enlever le Parallel.ForEach et le remplacer par une boucle for classique ;
- ou bien enlever tous les accès aux éléments graphiques de la boucle.
Au passage, l'usage de l'instruction lock est généralement à éviter au sein du thread graphique.

Il est possible pour un thread d'accéder aux éléments graphiques de l'UI (même si c'est déconseillé...). Il faut pour cela créer une méthode dédiée, et un callback.

Par exemple:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 private void UpdateDgv() { if (this.dataGridView1.InvokeRequired) { UpdateDgvCallback d = new UpdateDgvCallback(UpdateDgv); this.Invoke(d, new object[] { }); } else { this.dataGridView1.Update(); } }
Déclarer le callback ainsi:

Code:

delegate void UpdateDgvCallback();

Tu peux ainsi, dans tes thread, remplacer dataGridView1.Update(); par UpdateDgv();

Edit: regarde ici http://stackoverflow.com/questions/6...er-thread-in-c pour des méthodes plus générales

21/12/2016, 05h49
François DORIN

Citation:

Envoyé par nouanda

Il est possible pour un thread d'accéder aux éléments graphiques de l'UI (même si c'est déconseillé...). Il faut pour cela créer une méthode dédiée, et un callback.

Ce n'est pas du tout déconseillé. C'est la méthode à suivre dès lors qu'un thread autre que le thread graphique souhaite mettre à jour l'UI.

Par contre, vu l'utilisation de lock dans son code, je déconseillerai fortement cette approche, vu les risques d'obtenir des interblocages.

Bon ok je ne m'en sort pas. Donc je vais poser la question autrement.

J'ai les procédures suivantes qui fonctionne parfaitement.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
private List<Mark> lstMark = new List<Mark>();
 
private void button2_Click(object sender, EventArgs e)
{
	Mark m = new Mark();
	foreach (DataGridViewRow row in dataGridView1.SelectedRows)
	{
		m = row.DataBoundItem as Mark;			
		m.Titre = MarkNet.Tools.HtmlTools.gettitle(m.Href);
		if(m.Titre != "")
                {
                	Mark obj = Test.FirstOrDefault( x=> x.UId == m.UId);
			if (obj != null) obj.Titre = m.Titre;
			dataGridView1.Update();
                }
	}
}

Comment puis-je convertir cette procédure afin de faire la boucle sur plusieurs ligne en même temps?

Merci pour votre aide.

21/12/2016, 11h34
François DORIN

comment est définie la variable ou propriété Test ?
21/12/2016, 11h43
agparchitecture

Citation:

Envoyé par François DORIN

comment est définie la variable ou propriété Test ?

En fat c'est mon lstMark dans mon dernier post donc c'st ma list<Mark> que j'affiche dans mon datagridview par un datasource.

Voici un exemple de code. Attention, je ne l'ai pas testé, donc il peut y avoir des coquilles !
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 private List<Mark> lstMark = new List<Mark>(); private void button2_Click(object sender, EventArgs e) { foreach (DataGridViewRow row in dataGridView1.SelectedRows) { Mark m = row.DataBoundItem as Mark; string href = m.Href; Task.Run(() => { string titre = MarkNet.Tools.HtmlTools.gettitle(href); UpdateMark(m, titre); }); } } delegate void UpdateDgvCallback(Mark m, string titre); private void UpdateMark(Mark m, string titre) { if (this.dataGridView1.InvokeRequired) { UpdateDgvCallback d = new UpdateDgvCallback(UpdateDgv); this.Invoke(d, new object[] {m, titre}); } else { m.Titre = titre; if (m.Titre != "") { Mark obj = Test.FirstOrDefault(x=> x.UId == m.UId); if (obj != null) obj.Titre = m.Titre; dataGridView1.Update(); } } }
Voici en gros l'idée. J'ai supprimé le lock. Mais pour éviter qu'il y ait tout soucis, je me suis assuré que tous les accès aux instances de Mark se fassent dans le thread graphique. C'est pour cela que j'ai défini la méthode UpdateMark qui prend comme paramètre le Mark à modifier et le titre à utiliser. Cette méthode s'occupe ensuite de s'assurer que tout le traitement se fait dans le thread graphique (surtout, si tu as d'autres threads qui peuvent accéder à tes instances de Mark, alors le code ci-dessus n'est pas valide).

J'ai isolé la partie chronophage (le MarkNet.Tools.HtmlTools.gettitle) pour qu'elle soit exécuté dans des threads différents. J'utilise pour cela le pool de threads. Il y a une différence avec le code que tu as fourni, c'est que la, il n'y a pas de contrôle sur le nombre de traitements simultanés.

Si tu as vraiment besoin de cela, alors, tu peux utiliser Parallel.ForEach. Pour cela, il faut :

créer une liste de Mark ;
lancer une tâche (par exemple, via Task.Run) qui contiendra le Parallel.ForEach;
mettre le code actuellement dans Task.Run dans le Parallel.ForEach.

21/12/2016, 13h31
ZenZiTone

Citation:

Envoyé par nouanda

Il est possible pour un thread d'accéder aux éléments graphiques de l'UI (même si c'est déconseillé...). Il faut pour cela créer une méthode dédiée, et un callback.

Pour compléter ce qu'a dit @François DORIN, le this.Invoke(CallBack) est réalisé sur le thread principale de "this" et donc, le cas échéant, le thread principale de l'application (qui gère l'UI). Donc tu n'accèdes pas réellement aux composants graphiques dans un autres thread, mais bien dans celui qui gère ton UI.
21/12/2016, 16h21
agparchitecture

Merci ça fonctionne parfaitement

juste une coquille

Code:

UpdateDgvCallback d = new UpdateDgvCallback(UpdateDgv); à remplacer par UpdateDgvCallback d = new UpdateDgvCallback(UpdateMark);

Je mets en résolus et merci du coup de main