Discussion :

[Piolet]

Bonjour,

j'ai concu une popup dialog particulière.

Elle est appelée par une methode ayant cette signature :

Code :

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

1
2
 
public int showMessageBox(String message);

en gros cette methode affiche un JPanel en 1er plan de ma fenetre, donc ce qu'il y a dessous est inaccessible logiquement.

Pour faire encapsuler, je voudrais que l'utilisateur de cette méthode n'est pas d'evenement a lier sur le JPanel afficher.

En gros, l'appel de cette méthode afficherait le panel, et attendrait un clic (sur l'un des boutons present sur ce panel, "YES", "NO", etc...) et une fois que le clic a lieu renverrait la valeur associé a ce clic. en gros la méthode serait bloquante tant qu'on a pas cliqué.

Pour ceux qui connaissent c#, ca ressemble, bizarrement , aux MessageBox.show de c#

donc voila...j'ai pas l'idée pour le faire, alors que ca doit pas etre bien compliqué...

merci d'avance

[hydraland]

Salut,

Pourquoi n'utilises tu pas la classe JOptionPane? Elle fait exactement ce que tu veux.

A+
Hydraland

[Piolet]

Salut,

Pourquoi n'utilises tu pas la classe JOptionPane? Elle fait exactement ce que tu veux.

A+
Hydraland

euh...parce que c'est moche....je connais OptionPane...mais je veux ma propre "boite de dialogue" qui en fait n'est pas / plus une popup... you know

[bulbo]

Tu veux ça sous la forme d'une boite de dialogue (ie: une nouvelle petite fenêtre) qui s'ouvre devant l'autre ?

-> Dans ce cas utilise un JDialog modal

Tu veux ça comme un panel masquant (ou grisant) la fenêtre de ton application

-> Dans ce cas le mieux est d'utiliser un GlassPane (tu as une super démo de Gfx sur dvp)

Bulbo

[Piolet]

Tu veux ça sous la forme d'une boite de dialogue (ie: une nouvelle petite fenêtre) qui s'ouvre devant l'autre ?

-> Dans ce cas utilise un JDialog modal

Tu veux ça comme un panel masquant (ou grisant) la fenêtre de ton application

-> Dans ce cas le mieux est d'utiliser un GlassPane (tu as une super démo de Gfx sur dvp)

Bulbo

le panel est deja fait...et j'utilise le GlassPane ca ca marche...c'est meme bluffant comment ca rend bien...

mais je veux juste que ma méthode renvoie qq chose au clic sur un bouton...

je veux pas avoir a créer le panel, l'afficher puis m'abonner a un evenement et attendre l'evenement...

je veux juste que ces etapes soient encapsulées, comme le JOptionPane

[bulbo]

le panel est deja fait...et j'utilise le GlassPane ca ca marche...c'est meme bluffant comment ca rend bien...

mais je veux juste que ma méthode renvoie qq chose au clic sur un bouton...

je veux pas avoir a créer le panel, l'afficher puis m'abonner a un evenement et attendre l'evenement...

je veux juste que ces etapes soient encapsulées, comme le JOptionPane

Dans ce cas il va falloir faire un peu de synchro entre threads..

Il faut que ta méthode soit bloquante mais attention si tu es déjà sur l'EDT (le thread de swing) tu figes ta GUI.

Dans un cas similaire ce que j'ai fait c'est:

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
public int showMessageBox(String message)
{
  SwingUtilities.invokeLater(.....);
 
  synchronized(this)
  {
    try
    {
      this.wait();
    }
    catch (Exception e) {}
   }
 
  // ici tu récupères la valeur du bouton pressé et tu la retournes ..
}

Et dans l'action listener du bouton qui doit virer ton panel tu fais un:

Code :

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

1
2
3
4
5
...
synchronized(myMainClass.this)
{
  notifyAll();
}

Si tu es déjà sur l'EDT (dans l'action listener d'un bouton par exemple) passe par un nouveau thread pour appeler showMessageBox.

Bulbo

[Piolet]

ok je vais tester merci....

mais par contre...l'EDT...tu peux me la refaire....tu parles de 2 cas qui sont similaires, t'as pas oublié un "pas" ?

[bulbo]

ok je vais tester merci....

mais par contre...l'EDT...tu peux me la refaire....tu parles de 2 cas qui sont similaires, t'as pas oublié un "pas" ?

La méthode showMessageBox ne doit pas être appelée sur l'EDT.

C'est a dire que si tu as un code dans ton appli qui soudain décide de bloquer la GUI avec ton truc pas de problème tu appelles directement showMessageBox.

Par contre si tu veux afficher ton truc après que l'utilisateur ai clique qq part dans ta GUI par exemple, il faut passer par un nouveau thread.
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
public void actionPerformed(ActionEvent event)
{
  new SwingWorker<Integer, Object>()
  {
    protected Integer doInBackground()
    {
       return new Integer(showMessageBox(message));
    }
 
    public void done()
    {
       System.out.println("Le bouton appuyé est le : " + get());
    }
  }.execute();
}

Code écrit a la volée donc possible erreur mais l'idée est la..

Bulbo

[Piolet]

ok..donc vu que tout doit se passer a l'appel de la methode, ta première version passe....

[bulbo]

ok..donc vu que tout doit se passer a l'appel de la methode, ta première version passe....

Tout dépend d'où tu appelles ta méthode...

Exemple 1: Lorsqu'une appli essaye de se connecter a la tienne tu appelles la méthode pour demander si on accepte la connexion ...
Dans ce cas tu appelles ta méthode depuis le thread qui est bloqué sur l'accept du serveur socket donc pas de problème.

Exemple 2: tu cliques sur "Sauver" et l'appli te demande de confirmer en utilisant ta méthode, pas bon car tu es sur l'EDT, il faut donc passe par un SwingWorker..

J'ai l'impression qu'on se comprend pas c'est fou .. ou c'est moi qui suis fatigué

Bulbo

[Piolet]

Tout dépend d'où tu appelles ta méthode...

Exemple 1: Lorsqu'une appli essaye de se connecter a la tienne tu appelles la méthode pour demander si on accepte la connexion ...
Dans ce cas tu appelles ta méthode depuis le thread qui est bloqué sur l'accept du serveur socket donc pas de problème.

Exemple 2: tu cliques sur "Sauver" et l'appli te demande de confirmer en utilisant ta méthode, pas bon car tu es sur l'EDT, il faut donc passe par un SwingWorker..

J'ai l'impression qu'on se comprend pas c'est fou .. ou c'est moi qui suis fatigué

Bulbo

mais dans l'exemple 2...
le swingWorker peut etre contenu dans la méthode non ?
quand on appel un JOptionPane, on l'appel partout pareil ...

et je te rassure, je suis aussi fatigué...

sinon dans la continuité...comment je fais pour la touche tab n'affecte que mon panel supérieur et ne donne pas le focus au élément en dessous..???

[adiGuba]

Salut,


Je te conseillerais de rajouter une vérification au tout début de ta méthode showMessageBox() :

Code :

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

1
2
3
	if (SwingUtilities.isEventDispatchThread()) {
		throw new IllegalMonitorStateException("showMessageBox() ne peut pas être utilisé dans l'EDT");
	}

Cela permet d'éviter des deadlock sur l'EDT

quand on appel un JOptionPane, on l'appel partout pareil ...

Les JOptionPane utilise un JDialog modale qui bloque le thread courant, et qui gère le cas particulier de l'EDT en relançant les traitements de ce dernier pendant l'attente...

J'ai regardé un peu comment c'est fait mais cela utilise des classes non-public et je ne sais pas s'il est possible de reproduire ce comportement...

a++

[Piolet]

Salut,


Je te conseillerais de rajouter une vérification au tout début de ta méthode showMessageBox() :

Code :

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

1
2
3
	if (SwingUtilities.isEventDispatchThread()) {
		throw new IllegalMonitorStateException("showMessageBox() ne peut pas être utilisé dans l'EDT");
	}

Cela permet d'éviter des deadlock sur l'EDT



Les JOptionPane utilise un JDialog modale qui bloque le thread courant, et qui gère le cas particulier de l'EDT en relançant les traitements de ce dernier pendant l'attente...

J'ai regardé un peu comment c'est fait mais cela utilise des classes non-public et je ne sais pas s'il est possible de reproduire ce comportement...

a++

arf, en gros tu me dis que c'est la merde la

[adiGuba]

arf, en gros tu me dis que c'est la merde la

Heu... non ! Je confirme juste que tu ne peux utiliser cette méthode que depuis un thread différent de l'EDT... et la vérification permet d'éviter des deadlocks pas forcément évident à déceler...


Maintenant si tu veux pouvoir utiliser cette méthode même depuis l'EDT cela risque d'être plus compliqué...

a++

PS : j'ai trouvé un billet qui parle du fonctionnement des JOptionPane https://aptframework.dev.java.net/ed...ptionPane.html

[bulbo]

Non ça ne fonctionne pas avec le SwingWorker dans la méthode..

doInBackground doit être bloque jusqu'à la décision de l'utilisateur

grilled par adiGuba.. donc oui le mieux est de faire le test proposé au dessus au debut de la methode ...

Sur l'EDT il faudra faire attention et utiliser un SwingWorker ...

Attention le reste de ton code ne devra pas aller après le execute du SwingWorker mais dans la méthode done

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
public void actionPerformed(ActionEvent event)
{
  new SwingWorker<Integer, Object>()
  {
    protected Integer doInBackground()
    {
       return new Integer(showMessageBox(message));
    }
 
    public void done()
    {
       System.out.println("Le bouton appuyé est le : " + get());
 
        // La suite de ton code ici ...
    }
  }.execute();
  // ne mettre aucun code ici.
}

et la ça marche .. désolé je me suis trituré les méninges mais pas moyen de couper au SwingWorker dans ce cas ..

Bulbo

[bulbo]

Ah oui au fait j'avais oublié une des tes questions: comment éviter que tab se balade dans les composants "en dessous"

Ajoute ça dans le constructeur de ton JPanel:

Code :

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

1
2
3
4
5
6
7
8
9
 
    // This component keeps the focus until is made hidden
    setInputVerifier(new InputVerifier()
    {
      public boolean verify(JComponent input)
      {
        return !isVisible();
      }
    });

Bulbo

[Piolet]

et tout ca..je peux pas l'encapsuler dans une methode visiblement car ca implique mettre du code après le execute...donc c'est pas super pratique si a chaque fois que je veux appeller une popup je suis obligé de me 'rappeler" de tout ca ...

[bulbo]

Tu as lu l'article qu'a trouvé adiGuba ?

Quand tu vois le code qui a été pondu pour permettre ça , je suis pas sur que ça vaille la peine .. une IllegalStateException est suffisant et est très rapide a tester ...

Si tu as peur de ne pas te souvenir de la bidouille a faire pour l'EDT, mets ça dans les commentaires de ta méthode, ils sont la pour ça.

Bulbo

[Piolet]

Tu as lu l'article qu'a trouvé adiGuba ?

Quand tu vois le code qui a été pondu pour permettre ça , je suis pas sur que ça vaille la peine .. une IllegalStateException est suffisant et est très rapide a tester ...

Si tu as peur de ne pas te souvenir de la bidouille a faire pour l'EDT, mets ça dans les commentaires de ta méthode, ils sont la pour ça.

Bulbo

bah oui je suis d'accord, mais c'est tellement plus propre...
et merci pour le TAB

[Piolet]

Non ça ne fonctionne pas avec le SwingWorker dans la méthode..

doInBackground doit être bloque jusqu'à la décision de l'utilisateur

grilled par adiGuba.. donc oui le mieux est de faire le test proposé au dessus au debut de la methode ...

Sur l'EDT il faudra faire attention et utiliser un SwingWorker ...

Attention le reste de ton code ne devra pas aller après le execute du SwingWorker mais dans la méthode done

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
public void actionPerformed(ActionEvent event)
{
  new SwingWorker<Integer, Object>()
  {
    protected Integer doInBackground()
    {
       return new Integer(showMessageBox(message));
    }
 
    public void done()
    {
       System.out.println("Le bouton appuyé est le : " + get());
 
        // La suite de ton code ici ...
    }
  }.execute();
  // ne mettre aucun code ici.
}

et la ça marche .. désolé je me suis trituré les méninges mais pas moyen de couper au SwingWorker dans ce cas ..

Bulbo

si j'encapsule ca dans mon propre evenement, la méthode showMessage peut lancer mon evenement :

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
 
public int showMessage(string Message) {
   this.fireMonAction(MonActionEvent evt);
}
 
public void monActionListener(MonActionEvent evt) {
new SwingWorker<Integer, Object>()
  {
    protected Integer doInBackground()
    {
       return new Integer(showMessageBox(message));
    }
 
    public void done()
    {
       System.out.println("Le bouton appuyé est le : " + get());
       evt.set(le bouton appuyé);
 
        // La suite de ton code ici ...
    }
  }.execute();
}

rmq...ca va revenir au meme...ca va pas attendre ???