Limiter le nombre de Jframe ouvertes

Bonjour,

Voici mon probleme: un clic sur une image entraine l'ouverture d'une Jframe. Mais je voudrais interdire le fait d'en ouvrir deux. Autrement dit, quand l'utilisateur clic sur l'image, si une Jframe est deja ouverte, j'aimerai que cette Jframe se ferme, et qu'une nouvelle apparaisse.
J'ai donc procédé de la maniere suivante : avant de creer la Jframe, je teste si elle existe deja, et je la "dispose".

Code:

---

1 2	if (fenetre != null ) fenetre.dispose();

---

Malgres cela, et particulierement sur un ordinateur lent, (500 Mhz, 512 Mo de Ram ) en cliquant frenetiquement sur l'image, il arrive que plusieurs Jframe s'ouvrent. Y-aurait-il un moyen pour eviter cela ?

Merci d'avance

Pourrais tu mettre le code complet de ton image stp?

Bah pourquoi ne tout simplement pas changer l'image contenue dans la JFrame déjà ouverte?

J'ai du mal m'exprimer. L'image contenue dans le Jpanel ne doit pas changer. En cliquant sur cette image une sorte de pop-up s'ouvre (la Jframe). Cette Jframe contient deux listes.
Le probleme etant que je voudrais qu'il ne soit possible d'ouvrir qu'une seule de ces Jframe a la fois(et pas ouvrir une Jframe a chaque clic. Si l'utilisateur clic sur l'image et que la Jframe est deja ouverte, elle doit se fermer, et une nouvelle devrait s'ouvrir ).

En ce qui concerne le code :

Jpanel contenant l'image :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

this.addMouseListener(new java.awt.event.MouseAdapter() { public void mousePressed(final java.awt.event.MouseEvent e) { new SwingWorker() { public Object construct() { … startList(x, y);       return null; } }.start();   private void startList(int x, int y) { if (popup != null) { popup.dispose(); } else popup = new ListPopup(…);

---

(Popup etant la Jframe qui ne devrait s'ouvrir qu'une seule fois)

Pourquoi utilises-tu un SwingWorker ici, un EventQueue.involeLater suffirait largement il me semble ?

Dans le cas de clics frénétiques, je pense que tu es coincé par le délai entre la création d'une fenêtre et son affichage, et pareil pour le dispose. Ni la création d'une fenêtre, ni le dispose ne sont atomiques, même de loin. Je pense que ton code abouti à faire le dispose de la même fenêtre plusieurs fois, et / ou à créer plusieurs fenêtres alors que l'on croit qu'il n'y en a aucune.

Je pense que la solution la plus simple pour toi serait de mémoriser les informations de présence / absence de fenêtre dans un booléen, information certe redondante avec la fenêtre, mais qui réagit immédiatement.

Merci beaucoup pour ta réponse. J'ai commencé par remplacer mon swingworker par un EventQueue.invokeLater et du coup, la methode avec popup.dispose() marche, je n'ai plus de problème, la fenetre se referme systematiquement si elle etait auparavant ouverte.

Code:

---

1 2 3 4 5 6 7 8 9 10 11

this.addMouseListener(new java.awt.event.MouseAdapter() { public void mousePressed(final java.awt.event.MouseEvent e) { EventQueue.invokeLater(new Runnable() { public void run() { StartList(x, y);   } }); } });

---

Par contre j'ai l'impression que mon interface est moins reactive. L'ouverture de la fenetre entrainant differents acces a une base de donnée, mon interface "freeze" un peu plus. Alors que le fait d'utiliser invokeLater devrait justement eviter de contraindre l'EDT, non ? Bizarre ...

Ah bin peut être que ton SwingWorker était justifié alors :-) Ce n'est pas parce que je dis quelque chose qu'il faut le faire :frenchy: Surtout que au moment où je t'ai proposé ça je n'avais pas conscience qu'il y avait des accès aux bases de données, je voyais juste ton code de dispose et de création de frame.

Je pense que la situation s'est améliorée sans le SwingWorker, parce que l'intervalle ancien - nouveau est réduit, ou du moins qu'il y a moins de threads en cause, et donc qu'il y a moins de risques qu'un thread voit ancien alors que c'est déjà nouveau et vice versa.

Je te conseille de réfléchir à la situation sur le modèle des machines à état, et de bien vérifier que toutes les transitions sont synchrones sur tous les threads.