Thread appelé par un JDialog, problème avec bouton

**Marc_3** · 21/01/2013, 12h06

Bonjour,
J'essaie de commander un thread (class interne MetTimer) a travers une fenêtre JDialog (Classe Metronome). L’idée est de ne pouvoir appeler depuis la fenêtre principale qu'un seul et unique Metronome.
Voir code ci-dessous

Séparément les éléments fonctionnent mais la connexion entre les modules ne produit pas l'effet escomptée.

Le Thread est suppose être enfermé dans une boucle while avec une valeur boolean qui passe de true a false a travers les boutons start et cancel de la fenêtre de dialog et c'est la que j'ai le problème.

Le bouton start après le clic reste enfoncé et je n'ai plus de possibilité d'action avec le bouton cancel.

Comment est-ce que je pourrai remédier a cela?

Merci pour vos réponses

marc_3

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
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
 
package gamme;
 
import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Component;
import java.awt.Dimension;
import java.awt.FlowLayout;
import java.awt.GridBagLayout;
import java.awt.Insets;
import java.awt.Window;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.util.Date;
 
import javax.swing.JButton;
import javax.swing.JDialog;
import javax.swing.JLabel;
import javax.swing.JPanel;
 
//public class Metronome extends JDialog implements Runnable, MouseListener {
public class Metronome extends JDialog implements MouseListener {
 
	private Window owner;
	private int thisWidth = 300;
	private int thisHeight = 300;
	private JButton btnStart, btnCancel, btnEnd;
	private volatile boolean cancelProcess = true;
	private JPanel jpLayout, jpLayoutSouth, jpLayoutCenter, jPanelDessin;
	private JLabel jlabTitre;
 
	private MetTimer metTimer;
 
	public Metronome(Window owner)
	{
		this.owner = owner;
		initializeComponents();
	}
 
 
	private void initializeComponents()
	{
 
		this.setAlwaysOnTop(true);
		this.setModal(true);
		this.setDefaultCloseOperation(JDialog.DISPOSE_ON_CLOSE);
 
		jpLayout = new JPanel();
		jpLayout.setLayout(new BorderLayout());
		jpLayoutSouth = new JPanel();
		jpLayoutSouth.setLayout(new BorderLayout());
		jpLayoutCenter = new JPanel();
		jpLayoutCenter.setLayout(new BorderLayout());
		jpLayoutCenter.setBackground(Color.CYAN);
		jpLayoutCenter.setPreferredSize(new Dimension(270, 250));
		jpLayout.add(jpLayoutSouth, BorderLayout.SOUTH);
		jpLayout.add(jpLayoutCenter, BorderLayout.NORTH);
		this.add(jpLayout);
 
 
		jPanelDessin  = new JPanel();
		jPanelDessin.setPreferredSize(new Dimension(270, 250));
		jPanelDessin.setBackground(Color.gray);
		jPanelDessin.setVisible(true);
		jpLayoutCenter.add(jPanelDessin, BorderLayout.NORTH);
 
 
 
 
		jlabTitre = new JLabel("Titre");
		jlabTitre.setBounds(5, 10, 100, 20);
		jpLayoutCenter.add(jlabTitre, BorderLayout.NORTH);
 
		this.setResizable(false);
		this.setSize(new Dimension(thisWidth, thisHeight));
 
 
 
 
		btnStart = new JButton("start" );
		btnCancel = new JButton("Cancel");
		btnEnd  = new JButton("End");
		btnStart.setPreferredSize(new Dimension(100, 20));
		btnCancel.setPreferredSize(new Dimension(100,20));
		btnEnd.setPreferredSize(new Dimension(100,20));
		btnEnd.addMouseListener(this);
		btnCancel.addMouseListener(this);
		btnStart.addMouseListener(this);
		jpLayoutSouth.add(btnEnd, BorderLayout.EAST);
		jpLayoutSouth.add(btnCancel, BorderLayout.CENTER);
		jpLayoutSouth.add(btnStart, BorderLayout.WEST);
 
		//***************************************
		//***************************************
		metTimer= new MetTimer(this);
		metTimer.run();
		//***************************************
	}
 
 
 
 
	/**
         * Zeigt die Fortschrittsanzeige an.
         */
	public void Show()
	{
		centerOnParent(owner);
		this.setVisible(true);
	}
 
	/**
         * Schließt die Fortschrittsanzeige.
         */
	public void close()
	{
		this.dispose();
	}
 
	private void centerOnParent(Window owner)
	{
		this.setLocationRelativeTo(owner);
	}
 
	public boolean cancelRequested()
	{
		return this.cancelProcess = true;
	}
 
	public boolean startRequested()
	{
		return this.cancelProcess = false;
	}
 
 
	@Override
	public void mouseClicked(MouseEvent e) {
 
		if (e.getSource().equals(btnEnd))
		{
			System.out.println("btnEnd");	
			cancelRequested();
			this.dispose();
		}
		else if (e.getSource().equals(btnCancel))
		{
			cancelRequested();
			System.out.println("Dans btnCancel, valeur de cancelProcess : " + cancelProcess);
			System.out.println();
		}	
		else if (e.getSource().equals(btnStart))
		{
			System.out.println("btnStart");	
			startRequested();
			metTimer.run();
			System.out.println("Valeur cancelProcess dans btnStart : " + cancelProcess);
			System.out.println();
		}
	}
 
 
	@Override
	public void mouseEntered(MouseEvent arg0) {
		// TODO Auto-generated method stub
 
	}
 
 
	@Override
	public void mouseExited(MouseEvent arg0) {
		// TODO Auto-generated method stub
 
	}
 
 
	@Override
	public void mousePressed(MouseEvent arg0) {
		// TODO Auto-generated method stub
 
	}
 
 
	@Override
	public void mouseReleased(MouseEvent arg0) {
		// TODO Auto-generated method stub
 
	}
 
//	class MetTimer implements Runnable {
		class MetTimer extends Thread  {	
		Component owner;
 
		Metronome metronome;
 
		public MetTimer(Component owner)
		{
			this.owner = owner;
		}
 
 
		//
		@Override
		public void run() {
			System.out.println("Dans run valeur de cancelProcess : " + cancelProcess);
 
						  try
			  {
				  System.out.println("Dans run valeur de cancelProcess : " + cancelProcess);
 
				 while(!cancelProcess){  
			    //while(true){
				  Thread.sleep(1000);
			    long lDateTime = new Date().getTime();
			    System.out.println("Date() - Time in milliseconds: " + lDateTime);
			    }
			    }catch(InterruptedException e)
			  {
			    System.out.println("interrupted exception : "+e.getMessage());
			    e.printStackTrace();
			  }
		}
	}
 
 
 
 
}

**bulbo** · 21/01/2013, 12h25

Bonjour,

Pour démarrer un thread il faut utiliser la méthode start() et non la méthode run() !!

Et petite remarque en passant, s'il ne doit y avoir qu'un seul MetTimer dans l'appli, peut-être penser à en faire un singleton

Deuxième remarque en passant, pour les actions longues appelées depuis un listener, penser à utiliser un SwingWorker par exemple.
--> http://java.developpez.com/faq/gui/?...raitement_long

Bulbo

**Marc_3** · 21/01/2013, 13h02

Merci pour la réponse, j'ai repris en ne travaillant pas avec un JDialog mais un JFrame et en travaillant avec metTimer.start() et je n'ai plus de problème avec mon bouton.
Par contre je n'arrive pas a faire repartir le thread après avoir stoppé celui-ci avec le bouton cancel.
Comment le faire repartir?

J'ai l'impression que le thread est détruit et non pas suspendu en sortant de la boucle while

Est-ce correct?

**bulbo** · 21/01/2013, 13h20

Je n'avais pas vu que tu appelais 2 fois ta méthode run() sur MetTimer.. pas bon ça, vire les deux.. ça ira déjà sûrement mieux.

Bulbo

**Marc_3** · 21/01/2013, 13h32

oui j'ai vu et corrigé aussi le double appel, une idée pour mon redémarrage?

**Marc_3** · 21/01/2013, 14h25

Bon voila, j'ai repris ma class et cette fois cela fonctionne, voir la classe ci-dessous. Ce qui me turlupine encore est de savoir si le thread metTimer est détruit en sortant de la boucle while ou suspendu et encore existant quelque part.
Sinon merci pour les autres commentaires surement possible sur la class.
Je vais en faire un singleton en attendant.

Merci pour les réponses

PS J'ai d'autre question sur swingWorker, mais plus tard une chose a la fois pour ma pauvre petite tête.

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
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
 
 
package gamme;
 
import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Component;
import java.awt.Dimension;
import java.awt.FlowLayout;
import java.awt.GridBagLayout;
import java.awt.Insets;
import java.awt.Window;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.util.Date;
 
import javax.swing.JButton;
import javax.swing.JDialog;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
 
//public class Metronome extends JDialog implements Runnable, MouseListener {
//public class Metronome extends JDialog implements MouseListener {
//public class Metronome extends JDialog implements ActionListener {
public class Metronome extends JFrame implements ActionListener {
	private Window owner;
	private int thisWidth = 300;
	private int thisHeight = 300;
	private JButton btnStart, btnCancel, btnEnd;
	private volatile boolean cancelProcess = false;
	private JPanel jpLayout, jpLayoutSouth, jpLayoutCenter, jPanelDessin;
	private JLabel jlabTitre;
 
	private MetTimer metTimer;
 
	public Metronome(Window owner)
	{
		this.owner = owner;
		initializeComponents();
	}
 
 
	private void initializeComponents()
	{
 
		this.setAlwaysOnTop(true);
		//this.setModal(true);
		this.setDefaultCloseOperation(JDialog.DISPOSE_ON_CLOSE);
 
		jpLayout = new JPanel();
		jpLayout.setLayout(new BorderLayout());
		jpLayoutSouth = new JPanel();
		jpLayoutSouth.setLayout(new BorderLayout());
		jpLayoutCenter = new JPanel();
		jpLayoutCenter.setLayout(new BorderLayout());
		jpLayoutCenter.setBackground(Color.CYAN);
		jpLayoutCenter.setPreferredSize(new Dimension(270, 250));
		jpLayout.add(jpLayoutSouth, BorderLayout.SOUTH);
		jpLayout.add(jpLayoutCenter, BorderLayout.NORTH);
		this.add(jpLayout);
 
 
		jPanelDessin  = new JPanel();
		jPanelDessin.setPreferredSize(new Dimension(270, 250));
		jPanelDessin.setBackground(Color.gray);
		jPanelDessin.setVisible(true);
		jpLayoutCenter.add(jPanelDessin, BorderLayout.NORTH);
 
 
 
 
		jlabTitre = new JLabel("Titre");
		jlabTitre.setBounds(5, 10, 100, 20);
		jpLayoutCenter.add(jlabTitre, BorderLayout.NORTH);
 
		this.setResizable(false);
		this.setSize(new Dimension(thisWidth, thisHeight));
 
 
 
 
		btnStart = new JButton("start" );
		btnCancel = new JButton("Cancel");
		btnEnd  = new JButton("End");
		btnStart.setPreferredSize(new Dimension(100, 20));
		btnCancel.setPreferredSize(new Dimension(100,20));
		btnEnd.setPreferredSize(new Dimension(100,20));
//		btnEnd.addMouseListener(this);
//		btnCancel.addMouseListener(this);
//		btnStart.addMouseListener(this);
		btnEnd.addActionListener(this);
		btnCancel.addActionListener(this);
		btnStart.addActionListener(this);
 
 
		jpLayoutSouth.add(btnEnd, BorderLayout.EAST);
		jpLayoutSouth.add(btnCancel, BorderLayout.CENTER);
		jpLayoutSouth.add(btnStart, BorderLayout.WEST);
 
		//***************************************
		//***************************************
		//metTimer= new MetTimer(this);
		//metTimer.start();
		//metTimer.run();
		//***************************************
	}
 
 
 
 
	/**
         * Zeigt die Fortschrittsanzeige an.
         */
	public void Show()
	{
		centerOnParent(owner);
		this.setVisible(true);
	}
 
	/**
         * Schließt die Fortschrittsanzeige.
         */
	public void close()
	{
		this.dispose();
	}
 
	private void centerOnParent(Window owner)
	{
		this.setLocationRelativeTo(owner);
	}
 
	private void cancelRequested()
	{
		this.cancelProcess = true;
	}
 
	private void startRequested()
	{
		this.cancelProcess = false;
	}
 
	private boolean valueCancelProcess()
	{
		return cancelProcess;
	}
 
 
 
	class MetTimer extends Thread  {	
		Component owner;
 
		Metronome metronome;
 
		public MetTimer(Component owner)
		{
			this.owner = owner;
		}
 
 
		//
		@Override
		public void run() {
 
 
						  try
			  {
				  System.out.println("Dans run valeur de cancelProcess : " + cancelProcess);
 
				 while(!valueCancelProcess()){  
			    //while(true){
				  Thread.sleep(1000);
			    long lDateTime = new Date().getTime();
			    System.out.println("Date() - Time in milliseconds: " + lDateTime);
			    }
			    }catch(InterruptedException e)
			  {
			    System.out.println("interrupted exception : "+e.getMessage());
			    e.printStackTrace();
			  }
		}
	}
 
		@Override
		public void actionPerformed(ActionEvent e) {
			if (e.getSource().equals(btnEnd))
			{
				System.out.println("btnEnd");	
				cancelRequested();
				this.dispose();
			}
			else if (e.getSource().equals(btnCancel))
			{
				//metTimer.interrupt();
				cancelRequested();
				System.out.println("Dans btnCancel, valeur de cancelProcess : " + cancelProcess);
				System.out.println();
			}	
			else if (e.getSource().equals(btnStart))
			{
				startRequested();
				MetTimer metTimer = new MetTimer(this);
				metTimer.start();
			}
 
		}
 
 
 
 
}

**bulbo** · 21/01/2013, 15h11

L'execution d'un thread se termine à la fin de sa méthode run(), c'est à dire qu'à partir de ce moment là, aucun 'thread' de la machine virtuelle n'est plus lié à ton objet .. par contre les règles du GC s'appliquent comme pour le reste des objets donc si tu n'as plus de référence dessus, il sera détruit par le GC ..

**Marc_3** · 21/01/2013, 17h42

C'est donc résolu merci encore

Thread appelé par un JDialog, problème avec bouton

avec Java

Discussions similaires

Partager

Partager