Discussion :

[Iron Bull]

Salut

je suis en train de faire un programme qui contient un JTextPane et un un vecteur de JLabel. À chaque fois que le nombre de lignes dans le JTextPane change, j'ajoute un nouveau JLabel dans le vecteur qui correspond au numéro de la nouvelle ligne ou je supprime le JLabel qui correspond à la ligne supprimée.

Le problème est que, lorsque je supprime un JLabel, l'espace qui utilise n'est pas libérée et je ne comprends pas pourquoi puisque normalement, c'est le gc qui s'en occupe lorsqu'il n'y a plus de référence à l'objet.

Voici ma fonction setLinesNumber qui se charge de modifier les numéro de lignes.

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
36
37
38
39
40
41
42
private void setLinesNumber(int newLinesNumber){
	//Calcule la taille minimum du nombre pour combler le reste avec des zéros
	int newNumberLength = ("" + newLinesNumber).length(); 
	if(newNumberLength < 2){
		newNumberLength = 2;
	}
	int oldNumberLength = ("" + m_lblLinesNumberList.size()).length();
	if(oldNumberLength < 2){
		oldNumberLength = 2;
	}
 
	//Ajoute les zéro au label de l'ordre de grandeur précédante lors du changement d'ordre de grandeur
	if(newNumberLength != oldNumberLength){
		for(int i = 0; i < m_lblLinesNumberList.size(); i++){
			m_lblLinesNumberList.get(i).setText("0" + m_lblLinesNumberList.get(i).getText());
		}
	}
 
	//Ajoute le numéro de chaque nouvelle ligne
	if(newLinesNumber > m_lblLinesNumberList.size()){
		for(int i = m_lblLinesNumberList.size(); i < newLinesNumber; i++){
			String numberToAdd = "";
			for(int j = 0; j < newNumberLength - ("" + (i + 1)).length(); j++){
				numberToAdd += "0";
			}
			numberToAdd += i + 1;
 
			JLabel lblTemp = new JLabel(numberToAdd);
			lblTemp.setForeground(Color.WHITE);
			m_lblLinesNumberList.add(lblTemp);
			m_pnlLinesNumber.add(m_lblLinesNumberList.get(m_lblLinesNumberList.size() - 1));
		}
	}
	//Enlève le numéro de chaque ligne supprimée
	else if(newLinesNumber < m_lblLinesNumberList.size()){
		for(int i = m_lblLinesNumberList.size(); i > newLinesNumber; i--){
			m_pnlLinesNumber.remove(m_lblLinesNumberList.get(i - 1));
			m_pnlLinesNumber.updateUI();
			m_lblLinesNumberList.remove(i - 1);
		}
	}
}

[hydraland]

Salut,

Sur quoi te base tu pour dire que la mémoire n'est pas libéré?

A+
Hydraland

[adiGuba]

Salut,

je suis en train de faire un programme qui contient un JTextPane et un un vecteur de JLabel. À chaque fois que le nombre de lignes dans le JTextPane change, j'ajoute un nouveau JLabel dans le vecteur qui correspond au numéro de la nouvelle ligne ou je supprime le JLabel qui correspond à la ligne supprimée.

Quel est l'objectif de tout cela ?

Le problème est que, lorsque je supprime un JLabel, l'espace qui utilise n'est pas libérée et je ne comprends pas pourquoi puisque normalement, c'est le gc qui s'en occupe lorsqu'il n'y a plus de référence à l'objet.

Qu'est-ce qui te fait dire que les JLabels ne sont jamais libéré ?

a++

[tchize_]

n'oublie pas de retirer te JLable aussi de son conteneur graphique si tu ne veux plus le voir.

[Iron Bull]

Quel est l'objectif de tout cela ?

Vous connaissez notepad++? Je veux simplement afficher le numéro de chaque ligne devant celle-ci.

Qu'est-ce qui te fait dire que les JLabels ne sont jamais libéré ?

Dans le gestionnaire des tâches, lorsque je supprime les lignes, il n'y a aucune diminution de la mémoire utilisé par mon application. J'en déduis donc que la mémoire n'est pas libérée. Bien que se soit négligable pour quelques lignes, lorsqu'il sagit de plusieurs centaines de lignes, sa pose problème.

n'oublie pas de retirer te JLable aussi de son conteneur graphique si tu ne veux plus le voir.

Mon conteneur graphique est un JPanel et je retire en effet le JLabel du JPanel lorsque je supprime une ligne.

(Ce code est déjà écrit plus haut)

Code :

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

1
2
3
4
5
6
7
8
//Enlève le numéro de chaque ligne supprimée
	else if(newLinesNumber < m_lblLinesNumberList.size()){
		for(int i = m_lblLinesNumberList.size(); i > newLinesNumber; i--){
			m_pnlLinesNumber.remove(m_lblLinesNumberList.get(i - 1));
			m_pnlLinesNumber.updateUI();
			m_lblLinesNumberList.remove(i - 1);
		}
	}

Sinon, est-ce qu'il y aurait un meilleur moyen que d'utiliser un JLabel pour chaque ligne?
J'ai déjà essayé d'utiliser un JLabel unique, où chaque ligne est séparé par un <BR>, mais cela cause beaucoup de ralentissement dès qu'on dépasse 500 lignes (le texte du JLabel doit être réécrit à chaque fois).

[Invité]

Dans le gestionnaire des tâches, lorsque je supprime les lignes, il n'y a aucune diminution de la mémoire utilisé par mon application. J'en déduis donc que la mémoire n'est pas libérée. Bien que se soit négligable pour quelques lignes, lorsqu'il sagit de plusieurs centaines de lignes, sa pose problème.

La machine virtuelle réserve de la mémoire à l'avance mais rien ne te dit qu'elle est utilisée. Pour le savoir, utilise un outil tel que jvisualvm qui est incluet par défaut dans le JDK de Sun depuis la version 1.6_10. Là tu verras si tu as de réeles problèmes de mémoire.

A plus

[Iron Bull]

La machine virtuelle réserve de la mémoire à l'avance mais rien ne te dit qu'elle est utilisée. Pour le savoir, utilise un outil tel que jvisualvm qui est incluet par défaut dans le JDK de Sun depuis la version 1.6_10. Là tu verras si tu as de réeles problèmes de mémoire.

A plus

Merci, je vais regarder ça chez moi ce soir.

Donc si c'est le cas, lorsque je supprime les JLabel associés aux lignes supprimées, la jvm réserve de la mémoire. Mais lorsque je recrée les JLabel lors d'ajout de nouvelles lignes, pourquoi est-ce que la jvm n'utilise pas cette mémoire "réservé" au lieu de prendre encore plus de place en mémoire?
Comment faire pour libérer complètement la mémoire qu'il réserve puisqu'il ne semble pas s'en servir?

[tchize_]

C'est pas aussi simpel que ca. Le modèle de mémoire de la jvm fait qu'il n'y a pas de lien direct entre la mémoire système et la mémoire java. D'abord, quand tun objet n'est plus référencé, il est ramassable par le ramasse miettes, mais il ne va pas l'etre immédiatement. Cette analyse est assez couteuse et donc on limite autant que possible les nettoyages. C'est donc plus proche du gros nettoyage de printemps que du coup de balais vite fait. Par contre, un garantie il me semble de la jvm, c'est que le nettoyage va être fait en priorité avant d'augmenter la taille de la jvm. Si la mémoire continue d'augmenter, c'est que tu n'a pas supprimé toutes les références à ton JLabel (ou à d'autres objets, rien ne dit que le JLabel est en cause).

Si t'as un doute, crée tes JLabel comme çà:

Code :

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

1
2
3
4
5
new JLabel("texte"){
  protected void finalize() throws Throwable{
    System.out.println("nettoyage de "+getText());
  }
}

[Djobird]

Le GC ne passe pas dés qu'un objet n'est plus utilisé, mais selon d'autre alogrithm.

De plus, c'est pas forcément simple à voir sur le gestionnaire de tache, tu ne sais pas si la mémoire n'est pas alloué pour d'autre chose.
Si tu veux etre sur qu'ils sont bien pris par le gc, tu créé ton propre JLabel dans le quel tu met une variable static "compteur". Dans le constructeur de ton composant, tu incrémentes compteur. Tu overwrite ensuite la méthode finalize(), dans la quelle tu décrémentes compteur.

Tu regardes si ta variable compteur ne fait qu'augmenter au cours du temps, c'est que tes composants ne sont jamais détruits.