Discussion :

[jeremm]

Bonjour,

J'utilise un Timer pour actualiser des infos provenant de mon hardware. L'intervalle est toutes les 3 secondes.
Le problème est que toutes les 3 secondes, quand il interroge le hardware, la fenêtre se freeze le temps de récupérer les infos, genre 500ms.
Ces problèmes de lenteur sont embêtantes.
Pourrais-je à la place utiliser un thread à part qui se lancerait toute les 3 secondes et qui ne freezerait pas ma fenetre ?
Si oui comment ?
Mais, le "tick" du timer ne créer pas déjà un thread à part ?

Merci de m'éclairer sur la différence entre les deux, et quel serait le mieux niveau performance pour mon souci.

[azstar]

c normal que votre fenettre se friiizzz puisque le Timer utilise le thread principale de votre application ce lui qui cree et met a jour l'interface de ton application.

la solution que je te conseil de faire est utilise le backgroundworker avec bien sur votre Timer

Code :

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

1
2
3
4
5
6
 
//double click sur le backgroundworker 
private void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e)
        {
   ///ton code pour actualiser des infos provenant de mon hardware
        }

Code :

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

1
2
3
4
5
 
private void timer1_Tick(object sender, EventArgs e)
        {
           backgroundWorker1.RunWorkerAsync();
        }

[jeremm]

Merci pour ton aide.
Mais à vue d'œil quand le timer va appeler le backgroundworker, il va attendre la fin du backgroundworker non ? Donc freezer ?

[azstar]

Mais à vue d'œil quand le timer va appeler le backgroundworker, il va attendre la fin du backgroundworker non ? Donc freezer ?

la fonction RunWorkerAsync() lance le backgroundworker en mode asynchrone .alors en thread parallèle que le thread Princopale
donc pas de freezer de form(thread principale).

[jeremm]

ok, je vais tenter.

Sinon, est-ce qu'un thread avec un sleep() à l'intérieur ferait bien le boulot ??

[azstar]

oui si tu le fait sleeping pendent le 3s mais a condition que tu doit le faire maintenir meme apres la fin de son traitement soit avec

Code :

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

1
2
3
4
5
6
7
8
9
 
//dans la conception de form
 backgroundWorker1.RunWorkerCompleted += new RunWorkerCompletedEventHandler(this.RelancemonThread);
//dans ta form 
private void RelancemonThread(object sender, RunWorkerCompletedEventArgs  e)
        {
            backgroundWorker1.RunWorkerAsync();
 
        }

[jeremm]

J'ai tenté le code qui à chaque tick du timer appelle le BackgroundWorker, mais exception :

BackgroundWorker est actuellement occupé et ne peut pas exécuter plusieurs tâches simultanément.

Si je le Cancel avant de le lancer, je n'aurai pas mes infos !
Comment faire ?

[azstar]

c normal puisque le backgroundworker n'a pas encore termine son traitement
et il n'est pas encore completed

ça vous dire que le traitement de associe a votre backgroundworker nessicite plus que 3 s ou vous n'avez pas bien l'utlise dans ton code tu peut me donnee ton code ..
j'att votre reponse

[ppphil]

Et avec
System.Timers.Timer
à la place de
System.Windows.Forms.Timer
Ca n'irait pas mieux ?

[jeremm]

Quelle est la différence ?

[ppphil]

System.Windows.Forms.Timer tourne dans le thread de la Form dans laquelle il est déclaré.
System.Timers.Timer tourne dans un thread séparé.
Ce qui fait que si le traitement est long dans le Elapsed (pour Timers.Timer et Tick pour Forms.Timer) il ne freeze pas ta form.
Par contre il faut faire attention à l'accès aux composants de ta form qui ne sera plus threadsafe.

[azstar]

il faut faire des invokes pour appel les fonctions de mise des composantes bien sûr a travers des delegates

[jeremm]

Merci pour votre aide.
C'est justement ça que je ne parviens pas à faire, rafraichir un label qui est dans ma form en utilisant System.Timers.Timer ?
Avez-vous un bout de code, je ne trouve pas ça sur la msdn ?
Merci

[azstar]

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
 
        delegate void delegateaffiche (string msg);//vous declarez votre type delegate sur un type de fonction qui prend comme parametre un string et qui returne un void 
 
        delegateaffiche dle;//vous instancie un objet de ce type 
 
 
            dle = new delegateaffiche(affiche);//vous relie votre type dle avec la fonction affiche 
 
       // la fonction de mettre a jour votre appel 
        private void affiche(string msg)
        {
         label1.Text=msg ;
 
        }
 
 
 
 
//ici vous utilise la type delegate les paramettre de votre fonction mettre a jour votre label son entre sous form d'un table d'objet dans notre cas on un seul paramettre a lors , si vous avez plus alors object param={"param1","param2 ",parm3 } param3 deja declare ...
        private void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e)
        {
            object[] param = { "hi backgroundworker1" };
 
            label1.Invoke(dle, param);//pour invoker le delege de votre fonction de mise a jour 
        }

j'espere etre clear ...

[jeremm]

Merci pour ton aide.
Voilà mon code de test, mais erreur de compilation car mon label1 n'est pas en 'static'.

"Une référence d'objet est requise pour la propriété, la méthode ou le champ non statique"

Mon but n'est pas de le mettre en static..
Comment faire ?!

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
public partial class Form1 : Form
    {
        System.Timers.Timer testTimer = new System.Timers.Timer();
        delegate void delegateaffiche();
        delegateaffiche dle = new delegateaffiche(affiche);
 
        public Form1()
        {
            InitializeComponent();
 
            testTimer.Interval = 1000;
            testTimer.Elapsed += new System.Timers.ElapsedEventHandler(testTimer_Elapsed);
            testTimer.Start();
    }
 
        private static void affiche()
        {
// Erreur : Une référence d'objet est requise pour la propriété, la méthode ou le champ non statique
             label1.Text = DateTime.Now.ToShortTimeString();
        }
 
        private void testTimer_Elapsed(object sender, System.Timers.ElapsedEventArgs e)
        {
            label1.Invoke(dle);
        }
    }

[azstar]

j'ai teste votre code il est great ça marche bien
et j'ai remarque

Code :

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

1
2
 
delegateaffiche dle = new delegateaffiche(affiche);

l'initialisation de object se fait au constructeur ou au fonction load de votre form.

[jeremm]

Je l'ai mis dans le constructeur :

Code :

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

1
2
3
4
5
public Form1()
{
    dle = new delegateaffiche(affiche);
    ...
}

Mais toujours la meme erreur !

label1 : Une référence d'objet est requise pour la propriété, la méthode ou le champ non statique

[azstar]

j'ai teste ton code et ça marche pour moi vous me donner tout votre code form

[jeremm]

Pour ne pas pourir le forum, cf le fichier rar en piece jointe.
Merci !

[azstar]

ah bon j'ai pas vu que vous avez declare votre fonction affiche comme une fonction static enleve le static de votre fonction affiche .

Code :

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

1
2
3
4
5
6
7
 
 
private void affiche()
        {
 
             label1.Text = DateTime.Now.ToShortTimeString();
        }