Fuite de mémoire avec GUI Swing

**pouvoir triforce** · 02/06/2011, 18h56

Bonjour,

Je viens poster mon problème ici (pensant que c'est l'endroit le plus adapté du forum).

Etant en train de réaliser une interface graphique avec Swing, j'ai quelques soucis avec l'utilisation mémoire de mon programme Java qui augmente tout au long de l'utilisation de l'application.

J'ai remarqué que l'augmentation est constante du moment que le curseur de la souris est positionné sur mon application.
Utilisant des MouseListener, je me demande si cela ne viendrait pas de là.
J'ai lu sur un autre forum qu'il est possible d'avoir des fuites lorsque l'on utilise une classe spécifique pour créer les MouseListener, ce qui n'est pas mon cas.

Auparavant, j'ai également réalisé un petit programme avec également des MouseListener et j'avais le même problème...

Avez-vous des idées ?

Je vous remercie d'avance car je commence à perdre pied...

**lequebecois79** · 02/06/2011, 19h17

Envoyé par pouvoir triforce

Bonjour,

Je viens poster mon problème ici (pensant que c'est l'endroit le plus adapté du forum).

Etant en train de réaliser une interface graphique avec Swing, j'ai quelques soucis avec l'utilisation mémoire de mon programme Java qui augmente tout au long de l'utilisation de l'application.

J'ai remarqué que l'augmentation est constante du moment que le curseur de la souris est positionné sur mon application.
Utilisant des MouseListener, je me demande si cela ne viendrait pas de là.
J'ai lu sur un autre forum qu'il est possible d'avoir des fuites lorsque l'on utilise une classe spécifique pour créer les MouseListener, ce qui n'est pas mon cas.

Auparavant, j'ai également réalisé un petit programme avec également des MouseListener et j'avais le même problème...

Avez-vous des idées ?

Je vous remercie d'avance car je commence à perdre pied...

poste ton code, car c'est pas vraiment normale d'avoir des fuites

**pouvoir triforce** · 03/06/2011, 00h43

Il y a pas mal de classes dans mon code... Je vais poster seulement celle concernant l'interface graphique car c'est vraiment la seule sur laquelle j'ai un doute.
Voici le code de ma classe servant de Frame principale :

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
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
581
582
583
584
585
586
587
588
589
590
591
592
593
594
595
596
597
598
599
600
601
602
603
604
/*
 * MainFrame.java                    11 mai 2011
 * (c) Perso
 */
package systems.management.gui.elements;
 
import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Component;
import java.awt.Container;
import java.awt.Dimension;
import java.awt.FlowLayout;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.KeyEvent;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
 
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
 
import systems.management.gui.areas.AircraftMenuList;
 
/**
 * Classe de test des zones de l'interface graphique
 * @author Perso
 * @version 1.0
 */
public class MainFrame extends JFrame {
 
    /** Numéro serial */
    private static final long serialVersionUID = 1L;
 
    /** Variables des boutons */
    // Boutons de l'écran initial
    JButton loadButton = null;
    JButton searchButton = null;
    JButton compareButton = null;
    JButton exportButton = null;
    JButton newButton = null;
    JButton closeButton = null;
    // Boutons des écrans de consultation d'éléments
    JButton consultButton = null;
    JButton deleteButton = null;
    JButton createButton = null;
    JButton backButton = null;
    // Boutons d'ajout et de retrait d'élément
    JButton addButton = null;
    JButton removeButton = null;
 
    /** Information supérieure de la fenêtre */
    JLabel theTopInformation = null;
 
    /** Information inférieure de la fenêtre */
    JLabel theBottomIndication = null;
 
    /** Liste des fichiers avions */
    AircraftMenuList theAircraftList = null;
 
    // Création de la frame principale
    public MainFrame() {
	super("SIManagement v1.0");
	setDefaultCloseOperation(DISPOSE_ON_CLOSE);
	setPreferredSize(new Dimension(640, 400));
 
	// Paramétrage des indications
	theTopInformation = new JLabel();
	theBottomIndication = new JLabel();
	setTheTopInformation(
		"Select an aircraft file to work with:");
	reinitializeTheBottomIndication();
	theBottomIndication.setForeground(Color.gray);
 
	// Création de la liste
	theAircraftList = new AircraftMenuList();
 
	// Création des boutons initiaux
	createLoadBtn();
	createSearchBtn();
	createCompareBtn();
	createExportBtn();
	createCreationBtn();
	createCloseBtn();
 
	// Création des panneaux intermédiaires
	JPanel topPanel = new JPanel(new FlowLayout());
	JPanel bottomPanel = new JPanel(new FlowLayout());
	JPanel centerPanel = new JPanel(new FlowLayout());
	JPanel btnPanel = new JPanel(new GridLayout(6, 1));
	JScrollPane theScrollList = new JScrollPane();
	theScrollList.setViewportView(theAircraftList);
 
	// Récupération du panneau
	Container contentPane = getContentPane();
	contentPane.setLayout(new BorderLayout());
	// Ajout des boutons au panneau
	btnPanel.add(loadButton);
	btnPanel.add(searchButton);
	btnPanel.add(compareButton);
	btnPanel.add(exportButton);
	btnPanel.add(newButton);
	btnPanel.add(closeButton);
	// Ajout des indications à leur panneau
	topPanel.add(theTopInformation);
	bottomPanel.add(theBottomIndication);
	// Ajout du contenu principal de la fenêtre à son panneau
	centerPanel.add(theScrollList);
	//centerPanel.add(theAircraftList);
	centerPanel.add(btnPanel);
	// Ajout au panneau pricipal
	contentPane.add(topPanel, BorderLayout.NORTH);
	contentPane.add(bottomPanel, BorderLayout.SOUTH);
	contentPane.add(centerPanel, BorderLayout.CENTER);
	// Alignement des boutons
	loadButton.setAlignmentX(Component.RIGHT_ALIGNMENT);
	searchButton.setAlignmentX(Component.RIGHT_ALIGNMENT);
	compareButton.setAlignmentX(Component.RIGHT_ALIGNMENT);
	exportButton.setAlignmentX(Component.RIGHT_ALIGNMENT);
	newButton.setAlignmentX(Component.RIGHT_ALIGNMENT);
	closeButton.setAlignmentX(Component.RIGHT_ALIGNMENT);
 
	// Affichage de la fenêtre paramétrée
	pack();
	setResizable(false);
	setVisible(true);
 
    }
 
    /**
     * Méthode de création du bouton chargement
     */
    public void createLoadBtn() {
	loadButton = new JButton("Load");
	loadButton.setMnemonic(KeyEvent.VK_L);
	loadButton.addMouseListener(new MouseListener() {
 
	    public void mouseReleased(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mousePressed(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mouseExited(MouseEvent e) {
		reinitializeTheBottomIndication();
 
	    }
 
	    public void mouseEntered(MouseEvent e) {
		setTheBottomIndication("Load a selected aircraft");
	    }
 
	    public void mouseClicked(MouseEvent e) {
		// TODO
 
	    }
	});
    }
 
    /**
     * Méthode de création du bouton de recherche
     */
    public void createSearchBtn() {
	searchButton = new JButton("Search");
	searchButton.setMnemonic(KeyEvent.VK_S);
	searchButton.addMouseListener(new MouseListener() {
 
	    public void mouseReleased(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mousePressed(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mouseExited(MouseEvent e) {
		reinitializeTheBottomIndication();
 
	    }
 
	    public void mouseEntered(MouseEvent e) {
		setTheBottomIndication("Search element concerning " +
			"the selected aircraft");
 
	    }
 
	    public void mouseClicked(MouseEvent e) {
		// TODO
 
	    }
	});
    }
 
    /**
     * Méthode de création du bouton de comparaison
     */
    public void createCompareBtn() {
	compareButton = new JButton("Compare");
	compareButton.setMnemonic(KeyEvent.VK_M);
	compareButton.addMouseListener(new MouseListener() {
 
	    public void mouseReleased(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mousePressed(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mouseExited(MouseEvent e) {
		reinitializeTheBottomIndication();
 
	    }
 
	    public void mouseEntered(MouseEvent e) {
		setTheBottomIndication("Compare an aircraft " +
			"configuration file with the selected " +
			"aircraft configuration");
 
	    }
 
	    public void mouseClicked(MouseEvent e) {
		// TODO
 
	    }
	});
    }
 
    /**
     * Méthode de création du bouton d'exportation
     */
    public void createExportBtn() {
	exportButton = new JButton("Export");
	exportButton.setMnemonic(KeyEvent.VK_X);
	exportButton.addMouseListener(new MouseListener() {
 
	    public void mouseReleased(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mousePressed(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mouseExited(MouseEvent e) {
		reinitializeTheBottomIndication();
 
	    }
 
	    public void mouseEntered(MouseEvent e) {
		setTheBottomIndication("Export the selected " +
			"aircraft configuration to an Excel file");
 
	    }
 
	    public void mouseClicked(MouseEvent e) {
		// TODO
 
	    }
	});
    }
 
    /**
     * Méthode de création du bouton de création d'avion
     */
    public void createCreationBtn() {
	newButton = new JButton("Create");
	newButton.setMnemonic(KeyEvent.VK_C);
	newButton.addMouseListener(new MouseListener() {
 
	    public void mouseReleased(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mousePressed(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mouseExited(MouseEvent e) {
		reinitializeTheBottomIndication();
 
	    }
 
	    public void mouseEntered(MouseEvent e) {
		setTheBottomIndication("Create a new aircraft");
 
	    }
 
	    public void mouseClicked(MouseEvent e) {
		// TODO
 
	    }
	});
    }
 
    /**
     * Méthode de création du bouton de fermeture de l'application
     */
    public void createCloseBtn() {
	closeButton = new JButton("Quit");
	closeButton.setMnemonic(KeyEvent.VK_Q);
	closeButton.addMouseListener(new MouseListener() {
 
	    public void mouseReleased(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mousePressed(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mouseExited(MouseEvent e) {
		reinitializeTheBottomIndication();
 
	    }
 
	    public void mouseEntered(MouseEvent e) {
		setTheBottomIndication("Close the application");
 
	    }
 
	    public void mouseClicked(MouseEvent e) {
		setVisible(false);
		dispose();
 
	    }
	});
    }
 
    /**
     * Méthode de création du bouton de consultation
     */
    public void createConsultBtn() {
	consultButton = new JButton("Consult");
	consultButton.setMnemonic(KeyEvent.VK_U);
	consultButton.addMouseListener(new MouseListener() {
 
	    public void mouseReleased(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mousePressed(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mouseExited(MouseEvent e) {
		reinitializeTheBottomIndication();
 
	    }
 
	    public void mouseEntered(MouseEvent e) {
		setTheBottomIndication(
			"Consult the selected element");
 
	    }
 
	    public void mouseClicked(MouseEvent e) {
		// TODO
 
	    }
	});
    }
 
    /**
     * Méthode de création du bouton de suppression
     */
    public void createDeleteBtn() {
	deleteButton = new JButton("Delete");
	deleteButton.setMnemonic(KeyEvent.VK_D);
	deleteButton.addMouseListener(new MouseListener() {
 
	    public void mouseReleased(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mousePressed(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mouseExited(MouseEvent e) {
		reinitializeTheBottomIndication();
 
	    }
 
	    public void mouseEntered(MouseEvent e) {
		setTheBottomIndication(
			"Delete the selected element");
 
	    }
 
	    public void mouseClicked(MouseEvent e) {
		// TODO
 
	    }
	});
    }
 
    /**
     * Méthode de création du bouton de création d'élément
     */
    public void createElemCreationBtn() {
	createButton = new JButton("New");
	createButton.setMnemonic(KeyEvent.VK_N);
	createButton.addMouseListener(new MouseListener() {
 
	    public void mouseReleased(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mousePressed(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mouseExited(MouseEvent e) {
		reinitializeTheBottomIndication();
 
	    }
 
	    public void mouseEntered(MouseEvent e) {
		setTheBottomIndication("Create an new element");
 
	    }
 
	    public void mouseClicked(MouseEvent e) {
		// TODO
 
	    }
	});
    }
 
    /**
     * Méthode de création du bouton de retour
     */
    public void createBackButton() {
	backButton = new JButton("Back");
	backButton.setMnemonic(KeyEvent.VK_B);
	backButton.addMouseListener(new MouseListener() {
 
	    public void mouseReleased(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mousePressed(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mouseExited(MouseEvent e) {
		reinitializeTheBottomIndication();
 
	    }
 
	    public void mouseEntered(MouseEvent e) {
		setTheBottomIndication(
			"Back to the previous screen");
 
	    }
 
	    public void mouseClicked(MouseEvent e) {
		// TODO
 
	    }
	});
    }
 
    /**
     * Méthode de création du bouton d'ajout
     */
    public void createAddButton() {
	addButton = new JButton("+");
	addButton.setMnemonic(KeyEvent.VK_PLUS);
	addButton.setActionCommand("add");
	addButton.setToolTipText("Add existing element to the list");
	addButton.addMouseListener(new MouseListener() {
 
	    public void mouseReleased(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mousePressed(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mouseExited(MouseEvent e) {
		reinitializeTheBottomIndication();
 
	    }
 
	    public void mouseEntered(MouseEvent e) {
		setTheBottomIndication(
			"Add existing element to the list");
 
	    }
 
	    public void mouseClicked(MouseEvent e) {
		// TODO
 
	    }
	});
    }
 
    /**
     * Méthode de création du bouton de retrait
     */
    public void createRemoveButton() {
	removeButton = new JButton("-");
	removeButton.setMnemonic(KeyEvent.VK_MINUS);
	removeButton.addMouseListener(new MouseListener() {
 
	    public void mouseReleased(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mousePressed(MouseEvent e) {
		// TODO Auto-generated method stub
 
	    }
 
	    public void mouseExited(MouseEvent e) {
		reinitializeTheBottomIndication();
 
	    }
 
	    public void mouseEntered(MouseEvent e) {
		setTheBottomIndication(
			"Remove selected element from the list");
 
	    }
 
	    public void mouseClicked(MouseEvent e) {
		// TODO
 
	    }
	});
    }
 
    /**
     * Méthode de réinitialisation de l'indication inférieure
     */
    public void reinitializeTheBottomIndication() {
	setTheBottomIndication("Select a file and click on an " +
			"action to process");
    }
 
    /**
     * Accesseur à l'information supérieure
     * @return l'information supérieure
     */
    public JLabel getTheTopInformation() {
        return theTopInformation;
    }
 
    /**
     * Modifieur de l'information supérieure
     * @param l'information supérieure à modifier
     */
    public void setTheTopInformation(String theTopInformation) {
        this.theTopInformation.setText(theTopInformation);
    }
 
    /**
     * Accesseur à l'information inférieure
     * @return l'information inférieure
     */
    public JLabel getTheBottomIndication() {
        return theBottomIndication;
    }
 
    /**
     * Modifieur de l'information inférieure
     * @param l'information inférieure à modifier
     */
    public void setTheBottomIndication(String theBottomIndication) {
        this.theBottomIndication.setText(theBottomIndication);
    }
 
}

Et voilà ma classe comportant ma méthode main() :

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
/*
 * TestUI.java                    10 mai 2011
 * (c) Perso
 */
package systems.management.engine.tests;
 
import systems.management.gui.elements.MainFrame;
 
/**
 * Classe de test de l'interface graphique
 * @author Perso
 * @version 1.0
 */
public class TestUI {
 
    /**
     * Méthode principale d'instanciation des classes d'interface
     * graphique
     * @param args Non utilisé
     */
    public static void main(String[] args) {
	//Schedule a job for the event-dispatching thread:
	//creating and showing this application's GUI.
	javax.swing.SwingUtilities.invokeLater(new Runnable() {
	    public void run() {
		MainFrame theMainFrame = new MainFrame();
	    }
	});
 
    }
 
}

Si vous avez des conseils, n'hésitez pas.

Je vous remercie d'avance...

**babycart** · 07/06/2011, 00h16

Bonsoir,

D'après le code fourni, je pense qu'il n'y a rien de bien grave. J'observe effectivement une augmentation de la mémoire assez importante lorsque l'on déplace la souris sur les boutons, mais tout revient dans l'ordre à intervalle régulier (lorsque le garbage collector fait son boulot). Donc pas de fuite mémoire à priori.
J'ai pu obtenir une amélioration sensible en remplaçant les valeurs des Strings (en dur dans ton code) par des constantes. Les allocations de String peuvent être assez couteuses, en particulier quant tu fait des concaténations en "dur", par exemple :

Code :

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

1
2
3
4
setTheBottomIndication("Compare an aircraft " +
                                    "configuration file with the selected " +
                                    "aircraft configuration");

A éviter !

**pouvoir triforce** · 07/06/2011, 11h46

Je te remercie pour ton aide.
J'ai testé ce que tu m'as conseillé, et j'avoue qu'il y a quelques améliorations.
Je pense que les MouseListener font également pas mal varier l'utilisation de la mémoire.

Je considère le sujet comme résolu.

**-gma-** · 08/06/2011, 16h31

Bonjour,

Je m'excuse d'intervenir sur un sujet marqué résolu mais je ne vois pas vraiment ce que ça change de créer des constantes pour des String (d'un point de vue des performances uniquement) qui sont déjà "en dur".

Chaque constante (donc String) est déjà stockée dans le pool de constantes de la classe et normalement javac est suffisamment grand pour savoir que, par exemple :

Code :

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

toto("a" + "b" + "c")

c'est strictement la même chose que

Code :

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

toto("abc")

et donc ne générer qu'une seule constante "abc" dans ce cas.

**babycart** · 17/06/2011, 00h26

-gma-, désolé de répondre si tardivement, je n'avais pas vu ton message.

En fait tu as raison, et si j'ai bien compris, c'est vrai que quand les strings peuvent être résolues au moment de la compilation, le compilateur va faire son boulot et ne générer qu'une seule string au final.

Je ne m'explique pas du coup pourquoi j'ai observé une amélioration en ce qui concerne l'utilisation de la mémoire. Mais peu importe, j'aurai appris quelque chose grâce à toi.
Merci !

Fuite de mémoire avec GUI Swing

AWT/Swing Java

Vue hybride

Discussions similaires

Partager

Partager