Discussion :

[docv266]

Bonjour,

J'ai un lecteur de musique, et mon JSlider qui doit avancer en suivant la musique (classique quoi).

Alors j'ai une boucle while qui tourne tout le long de la chanson :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
while (!player.isComplete()) 
			{
				slider.setValeur(player.getPosition());
				try 
				{
					Thread.sleep(1000);					
				} 
				catch (Exception e)
				{
					e.printStackTrace();
				}
			}

Cette boucle appelle donc toutes les secondes la méthode setValeur, qui appelle elle-même la méthode setValue de JSlider à partir d'un Thread :

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 Curseur extends JPanel
{
        public void setValeur(int i)
	{
		SliderThread pt = new SliderThread(i);
		pt.start();
	}
 
	class SliderThread extends Thread 
	{
		int val;
 
		public SliderThread (int i)
		{
			val = i;
		}
 
		public void run() 
		{
			slider1.setValue(val);
		}
	}
}

(Je mets que le code nécessaire)

Le truc c'est que ça marche aléatoirement, parfois ça se bloque même en plein milieu etc...

Quelqu'un aurait une méthode efficace, ou pourrait trouver l'erreur dans mon code?

Merci d'avance!

[kwxyz]

Bonjour,

De ce que je lit, tu lances un nouveau Thread toutes les secondes, c'est pas très économique...

D'ailleurs je ne vois pas l'intéret du Thread juste pour modifier la valeur du slider.
Ce qui doit être dans un Thread, c'est la boucle de lecture de ta musique pour ne pas bloquer l'EDT, mais je suppose que c'est déja le cas.

Le mieux, c'est de créer un Timer qui va s'exécuter toutes les x millisecondes, et ou tu modifies le slider en fonction de la position du player, comme ça tu n'as toujours qu'un seul thread pour la mise à jour du slider.

Regarde : java.util.Timer.scheduleAtFixedRate()

Fred

[docv266]

Merci de ta réponse.

Alors j'ai fait ce que tu m'as dit :

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
 
public void setValeur(int i)
	{	
		valeur = i;
		Timer t = new Timer(); 
 
		GregorianCalendar gc = new GregorianCalendar(); 
		gc.add(Calendar.SECOND, 3); 
		t.scheduleAtFixedRate(new MyTask(), gc.getTime(), 1000); 
	}
 
	class MyTask extends TimerTask 
	{ 
		public void run()
		{
			slider1.setValue(valeur);
		}
	}

Alors en fait ça fonctionne, mais comme avant, c'est aléatoire!

Parfois ça se bloque ou ça ne démarre même pas.

C'est pas un problème de mise à jour du JPanel qui contient le JSlider?

[kwxyz]

Alors j'ai fait ce que tu m'as dit :

Pas tout à fait, tu ne dois pas créer ton timer dans le setValue().
La création du timer ne doit être faite qu'une seule fois.

Si ta boucle de lecture ne servait qu'a ça, tu peux la supprimer.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
// Lancement du thread de lecture audio
 
// Création du timer de mise à jour du slider
Timer timer = new Timer();
timer.schedule(new SliderUpdateTask(), 1000);
 
public class SliderUpdateTask extends TimerTask 
{ 
   public void run()
   {
     slider1.setValue(player.getPosition());
   }
}

J'imaginais plutôt quelque chose dans ce style.

Fred