Discussion :

[thomas55]

J'essaye d'utiliser un thread dans une Windows Forms, mais je rencontre un problème lorsque j'utilise la croix pour fermer la fenêtre avant la fin du thread. Le thread ne semble pas s'interrompre correctement.
Cela me lance une exception : 'Object Disposed Exception' cannot access a dispoded object

Merci de votre aide.

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
 
     public partial class Form1 : Form
    {
 
        private bool thread_stop = false;
        Thread t=null;
        public delegate void deleg(string x);
 
        private void button1_Click(object sender, EventArgs e)
        {
            t = new Thread(new ThreadStart(le_thread));
            thread_stop = false;
            t.Start();
        }
 
        public void le_thread()
        {
            Random r = new Random();
            int res = 0;
            for (int i = 1; i < 1000; i++)
            {
                if (thread_stop) { return; }
 
                Invoke((deleg)maj_txt, i.ToString());
 
                for (int j = 1; j < 1000; j++)
                {
                    for (int k = 1; k < 1000; k++)
                    {
                        res = k + r.Next();
                    }
                }
            }
        }
 
        public void maj_txt(string x)
        {
            info.Text = x;
        }
 
        private void Form1_FormClosing(object sender, FormClosingEventArgs e)
        {
            thread_stop = true;
            if(t!=null)
                if (t.IsAlive || t.ThreadState == System.Threading.ThreadState.Running)
                {
                    t.Join(1000);
                }
 
        }
    }

[clemou01]

Bonjour,

D'après ce que j'ai lu, normalement les Threads se ferment automatiquement. Donc es-tu sûr que c'est ton Thred qui pose un problème.
Deuxièmement, es-tu sûr de cette syntaxe :

Code :

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

t.Join(1000);

En regardant la méthode sur le MSDN, on dirait que ta méthode n'attend pas d'attribut.

Cordialement,
Clemou01

[thomas55]

Bonjour Clemou01,

Mon problème vient du fait que je veux interrompre le thread lors de l'évènement 'FormClosing' qui est déclenché lorsque l'on clique sur la croix rouge de la fenêtre, si je le stop à un autre moment ça marche.
Pour le Join j'ai essayé sans argument la fenêtre ne se ferme pas et reste bloquée.

Merci

[PixelJuice]

Join() n'est pas vraiment fait pour ça , essaye plutôt ça :

Code :

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

t.Abort();

[Jaco67]

Salut,

Le soucis se trouve dans ta méthode Invoke() je pense : lorsque tu cliques sur la croix pour fermer l'application, la textbox est "disposed", et lorsque tu veux y affecter ta valeur il te balance l'exception.

Si j'étais toi je ne m'embêterais pas avec les threads : regarde du côté du BackgroundWorker.

[Invité]

Bonjour,
Ton idée de fermer proprement le thread est une bonne idée. Après, il y a 2 soucis qui sont combinés : dans ta grosse boucle, tu testes ta variables "thread_stop" à un niveau trop haut, du coup, lorsque tu fais le "Join", tu n'attends qu'une seconde et tu sors tout de suite... Pendant ce temps, dans le thread, avec "Invoke" tu tentes d'accéder à un contrôle qui n'existe plus.
La solution rapide dans ce cas, consiste à déplacer le test du "thread_stop" à un niveau plus bas dans l'imbrication des boucles :

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
public void le_thread()
        {
            Random r = new Random();
            int res = 0;
            for (int i = 1; i < 1000; i++)
            {
 
                Invoke((deleg)maj_txt, i.ToString());
 
                for (int j = 1; j < 1000; j++)
                {
                    for (int k = 1; k < 1000; k++)
                    {
                        if (thread_stop) { return; }
                        res = k + r.Next();
                    }
                }
            }
        }

Après, il va faire les tests plus souvent mais bon...

Moi je te conseille d'utiliser la classe "Task" à la place de la classe Thread...

Au passage, petit commentaire sur les autres réponses

Bonjour,

D'après ce que j'ai lu, normalement les Threads se ferment automatiquement. Donc es-tu sûr que c'est ton Thred qui pose un problème.
Deuxièmement, es-tu sûr de cette syntaxe :

Code :

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

t.Join(1000);

En regardant la méthode sur le MSDN, on dirait que ta méthode n'attend pas d'attribut.

Cordialement,
Clemou01

Tu as râté la surcharge avec un entier ici. La surcharge de thomas existait bien...

Pour le Join j'ai essayé sans argument la fenêtre ne se ferme pas et reste bloquée.

Oui, en effet pour la raison évoquée plus haut...

Join() n'est pas vraiment fait pour ça , essaye plutôt ça :

Code :

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

t.Abort();

Ah oui, ça fonctionne pour cette situation mais c'est très violent et ce n'est plus recommandé par Microsoft. Dans ce cas-ci, ça ne pose pas de problème mais si on est dans un contexte d'écriture dans un fichier ou une base de données par exemple cela risque de compromettre l'intégrité des données.
Encore une fois, la classe "Task" et le "Task-based Asynchronous Pattern" sont recommandés dans tous les cas, les classes "Thread" et "BackgroundWorker" sont hasbeen. Le "TAP" permet d'inclure un jeton d'annulation permettant d'indiquer juste à la tâche qu'on demande son arrêt mais c'est la tâche qui va décider quel est le moment le plus opportun pour s'arrêter.

Si j'étais toi je ne m'embêterais pas avec les threads : regarde du côté du BackgroundWorker.

BackgoundWorker est hasbeen maintenant. Voici le lien que je montrerai plutôt :
http://msdn.microsoft.com/fr-fr/libr...v=vs.110).aspx