Probleme d'instances de JLabel

**Crypt** · 19/04/2006, 16h13

J'ai un probleme avec le nombre d'instances de JLabel dans mon soft.
Rapidement : s'agissant d'un jeu j'ai un JPanel servant à afficher l'équipement du personnage. A chaque fois qu'il y a une modification dans cette équipement, la méthode updateEquipementList est utilisée pour modifier une JList (une JListBackdrop qui étend JList.) La méthode en question permet d'afficher à la fois une image et des informations telles que le nom de l'objet qui est dans l'équipement (ici j'ai enlevé les autres informations pour plus de lisibilité.)

J'ai fait une analyse du soft avec JProfiler et il apparait que les instances de JLabel augmentent régulièrement mais ne diminuent jamais. Fatalement au bout d'un moment ça pose un probleme (de mémoire et de CPU. Ici le CPU sature avant la mémoire, soit dit en passant.)

Il semblerait que les JPanel crées dans updateEquipementList ne soient jamais supprimés. Je suppose donc qu'ils se chevauchent à l'affichage.

J'ai essayé divers trucs mais rien n'y fait. Tout ce que j'ai réussit à faire c'est retarder la saturation avec ma méthode verifModif.

éventuellement pour plus de compréhension, il s'agit, dans ce screenshot, du JPanel tout à droite:
http://cryptmaster.free.fr/cryptrl/S...crl_0529_2.jpg
(ça n'utilisait pas JListBackdrop à l'époque donc pas d'image de fond)

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
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
import javax.swing.*;
import java.util.*;
import java.awt.*;
import java.awt.event.*;
 
 
 
class EquipementPanel extends JPanel implements FocusListener{
 
	private Vector ancienMatos=new Vector();////Pour comparaison vectors
 
	private Vector vector = new Vector();
	private JPanel jp = new JPanel();
	private JLabel jlab=new JLabel();
	private ImageIcon imIcon=new ImageIcon();
	private Enumeration en;
 
 
	private JScrollPane scroll;
	private JListBackdrop matosJList=new JListBackdrop();
	private Entite objet=new Entite();
	private PanneauJeu jeuPan;
	private Font police;
 
 
 
	public EquipementPanel(PanneauJeu pj){
 
		police=pj.getPolice();
 
		jeuPan=pj;
 
		matosJList.setCellRenderer(new CustomCellRenderer()); 
 
		setLayout(null);
 
		scroll = new JScrollPane(matosJList);
		scroll.setBounds(0,0,195,450);
 
		add(scroll);
 
		matosJList.addFocusListener(this);
 
	}
 
 
///////VERIFIE S'IL Y A UNE MODIFICATION DE L'EQUIPEMENT/////
	public void verifModif(Vector matosVct){
 
		if(!matosVct.isEmpty()){
 
			if(ancienMatos.isEmpty()){
				updateEquipementList(matosVct);
 				System.out.println("CHANGEMENT EQUIPEMENT");
 
 			}
 
			else{
				for(en = matosVct.elements();en.hasMoreElements();){
 
	 			  try{
	 			     objet=(Entite)en.nextElement();
 
 
	 			     if(!ancienMatos.elementAt(matosVct.indexOf(objet)).equals(objet)){
	 				updateEquipementList(matosVct);
	 				System.out.println("CHANGEMENT EQUIPEMENT");
	 				break;
	 			       }
 
				      }
	 			     catch(NoSuchElementException exc){};
	 			}
	 		}	
 
 		}
 
	}
 
 
 
///////////LA METHODE INCRIMINEE/////////////////////
	public void updateEquipementList(Vector matosVct){
 
//matosVct est le vector equipement du personnage.
 
 
		////////j'ai ajouté ça pour essayer de tout vider 
                 ////////mais ça ne change rien
		this.removeAll();
		scroll.removeAll();
		scroll = new JScrollPane(matosJList);
		scroll.setBounds(0,0,195,450);
		this.add(scroll);
		//////////////////////////////
 
 
		ancienMatos=matosVct;
		vector.clear();
 
 
		if(!matosVct.isEmpty()){
 
			for(en = matosVct.elements();en.hasMoreElements();){
 
 				try{
 					objet=(Entite) en.nextElement();
 
 					jp = new JPanel();
 
                                          //IMAGE DE L'OBJET
 					imIcon=new ImageIcon(objet.getTileStr());
 					jlab=new JLabel(imIcon);
 					jp.add(jlab);
 
    				        //NOM DE L'OBJET
    				        jlab=new JLabel(objet.getNom());
    				        jlab.setFont(police);
    				        jp.add(jlab);
 
 
					jp.setLayout(new FlowLayout(FlowLayout.LEFT,5,2));
					vector.addElement(jp);
 
 				}
 				catch(NoSuchElementException exc){};
 			}
 		}
 
		matosJList.setListData(vector);
 
		System.out.println("MISE A JOUR EQUIPEMENT");
		System.out.println("TAILLE Vector "+vector.size() );
 
	}
 
 
	public void focusGained(FocusEvent e){
		jeuPan.requestFocus();
	}
 
	public void focusLost(FocusEvent e){
 
	}	
 
}
 
/////////////////////////////////////////////////////////////////////////

La classe JListBackdrop dont l'unique intérêt ici est de me permettre de mettre une image de fond. Je doute que ça ait un rapport avec mon probleme mais en ne sait jamais... =

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
import javax.swing.*;
import java.awt.*;
 
 
class JListBackdrop extends JList {
 
	private ImageIcon fond;
 
 
	public JListBackdrop(){
		fond=new ImageIcon("./images/parcheminlisting.jpg");	
	}
 
	public void paintComponent (Graphics g){
 
		Dimension d = getSize();
		g.drawImage(fond.getImage(), 0, 0, d.width, d.height, null);
 
		setOpaque( false );
		super.paintComponent(g);	
 
	}
 
 
	public void update(Graphics g){	
		paintComponent(g);
	}
 
}

**PoPeio** · 19/04/2006, 16h24

en fait le soucis c'est que tu instancie a chaque fois une nouvelle imageIcon et un nouveau JLabel...

Ce qu'il faut que tu fasses c'est les instancier en global .. et puis les setter avec les bonnes valeurs... autant pour les JLabel que pour les ImagesIcon

OK ?

**Crypt** · 19/04/2006, 16h32

Envoyé par PoPeio

en fait le soucis c'est que tu instancie a chaque fois une nouvelle imageIcon et un nouveau JLabel...

Ce qu'il faut que tu fasses c'est les instancier en global .. et puis les setter avec les bonnes valeurs... autant pour les JLabel que pour les ImagesIcon

OK ?

oui, je vois, j'avais déjà testé un truc dans le genre

(de mémoire)

en global =
Icon imIcon=new ImageIcon();
JLabel jlab=new JLabel()

dans la méthode =
jlab.setIcon(imIcon);

ou

jlab.setText("chose");

Ca posait d'autres problemes (mais manifestement il va falloir que je recreuse de se côté.)

Cela dit je trouve tout de même très étrange que le GC ne supprime pas les JLabel., même avec
this.removeAll();
scroll.removeAll();

**Crypt** · 19/04/2006, 16h39

Cela dit je crois qu'il va de toute façon y avoir un probleme car il peut y avoir plusieurs objets dans l'équipement.

Si je n'instancie par un nouveau JLabel dans ma méthode (quand bien même je ferais les choses à l'économie en utilisant le constructeur (icon, string, alignement horizontal) pour mettre la description et l'image dans le même JLabel ) alors je vais me retrouver , à l'affichage, avec un nombre correct d'objets d'équipement mais ils seront visuellement tous les même (en l'occurence le dernier de la liste)

**PoPeio** · 19/04/2006, 17h14

OK tu as raison ...
mais le removeAll() libère tous les composants de ton container.. Il ne sont pas détruit.. Il faut attendre le passage du garbage collector

Cependant tu peut forcer le garbage collector à travailler !!! Je dois avoir un bout de code ;

**Crypt** · 19/04/2006, 17h21

Envoyé par PoPeio

OK tu as raison ...
mais le removeAll() libère tous les composants de ton container.. Il ne sont pas détruit.. Il faut attendre le passage du garbage collector

Cependant tu peut forcer le garbage collector à travailler !!! Je dois avoir un bout de code ;

Je m'étais fait un petit raccourcis pour lancer manuellement Runtime.getRuntime().gc();

Malheureusement il semblerait que le fait de libérer les composants du container ne suffise pas à les rendre candidats au GC car, après test dans JProfiler les symptômes restent les mêmes (les instances de JLabel ne baissent pas)

A vue de nez une solution serait que l'équipement soit affiché dans une JDialog sur demande du joueur et non pas en permanence dans la JFrame.
Avec le dispose() d'une JDialog je pense que ça serait radical (enfin j'espère quand même que les JLabel seraient cette fois réellement candidats au GC !!.) Mais je trouverai dommage de ne pas avoir l'équipement en permanence sous les yeux

////////////////////////////////
J'ai l'impression que je vais devoir totalement changer de méthode et utiliser une image (avec toolkit, buffer et tout ce que ça suppose.)
Ce que j'ignore par contre c'est s'il est facile de transformer un String en image (ou en tout cas de faire une image à partir d'un String.) J'en aurai besoin car il faut au moins que le nom de l'objet apparaisse.
L'autre probleme qui va se poser avec cette méthode = sera t'il possible de la placer dans un JScrollPane ?

**PoPeio** · 19/04/2006, 17h31

petite question as tu un nombre max d'équipements ??

Si oui .. et que ce nombre n'est pas dantesque instancie tout en variable globale et fait set
Ensuite tu fais un contrôle sur ton équipement pour savoir si tu l'ajoute ou non au panel ..

parce que si tu dois tout te dessiner ... y' aura du TAF

Bonne chance

**Crypt** · 19/04/2006, 17h42

Envoyé par PoPeio

petite question as tu un nombre max d'équipements ??

Si oui .. et que ce nombre n'est pas dantesque instancie tout en variable globale et fait set
Ensuite tu fais un contrôle sur ton équipement pour savoir si tu l'ajoute ou non au panel ..

parce que si tu dois tout te dessiner ... y' aura du TAF

Bonne chance

Le nombre d'objet pouvant être ramassé par le personnage n'est limité que par sa force. De toute façon la version finale (dans plusieurs mois / années ?) comportera un grand nombre d'objets utilisables (c'est un Roguelike...)

Du coup cette solution n'est malheureusement pas envisageable

Mais c'est pas grave

, je vais faire autrement = l'inventaire ne sera accessible qu'en JDialog et l'équipement (affiché en permanence) ne sera plus un listing de l'inventaire mais une image (buffer, toolkit, etc) représentant une silhouette avec les ImageIcon (uniquement) des armes et armures portées.

Je pense que ça sera plus propre.