Discussion :

[la_praline]

Bonjour, je souhaiterai réaliser un affichage rafraichi toutes les 100ms.
J'ai dans une classe l'ensemble du code d'affichage et dans l'autre les méthodes de traitements le tout s'articule ainsi

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
 
public partial class DiagnosticAffichage : UserControl
    {
        private System.Timers.Timer temps;
        private Diagnostic diagnostic;
 
        public DiagnosticAffichage()
        {
            InitializeComponent();
            diagnostic = new Diagnostic();
            TimerAction();
        }
 
        public void TimerAction()
        {
            temps = new System.Timers.Timer();
            temps.Interval = 200;
            temps.Elapsed += new ElapsedEventHandler(AppelAffichage);
            temps.Enabled = true;
            temps.Start();
        }
 
        public void AppelAffichage(object source, ElapsedEventArgs e)
        {
            Affichage();
        }
 
        public void Affichage()
        {
            this.ramLibre.Text = diagnostic.getMemoire();
            this.processeurPourcentage.Text = diagnostic.getProcesseur();
            this.disqueLibre.Text = diagnostic.getDisque();
            this.kbytesSec.Text = diagnostic.getReseau();
        }
    }

J'ai l'erreur suivante qui apparaît :

Opération inter-threads non valide : le contrôle 'ramLibre' a fait l'objet d'un accès à partir d'un thread autre que celui sur lequel il a été créé.

Je regarde mais j'ai du mal à trouver quelque chose de compréhensible pour savoir comment dans mon cas je dois géré cette communication inter-thread.
Merci d'avance.

[Idrakis]

Salut,

Le problème ne provient probablement pas du bout de code que tu nous montre. Est-ce que tu manipules ton objet "ramLibre" à partir d'un autre endroit dans ton code ?

Je pense que tu dois le faire en passant directement ton objet ramLibre, or celui-ci s'éxecute déja sur un thread.
Si tu as besoin de manipuler cet objet depuis l'exterieur de ta form, tu va devoir créer un event.

Tu déclare ton event sur ta form.
Ta fonction qui manipule "ramLibre" sera, de préférence, également sur ta form, afin qu'elle connaisse son éxistence.
Et toi, de l'éxterieur, tu n'a plus qu'a déclenché cet event, en lui passant les paramètre dont tu as besoin.


Je crois que ceci devrait résoudre ton problème.

[la_praline]

ramLibre est un label qui fait partie de la classe DiagnosticAffichage qui est un UserControl.

DiagnosticAffichage est appellé dans une classe Fenetre qui est mon windows.form principal.

Les méthodes getMemoire(), getProcesseur() sont des méthodes implémentés dans la classe Diagnostic mais cette classe est un membres de DiagnosticAffichage donc ca m'étonnai aussi qu'on me parle de thread.

Le problème qui se pose pour ramLibre se posera également pour les 3 instructions suivantes.

C'est la première fois que j'appréhende les Threads j'imagine que doit faire communiquer le thread de mon windows.form avec ma classe DiagnosticAffichage. Ai je raison ou je fait fausse route ? Et comment faut il s'y prendre ?

[matdur]

Le problème c'est que tu ne peux pas modifier un contrôle à partir d'un autre thread que celui de l'IHM. Pour renvoyer le traitement vers le thread de l'IHM il faut appeler les fonctions Control.Invoke() ou Control.BeginInvoke()

[la_praline]

Ok je regarde depuis un long moment comment m'en servir donc si j'ai bien compris.

Dans mon windows.form je déclare en variable un Thread et puis je défini ce thread de cette manière :

Code :

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

1
2
3
 
myThread = new Thread(new ThreadStart(fonctionThread));
myThread.Start();

dans fonctionThread on a :

Code :

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

1
2
3
4
5
6
7
 
private void fonctionThread()
{
      diagnostic = new DiagnosticAffichage(this);
      this.layout.Controls.Add(diagnostic);
      diagnostic.Run();
}

et enfin dans ma classe DiagnosticAffichage

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
 
public DiagnosticAffichage(Fenetre fenetre)
        {
            InitializeComponent();
            diagnostic = new Diagnostic();
            this.fenetre = fenetre;
        }
 
        public void Run()
        {
            temps = new System.Timers.Timer();
            temps.Interval = 200;
            temps.Elapsed += new ElapsedEventHandler(AppelAffichage);
            temps.Enabled = true;
            temps.Start();
        }
 
        public void AppelAffichage(object source, ElapsedEventArgs e)
        {
            this.ramLibre.Text = "12";
        }
 
        public void Affichage()
        {
 
            this.ramLibre.Text = diagnostic.getMemoire();
            this.processeurPourcentage.Text = diagnostic.getProcesseur();
            this.disqueLibre.Text = diagnostic.getDisque();
            this.kbytesSec.Text = diagnostic.getReseau();
        }

Ce que je ne comprends pas c'est à quel moment j'appelle Control.Invoke et comment ?

[matdur]

Je pense que le code suivant tu peux le supprimer

Code :

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

1
2
3
 
myThread = new Thread(new ThreadStart(fonctionThread));
myThread.Start();

et tu le remplaces directement par

Code :

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

1
2
3
4
 
      diagnostic = new DiagnosticAffichage(this);
      this.layout.Controls.Add(diagnostic);
      diagnostic.Run();

Le Invoke() ou BeginInvoke() tu l'appelles dans AppelAffichage(), tu fais un truc du style

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
 public void AppelAffichage(object source, ElapsedEventArgs e)
        {
            if ( this.InvokeRequire)
            {
                 BeginInvoke( AppelAffichage )
            }
            else
            {
                 this.ramLibre.Text = "12";
            }
        }