Thread en C#

Version imprimable

Thread en C#

Bonjour,
J’ai trouvé sur le net un tuto c# sur les threads, mais il ne semble pas fonctionner. Il compile, mais au lancement, il y a une erreur qui dit que mon thread essaye de modifier un objet qui n'a pas été créée dans ce thread (c'est pourtant bien ce que je souhaite faire).
L'url du tuto : http://csharpro.developpez.com/page9.php
Quelqu'un voit-il un problème dans ce code?
Dois-je utiliser des délégués?

Merci beaucoup,

David.

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
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Text;
using System.Windows.Forms;
using System.IO;
using System.Threading;
 
namespace WindowsApplication1
{
 
  public partial class Form1 : Form
  {
    private int ti, tj;
 
    public Form1()
    {
      InitializeComponent();
    }
 
 
    private void Form1_Load(object sender, EventArgs e)
    {
      Thread t1, t2;
 
      ti = 0;
      tj = 0;
      t1 = new Thread(new ThreadStart(implemente_ti));
      t1.Start();
 
      t2 = new Thread(new ThreadStart(implemente_tj));
      t2.Start();
    }
 
    private void implemente_ti()
    {
      ti++;
      this.textBox2.AppendText("\r\n" + ti.ToString());
      if (ti <= 10000)
        implemente_ti();
    }
 
    private void implemente_tj()
    {
      tj++;
      textBox3.AppendText("\r\n" + tj.ToString());
      if (tj <= 10000)
        implemente_tj();
    }
  }
}

Le problème de ton code, c'est que (comme te l'indique l'exception qui est levée) tu cherches à modifier un contrôle dans une Form, en dehors du thread qui fait tourner la Form (ce qui n'est pas autorisé).

La solution est donc (comme indiqué dans les détails de l'exception qui est levée) de modifier le code, par exemple de la façon suivante (pour la premiere méthode) :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 // Delegué pour la méthode private delegate void implemente_tiDelegate(); private void implemente_ti() { if(this.textBox2.InvokeRequired) { // On est pas sur le bon thread implemente_tiDelegate d = new implemente_tiDelegate(implemente_ti); // on rapelle la méthode depuis le thread courant du contrôle this.textBox2.Invoke(d); } else { // On est sur le bon thread, on continue ti++; this.textBox2.AppendText("\r\n" + ti.ToString()); if (ti <= 10000) implemente_ti(); } }

03/01/2008, 17h11
mrrenard

Tout les control lance une exception si on les modifie depuis un autre thread que celui dans lequel ils ont été créés. Pour modifier un control depuis un thread différent il faut utiliser la méthodes Invoke().

Exemple :

//Délégué
private delegate void SetTextDelegate(string text);

//Modifie le text d'un richTextBox tmpRTB
private void SetText(string text)
{
if(tmpRTB.InvokeRequired)
{
SetTextDelegate tmpDelegate = new SetTextDelegate(SetText);
tmpRTB.Invoke(tmpDelegate, new object[]{text});
}
else
{
tmpRTB.Text = text;
}
}

Plus élégant :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
private void implemente_ti()
    {
      if(this.textBox2.InvokeRequired)
      {
        this.Invoke(new MethodInvoker(delegate
        { implemente_ti(); }));
      }
      else
      {
        // On est sur le bon thread, on continue
        ti++;
        this.textBox2.AppendText("\r\n" + ti.ToString());
        if (ti <= 10000)
          implemente_ti();
      }
    }

Pas besoin de créer un nouveau type de delegate ou autre

Citation:

Envoyé par smyley

Plus élégant :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
private void implemente_ti()
    {
      if(this.textBox2.InvokeRequired)
      {
        this.Invoke(new MethodInvoker(delegate
        { implemente_ti(); }));
      }
      else
      {
        // On est sur le bon thread, on continue
        ti++;
        this.textBox2.AppendText("\r\n" + ti.ToString());
        if (ti <= 10000)
          implemente_ti();
      }
    }

Pas besoin de créer un nouveau type de delegate ou autre

En effet c'est mieux comme ça, je ne connaissais pas cette syntaxe, mais je l'adopte sur le champ :)

03/01/2008, 17h43
ZashOne

Ou tout simplement oublier les Thread dans ce cas et utiliser un BAckgroundWorker qui est bien plus adequat.
03/01/2008, 17h46
dahoo

Merci pour votre aide.
J'ai bien compris le truc je crois.