Discussion :

[sepas]

Salut,

voici un extrait du code d'une petite application où Scenario est un JFrame :

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
 
	private boolean doIt() {
 
 		if (this.file==null) this.file = new File(System.getProperty("user.dir")) ;
		if (this.scenario == null) {
			this.file = selectFile() ;
			if (this.file==null) return false ;
			final String text = readFile(this.file) ;
			if (text == null) return true ;
			try { SwingUtilities. invokeLater(new Runnable(){
				public void run(){
					James.this.scenario = new Scenario(James.this, James.this.file, text) ;
					}
				}) ; }
			catch (Exception e) { e.printStackTrace(); }
			}
		try {Thread.sleep(1000);} catch (InterruptedException e) {}
		return true ;
		}
 
	/*
	main
	*/
 
	public static void main(String params[]) {
		James james = new James() ;
		while (james.doIt()) ;
		}

Ce que je voudrais, c'est que lorsque l'utilisateur ferme le scenario, la methode doIt() lui demande de choisir un autre fichier.
Ce que j'ai fait ne marche pas, car l'appel à scenario.dispose() ne suffit pas à rendre scenario null.

Comment dois-je faire pour informer le thread principal que la fenêtre scenario a été fermée par l'utilisateur ?

[Balbuzard]

Si je comprends bien, lors de la fermeture de ta fenêtre, tu veux qu'il y ait affichage d'une autre fenêtre pour que l'utilisateur continue de faire quelque chose.

A ta place, je ferais étendre ma classe de WindowListener. Cette interface propose une méthode windowClosed.

va ici si tu veux voir l'api java

lorsque ta fenêtre est fermée, la méthode windowClosed s'exécute, et en la surchargeant, tu peux demander à ce qu'une autre fenêtre s'ouvre.

Enfin, pour fermer une fenêtre, j'utilise plutôt :

Code :

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

maFenetre.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

qui marche mieux

Voilà, j'espère t'avoir aidé :-)

[visiwi]

l'appel à scenario.dispose() ne suffit pas à rendre scenario null.

Un dispose() est fait pour détruire une fenêtre, pas pour rendre ton objet null.

Pour maFenetre.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); ce n'est peut-être pas ce que tu cherche car cela fermera complètement ton application.

Je suis assez d'accord avec le WindowListener.

Voici un exemple qui peut-être t'aidera :

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
import java.awt.Dialog.ModalityType;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.WindowEvent;
import java.awt.event.WindowListener;
import javax.swing.JButton;
import javax.swing.JDialog;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JOptionPane;
 
@SuppressWarnings("serial")
public class Test extends JFrame implements ActionListener, WindowListener {
 
	private JButton btn = new JButton("Choose a file");
	private JDialog chooser = new JDialog();
 
	public static void main(String[] args) {
		new Test();
	}
 
	public Test() {
		chooser.setModalityType(ModalityType.APPLICATION_MODAL);
		chooser.add(new JLabel("Simulation du choix d'un fichier"));
		chooser.setDefaultCloseOperation(JDialog.DISPOSE_ON_CLOSE);
		chooser.addWindowListener(this);
		btn.addActionListener(this);
 
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		add(btn);
		pack();
		setLocationRelativeTo(null);
		setVisible(true);
	}
 
	@Override
	public void actionPerformed(ActionEvent e) {
		chooser.pack();
		chooser.setLocationRelativeTo(this);
		chooser.setVisible(true);
	}
 
	@Override
	public void windowActivated(WindowEvent e) { }
 
	@Override
	public void windowClosed(WindowEvent e) {
 
	}
 
	@Override
	public void windowClosing(WindowEvent e) {
		JOptionPane.showMessageDialog(this, 
		         "Choix d'un autre fichier",
		         "Avertissement",
		         JOptionPane.WARNING_MESSAGE);
	}
 
	@Override
	public void windowDeactivated(WindowEvent e) { }
 
	@Override
	public void windowDeiconified(WindowEvent e) { }
 
	@Override
	public void windowIconified(WindowEvent e) { }
 
	@Override
	public void windowOpened(WindowEvent e) { }
 
}

[Balbuzard]

En effet, si tu as deux fenêtres et que si l'une d'elles tu exécutes la méthode

Code :

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

1
2
 
maFenetre.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

alors, toute ton application se fermera. Il faut que tu vérifies combien il reste de JFrame ouverte et si il n'y en plus qu'une seule, l'exécuter, sinon, faire

Code :

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

1
2
 
maFenetre.dispose();

:-)

[sepas]

Bon, regardez comment je m'en sors avec un windowListener.
Lorsque la fenetre scenario est fermée, elle le signale au thread principal :

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
 
	private void addListeners() {
		addWindowListener(new WindowAdapter() { 
			public void windowClosing(WindowEvent e) {
				if (Scenario.this.text.equals(Scenario.this.textArea.getText())) close() ;
				else {
					int rep = JOptionPane.showConfirmDialog(Scenario.this, "Save changes before closing ?", "Scenario modified", JOptionPane.YES_NO_CANCEL_OPTION, JOptionPane.WARNING_MESSAGE) ;
					switch(rep) {
						case JOptionPane.YES_OPTION :
							saveScenario(Scenario.this.file) ;
						case JOptionPane.NO_OPTION :
							close() ;
						case JOptionPane.CANCEL_OPTION :
							break ; 
						}
					}
				}
			});
		}
 
	private void close() {
		setVisible(false) ;
		dispose() ;
		Scenario.this.parent.scenarioClosed() ;
		}

Lorsque le thread principal constate la fermeture du scenario, il propose l'ouverture d'un autre.
Si l'utilisateur abandonne, alors l'application quitte.

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
 
	public void scenarioClosed() {
		this.scenarioOpen = false ;
		}
 
	private boolean doIt() {
 
 		if (this.file==null) this.file = new File(System.getProperty("user.dir")) ;
		if (!this.scenarioOpen) {
			this.file = selectFile() ;
			if (this.file==null) return false ;
			final String text = readFile(this.file) ;
			if (text == null) return true ;
			SwingUtilities.invokeLater(new Runnable(){
				public void run(){
					James.this.scenario = new Scenario(James.this, James.this.file, text) ;
					James.this.scenarioOpen = true ;
					}
				}) ;
			}
		try {Thread.sleep(1000);} catch (InterruptedException e) {}
		return true ;
		}
 
	public static void main(String params[]) {
		James james = new James() ;
		while (james.doIt()) ;
		}
 
	}

Ca marche, mais est-ce qu'il n'y a pas un inconvénient que je n'aurais pas vu à s'y prendre comme ça, en particulier du point de vue des threads ?