Discussion :

[jeremm]

Bonjour,

J'essaye de lire des données provenant de mon hardware et de les afficher dans un label, le tout dans un timer.
J'utilise System.Timers.Timer pour cela :

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 partial class Form1 : Form {
     delegate void delegateDisplay();
     delegateDisplay dlgLbl;
 
     public Form1() {
          InitializeComponent();
          dlgLbl = new delegateDisplay(this.RefreshInfos);
     }
 
     private void RefreshInfos() {
 
          // récupère ici les infos du harware
          // cela prend environ 2,5 sec
          // ensuite j'affiche le résultat dans un label : 
          label1.Text = myHardware.DriverInfo.ToString()  // exemple
     }
 
     private void timer1_Elapsed(object sender, System.Timers.ElapsedEventArgs e) {
 
          label1.Invoke(dlgLbl);
     }
}

Problème :
Le temps de récupérer les infos du hardware prend environ 2,5 sec.
Pendant ce temps, la fenêtre où j'affiche mon label freeze, ne répond pas, puisqu'elle communique avec le hardware pour récupérer les infos.
Pourtant je croyais qu'en utilisant Invoke je passais par un thread et ça m'éviterait ce genre de problème d'attente ?

Donc, comment pourrais-je modifier mon code pour que la fenêtre ne freeze pas ?

Merci.

[chamamo]

Utilise BeginInvoke au lieu de Invoke

BeginInvoke réalise cet appel de manière asynchrone (le programme n'attend pas qu'elle soit terminée pour continuer son exécution), alors qu'Invoke le fait de manière synchrone (le programme bloque tant que la fonction n'est pas terminée)

[jeremm]

Ca ne change rien.
Le problème est toujours là, il freeze pendant 2secondes le temps de récupérer les infos.

[chamamo]

essaie:

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
 
 
void RefreshInfos()
        {
            if (label1.InvokeRequired)
            {
                label1.BeginInvoke(new delegateDisplay(RefreshInfos));
                // ou
                label1.Invoke(new delegateDisplay(RefreshInfos));
            }
            else
            {
                label1.Text = myHardware.DriverInfo.ToString();
            }
        }

[azstar]

le mode asynchrone s'exécute dans un thread parallèle, ce qui nous permet une bon performance au niveau des traitements .

tu peut soit utilise le mode asynchrone ou utilise un thread qui ce charge de récupère les infos de ton hardward ;pour éviter de faire le traitement dans un timer .
car le timer s'exécute dans le thread principale de ton application y'a compri ton interface pour ce la vous avez un freeez dans votre form

[Ed.11]

Effectivement les timers de la classe System.Timers bloquent le thread principal. En utilisant les timers de System.Threading tu lances ton timer en arrière plan.

Essaye si tu veux:

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
 
 
private System.Windows.Forms.Label m_label;
private System.Threading.Timer m_myTimer;
private bool pendule_lance = false;
 
public void startPendule(System.Windows.Forms.Label label, int periode)
		{
			m_label = label; //le label que tu veux actualiser
 
			TimerCallback timerDelegate = new TimerCallback(this.OnTimeEvent);
 
			//Crée le timer qui lance le délégué 
			m_myTimer = new System.Threading.Timer(timerDelegate, null, periode, periode);
			pendule_lance = true;
		}
 
		private void OnTimeEvent(Object objet)
		{
			// récupère ici les infos du harware
                        // cela prend environ 2,5 sec
                        // ensuite j'affiche le résultat dans un label : 
                        m_label.Text = myHardware.DriverInfo.ToString();  // exemple
 
		}
 
                // à appeler dans le forme principale
		public void stopPendule()
		{
			if (pendule_lance)
			{
				m_myTimer.Dispose();
			}
		}

Ensuite tu lances startPendule dans ta forme principale, et le stopPendule dans le Dispose de la forme.

[jeremm]

chamamo > Merci, mais ca ne change rien car il ne rentre jamais dans le "if (label1.InvokeRequired)"..

Ed.11 > Je vais tester merci.

[jeremm]

Effectivement les timers de la classe System.Timers bloquent le thread principal. En utilisant les timers de System.Threading tu lances ton timer en arrière plan.

J'ai essayé ton code, mais j'ai une exception ici :

Code :

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

1
2
3
4
5
private void OnTimeEvent(Object objet)
{
// "Opération inter-threads non valide : le contrôle 'label1' a fait l'objet d'un accès à partir d'un thread autre que celui sur lequel il a été créé."
         m_label.Text = "test";
}

Ci-joint mon code complet.

Merci pour ton aide..