Discussion :

[lemordore]

Bonjour, Je charge des donnees d'une base dans un dataset ,le traitement est assez lourd. Pour ne pas que l'utilisateur pense que l'appli ne repond pas j'ai essayé d'integrer une Progressbar ca fait 2 semaines que je planche sur le sujet. Mais là je n'ais plus de visibilité et je m'en mêle les pinceaux.

Voila où j'en suis: L'importation des données se fait au clique d'un bouton.

Voici mon code:

Code :

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

1
2
3
4
5
6
7
8
 // Le bouton
private void Bt_exp_Click(object sender, EventArgs e)
 {

        this.DgResultat.DataSource = null;
        Thread t = new Thread(new ThreadStart(ThreadMethod));
        t.Start();
 }

// Le traitement dasn le Thread(base de donnees -> Datagridview)

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
 private void actualiser_dg(){
               
            MonDataAdapter.Fill(ds, flux);                  
            DgResultat.DataSource = ds.Tables[flux].DefaultView;
            foreach (DataGridViewRow currentRow in DgResultat.Rows)
            {
                currentRow.Selected = false;
                if (currentRow.IsNewRow == false)
                {
                   //Traitement lourd... 
                }
                Show_ProgressBar();
            }
            MyProgressBar.ReInit();
}

// La progressbar

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12

private void Show_ProgressBar()
 {
    if (InvokeRequired) //si on n'est pas dans le thread principal
    {
       Invoke(new progress(Show_ProgressBar));             
     } else{
         MyProgressBar.Max = DgResultat.RowCount;
         MyProgressBar.PerformSteps();  
      }             
}

// La méthode appelé dans le thread

Code :

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

1
2
3
4
5
6
7

private void ThreadMethod()
 {
         this.Invoke(new progress(actualiser_dg));

   }

La progressbar evolue bien pendant le chargement mais ralentie considerablement le temps de chargement.

[lemordore]

Je suis totalement bloqué ,quelqu'un pourrait il me donner un coup de pouce, Merci d'avance pour votre aide.

[harz62]

Bonjour,

Utilise la classe BackgroundWorker, je pense qu'elle doit est adaptée dans ton cas.

[lemordore]

Merci de votre intervention mais desole harzl62 ,si pres du but je ne peux renoncer ,ca serait vraiment dommage de faire demi tour pour reprendre le chemin le plus simple, je pense (et ne suis pas le seul d'apres le nombre de post sur ce forum) que les threads est un bon moyen pour arriver à mes fins.

Ce que je ne comprends pas c'est que le code que je vous ais presenter semble fonctionner lorsque je recupere 2000 lignes d'une base de donnee, mais dès que le nombre depasse les 150 000 alors la progressbar met du temps avant de se lancer (au moins 20 secondes )

Je ne comprends vraiment pas ce qui peut bien se passer.

[lemordore]

Bonjour,
l'utilisation d'un second thread est censé reduire le temps de traitement,ce n'est pas du tout le cas pour mois, au contraire avec un second thread le temps double voir triple.

Quelqu'un aurait il cerné mon probleme ?

merci de votre aide.

[Graffito]

Bonjour,

Essaye de mettre une instruction Application.DoEvents() à la fin de Show_ProgressBar().
Attention, il faudra peut-être entourer l'instruction par une sauvegarde/restauration de la forme du curseur.

[lemordore]

Bonjour graffito, ton message m'a redonné espoir. j'ais ajouté l'element,
dans le else de la methode Show_ProgressBar.

Code :

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

System.Windows.Forms.Application.DoEvents();

Mais ca ne change rien c'est toujours aussi lent.
Merci

[Graffito]

MyProgressBar.PerformSteps();

Est-ce une ProgressBar non standard?
Si oui, que se passe-t-il en la remplaçant par la ProgressBar standard des winForms?

[SirJulio]

Salut,

ton traitement ici, n'est, in fine, pas threadé.

Tu lances un nouveau thread (jusque la pas de probleme) qui invoke la methode de remplissage sur le thread de presentation. Au final, ton traitement se fait comme si tu l'appelais directement sur le thread d'UI. Je verrais plus (pour le peu que j'ai compris du traitement =p) :

- Tu cliques sur le bouton, et tu lances en async le remplissage du dataset.
- Tu traites chaque row de ton Set, avec ton traitement lourd
- et finalement tu affiches tes resultats.

Exemple (vite fait à tester) :

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
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
//ttest etant une table du dataset Dataset1, l'adapter etant celui generé par Visual
/*La form comprend 3 controls
 - un button
 - une progress
 - une DGV*/
    public partial class Form1 : Form
    {
        delegate void InitProgressDel(int rowCount);
        DataSet1 ds = new DataSet1();
 
        public Form1()
        {
            InitializeComponent();
        }
 
        private void button1_Click(object sender, EventArgs e)
        {
            dataGridView1.DataSource = null;
            Thread t = new Thread(ThreadStartMeth);
            t.Start();
        }
 
        private void ThreadStartMeth()
        {
            using (DataSet1TableAdapters.ttestTableAdapter adapt = new WindowsFormsApplication1.DataSet1TableAdapters.ttestTableAdapter())
            {
                adapt.Fill(ds.ttest);
            }
 
            InitProgress(ds.ttest.Rows.Count);
 
            foreach (DataSet1.ttestRow item in ds.ttest)
            {
                //traitement lourd
 
                ShowProgress();
            }
 
            DisplayData();
        }
 
 
        private void InitProgress(int rowCount)
        {
            if (this.InvokeRequired)
            {
                this.Invoke(new IntProgressDel(InitProgress), rowCount);
            }
            else
            {
                progressBar1.Maximum = rowCount;
                progressBar1.Step = 1;
            }
        }
 
        private void ShowProgress()
        {
            if (this.InvokeRequired)
            {
                this.Invoke(new MethodInvoker(ShowProgress));
            }
            else
            {
                progressBar1.PerformStep();
            }
        }
 
        private void DisplayData()
        {
            if (this.InvokeRequired)
            {
                this.Invoke(new MethodInvoker(DisplayData));
            }
            else
            {
                dataGridView1.DataSource = ds.ttest.DefaultView;
            }
 
        }
 
    }

Ici, je n'appelle le thread de presentation que pour initialiser la progressbar, la mettre à jour et afficher les données, le reste etant threadé. Mais bien comprendre qu'un thread ne va pas forcement accelerer le chargement des données, il est la pour garder ton UI reactive pendant le chargement des dites données (ce qui est le but recherché apparement selon ton premier message). Si le chargement est toujours lourd et consommateur de temps, il faudra voir à optimiser ce traitement, les threads ne pourront pas plus t'aider (du moins, pas sans en savoir plus sur ce traitement =)).

Bon courage, en esperant que cela t'aide.

[lemordore]

Salut SirJulio,
Merci ,tes indications mon beaucoup aider,
je me suis inspiré de la structure de ton code,
sauf que j'ais placé le foreach au niveau du gridview et non de la datatable (j'applique une couleur de fond sur certaine lignes)
et en effet mon UI est maintenant reactive c'est bien ce que je voulais.
Mais le chargement des donnees est terminé alors que la progressbar continu à s'incrementer.