Discussion :

[calogerogigante]

J'ai fait des recherches sur le forum concernant les JDialog, mais je n'ai pas trouvé de réponses très précises à ma question.

J'ai créé une boite de dialogue un peu compliquée (les JOptionPane ne suffisaient pas). Dans cette boite (que je rends modale), j'ai pas mal d'infos à récupérer.

Donc (si j'ai bien compris), contrairement aux JOptionPanes, la JDialog doit encore exister après sa fermeture par l'utilisateur, c'est ça ?
Elle ne renvoit pas une valeur à sa fermeture, il faut la chercher soi-même en prévoyant des getters ? Je me trompe ?

Un exemple complet est donné ici (trouvé sur le net) :

example

Est-ce que mon raisonnement est juste ?
C'est comme ça qu'il faut faire ?

C'est seulement quand on est sûr d'avoir tout récupéré, qu'on peut détruire une JDialog par la méthode .dispose() ?

[iuz]

La JDialog ne doit pas exister après sa fermeture par l'utilisateur, mais elle le PEUT.

Personnellement quand je récupère plusieurs données je m'y prend comme suit, en changeant l'ArrayList par un tableau quand il s'agit de plusieurs données d'un même type :

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
 
public class MaJDialog extends JDialog implements ActionListener {
  private static ArrayList liste;
 
  public MaJDialog(Frame frame, String title, boolean modal) {
    super(frame, title, modal);
    /** création de la JDialog*/
    this.setVisible(true);
  }
  public static ArrayList ShowMaJDialog(Frame frame, String title, boolean modal) {
    new MaJDialog(frame, title, modal);
    return liste;
  }
  public void actionPerformed(ActionEvent e) {
    if (e.getActionCommand().equals("valider")) {// Les informations sont validées par l'utilisateur
      /** On place toutes les données dans l'ArrayList */
    }
    this.setVisible(false);
    this.dispose();
  }
}

Comme ça on n'a plus besoin de gérer sa fermeture, juste de traiter les données renvoyées.

[bubu56]

Personnellement je te conseille plutôt la méthode que tu décris avec les getters…
une fois que tu as fini avec les getter tu peux appeler dispose...

surtout que la méthode ShowMaJDialog renvoit une reference à la liste, et rien ne te garantie qu'une autre instance du dialogue (ou une autre classe) ne va pas modifier cette même liste par la suite...

[iuz]

C'est vrai qu'il fallait utiliser ma boite de dialogue avec une seule Frame et en modale. Mais je préfère ne plus m'occuper de la boite de dialogue une fois qu'elle a rempli son office, d'autant que j'utilise toujours la même, je change juste le panneau à afficher avec un paramètre supplémentaire.

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
 
public class MaJDialog extends JDialog implements ActionListener {
  private ArrayList liste;
 
  public MaJDialog(Frame frame, String title, boolean modal) {
    super(frame, title, modal);
    /** création de la JDialog*/
    this.setVisible(true);
  }
  public static ArrayList ShowMaJDialog(Frame frame, String title, boolean modal) {
    MaJDialog maJD = new MaJDialog(frame, title, modal);
    return maJD.getListe();
  }
  public void actionPerformed(ActionEvent e) {
    if (e.getActionCommand().equals("valider")) {// Les informations sont validées par l'utilisateur
      /** On place toutes les données dans l'ArrayList */
    }
    this.setVisible(false);
    this.dispose();
  }
  public ArrayList getListe() {
    return liste;
  }
}[/LEFT]

La remarque de bubu56 va me permettre d'épurer mon programme d'un problème, merci

[bubu56]

cette méthode est bonne si tu n'as qu'un seul objet à récupérer et dans la plupart des cas, cela suffit, mais si tu as plusieurs objets à récupérer tu seras obligé de passer par la première méthode décrite.

avec la méthode de iuz il faut mettre le constructeur de MaJDialog private afin d'être sur que le développeur utilise la classe MaJDialog correctement (c'est à dire en appelant ShowMaJDialog).

Autre chose, les noms de méthodes sont sensés commencer par des minuscules donc il faudrait renommer ShowMaJDialog en showMaJDialog

[calogerogigante]

Merci pour vos explications et vos propositions.

J'ajoute le témoignage d'un exemple vu dans le bouquin de C. Delannoy.

Il crée une JDialog maDialog depuis la JFrame qui la lance.

Ensuite, depuis la JFrame même, il fait

maDialog.setVisible(true);

et ceci suffit à mettre la JDialogue en attente de réaction par l'utilisateur (surtout si elle est modale).

Ensuite, directement après (la ligne de code juste après), il récupère ses données via une classe Info qui sert de transmission de valeurs de différentes variables, ensuite il détruit la JDialog (la ligne juste après encore).

Je crois que je vais donc aussi opter pour ce genre de fonctionnement.

Merci à tous. Je mets ce post en résolu !

Long live to Developpez.com