Discussion :

[agparchitecture]

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 :

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

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

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

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

Par exemple:

Code :

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

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 :

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

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

[François DORIN]

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.

[agparchitecture]

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 :

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

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.

[François DORIN]

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

[ZenZiTone]

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.

[agparchitecture]

Merci ça fonctionne parfaitement

juste une coquille

Code :

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

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

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