détecter la fermeture de la fenêtre

Salut,

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

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

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 ?

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:

---

maFenetre.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

---

qui marche mieux

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

Citation:

---

Envoyé par sepas

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:

---

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) { }   }

---

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

Code:

---

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:

---

1 2	maFenetre.dispose();

---

:-)

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

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

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:

---

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 ?