Discussion :

[keub51]

Bonjour,

Je souhaite afficher une barre de defilement JProgressBar qui indique le traitement de fichier en cours ( lecture de fichier .txt ).

On m'a donc conseillé les threads et SwingWorker pour que l'interface graphique ne se fige pas et que ma barre de progression évolue en même temps que le traitement.

Mais voilà je suis bloqué je voudrais savoir où est mon erreur car ma barre de progression ne s'affiche qu'à la fin de mon traitement :s

J'ai donc deux threads différents :
- Un thread pour la JProgressBar
- Un thread pour le traitement de lecture ( le traitement long donc ... )

je les apelle ainsi :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
	public void setRepertory(String reponse) throws IOException {
 
		this.Repertory=reponse;//on definit le workspace
		// il faut lire le répertoire
		ThreadIG myThreadIG = new ThreadIG();		 
		MonThread myThread = new MonThread(new Read(this,this.Repertory));
		myThreadIG.setThread(myThread);
		myThreadIG.construct();
		myThread.run();
		myThreadIG.interrupt();		
	}                         // IL FAUT TUER LES THREADS comment on fait ? :)

Déja est ce que j'apelle les threads de la bonne façon ? et comment je les kill ?

voici le code de chacun des threads :

Thread pour le traitement long :

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
 
public class MonThread extends Thread {
 
    Read MyRead= null; 
 
  public MonThread(Read r) {
		MyRead=r;
  }
  public void run() {
	try {
		MyRead.Updating();
	} catch (IOException e) {
		// TODO Bloc catch auto-généré
		e.printStackTrace();
	} 
  }
}

Thread pour la progressBar :

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
public class ThreadIG extends SwingWorker {
    //...
 
	MonThread t=null;
 
    public Object construct(){
 
		MaProgressBar monTest = new MaProgressBar();
			while(this.t.isAlive())
			{			
			  monTest.maBarre.setValue(/* il faut récupérer un indice qui indique		  
			   *                          la progression du traitement...
			   *  */1);
			  Thread.yield();
			}  
        return (Object)monTest;
    }
    //...
 
    public void setThread(MonThread t)
    {
    	this.t = t;
    }
}

Enfin voila où j'en suis j'aimerai vos avis et vos suggestions pour que cette JProgressBar avance

Merciii

[sinok]

Utiliser à la fois des Threads et SwingWorker çà fait un peu beaucoup redondance....

Utilise plutôt le doInBackground(...) du SwingWorker pour effectuer les tâches lourdes, à la place de mettre ce traitement dans des threads...

[keub51]

Utiliser à la fois des Threads et SwingWorker çà fait un peu beaucoup redondance....

Utilise plutôt le doInBackground(...) du SwingWorker pour effectuer les tâches lourdes, à la place de mettre ce traitement dans des threads...

ok jregarde ca et jreviens dans quelques temps lol

[let_me_in]

salut,

pourquoi tu ne passe pas ta JProgressBar a ta methode readFile, tu pourra ainsi augmenter sa valeur au fur et mesure que tu lise tes fichier, ou bien plus simple mets ça dans ta boucle :

Code :

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

1
2
3
4
5
6
7
for(int i=0;i<count;i++)
	{	
	if(tab[i].isFile()){
		this.readFile(tab[i]);
                 maProgressBar.setValue(100/(count-i));
                 }
	}

ça fait un peu bricolage mais ça devrait marcher.

PS : coté conception, elle est vraiment nulle cette solution, mais on peut dire de même pour la tienne.

[keub51]

voila ce que j'ai :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
if(count!=0)
				{
					log.info.setText("");
					log.addInfo("Le repertoire contient "+ count + " fichier(s) de log à analyser.");
					canRead = true;	
					File[] tab = repertoire.listFiles();		
 
					MonThread t = new MonThread();
					t.setParameters(log,tab,count,this,error);
					t.construct();
 
 
 
				}

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
public class MonThread extends SwingWorker {
	//...
 
	public KeubLog log=null;
	public File[] tab=null;
	public int count=0;
	public Read r;
	public LinkedList e;
	public Object construct(){
		//Construction graphique...
		return null;
	}
	//...
	public Object doInBackground()
	{
		for(int i=0;i<count;i++)
		{											
			if(tab[i].isFile() && tab[i].getName().indexOf(".log")>0)
				try {
					r.readFile(tab[i]);
				} catch (IOException e1) {					
					e1.printStackTrace();
				}
			log.addInfo("Lecture effectué avec succes");
			log.ConstruireArbre(e,tab[i].getName());							 
		}	
		return null;
	}
 
	public void setParameters(KeubLog log,File[] tab,int count,Read r, LinkedList e)
	{
		this.log=log;
		this.tab=tab;
		this.count=count;
		this.r=r;
		this.e=e;
	}
 
}

comment execute t'on ce qui est dans la methode doInBackground(); ????

[fr1man]

Lis au moins les liens qui te sont donnés !

http://rom.developpez.com/java-swingworker/

[Gfx]

Il ne faut pas modifier la progress bar dans doInBackground(). Va lire la doc et regarde bien les méthodes publish() et process().

[Kh4iN3]

Il ne faut pas modifier la progress bar dans doInBackground(). Va lire la doc et regarde bien les méthodes publish() et process().

+1, sinon les SwingWorker ne serviraient à pas grand chose, un Thread suffirait.

[SpecialCharacter]

Sans vouloir concurrencer GFX, ce tutoriel est également très bien fait et clair (en)http://java.sun.com/.../swingworker