Timer s'executant 2 fois simultanément

Version imprimable

Bonjour,

J'ai ce code très simple :
Code:

1 2 3 4 5 6 7 8 ActionListener taskPerformer = new ActionListener() { public void actionPerformed(ActionEvent evt) { JOptionPane.showMessageDialog(null, "alerte"); } }; Timer temp = new Timer(2000, taskPerformer); temp.start();
Et ce que je ne comprend pas c'est pourquoi le timer ne s'execute pas correctement. En effet, quand je lance cela j'ai 2 messages d'erreur qui s'affiche l'un derrière l'autre puis 2 secondes plus tard 2 autres s'affichant ensemble et ainsi de suite.

Or moi j'ai besoin que l'action s'execute une fois, pas en double

21/08/2010, 19h17
slim_java

salut.
c'est quoi l'erreur que tu obtiens ?
et si tu veux connaitre la trace de l'exécution, utilise un System.out.println(....) plutôt qu'un JOptionPane
21/08/2010, 20h32
Ceubex

Il n'y a aucune erreur, le problème n'est pas que le programme bug mais qu'il ne s'execute pas correctement. Je n'ai mis que le JOptionPane pour montrer que cela ne fonctionnait pas et que l'action du timer s'executait 2 fois toutes les 2 secondes au lieu d'une toute les 2 secondes
23/08/2010, 14h12
Caalador

Alors, l'erreur avec ton code, ce n'est pas que le message s'affiche 2 fois au lieu de 1, mais plutôt qu'il ne s'affiche QUE 2 fois, et non pas à l'infini !

Une fois que tu fais start sur ton Timer, il va exécuter l'actionPerformed() en boucle jusqu'à ce que tu fasses un stop.

Dans ton cas, je suppose qu'il s'arrête à 2 car ton programme se termine.

Ajoute une JFrame que tu laisses ouverte afin que ton programme ne se termine pas, et tu auras ton message toutes les 2 secondes non stop.

Si tu veux que le traitement ne se fasse qu'une fois, met un stop à la fin de ton actionPerformed sur ton timer, ou alors utilise les Thread (car je pense que c'est plutôt cela que tu essaies de faire).

Décidement, j'ai fait un Thread mais j'ai toujours le même problème

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
class Clignote extends Thread
{
	public void run()
	{
		JOptionPane.showMessageDialog(null, "alerte");
 
  		  try {
  	        // pause
  	        Thread.sleep(2000);
  	        Fenêtre.clignoter();
  	      }
  	      catch (InterruptedException ex) {}
	}
}

Dans Fenêtre :

Code:

1
2
3
4
5
	static public void clignoter()
	{
		Thread clignement=new Clignote();
		clignement.start();
	}

23/08/2010, 16h32
Caalador

Alors c'est tout à fait normal !
Tu lances ton thread qui se relance lui même puis qui se relance, qui se relance, ... ZZzZz...

J'avais compris que tu voulais faire exécuter une tâches 1 fois, mais en dehors du Thread actif. Par contre, je vois que tu as une méthode clignoter(), du coup j'ai un doute.

Tu cherches à faire quoi ? Car je me demande si j'avais bien compris la première fois.
23/08/2010, 17h32
Ceubex

En une phrase : je veux qu'une (et une seule) boite de message apparaisse une fois toute les 2 secondes.

Que dois-je mettre pour faire cela ?
23/08/2010, 17h46
Caalador

Ah !! alors en effet, je n'avais rien compris.

Le moyen le plus simple c'est d'avoir une fenêtre statique je pense.
Tu peux utiliser le design pattern singleton.

Et ensuite tu joues avec un simple setVisible() afin d'afficher ou de masquer la fenêtre.

Voilà un petit exemple :

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
 
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.Timer;
 
 
@SuppressWarnings("serial")
public class Test2 extends JFrame {	
	private static JFrame instance = null;
 
	private Test2() {
		setTitle("Avertissement");
		add(new JLabel("Avertissement !!"));
		setSize(200, 200);
	}
 
	public static Test2 getInstance(){
		if(instance==null)instance = new Test2();
		return instance;
	}
 
	public static void main(String[] args) {
		JFrame frame = new JFrame();
		JButton bouton = new JButton("Start");
 
		bouton.addActionListener(new ActionListener() {	
			public void actionPerformed(ActionEvent e) {
				new Timer(2000, new ActionListener() {
					public void actionPerformed(ActionEvent e) {
						Test2.getInstance().setVisible(true);
					}
				}).start();				
			}
		});
 
		frame.add(bouton);
		frame.pack();
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setVisible(true);
 
 
	}
}

Comme cela, tu as un appel setVisible(true) qui s'effectue toujours sur la même instance.
Si elle est déjà ouverte, ca n'en ouvre pas d'autre.
Si elle est masquée, elle est réaffichée.

Je pense que cela correspond à ce que tu recherches.

Citation:

Envoyé par Ceubex

En une phrase : je veux qu'une (et une seule) boite de message apparaisse une fois toute les 2 secondes.

Que dois-je mettre pour faire cela ?

salut.
ton code est correcte mais apparemment tu appui pas sur le bouton OK de ton JOptionPane.

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
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
 
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JDialog;
import javax.swing.JFrame;
import javax.swing.Timer;
 
 
public class Fen extends JFrame {
 
    /**
     * author slim_java
     */
 
     int i  =0;
     JDialog boîte = new JDialog();
 
    public static void main(String args[]) {
        try {
            Fen frame = new Fen();
            frame.setVisible(true);
 
 
        } catch (Exception e) {
            e.printStackTrace();
        }
    }
 
 
    public Fen() {
        super();
        setBounds(100, 100, 500, 375);
        boîte.setBounds(this.getX()+50, this.getY()+50, 300, 200);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
 
 
        ActionListener taskPerformer = new ActionListener() {
 
            public void actionPerformed(ActionEvent evt) 
            {
                 i++;  
                 boîte.setVisible(true);
                 boîte.setTitle("boite de dialogue num : -- "+i+" --");                 
                 boîte.setVisible(false);             
             }
          };
          Timer temp = new Timer(2000, taskPerformer);
          temp.start();
    }
 
}

23/08/2010, 18h46
Ceubex

Ok bon alors mauvaise nouvelle, toutes vos solutions marchent quand je crée un projet pour les y mettre mais lorsque je les mets dans mon projet sous la même forme, ca le fait à chaque fois en double

Le problème ne viens pas du fait que je clique ou non sur OK, que je le fasse ou pas le problème survient, ou même si je fais toute autre action à la place de la boite de message, elle se fait en double.

Apparemment, il y a quelque chose dans la configuration de mon projet qui provoque ce double appel car ça le fait avec tous les codes. Mais je ne vois vraiment pas ce que cela pourrait être
24/08/2010, 10h50
Lady

Tu n'appelerais pas deux fois la methode clignoter() par hazard ??
24/08/2010, 16h12
Ceubex

Non il ne s'agit pas de ça, j'ai mis dans mon projet le code de slim_java et ça double l'appel, tandis que dans un nouveau projet ça se passe bien
24/08/2010, 16h16
Heimdal

Fais voir ton main...
24/08/2010, 17h01
Caalador

Dans un cas comme celui-ci, à mon avis, le mieux est encore de faire un debug, tu peux ainsi suivre l'exécution de ton code, et localiser ton problème.