Discussion :

[billynirvana]

Bonjour,

Je suis en cours de développement d'une application dont la classe qui contient le main possède grosomodo 3 méthodes principales:

- init()
- launchWindow()
- finalize()

Dans init(), j'ouvre pas mal de flux notamment des sockets.
Dans launchWindow(), je crée la fenetre graphique.
Dans finalize(), je libère les ressources utilisées (et initialisées dans init()).

Si je code de la facon suivante, ca plante:

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
public final class Appli {

	public final static void main(String[] args) {
		Appli appli = new Appli();

		appli.init();
		appli.launchWindow();

		appli.finalize(); <--- PROBLEME!!!
	}

	public Appli() {}

	public void init() { ... }
	public void launchWindow() { ... }
	public void finalize() { ... }
}

La raison est surement parce que launchWindow() n'est pas bloquant. finalize est tout de suite appelée apres launchWindow() et donc j'ai un nullPointerException


Je ne vois pas comment m'en sortir... Peut etre est ce du à une mauvaise conception...


Si vous avez des idées ou propositions pour résoudre mon problème, je suis preneur!!


Merci d'avance,

Billy

[maxf1]

C'est dans ta methode finalize qu'il y a un nullPointer.

Regarde bien la stackTrace de ton plantage et va à la ligne indiqué.

NUllPointer => Signifie que tu appel une methode sur un object null

Exemple :

Code :

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

1
2
String s = null;
s.indeOf("1"); // <= la il va y avoir un nullPointer

[Baptiste Wicht]

Il faut que tu appelles finalize() à la fermeture de ta fenêtre, pour cela, il faut que tu ajoutes un WindowListener et que lors de la l'appel de la méthode windowClosing, tu appelles finalize().

[billynirvana]

J'ai un windowListener dans la classe correspondant à ma fenetre. Je dois également ajouter cette implémentation dans ma classe Appli ?

[billynirvana]

J'ai essayé mais cela ne marche pas. je n'y arrive pas!

J'ai vérifié les logs Eclipse et le nullPointerException apparait dans l'appli lorsque je veux envoyer des données sur la socket [socket is null] (la méthode qui cradosse se trouve dans la classe qui étend JFrame)

[franck06]

J'ai essayé mais cela ne marche pas. je n'y arrive pas!

J'ai vérifié les logs Eclipse et le nullPointerException apparait dans l'appli lorsque je veux envoyer des données sur la socket [socket is null] (la méthode qui cradosse se trouve dans la classe qui étend JFrame)

t'aas une classe qui étend de JFrame
dans le constructeur de ta classe, lorsque tu définis la fenetre, a la fin, tu appelles initialise
tu appelles ensuite launchWindow
dans ta classe, tu définis un WindowListener pour la fermeture de ta fenetre
c'est la que tu mettras le finalize()

[le y@m's]

Attention, il est fortement recommandé de ne PAS utiliser la méthode finalize() pour libérer des ressources.
En effet, il n'est pas garantie que celle-ci soit appelée. Et si tu l'appelle explicitement (chose qu'il ne faut pas faire), il peut y avoir des problèmes si elle est réappelée lors de la destruction de l'objet (par exemple tentative d'un flux déjà fermé lors du précédent appel à finalize()).

Il vaut mieux écrire une méthode par toi même, par exemple nommée exit(), qui s'occupe de libérer les ressources (ou tout autre tâche de ton choix).

[billynirvana]

Ca ne me gene pas de renommer la méthode finalize en exit car elle n'existe pas dans la classe Object.

En fait, je crois que vous vous êtes embrouillé tout seul, ou que je me suis mal exprimé.

Voilà à quoi ressemble mes 2 principales classes. Les autres classes sont juste des composants swing customizés.

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
public class Appli extends Object {

	public final static void main(String[] args) {
		Appli appli = new Appli();

		appli.init();
		appli.launchWindow();

		appli.finalize(); <-- PROBLEME
	}

	private void init() {
		// ouverture socket

	}

	private void finalize() {
		// fermeture socket
	}

	private void launchWindow() {
		new AppliWindow();
	}

}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public class AppliWindow extends JFrame implements WindowListener, ActionListener {
 
	public AppliWindow() {
		// initialisation des composants graphiques + listener
	}
 
	// insertion des méthodes obligatores par les implementations
	public void windowClosing(WindowEvent arg0) { // par exemple
		dialogueBox("Vous nous quittez déjà?");
		...
	}
}

J'ai essayé de faire en sorte que Appli implemente aussi WindowListener, mais je n'ai pas obtenu satisfaction...

[nicØB]

Salut,

Ca veut dire quoi "je n'ai pas obtenu satisfaction..." ?

Tu fais bien un addWindowListener(this), quelque part dans le code de AppliWindow ?
Et penses à faire un setDefaultCloseOperation(JFrame.DO_NOTHING_ON_CLOSE), sur ta fenêtre.

Tu peux aussi voir du côté des fonctions wait() et notify(), si tu veux garder ta "façon de faire".
Tu bloques au niveau de launchWindow, puis tu débloques quand il faut fermer.

P.S: la méthode windowClosing est appelée quand la fenêtre reçoit l'ordre de se fermer, et windowClosed quand la fenêtre est fermée.

[billynirvana]

Je n'ai pas obtenu satisfaction signifie que

- Soit je dois passer en argument l'instance de Appli dans le constructeur de AppliWindow afin de pouvoir appeler la methode finalize dans WindowClosed (qd l'evenement de fermeture de fenetre se produit). Ca ne me plaisait pas comme conception.

- J'ai essayé d'ajouter un addWindowListener(---) dans Appli mais Eclipse refuse de me compiler.

[adiGuba]

Salut,


Tu peux utiliser un shutdown hook :

Code :

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

1
2
3
4
5
6
	Runtime.getRuntime().addShutdownHook(new Thread() {
		@Override
		public void run() {
			// Code a exécuter à la fermeture du programme
		}
	});

Par contre tu ne devrais pas utiliser le nom "finalize()" qui est utilisé par le GC...

a++

[billynirvana]

Bon, j'ai réussi avec les WindowListener. En fait j'avais utilisé windowClosing au lieu de windowClosed

Voici l'essentiel des opérations:

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 class Appli extends Object implements WindowListener {
 
	public final static void main(String[] args) {
		Appli appli = new Appli();
 
		appli.init();
		appli.launchWindow();
	}
 
	private void init() {
		// ouverture socket
 
	}
 
	private void cleanExit() {
		// fermeture socket
	}
 
	private void launchWindow() {
		AppliWindow appWindow = new AppliWindow();
		appWindow.addWindowListener(this);
	}
 
	public void windowClosed(WindowEvent arg0) {
		cleanExit();
	}
 
	public void windowOpened(WindowEvent arg0) {}
	public void windowClosing(WindowEvent arg0) {}
	public void windowIconified(WindowEvent arg0) {}
	public void windowDeiconified(WindowEvent arg0) {}
	public void windowActivated(WindowEvent arg0) {}
	public void windowDeactivated(WindowEvent arg0) {}
 
}

Code :

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

1
2
3
4
5
6
7
8
9
public class AppliWindow extends JFrame {
 
	public AppliWindow() {
		...
		/** non autorisation de la fermeture de la fenetre avec ALT-F4 */
		this.setDefaultCloseOperation(JFrame.DO_NOTHING_ON_CLOSE);
		...
	}
}

@adiguba: Cela marche également avec ton astuce.


MERCI A TOUS!




Billy