Discussion :

[Nephi]

Bonjour,

Je suis en train de développer une application multi thread avec le backgroundWorker pour établir une communication bluetooth avec un module Mindstorm NXT. Tout va bien sauf que mon application est considérablement ralentit. J'ai une boucle while dans Mon background worker qui tourne en permanence une fois la communication etablit. Du coup j'ai l'impression que le background worker a la priorité sur le thread de la mainform et du coup mon application n'est plus fluide du tout.
Comment rendre la priorité au thread du mainform afin que mon application reste fluide ou existe t'il un moyen de garder la main sur la mainform sans ralentir le background worker.

merci d'avance.

[tomlev]

Il y une propriété Priority dans la classe Thread. Mais à mon avis le thread du BGW n'a pas une plus grande priorité que le thread de l'UI... ça doit simplement être que ta boucle bouffe du CPU inutilement. Rajoute un petit Sleep(0) dans ta boucle, histoire que le thread du BackgroundWorker rende la main plus souvent.

[Nephi]

deja essayé ^^, j'avai mis un thread.sleep(250) mais aucun effet, je ne pense pas que le thread monopolise le cpu unitilement car vu le code dans le corp de la boucle while ya assez a faire XD. de plus disposant d'un intel core I7 a 2.80Ghz et le monitorant en direct il est pas surcharger du tout. a peine 10% d'augmentation sur un 1 core. par contre mon application est lente mais le reste du systeme lui est fluide c'est la que je ne comprend pas.

[tomlev]

Tu fais des appels au thread UI à partir de ton BGW (Invoke ou ReportProgress) ?

[kzrystof]

Salut,

Tu pourrais utiliser la classe Thread à la place du BackgroundWorker, d'attribuer la propriété adéquate (BelowNormal genre) tel que mentionné plus haut. Si tu as besoin de faire des mises à jour sur l'état des opérations dans le UI, tu n'auras qu'à pousser ces informations sur le Thread UI depuis ton Thread de traitement.

Qu'en penses-tu ?

[tomlev]

Tu pourrais utiliser la classe Thread à la place du BackgroundWorker, d'attribuer la propriété adéquate (BelowNormal genre) tel que mentionné plus haut.

Pas la peine de créer un thread explicitement, tu peux tout à fait le faire dans un BackgroundWorker :

Code :

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

Thread.CurrentThread.Priority = ThreadPriority.BelowNormal;

[Nephi]

Bon j'ai essayer un peu tout, je retrouve un peu de fluidité en mettant un sleep avant l'appel du delegate mais mon appli est toujours saccader.
voila carrément les bout des code.
La partie delegate :

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
namespace NxtStatus10
{
    public partial class MainForm
    {
        bool DistribStatusThread = false;
        bool PnPStatusThread = false;
 
        private delegate void GetDistribStatus();
 
        private void DistribStatus()
        {
            Nxt1MotorASt = Nxt1.GetOutputState(MotorPort.PortA);
            Nxt1MotorBSt = Nxt1.GetOutputState(MotorPort.PortB);
            Nxt1MotorCSt = Nxt1.GetOutputState(MotorPort.PortC);
            LblNxtDistribBatLvl.Text = string.Format("{0} mV", Nxt1.GetBatteryLevel().ToString());
            LblNxt1MotAPowerLvl.Text = Nxt1MotorASt.Power.ToString();
            LblNxt1MotAModeLvl.Text = Nxt1MotorASt.Mode.ToString();
        }
    }
}

et la partie mainform avec l'appelle du delegate dans la boucle du bgworker

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
namespace NxtStatus10
{
    public partial class MainForm : Form
    {
      . 
      .
      .
        private void BgWkDistribStatus_DoWork(object sender, System.ComponentModel.DoWorkEventArgs e)
        {
            Thread.CurrentThread.Priority = ThreadPriority.BelowNormal;
            while (DistribStatusThread)
            {
                if (BgWkDistribStatus.CancellationPending)
                {
                    e.Cancel = true;
                    Nxt1.Disconnect();
                    DistribStatusThread = false;
                }
                else
                {
                    Thread.Sleep(300);
                    this.Invoke(new GetDistribStatus(DistribStatus));
                }
            }
        }
        .
        .
      }
}

Faut il que je fasse appel a la fonction ReportProgress du bgworker et que je traite l'appel du delegate dans la parie ProgressChanged ? ou y a t'il quelque chose que j'ai encore louper ?
merci de votre intérêt.

[fiberMan]

Bonjour,

Je suis en train de développer une application multi thread avec le backgroundWorker pour établir une communication bluetooth avec un module Mindstorm NXT. Tout va bien sauf que mon application est considérablement ralentit. J'ai une boucle while dans Mon background worker qui tourne en permanence une fois la communication etablit. Du coup j'ai l'impression que le background worker a la priorité sur le thread de la mainform et du coup mon application n'est plus fluide du tout.
Comment rendre la priorité au thread du mainform afin que mon application reste fluide ou existe t'il un moyen de garder la main sur la mainform sans ralentir le background worker.

Le fonctionnement de BackGroundWorker est limité par rapport à l'utilisation de la classe Thread. Si j'étais toi, j'utilise La Thread et utiliser XXX.Invoke(delegate) pour se communiquer avec l'UI

[tomlev]

voila carrément les bout des code.

Euh... là tu fais quasiment tout le boulot sur le thread UI !
Tout le boulot est fait dans DistribStatus, et tu exécutes cette méthode via Invoke, donc sur le thread UI... je suppose que c'est la méthode GetOutputState qui prend du temps ?

Dans ce cas tu devrais faire comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
        private void DistribStatus()
        {
            Nxt1MotorASt = Nxt1.GetOutputState(MotorPort.PortA);
            Nxt1MotorBSt = Nxt1.GetOutputState(MotorPort.PortB);
            Nxt1MotorCSt = Nxt1.GetOutputState(MotorPort.PortC);
            this.Invoke(new Action(UpdateUI));
        }
 
        void UpdateUI()
        {
            LblNxtDistribBatLvl.Text = string.Format("{0} mV", Nxt1.GetBatteryLevel().ToString());
            LblNxt1MotAPowerLvl.Text = Nxt1MotorASt.Power.ToString();
            LblNxt1MotAModeLvl.Text = Nxt1MotorASt.Mode.ToString();
        }

Et appeler directement DistribStatus à partir de ton BackgroundWorker, sans Invoke. De cette façon GetOutputState sera exécuté en arrière plan, et seule la mise à jour des Labels se fera sur le thread UI.

Le fonctionnement de BackGroundWorker est limité par rapport à l'utilisation de la classe Thread.

Limité en quoi ? Peux-tu préciser ta pensée ? A part le fait que tu ne peux pas préciser le ThreadApartment vu que ce n'est pas toi qui démarres le thread, je ne vois pas trop ce qu'il y a comme limite... Et le ThreadApartment, à moins que tu utilises des composants COM, tu t'en fous

[Nephi]

Bon effectivement c’était un problème de répartition des tache. je me suis fourvoyer en pensant que le delegate était exécuter par le BGWorker en non l'UI.
Du coup c'est bon tout fonctionne impec.
merci a vous