Discussion :

[Yokooo]

Bonjour,
mon problème est que je n'arrive pas à fermer ma fenêtre, je sais qu'il faut utiliser frame.dispose() mais je dois faire quelque chose de mal car il me souligne en rouge frame et dispose() en marquant "Cannot find symbol"
Voici mon code :

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
25
26
27
28
29
30
31
32
33
34
35
public static void main() {
		SwingUtilities.invokeLater(new Runnable() {
			public void run() {
				JFrame frame = new JFrame();
				SelectCourb inst = new SelectCourb();
				frame.getContentPane().add(inst);
				((JComponent)frame.getContentPane()).setPreferredSize(inst.getSize());
				frame.pack();
				frame.setVisible(true);
                                System.out.println("Test");
			}
		});
 
	}
 
	public SelectCourb() {
		super();
		initGUI();
	}
 
	private void initGUI() {
		try {
			GroupLayout thisLayout = new GroupLayout((JComponent)getContentPane());
			getContentPane().setLayout(thisLayout);
			setSize(new Dimension(400, 300));
{
				jButton1 = new JButton();
				jButton1.setText("Valider");
                                jButton1.addActionListener(new java.awt.event.ActionListener() {
                                    public void actionPerformed(java.awt.event.ActionEvent evt) {
                                    frame.dispose(); 
                                    }
                                });
			}
...................................

Voila je vous remercie d'avance pour votre aide.

[bouye]

Il n'y a pas de variable frame definie dans la classe. Dans ton code, ta seule variable frame est locale a la methode run() du Runnable dans ton main().

[Yokooo]

ça ne marche toujours pas il ne connait pas la méthode, j'avais essayé this.dispose()

Donc ce qui veut dire que si je veux fermer cette fenêtre il va falloir que je le fasse à partir du run ? ou que je déplace mon ouverture de frame dans :

Code :

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

private void initGUI()

[Eric73]

Bonjour,

Tu ne peux pas appeler dispose() dans ta méthode actionPerformed() puisque tu te trouves alors dans l'ActionListener anonyme. "this" se réfère alors à lui. Tu peux référencer le this de ta fenêtre en préfixant "this" par le nom de la classe de ta fenêtre. Ainsi, si la classe de ta fenêtre est "MyWindow", tu peux écrire :

Code :

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

MyWindow.this.dispose();

[slim_java]

Bonjour,

Tu ne peux pas appeler dispose() dans ta méthode actionPerformed() puisque tu te trouves alors dans l'ActionListener anonyme. "this" se réfère alors à lui.

Code :

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

MyWindow.this.dispose();

ou bien ne pas utiliser une classe anonyme et créer une classe " private implements ActionListener "

[Eric73]

Bonjour,

ou bien ne pas utiliser une classe anonyme et créer une classe " private implements ActionListener "

Non, car cela ne changerait rien à son problème : il n'aura toujours pas accès à la référence de sa fenêtre directement, si ce n'est qu'en plus il aura créé une classe (qu'il devra nommer) exprès. L'intérêt des classes anonymes (et bientôt, des closures) est justement de ne pas s'embêter à créer des classes pour des choses aussi triviales. D'autant que pour ce cas, ça réduit même la lisiblité :

Code :

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

jButton1.addActionListener(new MyListener());

Que fait MyListener ? Il faut aller voir pour le savoir. Et dans MyListener, il faudra bien qu'il écrive ça, comme dans la classe anonyme :

Code :

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

MyWindow.this.dispose();

[slim_java]

Non, car cela ne changerait rien à son problème

pourquoi j'ai mis private ?
ben tous simplement pour dire que la classe qu'il va créer sera interne et dans ce cas tout est arrangé avec un petit dispose().

( je dois apprendre a être explicite)

[tchize_]

Bonjour,

Tu ne peux pas appeler dispose() dans ta méthode actionPerformed() puisque tu te trouves alors dans l'ActionListener anonyme. "this" se réfère alors à lui. Tu peux référencer le this de ta fenêtre en préfixant "this" par le nom de la classe de ta fenêtre. Ainsi, si la classe de ta fenêtre est "MyWindow", tu peux écrire :

Code :

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

MyWindow.this.dispose();

le this n'est pas nécesaire. Ni le MyWindow. Ce code devrais fonctionner sans problème:

Code :

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

1
2
3
4
5
6
7
8
9
10
public class MyFrame extends JFrame{
  .....
  button.addActionListener(new ActionListener(){
       ...
       dispose();
       ...
  }
  .....
}
}

Car les inner class voient toutes les méthodes de la outer class. Le problème de départ c'est que frame n'est pas visible. Et pour cause, frame est une variable locale à la méthode main. Tout le monde a l'air de supposer que sa classe étend JFrame, ce qui n'est pas le cas il me semble. Une solution est de faire ceci

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
 
// main
				JFrame frame = new JFrame();
				SelectCourb inst = new SelectCourb(frame);
//....
//constructeur
        private JFrame parent;
	public SelectCourb(JFrame parent) {
                this.parent=parent;
		initGUI();
	}
// initGui (on change frame en parent)
 
                                jButton1.addActionListener(new java.awt.event.ActionListener() {
                                    public void actionPerformed(java.awt.event.ActionEvent evt) {
                                    parent.dispose(); 
                                    }
                                });

[Eric73]

Salut,

Mmh en effet tchize_, j'avoue ne pas avoir pensé un instant que sa classe pouvait ne pas être une JFrame. C'est pour ça que j'ai pensé qu'il était nécessaire de préfixer. Autant pour moi .