Détection de l'affichage d'une JFrame.

Version imprimable

21/02/2008, 21h12
FredPsy

Détection de l'affichage d'une JFrame.

Bonsoir,
j'explique mon problème. J'ai un JFrame qui ne comporte que des boutons ayant pour but d'ouvrir d'autre JFrames. Un espèce de tableau de bord en fait.

Lorsque je clique sur un bouton X, la JFrame X s'ouvre. Le soucis, c'est que si j'appui 10 fois sur le bouton X, j'aurai 10 fois la JFrame X d'ouverte.

Ma question, c'est peut on savoir si une JFrame est déjà ouverte de façon à ce qu'elle ne se ré-ouvre pas au prochain clique ?

J'ai essayé isShowing, isVisible, sans résultat, ou alors, j'ai mal fais.

Merci.

Un imprime écran vaut mieux que de longue explication.
J'ai cliqué trois fois sur le premier bouton.

http://fred.bty.free.fr/imagesForum/Capture.png
21/02/2008, 23h47
bouye

Tu a probablement un probleme de conception et de manipulation de variable. Peux-tu nous mettre du code s'il te plait ?

Sinon un autre moyen de pallier a ce probleme est de desactiver le bouton et de ne le reactiver que quand la fenetre est fermee, empechant ainsi d'en creer d'autres.
22/02/2008, 00h09
FredPsy
En fait, pour la conception, je viens de commencer ce projet avec presque une année sans mettre les mains dans le Java. Et décidément, c'est pas comme le vélo, ça se perd.

Pour ce qui est du code, il est sur mon autre PC, qui est éteint, donc je verais demain. Mais en gros :
Mon bouton est dans un observateur d'événement et lorsque je cliques dessus, j'ai un morceau de code comme suit :
Code:

1 2 3 4 5 6 if (e.getSource() == NomBouton) { /* Autre classe, fichier différent*/ new AutreJFrame();// Pour l'exemple, j'ai mis AutreJFrame }
Voilà.

Citation:

Sinon un autre moyen de pallier a ce probleme est de desactiver le bouton et de ne le reactiver que quand la fenetre est fermee, empechant ainsi d'en creer d'autres.

Le problème sera le même, les deux JFrame sont indépendantes, donc il faut que je récupère l'état d'une des deux pour intéragir avec l'autre.

En fait, non, je pourrais désactiver le bouton lors de l'ouverture de l'autre JFrame, mais à la fermeture de cette dernière, je ne sais pas comment réactiver le bouton.

Je viens de refaire ça vite fait sur Eclipse.
Mon premier fichier Boarding.java

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
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
 
package test.ihm;
 
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.Rectangle;
 
import javax.swing.SwingUtilities;
import javax.swing.JPanel;
import javax.swing.JFrame;
import javax.swing.JButton;
 
 
public class Boarding extends JFrame implements ActionListener {
 
    private static final long serialVersionUID = 1L;
 
    private JPanel jContentPane = null;
 
    private JButton Button = null;
 
    /**
     * This method initializes Button    
     *     
     * @return javax.swing.JButton    
     */
    private JButton getButton() {
        if (Button == null) {
            Button = new JButton("Ouvrir");
            Button.setBounds(new Rectangle(4, 4, 106, 37));
            Button.addActionListener(this);
        }
        return Button;
    }
 
    /**
     * @param args
     */
    public static void main(String[] args) {
        // TODO Raccord de méthode auto-généré
        SwingUtilities.invokeLater(new Runnable() {
            public void run() {
                Boarding thisClass = new Boarding();
                thisClass.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
                thisClass.setVisible(true);
            }
        });
    }
 
    /**
     * This is the default constructor
     */
    public Boarding() {
        super();
        initialize();
    }
 
    /**
     * This method initializes this
     * 
     * @return void
     */
    private void initialize() {
        this.setSize(123, 80);
        this.setContentPane(getJContentPane());
        this.setTitle("Mainboard");
        this.setLocation(50, 50);
    }
 
    /**
     * This method initializes jContentPane
     * 
     * @return javax.swing.JPanel
     */
    private JPanel getJContentPane() {
        if (jContentPane == null) {
            jContentPane = new JPanel();
            jContentPane.setLayout(null);
            jContentPane.add(getButton(), null);
        }
        return jContentPane;
    }
 
    public void actionPerformed(ActionEvent e) {
        // TODO Raccord de méthode auto-généré
        if (e.getSource() == Button)
            {
                new Ihm();
            }
 
    }
 
}

Le deuxième fichier Ihm.java (deuxième 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
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
 
package test.ihm;
 
import javax.swing.JPanel;
import javax.swing.JFrame;
import javax.swing.JButton;
import java.awt.Rectangle;
 
public class Ihm extends JFrame {
 
    private static final long serialVersionUID = 1L;
 
    private JPanel jContentPane = null;
 
    private JButton ButtonClose = null;
 
    /**
     * This is the default constructor
     */
    public Ihm() {
        super();
        initialize();
    }
 
    /**
     * This method initializes this
     * 
     * @return void
     */
    private void initialize() {
        this.setSize(300, 200);
        this.setContentPane(getJContentPane());
        this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        this.setTitle("JFrame");
        this.setVisible(true);
        this.setLocation(150, 150);
    }
 
    /**
     * This method initializes jContentPane
     * 
     * @return javax.swing.JPanel
     */
    private JPanel getJContentPane() {
        if (jContentPane == null) {
            jContentPane = new JPanel();
            jContentPane.setLayout(null);
            jContentPane.add(getButtonClose(), null);
        }
        return jContentPane;
    }
 
    /**
     * This method initializes ButtonClose    
     *     
     * @return javax.swing.JButton    
     */
    private JButton getButtonClose() {
        if (ButtonClose == null) {
            ButtonClose = new JButton("Fermer");
            ButtonClose.setBounds(new Rectangle(195, 121, 75, 27));
            ButtonClose.addActionListener(new java.awt.event.ActionListener() {
                public void actionPerformed(java.awt.event.ActionEvent e) {
                    Fermer();
                }
            });
        }
        return ButtonClose;
    }
    public void Fermer()
    {
        this.setVisible(false);
    }
 
}

22/02/2008, 03h42
bouye

Citation:

Envoyé par FredPsy

En fait, non, je pourrais désactiver le bouton lors de l'ouverture de l'autre JFrame, mais à la fermeture de cette dernière, je ne sais pas comment réactiver le bouton.

Avec un WindowListener place sur la nouvelle fenetre que tu viens d'ouvrir tout simplement. Dans ce cas meme plus besoin de garder une reference sur la nouvelle fenetre ouverte.

Que tes fenêtres soient des singletons!!

:yaisse2:
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 public class Home extends JFrame { private JMenuBar menuBar; private JMenuItem communityMenuItem; private static Home instance; public static Home getInstance() { if (intance == null) { instance = new Home(); } } private Home() { initComponents(); ... } private void initComponents() { menu = new javax.swing.JMenuBar(); ... communityMenuItem = new javax.swing.JMenuItem(); ... communityMenuItem.addActionListener(new java.awt.event.ActionListener() { public void actionPerformed(java.awt.event.ActionEvent evt) { communityMenuItemActionPerformed(evt); } }); } private void communityMenuItemActionPerformed(java.awt.event.ActionEvent evt) { try { // Pour ouvrir une fenêtre JInternalFrame de type Community Community community = Community.getInstance(); this.add(community); community.setVisible(true); } catch (PapoException pe) { pe.printStackTrace(); } } ... }
C'est beaucoup plus simple et tu n'as alors aucun contrôle du style "est-ce que la frame est déjà affiché ou pas?"

TaFenetre.getInstance() au lieu new TaFenetre(), en créeant getInstance comme dans mon code.
Le constructeur est private réservé à lui-même mais accessible par getInstance() qui va te retourner une instance de TaFenetre.

De cette façon, tu n'as aucun truc à faire au niveau de tes boutons. TU auras beau cliquer dessus ce sera toujours la même interface qui apparaîtra, pas un clone ou une nouvelle instance, la MEME :king:
Après tu peux rajouter des comportements d'enfant unique à ta JFrame unique comme le fait de pouvoir se réinitialiser, de sauvegarder sa dernière position, etc...

Désactiver un bouton pour éviter le "double" clic dessus, c'est souvent une rustine qu'il vaut mieux éviter à moins de ne pas avoir le choix!

22/02/2008, 10h15
FredPsy

Bonjour,

Désolé MickaelBW, mais là, je ne comprends strictement rien à ce que tu m'exposes.

Tu pourrais me faire un exemple de code complet afin que je puisses l'étudier sans mettre trop de chose.

Genre une JFrame avec un bouton, qui ouvre une autre JFrame (et non une JInternalFrame), sans plus.

Merci Beaucoup.

PS : J'étudie déjà ce que tu m'as envoyé.

Dans le code que je t'ai mis j'ouvre une JInternalFrame mais tu peux remplacer par une JFrame, çà change rien.

La partie importante c'est çà :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 private static Home instance; public static Home getInstance() { if (intance == null) { instance = new Home(); } } private Home() { initComponents(); ... }
- Le constructeur de la JFrame devient private.
- Il y a maintenant une méthode static getInstance() qui va renvoyer une instance de Home en appellant ce constructeur private, sauf que getInstance() vérifie d'abord si une instance de Home n'existe pas déjà.
- La variable static Home instance contient donc une instance de Home.

Dans le code du bouton qui est censer ouvrir ta JFrame tu n'as plus que çà à faire :
Code:

1 2 3 Home home = Home.getInstance(); home.visible(true);
Fait que home soit DISPOSE_ON_CLOSE et voilà :)

EDIT: Oublie tout ce que je déclare dans Home au niveau des menu et actions. Home est une JFrame comme une autre sauf que pour ton application il n'y a jamais plus d'une seul instance de Home.

Bon, je t'ai çà rapidos :

Menu.java (une JFrame avec un bouton qui ouvre une JFrame Ecran)

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
72
73
74
 
/*
 * Menu.java
 *
 * Created on 22 February 2008, 09:31
 */
 
package testappli;
 
/**
 *
 * @author  Blackwings
 */
public class Menu extends javax.swing.JFrame {
 
  /** Creates new form Menu */
  public Menu() {
    initComponents();
  }
 
  /** This method is called from within the constructor to
   * initialize the form.
   * WARNING: Do NOT modify this code. The content of this method is
   * always regenerated by the Form Editor.
   */
  // <editor-fold defaultstate="collapsed" desc=" Generated Code ">                          
  private void initComponents() {
    jButton1 = new javax.swing.JButton();
 
    setDefaultCloseOperation(javax.swing.WindowConstants.EXIT_ON_CLOSE);
    jButton1.setText("Clic");
    jButton1.addActionListener(new java.awt.event.ActionListener() {
      public void actionPerformed(java.awt.event.ActionEvent evt) {
        perfomClic(evt);
      }
    });
 
    org.jdesktop.layout.GroupLayout layout = new org.jdesktop.layout.GroupLayout(getContentPane());
    getContentPane().setLayout(layout);
    layout.setHorizontalGroup(
      layout.createParallelGroup(org.jdesktop.layout.GroupLayout.LEADING)
      .add(layout.createSequentialGroup()
        .addContainerGap()
        .add(jButton1, org.jdesktop.layout.GroupLayout.DEFAULT_SIZE, 75, Short.MAX_VALUE)
        .addContainerGap())
    );
    layout.setVerticalGroup(
      layout.createParallelGroup(org.jdesktop.layout.GroupLayout.LEADING)
      .add(jButton1)
    );
    pack();
  }// </editor-fold>                        
 
  private void perfomClic(java.awt.event.ActionEvent evt) {                            
    Ecran ecran = Ecran.getInstance();
    ecran.setVisible(true);
  }                           
 
  /**
   * @param args the command line arguments
   */
  public static void main(String args[]) {
    java.awt.EventQueue.invokeLater(new Runnable() {
      public void run() {
        new Menu().setVisible(true);
      }
    });
  }
 
  // Variables declaration - do not modify                     
  private javax.swing.JButton jButton1;
  // End of variables declaration                   
 
}

Ecran.java

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
72
73
74
75
76
77
78
79
80
81
82
83
84
85
 
/*
 * Ecran.java
 *
 * Created on 22 February 2008, 09:32
 */
 
package testappli;
 
/**
 *
 * @author  Blackwings
 */
public class Ecran extends javax.swing.JFrame {
  private static Ecran instance;
  public static Ecran getInstance() {
    if (instance == null) {
      instance = new Ecran();
    }
    return instance;
  }
 
  /** Creates new form Ecran */
  private Ecran() {
    initComponents();
  }
 
  /** This method is called from within the constructor to
   * initialize the form.
   * WARNING: Do NOT modify this code. The content of this method is
   * always regenerated by the Form Editor.
   */
  // <editor-fold defaultstate="collapsed" desc=" Generated Code ">                          
  private void initComponents() {
    jLabel1 = new javax.swing.JLabel();
    jTextField1 = new javax.swing.JTextField();
 
    setDefaultCloseOperation(javax.swing.WindowConstants.DISPOSE_ON_CLOSE);
    jLabel1.setText("Hello world");
 
    jTextField1.setText("ecrit un truc");
 
    org.jdesktop.layout.GroupLayout layout = new org.jdesktop.layout.GroupLayout(getContentPane());
    getContentPane().setLayout(layout);
    layout.setHorizontalGroup(
      layout.createParallelGroup(org.jdesktop.layout.GroupLayout.LEADING)
      .add(layout.createSequentialGroup()
        .addContainerGap()
        .add(layout.createParallelGroup(org.jdesktop.layout.GroupLayout.LEADING)
          .add(layout.createSequentialGroup()
            .add(jLabel1)
            .addContainerGap())
          .add(layout.createSequentialGroup()
            .add(jTextField1, org.jdesktop.layout.GroupLayout.DEFAULT_SIZE, 69, Short.MAX_VALUE)
            .add(39, 39, 39))))
    );
    layout.setVerticalGroup(
      layout.createParallelGroup(org.jdesktop.layout.GroupLayout.LEADING)
      .add(layout.createSequentialGroup()
        .addContainerGap()
        .add(jLabel1)
        .addPreferredGap(org.jdesktop.layout.LayoutStyle.RELATED)
        .add(jTextField1, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE, org.jdesktop.layout.GroupLayout.DEFAULT_SIZE, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE)
        .addContainerGap(20, Short.MAX_VALUE))
    );
    pack();
  }// </editor-fold>                        
 
  /**
   * @param args the command line arguments
   */
  public static void main(String args[]) {
    java.awt.EventQueue.invokeLater(new Runnable() {
      public void run() {
        new Ecran().setVisible(true);
      }
    });
  }
 
  // Variables declaration - do not modify                     
  private javax.swing.JLabel jLabel1;
  private javax.swing.JTextField jTextField1;
  // End of variables declaration                   
 
}

C'est du code généré. Le truc important est au début de Ecran.java comme indiqué dans mon message précédent et dans la méthode performClic de Menu.java.

Pour le tester, utilise-le tel quel, clic sur le bouton, la JFrame apparait, modifie le champ texte, ferme la fenetre et reclic :)

Ou clic un millier de fois sur le bouton :king:

Citation:
Envoyé par MichaelBW
Dans le code que je t'ai mis j'ouvre une JInternalFrame mais tu peux remplacer par une JFrame, çà change rien.

La partie importante c'est çà :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 private static Home instance; public static Home getInstance() { if (intance == null) { instance = new Home(); } } private Home() { initComponents(); ... }
, etc ...

Ok, merci, ça marche, j'ai regardé tout ceci et ce n'est pas compliqué.
Merci beaucoup pour toutes ces précisions.

Bonne journée.