Tuer un timer

Version imprimable

10/09/2008, 09h51
donnadieujulien

Tuer un timer

Bonjour,

je déclare un timer dans une Jdialog.

Ce timer démarre à l'ouverture de la fenètre, et doit s'arrêter et être détruit à sa fermeture.

Je ferme cette fenetre via this.dispose();

Or je constate qu'en faisant comme ca, le timer reste vivant même après la fermeture de cette fenetre.

Comment est ce possible?

Faut il le finaliser pour résoudre le problème?
Sachant que je pourrait très bien faire MonTimer.stop(), mais je ne suis pas sur que l'objet soit bien effacé de la mémoire, à moins que le garbage collector voit quand un objet n'est plus actif et le vire automatiquement?

Aidez moi svp, car comme mon appli est déjà gourmande en mémoire, je voudrai l'alléger un peu.

Merci
10/09/2008, 10h45
dinobogan

Tant que le Thread n'a pas terminé son traitement, il va rester vivant et en mémoire.
Quel est le code de ton timer ?

Code

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
private Timer createTimer (){
    ActionListener action = new ActionListener (){
        public void actionPerformed(ActionEvent event){
            if(compteur < 240){
                transactions.setListData(datas.getTransactions());
                repaint();
                compteur++;
            }
            else{
                timer.stop();
                timer = null;
                alerte(); //JDialog "Temps dépassé"
            }
        }
    }
}

Il sert à rafraichir un l'affichage des transactions en cours sur DB2, et à décompter 20 min de connexion max sur l'interface
merci

Desolé

J'avais zappé une partie :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
private Timer createTimer (){
	ActionListener action = new ActionListener (){
		public void actionPerformed (ActionEvent event){
			if(compteur < 240){					transactions.setListData(datas.get_Transactions());
				repaint();
				compteur ++;
			}
			else{
				timer.stop();
				timer = null;
				alerte();
			}
		}
	};
	return new Timer (5000, action);
}

Salut,

Citation:

Je ferme cette fenetre via this.dispose();

Or je constate qu'en faisant comme ca, le timer reste vivant même après la fermeture de cette fenetre.

Comment est ce possible?

Le Timer est un processus qui est indépendant de tous les autres une fois crée. Si tu veux résoudre le problème, esaie de le déclarer Daemon. Cela signifie qu'il sera détruit si plus aucune application provenant de ton code ne s'exécute. ça ne marchera pas si ton programme continue de tourner (encore des calculs en cours, d'autres fenêtres...) après la fermeture de ta JDialog.

Citation:

Sachant que je pourrait très bien faire MonTimer.stop(), mais je ne suis pas sur que l'objet soit bien effacé de la mémoire, à moins que le garbage collector voit quand un objet n'est plus actif et le vire automatiquement?

Si tu fais monTimer.stop(), alors il stoppera l'écoute des évènements qui jusqu'alors déclanchaient ses actions. Mais ton Timer ne sera pas mort pour autant, tu peux toujours faire la méthode restart() pour t'en convaincre. De plus, si ton Timer arrête d'écouter les évènements, il continuera quoiqu'il arrive d'exécuter sa méthode actionPerformed().
Par contre, si plus rien ne pointe sur ton objet, le GarbageCollector s'en chargera :-)

A mon avis, le plus simple est de faire un truc du genre:
Code:

1 2 3 4 5 6 7 8 9 10 11 public void killTimer() { killTimer=true; } public void actionPerformed(ActionEvent evt) { if(getKillTimer()) { return; } else { //...Perform a task... } }
Lorque ton Timer a finit d'exécuter la méthode run(), tu peux alors le finaliser.

J'espère t'avoir aidé ;-)

Cordialement

11/09/2008, 09h27
donnadieujulien

Comment

Comment faire pour le finaliser?

C'est la que je ne sais pas faire... pas l'habitude avec java.

"plus rien ne pointe sur le timer" ca signifie timer = null; ou c'est plus compliqué?
11/09/2008, 10h10
Balbuzard

Une condition suffisante pour que plus rien ne pointe sur ton objet est de vérifier que plus aucune méthode ne l'appelle.

En même temps, si tu mets timer=null, et que tu tentes de l'appeler, il se peut qu'il y ait une exception de type : nullPointerException.

Enfin, il y a une méthode pour finaliser ton Timer : monTimer.finalize(), mais elle est appleée normalement par le Garbage-Collector et je ne suis pas sûr que tu puisse l'utiliser toi-même.

Si tu es sûr que plus rien ne fais appel à ton Timer, tu peux exécuter l'instruction System.gc(); qui fera appel au garbage collector.

Le problème du Timer, c'est qu'il peut avoir à faire des tâches périodiques, et du coup, fonctionner indéfiniment...

Bon courage :-)
11/09/2008, 10h52
donnadieujulien

Ouais

Une fois que j'ai fais le this.dispose(); sur ma fenetre, après timer.stop();

je n'utilise plus le timer, donc logiquement, il doit être détruit.

Donc en fait, il n'y a rien a faire si j'ai bien compris, à part d'être sur que je ne l'utilise plus.

J'en usis sur car comme je fais timer = null; et que j'ai pas d'exceptions, ca veut dire plus utilisé.