Discussion :

[donnadieujulien]

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

[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 ?

[donnadieujulien]

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
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

[donnadieujulien]

J'avais zappé une partie :

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

[Balbuzard]

Salut,

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.

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 :

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

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

[donnadieujulien]

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é?