Discussion :

[vins86]

Bonsoir,

Je viens vers vous car je n'arrive pas à faire en sorte que mon bouton d'arrêt d'un while soit accessible. J'ai testé sur deux threads, mais aucun changement.

Voici mon code:

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
static class ArreterListener implements ActionListener {
		// Redéfinition de la méthode actionPerformed()
		public void actionPerformed(ActionEvent e) {
			traitement traiter = new traitement();
			Thread thread = new Thread (traiter);
			thread.start();
			traiter.cancel();
			System.out.println("Arret de la synchro");
 
		}
	}
 
	static class BoutonListener implements ActionListener {
		// Redéfinition de la méthode actionPerformed()
		public void actionPerformed(ActionEvent e) {
			if(!jtf.getText().isEmpty())
			{
			traitement traiter1 = new traitement();
			Thread thread1 = new Thread (traiter1);
			thread1.start();
			traiter1.run(jtf.getText());
			}
		}
	}

Et mon code dans un thread faisant le traitement :

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
public class traitement extends Thread{
 
	private volatile boolean cancelled;
	static StorageManager stMan;
	public void run(String chemin) {
		// TODO Auto-generated method stub
		   while (!cancelled)
	        {
			   try {
					getCoureurs(chemin);
				} catch (ClassNotFoundException | SQLException | IOException e1) {
					// TODO Auto-generated catch block
					e1.printStackTrace();
				}
			   try
	            {
	                Thread.sleep (2000);
	            }
	            catch (InterruptedException exception){}
	        }
	}
	 public void cancel()
	    {
	        cancelled = true;
	    }

Merci d'avance de votre aide.

Cordialement,

Vins86

[joel.drigo]

Salut,

1. C'est normal que ça ne fonctionne pas : ton écouteur BoutonListener créé une instance de thread et de Traitement et ton écouteur ArreterListener en crée d'autres : il n'y aucun lien entre ces instances : modifier la valeur d'un attribut de l'une n'a aucun effet sur l'autre. Il ne faut qu'une seule instance de classe Traitement accessible par les deux écouteurs, et seul le BoutonListener doit en créer.
2. Ne jamais manipuler des éléments de Swing dans un thread autre que l'Event Dispatch Thread. Soit utiliser des SwngUtilities.invokeLater dans tes threads pour faire les manipulations Swing, soit utiliser un SwingWorker plutôt qu'un Thread, ou un Tmer Swing dans ton cas (puisqu'il s'agit d'une tâche qui s'exécute à intervalle régulier

[vins86]

Salut Joel,

Merci beaucoup de ta réponse. Par contre Je ne vois pas comment faire le même thread pour les deux.

J'ai vraiment chercher mais rien du tout.

Je serais preneur d'un petit coup de main si possible.

Cordialement,

Vins86

[joel.drigo]

Il faut pouvoir manipuler la même instance de classe. Le plus simple est d'avoir qu'une seule classe de listener avec une variable de classe pour ton instance de Traitement. Tu en créées une seule instance que tu enregistres sur les deux boutons et tu distingues le lancement de l'arrêt avec la source de l’événement. Ou alors tu utilises de classes internes de ActionListener et la variable partagée est dans la classe englobante (celle de JPanel, de JFrame ou quoique tu utilises).

Voici un exemple, avec deux classes d'ActionListener internes, une variable de traitement partagée (une instance de SwingWorker ici, pour l'exemple) :

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
import java.awt.BorderLayout;
import java.awt.FlowLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.List;
 
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.SwingConstants;
import javax.swing.SwingWorker;
 
public class ExempleSwingWorker extends JPanel {
 
	private JLabel resultat;
	private JButton buttonLancer;
	private JButton buttonArreter;
	private Worker swingworker;
	private int counter=0;
 
	public ExempleSwingWorker() {
		super(new BorderLayout());
		resultat = new JLabel();
		resultat.setHorizontalAlignment(SwingConstants.CENTER);;
		add(resultat, BorderLayout.CENTER);
		JPanel buttons = new JPanel(new FlowLayout(FlowLayout.CENTER,5,5));
		buttonLancer = new JButton("Lancer");
		buttonArreter = new JButton("Arreter");
		buttonArreter.setEnabled(false);
		buttons.add(buttonLancer);
		buttons.add(buttonArreter);
		add(buttons, BorderLayout.SOUTH);
		buttonLancer.addActionListener(new LancerListener());
		buttonArreter.addActionListener(new ArreterListener());
	}
 
	public static void main(String[] args) {
 
		JFrame frame = new JFrame("Démo");
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
 
		frame.add(new ExempleSwingWorker());
 
		frame.setSize(300, 300);
		frame.setLocationRelativeTo(null);
		frame.setVisible(true);
 
	}
 
	private class Worker extends SwingWorker<Object, String> {
 
 
 
		@Override
		protected Object doInBackground() throws Exception {
			while(!isCancelled()) {
				counter++;
				publish(String.valueOf(counter));
				try {
					Thread.sleep(1000);
				}
				catch(InterruptedException e) {
					break;
				}
			}
			return null;
		}
 
		@Override
		protected void process(List<String> chunks) {
			if ( !chunks.isEmpty() ) {
				resultat.setText(chunks.get(chunks.size()-1));
			}
		}
 
	}
 
	private class LancerListener implements ActionListener {
 
		@Override
		public void actionPerformed(ActionEvent e) {
			swingworker = new Worker();
			swingworker.execute();
			buttonLancer.setEnabled(false);
			buttonArreter.setEnabled(true);
		}
 
	}
 
	private class ArreterListener implements ActionListener {
 
		@Override
		public void actionPerformed(ActionEvent e) {
			swingworker.cancel(true);
			buttonArreter.setEnabled(false);
			buttonLancer.setEnabled(true);
		}
 
	}
 
}