Problème fermeture fenêtre (frame.dispose())

Version imprimable

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:

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.

26/08/2009, 02h40
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().
26/08/2009, 13h04
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:

private void initGUI()
28/08/2009, 19h01
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:

MyWindow.this.dispose();
29/08/2009, 02h52
slim_java

Citation:

Envoyé par 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.

Code:

MyWindow.this.dispose();

ou bien ne pas utiliser une classe anonyme et créer une classe " private implements ActionListener "
29/08/2009, 14h23
Eric73

Bonjour,

Citation:

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:

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:

MyWindow.this.dispose();
29/08/2009, 16h06
slim_java

Citation:

Envoyé par Eric73

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

pourquoi j'ai mis private ? 8O
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)

Citation:

Envoyé par 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:

MyWindow.this.dispose();

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

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:

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(); } });

30/08/2009, 11h24
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 :).