Discussion :

[TraPpeur]

Bonjour à tous.

J'ai une classe qui lance un thread dans le constructeur, thread qui tourne en permanence.

Ce thread doit s'arrêter lorsque'il n'existe plus de référence de l'objet qui l'a lancé. Ce qui n'est pas le cas avec le code suivant :

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
 class Toto
    {
        Thread t;
 
 
        volatile bool stop = false;
        public Toto()
        {
            t = new Thread(new ThreadStart(executingMethod));
            t.IsBackground = true;
 
            t.Start();
        }
 
        private void executingMethod()
        {
            Thread.Sleep(1000);
            while (! stop)
            {
                Console.WriteLine("Toto active");
                Thread.Sleep(1000);
            }
        }
 
        private void Stop()
        {
            Console.WriteLine("Stop called");
            stop = true;
        }
 
    }
    class Program
    {
 
        static void Main(string[] args)
        {
            Toto t = new Toto();
            Console.ReadLine();
            t = null;
 
            // J'aimerai que le thread s'arrete à ce moment la...
 
           Console.ReadLine();
 
        }
    }

J'aimerais que le thread s'arrête quand on n'utilise plus l'objet. Est-ce qu'il y a un moyen de savoir que l'objet n'a plus de référence vers lui, pour arrêter le thread automatiquement ?

Je ne sais pas si j'ai été clair.

Merci d'avance pour vos réponses !

[Themacleod1980]

salut,
pourquoi ne pas mettre Stop() en public et faire un t.Stop() ?

sinon je dirais faire une fonction qui dés-allouerait la référence (dispose() ?)et déclenchant un évènement qui exécuterait ce Stop().
(en même temps.. autant faire un t.Stop() directement ...)

hum je m'embrouille...
mais je ne pense pas que ton thread puisse savoir que ton main n'a plus de référence vers lui...

[TraPpeur]

Bonjour, merci pour ta réponse !

Le code que j'ai donnée est un exemple simplifié pour comprendre la problématique.

Je voudrais justement que si un appel vers le "stop" ou le "dispose" n'est pas fait (cas d'un oubli), le thread s'arrête bien...

[ppphil]

Et si tu oublies de faire t = null; ????
C'est pas mieux que d'oublier de faire un joli Dispose()....

[TraPpeur]

Comme je disais, ce code est la pour le test et l'exemple.

Il faut imaginer le cas ou ma classe "Toto" qui inclus le thread est embarqué dans une dll "utilitaire".

Un programme se sert de l'utilitaire, puis quand il n'en a plus besoin, la référence de l'instance toto qu'il utilisait est perdue. A ce moment la, je voudrais être sur que le thread s'arrêtera.

Je préfèrerais masquer au programme qui se servirait de "Toto" qu'il y a un thread qui tourne derrière et qu'il doit appeler la méthode "stop" avant de perdre la référence...

[nicolas.sitbon]

Tu peux utiliser le finaliser le l'objet qui à lancer le thread, mais son appel n'est déterministe. Voici un petit programme qui illustre le principe, je force simplement le garbage collector, afin que celui ci appelle bien le finaliseur.

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
using System.Net;
using System.Threading;
using System;
 
namespace ConsoleApplication1
{
    class Toto
    {
        Thread thread = new Thread(ThreadMain);
 
        static public void ThreadMain()
        {
            while (true)
            {
                Console.WriteLine("thread running");
                Thread.Sleep(1000);
            }
        }
 
        public Toto()
        {
            Console.WriteLine("start thread");
            thread.Start();
        }
 
        ~Toto()
        {
            Console.WriteLine("abort thread");
            thread.Abort();
        }
 
    }
 
    class Program
    {
        static void CreateStarterObject()
        {
            Toto t = new Toto();
            Thread.Sleep(1000);
        }
 
        static void Main(string[] args)
        {
            Console.WriteLine("Creating object");
            CreateStarterObject();
            Console.WriteLine("forcing garbage collection");
            GC.Collect();      
            Console.ReadLine();
        }
    }
}

Cordialement.