Discussion :

[awabi]

Bonjour,

J'essaye de faire un chronomètre en java. Mon problème c'est une fois que j'appuie sur le bouton start, impossible d'appuyer sur stop ou reset, dans les actionperformed() j'ai fait un while(True) pour le bouton start.
Est ce pour ça que ca bloque le reste?
Comment je peux faire?

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
 
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.*;
import java.awt.*;
 
public class LesBoutons extends JPanel implements ActionListener{
    private JButton start = new JButton("Start");
    private JButton reset = new JButton("Reset");
    private JButton stop = new JButton("Stop");
 
    private Cadre  ardoise;
   boolean compt=false;
    public LesBoutons(){
        ardoise = new Cadre();
     setLayout(new BorderLayout(5, 5));
      JPanel lesBoutons = new JPanel();
      lesBoutons.add(start);
      lesBoutons.add(stop);
      lesBoutons.add(reset);
 
      add(lesBoutons, BorderLayout.NORTH);
      add(ardoise, BorderLayout.CENTER);
      start.addActionListener(this);
      stop.addActionListener(this);
      reset.addActionListener(this);
    }
     public void actionPerformed(ActionEvent e) {
      if (e.getSource().equals(start)) {
          compt=true;
           while(true){
            // ardoise.run();
               System.out.println("demarre");
               ardoise.run();
          }
      }
      else if (e.getSource() == reset){
          compt=false;
        System.out.println("recommencer");}
//ardoise.arret();
      else if (e.getSource() == stop){
          compt=false;
       System.out.println("stop");}
 //ardoise.reset();
      ardoise.repaint();
    }
}

[wax78]

C'est normal, ton traitement se fait dans le thread de l'edt de swing.
Et comme tu mets une boucle, et que tu ne pourras plus appuyez sur rien, alors ton programme va freezer.

C'est un problème frequement posé. Et il y'a plusieurs solutions dont :

-Un petit tutorial ici :
http://rom.developpez.com/java-swingworker/

-Sinon y'a aussi SwingUtilities.invokeLater qui marche bien aussi avec une thread :
http://gfx.developpez.com/tutoriel/j...ing-threading/

Puis peut etre :
http://java.developpez.com/faq/gui/?...alitesAWTSwing

[wax78]

En refaisant ton code vite fait (mal fait), voici a quoi ca devrait ressembler et fonctionner.
Attention que tu px appuyez plusieur fois sur start et lancer plusieur thread donc mefiance (limite grisé les boutons pour eviter que l'user ne lancer 2 fois start).

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
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.*;
import java.awt.*;
 
public class LesBoutons extends JPanel implements ActionListener
{
	private JButton	start	= new JButton("Start");
	private JButton	reset	= new JButton("Reset");
	private JButton	stop	= new JButton("Stop");
 
	boolean			compt	= false;
 
	public LesBoutons()
	{
		//ardoise = new Cadre();
		setLayout(new BorderLayout(5, 5));
		JPanel lesBoutons = new JPanel();
		lesBoutons.add(start);
		lesBoutons.add(stop);
		lesBoutons.add(reset);
 
		add(lesBoutons, BorderLayout.NORTH);
		//  add(ardoise, BorderLayout.CENTER);
		start.addActionListener(this);
		stop.addActionListener(this);
		reset.addActionListener(this);
	}
 
	public void actionPerformed(ActionEvent e)
	{
		if (e.getSource().equals(start))
		{
 
			Thread t = new Thread()
			{
				public void run()
				{
					LesBoutons.this.start();
				}
			};
			t.start();
 
		} else if (e.getSource() == reset)
		{
			setCompt(false);
			System.out.println("recommencer");
		}
		//ardoise.arret();
		else if (e.getSource() == stop)
		{
			setCompt(false);
			System.out.println("stop");
		}
		//ardoise.reset();
		//  ardoise.repaint();
	}
 
	private synchronized void setCompt(boolean b)
	{
		compt = false;
	}
 
	private void start()
	{
		compt = true;
		while (isCompt())
		{
			// ardoise.run();
			System.out.println("demarré");
			/*
			 * // Ici on va imaginer que tu doivens changer un label ou une
			 * progressbar, donc il faut un invokeLater pour toucher les element
			 * de swing a l'interieur d'un thread qui n'est pas l'EDT.
			 * SwingUtilities.invokeLater(new Runnable() 
			 * { public void run() {
			 * jLabel.setText("balbalbla"); } });
			 */
 
			try
			{
				Thread.sleep(1000);
			} catch (InterruptedException e1)
			{
				// TODO Auto-generated catch block
				e1.printStackTrace();
			}
			//      ardoise.run();
		}
	}
 
	private synchronized boolean isCompt()
	{
		return compt;
	}
}

[awabi]

Merci, beaucoup le chronomètre tourne!!

Mais j'ai encore un petit souci, à chaque fois que j'appuie sur start il refait d'autre bouton au dessus le champ chronomètre aussi est dédoublé.

J'essaye de comprendre pourquoi.
Résultat en PJ

[awabi]

Le problème est réglé c'est la surcharge paint() qui faisait défaut, donc j'ai utilisé un label avec settext() à la place et tout marche nikel confer PJ.

Merci.