Discussion :

[sCove]

Bonjour,

Je développe une application industrielle (framwork .Net 1.1 & visual studio 2003) et je rencontre un problème avec mes threads.

Voici grossièrement le fonctionnement souhaité :
j'ai une classe "camera" que j'instancie 3x (camera1, camera2 & camera3). Dans la définition de la classe camera j'ai une fonction membre qui doit être executé dans un thread lors de la création de l'objet, mais je voudrais créer ce thread qu'une seule fois. Je l'ai donc déclaré en static (ainsi que toutes les variables dont le thread à besoin).

Le problème que je rencontre est que lors du débuggage je vois 3 threads tourner. La question que je me pose (et que je vous soumets ) est : l'attribut statique fontionne t'il sur les threads de la même façon que sur les variables ? Ou alors je m'y prends mal en instanciant et demarrant mon thread dans le constructeur de ma classe et il faut que je le fasse par une methode publique appelé postérieurement ?

Merci d'avance

[StormimOn]

Pour te répondre le plus simple serait de voir un peu de code, l'ossature globale, de manière à bien comprendre ce que tu fais. Parce que comme ca, ce n'est pas facile de voir ^^

[sCove]

Structure de ma classe camera

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
 
public class camera
{
 private static Thread gestionGrafcet;
 ...
 
 private static void threadGrafcet()
 {
   ...
 }
 
 ...
 public camera()
 {
   camera.gestionGrafcet = new Thread(threadGrafcet);
   camera.gestionGrafcet.Start();
 }
}

classe appelante

Code :

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

1
2
3
4
5
6
 
...
camera maCamera1 = new camera();
camera maCamera2 = new camera();
camera maCamera3 = new camera();
...

J'espère que cela sera plus compréhensible que mon explication précédente (déja pas très claire pour moi :p )

Avec ce code je me retrouve avec 3 threads "threadGrafcet" alors qu'en passant le thread en static je pensais en obtenir qu'un seul.

[scifire]

Salut.
A chaque fois que tu appelles le constructeur tu crees un nouveau Thread et la reference est assigne a la variable gestionGrafcet. Alors dans ton exemple tu finis par creer trois threads et seulement la derniere reference est gardee. Les deux premiers threads tournent mais tu ne peux rien faire avec car tu n'as pas des references sur eux
Je te propose d'utiliser la classe suivante

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
    public class Camera
    {
        private static object keyLock = new object();
        private static Thread gestionGrafcet = null;
 
        private static void threadGrafcet()
        {
 
        }
 
        public Camera()
        {
            lock (keyLock)
            {
                if (gestionGrafcet == null)
                {
                    gestionGrafcet = new Thread(threadGrafcet);
                    gestionGrafcet.Start();
                }
            }
        }
    }

Bon courage

[scifire]

J'ai pense un petit peu et si moi j'etais a ta place je vais creer un seul gestionnaire(singleton) qui peut gerer un nombre variable de cameras. Il est possible d'ajouter/supprimer des cameras; arreter/redemarrer le gestionnaire.

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
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
public class GestionGrafcet
{
    private Thread task = null;
    private volatile bool running = false;
 
    private static readonly object keyLock = new object();
    private static GestionGrafcet instance = null;
    public static GestionGrafcet Instance
    {
        get
        {
            lock (keyLock)
            {
                if (instance == null)
                {
                    instance = new GestionGrafcet();
                }
            }
            return instance;
        }
    }
 
    private GestionGrafcet()
    {
        this.cameras = new List<Camera>();
    }
 
    public void Start()
    {
        if (!running)
        {
            running = !running;
 
            task = new Thread(threadGrafcet);
            task.Start();
        }
    }
 
    public void Stop()
    {
        if (running)
        {
            running = !running;
 
            task.Join();
        }
    }
 
    private void threadGrafcet()
    {
        while (running)
        {
            foreach (Camera c in this.Cameras)
            {
                // ...
            }
        }
    }
 
    private List<Camera> cameras;
 
    public List<Camera> Cameras
    {
        get { return cameras; }
        set { cameras = value; }
    }
}
 
public class Camera
{
}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
            Camera c = new Camera();
            Camera c2 = new Camera();
 
            GestionGrafcet gestion = GestionGrafcet.Instance;
            gestion.Start();
 
            gestion.Cameras.Add(c);
            gestion.Cameras.Add(c2);
 
            // ...
            gestion.Cameras.Remove(c);
 
            gestion.Stop();

Tu peux bien sur modifier ce code pour mieux correspondre a tes besoins.

A+

[sCove]

Merci, en effet ton code me semble plus cohérent que le mien.

Merci encore