probleme liberation memoire avec JTextArea

Version imprimable

11/12/2006, 10h58
iutcien

probleme liberation memoire avec JTextArea

Bonjour,
J'utilise un JtextArea dans une fenetre independante pour afficher un texte d'environ 20000 lignes.Celui-ci est appelé grace à un bouton dans la fenetre principale de mon application.
Je me sert de dispose() pour fermer la fenetre d'apercu.
Lors du premier chargement du texte dans mon JtextArea, il n'y a pas de probleme. Ensuite je ferme l'apercu et je le relance et la erreur java heap space par manque de memoire...
Comment faire pour liberer l'espace memoire (System.gc() ne marche pas)?
11/12/2006, 11h06
iohack
Lors de la fermeture de ta fenêtre essai de passer à null les objets qui consomment de la mémoire :
Code:

1 2 3 4 5 6 7 public void windowClosing(WindowEvent e){ monTextArea.setText(""); monTextArea=null; ... this.dispose(); }
Si cela ne marche pas, laisse tomber ,le System.gc() qui ne garanti pas à 100% que le garbage collector fasse son travail ( tu risques d'avoir ce soucis de manière aléatoire ).
En dernier recours, il faut que tu augmentes la mémoire de la JVM en lui spécifiant par des arguments :
java -Xms memoireDeDepart -Xmx memoireMaximum MonProgramme
memoireDeDepart et memoireMaximum sont des valeurs numériques représentant des MegaOctets.
11/12/2006, 12h00
spekal

Je ne suis que partiellement d'accord : le gc garantit que la mémoire est libérée, si elle conserve des objets devenus innaccessibles, et si c'est nécessaire.

Donc, si ces deux conditions sont remplies, alors le gc libère la mémoire, même sans appel à System.gc(), qui est 99 fois sur 100 inutile.

Donc cela signifie que tu n'as pas vraiment libéré la mémoire suite à ton JTextArea, c'est à dire que quelque chose de ton JTextArea reste accessible. Le dispose libère les ressources système pour afficher une fenêtre, mais si tu as chargé l'ensemble du fichier visualisé dans un tableau de caractères, ce tableau n'est absolument pas libéré par un dispose.

Il faut que tu regardes si toutes les références à ton JTexteArea sont devenues invalides. Si tu n'y arrives pas, décrit nous un peu plus la situation de ton JTextArea pour voir.
11/12/2006, 16h31
iutcien

Mon JtextArea est contenu dans un JPanel, lui meme contenu dans un JTabbedPane, lui meme issu de la JFrame. Mon textArea est privé est aucune reference n'est transmise à une autre classe.
11/12/2006, 17h03
iutcien

Un truc que je comprend pas c'est que lorsque j'execute une autre action (je click sur un autre bouton) dans ma fenetre principale et que je relance l'aperçu alors là ça marche...
11/12/2006, 17h32
spekal

Il ne suffit pas qu'il soit privé, puisqu'il reste accessible depuis l'objet dont il est un attribut.

Normalement, si c'est un attribut d'instance, il suffit de remplacer l'ancienne référence par la nouvelle. Cependant, fait gaffe à tout ce qui est listener et classe interne, qui peuvent conserver des références vers l'ancien discrètement.
12/12/2006, 07h41
Baptiste Wicht

Citation:

Envoyé par spekal

Je ne suis que partiellement d'accord : le gc garantit que la mémoire est libérée, si elle conserve des objets devenus innaccessibles, et si c'est nécessaire.

Donc, si ces deux conditions sont remplies, alors le gc libère la mémoire, même sans appel à System.gc(), qui est 99 fois sur 100 inutile.

Donc cela signifie que tu n'as pas vraiment libéré la mémoire suite à ton JTextArea, c'est à dire que quelque chose de ton JTextArea reste accessible. Le dispose libère les ressources système pour afficher une fenêtre, mais si tu as chargé l'ensemble du fichier visualisé dans un tableau de caractères, ce tableau n'est absolument pas libéré par un dispose.

Tout ce qui dit iohack, c'est que l'appel à System.gc() est tout à fait inutile et peut même se révéler dangereuse. Il ne garantit en rien que le gc s'active pour venir faire un tour.
12/12/2006, 08h26
spekal

Citation:

Envoyé par wichtounet

Tout ce qui dit iohack, c'est que l'appel à System.gc() est tout à fait inutile et peut même se révéler dangereuse. Il ne garantit en rien que le gc s'active pour venir faire un tour.

Oui, c'est vrai, je me suis sans doute mal exprimé, car c'est surtout sur la suite que je suis réservé (décidemment :lol: ) : il conseille d'augmenter la mémoire disponible, or il me semble que ici il vaudrait mieux vérifier que les références deviennent bien invalides. La mise à null comme il le conseille ne suffit pas (mais c'est déjà un début, je l'admets).

Mais il faudrait que iutcien nous donne plus de détails. Attendons.

d'accord je vous met un peu de 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
 
//Visualiseur de texte.
		JTextArea jtext=new JTextArea();
//Declaration de l'objet permettant de lire un fichier.
		FileReader reader=null;
		try
		{
//Lecture du fichier.
			reader=new FileReader(fichier);
			jtext.read(reader, fichier);
 
 
		} catch (Exception e)
		{}
		finally
		{
			if(reader!=null)
				try
				{
//Fermeture de l'objet de lecture.
				reader.close();
				} catch (IOException e)
				{}
		}
 
 
		JScrollPane pan=new JScrollPane(jtext);

Voila là c le jpanel contenant le textArea. Je le rempli en me servant de read().

Je l'ajoute à mes onglet par un :

Code:

tabs.addTab("Viking 1er sens", prepareApercu(fichiers.get(0)));

pour l'utiliser:

Code:

1
2
3
4
5
6
7
8
9
 
//Filtre retournant la liste des fichiers à afficher. 
Vector<String> fic=mdlTab.getFiltre();
			if(fen!=null) 
				{
				fen.dispose();
				fen=null;
				}
			fen=new FenetreApercu(fic);

12/12/2006, 10h40
spekal

L'idéal serait que tu regardes toi même... Il faut vraiment suivre tous les risques de conservation de référence tout au long du code, et c'est un boulot. Avant de le faire, il faut juste comprendre le principe d'un objet accessible ou pas.

Par exemple une instruction à première vue aussi innofensive que JScrollPane pan=new JScrollPane(jtext); provoque la conservation de jtext dans le JScrollPane ; à partir de là il ne suffira pas, par la suite, de faire jtext = null, mais il faudra de plus s'assurer que le JScrollPan devient lui aussi innaccessible, ou lui donner une autre vue.

Et pareil pour le JScrollPane qui est peut être à son tour conservé quelque part ; c'est un boulot de vigilance sur l'ensemble de ton code.

Normalement un enchaînement simple (on ajoute, on retire) suffit à tout remettre d'équerre, mais il peut y avoir des combinaisons particulières.
12/12/2006, 12h03
iutcien

mais pour moi c'est assez simple, j'ai une fenetre totalement independante qui se gere toute seul, et qui est detruite à chaque fois. Mais voila, il n'y à pas d'espace memoire liberer et là je bloque!!!
12/12/2006, 12h03
iutcien

mais pour moi c'est assez simple, j'ai une fenetre totalement independante qui se gere toute seul, et qui est detruite à chaque fois. Mais voila, il n'y à pas d'espace memoire liberer et là je bloque!!!
12/12/2006, 13h06
spekal

Est-ce que dans ta fenêtre principale tu ne mets pas un écouteur de ta fenêtre avec le JTextArea ?