[Thread] Form ProgressBar qui ne bloque pas le traitement

Version imprimable

31/08/2010, 10h44
alavoler

[Thread] Form ProgressBar qui ne bloque pas le traitement
Bonjour,

je suis en .NET 2.0 C#

J'aimerai faire une petite classe générique qui permet de faire un truc somme toute classique: afficher les étapes d'un traitement en cours avec une barre de progression (progressbar)

Comment m'y prendre pour que ca ne bloque pas le traitement qui tourne derrière ?
Code:

1 2 3 4 5 using (StepForm f = new StepForm ()) { f.Show(); }
je sais que Show permet de faire en sorte que ce soit non modal... mais après je ne sais pas comment réaliser le reste.
Que me conseillez vous ? Quelle est la bonne pratique ?
D'utiliser Invoke, un backgroundWorker ?

Merci du coup de main & Bonne journée à tous
31/08/2010, 14h04
rori

Utiliser les Threads est une bonne idée. Vous avez un exemple dans les tuto :
http://drq.developpez.com/dotnet/articles/threads/#L5
31/08/2010, 14h46
alavoler

rori
Et bien c'était pile poil ce que je recherche !
(j'ai du mal chercher)

Je te remercie et je te souhaite de passer une agréable journée !

Hum par contre je souhaite mettre un bouton annuler dans ma Form ou il y a la progression, seulement celui si est "bloqué" pour le moment pendant la progression...

Donc j'ai 2 form:
Une principal qui appel la form de progression ainsi:

Code:

1
2
3
4
5
6
7
8
9
10
11
            using (StepForm f = new StepForm ())
            {
                f.Show();
                int progression = 1;
                while (f.GoToStep(progression) && progression < 100)
                {
                    System.Threading.Thread.Sleep(50);
                    ++progression;
                }
                f.Close();
            }

dans ma form de progression il y a:

Code:

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
    public partial class StepForm : Form
    {
 
        private delegate void ProgressBarDelegateHandler(int step);
        private ProgressBarDelegateHandler ProgressBarDelegate;
 
        public StepForm()
        {
            InitializeComponent();
 
            ProgressBarDelegate = new ProgressBarDelegateHandler(UpdateProgressBar);
        }
 
        private void UpdateProgressBar(int step)
        {
            progressBar1.Value = step;
        }
 
        public bool GoToStep(int progression)
        {
            this.Invoke(this.ProgressBarDelegate, new object[] { progression });
            this.Update();
            return true;
        }
    }

EDIT/
BeginInvoke ?

Arf j'ai essayé avec la methode BeginInvoke... mais le bouton est toujours bloqué....

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
        private delegate void InvokeMethod(int step);
 
        public StepForm()
        {
            InitializeComponent();
 
        }
 
        private void UpdateProgressBar(int step)
        {
            progressBar1.Value = step;
        }
 
        public bool GoToStep(int progression)
        {
 
            IAsyncResult res = progressBar1.BeginInvoke(new InvokeMethod(UpdateProgressBar), new object[] { progression });
            progressBar1.EndInvoke(res);
            this.Update();
            return true;
        }

C'est normal, vous n'utilisez pas les Threads !

Dans votre exemple, tout se fait sur le Thread de votre Main.

Regarder bien dans le tutorial que je vous ai link :
Code:

1 2 3 4 5 6 7 8 9 10 11 Thread t = new Thread(new ThreadStart(ThreadProcess)); t.Start(); private void ThreadProcess() { for(int i = 0 ; i < 11 ; i++) { // Déclenche la mise à jour de l'interface this.Invoke(this.ProgressBarDelegate,new object[] {i}); Thread.Sleep(1000); } }
Ici c'est bien un nouveau thread (t) qui gére la mise à jour de la ProgressBar.

Dans l'exemple il utilise tout depuis une meme form

Moi j'ai 2 forms: la principale qui 'control' la seconde en lui indiquant à quelle étape elle en est...
Bon j'ai essayé de threader l'appel sur la form principal:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
        private void TesterStepForm_Click(object sender, EventArgs e)
        {
            Thread t = new Thread(new ThreadStart(ThreadProcess));
            t.Start();
        }
 
        private void ThreadProcess()
        {
 
            using (ProgressBarEtapeForm f = new ProgressBarEtapeForm())
            {
                f.Show();
                int progression = 1;
                while (progression < 100)
                {
                    f.Invoke(f.ProgressBarDelegate, new object[] { progression });
                    System.Threading.Thread.Sleep(50);
                    ++progression;
                }
                f.Close();
            }
        }

et sur la StepForm:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
        public bool GoToStep(int progression)
        {
 
            Thread t = new Thread(new ThreadStart(ThreadProcess));
            t.Start();
            return true;
 
        }
 
        private void ThreadProcess()
        {
            IAsyncResult res = progressBar1.BeginInvoke(new InvokeMethod(UpdateProgressBar));
            progressBar1.EndInvoke(res);
            this.Update();
        }

bon jdois etre à coté de la plaque...

:::: EDIT ::::
// Là je suis un peu dans les choux niveau code, si c'est possible pourriez vous me dire à quel endroit il faut threader en reprenant mon précédent code

Voilà comment je m'y prendrai :

- Votre "StepForm" qui contient la progressBar possède une méthode pour la mise à jour de sa ProgressBar
Code:

1 2 3 4 5 public void UpdateProgressBar(int step) { progressBar1.Value = step; }
- La Form principale :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 public Form1() { InitializeComponent(); f = new StepForm(); f.Show(); Thread t = new Thread(new ThreadStart(MyThread)); t.Start(); } private void MyThread() { for (int i = 0; i <= 100; i+= 10) { f.UpdateProgressBar(i); Thread.Sleep(1000); } }

rori Merci de ta patience & de ta réponse c'est sympa

par contre:

Citation:

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

Avec ton code j'ai été obligé de mettre f en varibale d'instance !
Peut on passer une variable à un thread ??
typiquement un truc du genre:

Code:

Thread t = new Thread(new ThreadStart(MyThread), f);

::: EDIT :::
Jvais tester ca et je reviens:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 static void Main(string[] args) { Thread th = new Thread(new ParameterizedThreadStart(Afficher)); Thread th2 = new Thread(new ParameterizedThreadStart(Afficher)); th.Start("A"); //On attend la fin du thread A avant de commencer le thread B. th.Join(); th2.Start("B"); Console.ReadKey(); } static void Afficher(object texte) { for (int i = 0; i < 10000; i++) { Console.Write((string) texte); } }
:::: EDIT2 ::::

Citation:

Erreur 1 Aucune surcharge pour 'MyThread' ne correspond au délégué 'System.Threading.ParameterizedThreadStart'

je suis en 2.0

::: EDIT 3 :::

J'ai rien dis il faut mettre object en param,
mais j'ai toujours l'erreur:

Citation:

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

01/09/2010, 09h59
alavoler

Ok pour le moment j'ai un truc qui à l'air de fonctionner,
jvais mettre au propre et le poster ici pour que vous me donniez votre avis d'ici peu

Je te remercie rori pour ton aide précieuse!
01/09/2010, 10h19
rori

Citation:

Envoyé par alavoler

::: EDIT 3 :::

J'ai rien dis il faut mettre object en param,
mais j'ai toujours l'erreur:
Opération inter-threads non valide : le contrôle 'progressBar1' a fait l'objet d'un accès à partir d'un thread autre que celui sur lequel il a été créé.

Il faut faire un Invoke() sur le delegué de la méthode qui met à jour la progresseBar

Bon je poste mon code, si quelqu'un peu me dire ce qu'il en pense (si ca vous parait coherent & élégant) je suis preneur

La form qui appel ma form progress bar et qui lance un 'traitement':

Code:

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
        private void TesterStepForm_Click(object sender, EventArgs e)
        {
 
            using (StepForm f = new StepForm())
            {
                Thread t = new Thread(new ParameterizedThreadStart(MyThread));
                t.Start(f);
                if (f.ShowDialog() == DialogResult.OK)
                    MessageBox.Show("Operation OK");
                else if (f.DialogResult == DialogResult.Cancel)
                    MessageBox.Show("Operation cancelled");
            }
        }
 
        private static void MyThread(object o)
        {
            StepForm f = (StepForm) o;
            for (int i = 0; i <= 100; ++i)
            {
                f.GoToStep(i);
                Thread.Sleep(20);
            }
            f.DialogResult = DialogResult.OK;
            f.Invalidate();
        }

le code la form qui s'occupe de l'affichage des étapes:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
        private void UpdateProgressBar(int step)
        {
            progressBar1.Value = step;
        }
 
        private delegate void MyProc<T>(T p1);
 
        public bool GoToStep(int progression)
        {
            if (InvokeRequired)
                Invoke(new MyProc<int>(UpdateProgressBar), progression);
            return true;
        }

Merci

01/09/2010, 14h20
alavoler

re,

sur mes Invoke parfois j'ai ce message quand je clic sur "Annuler" de ma StepForm:

Citation:

ObjectDisposedException
Impossible d'accéder à un objet supprimé.

J'ai cru dans un premier temps que le if (InvokeRequired) suffirait mais visiblement pas dans tous les cas !

De plus j'ai parfois une certaine latence entre le moment ou je fais un

Code:

f.DialogResult = DialogResult.OK;

dans mon thread, et que

Code:

MessageBox.Show("Operation OK");

s'affiche
01/09/2010, 15h27
alavoler

Hum... pour l'effet de 'latence', en fait j'ai réussi à reproduire:
Pour que MessageBox.Show("Operation OK") s'affiche, il faut que je bouge simplement la souris et la ca close l'ecran StepForm, et ca m'affiche bien la MsgBox....

Donc si quequ'un peu relire mon code je suis preneur, je continue à chercher de mon coté, merci
01/09/2010, 16h22
alavoler

Bon en mettant un f.Invalidate(); ca résout le problème du refresh (mais ca complique l'implémentation du coté des gens qui utilise ce pti composant StepForm !....)

je suis toujours à la recherche d'un relecteur de code... // critique & co
Et j'ai toujours mon problème de "Impossible d'accéder à un objet supprimé." sur mes invokes !

je met à jours mon code 3 postes plus haut