Problème mémoire objet

Version imprimable

04/12/2006, 13h02
Carrel

Problème mémoire objet

Bonjour,
J'ai un problème mémoire dans des traitements batchs sur un ensemble d'objet.

Mes batchs initialise un ensemble d'objet qui, les unes après les autres vont réaliser un traitement.

Pour ce faire, j'ai une collections d'objet avec pour chaque objet :
- Initialisation de l'objet
- Traitement de l'objet
- Mettre à null l'objet

Malgré cela j'ai des problèmes de mémoire. Après plusieurs traitements j'ai ma mémoire qui s'atture malgré que j'ai agrandi mes paramètres java (-Xmx512M ) et j'aimerai pas trop utiliser cette solution puisque mes batchs vont augmenter et cette solution est provisoire.

Pour votre information. L'initialistion des objets ce fait avec des références sur des objets persistant ou qu'ils peuvent se rappeler (un peu complexe mais le but cela fonctionne très bien)

Ma question est : comment je peux libérer proprement la mémoire d'un objet afin que les suivants prennent cet emplacement mémoire ?

Merci d'avance
04/12/2006, 14h07
woodwai

Pour qu'un object soit éligible au garbage collection, il faut que plus aucune référence pointe vers lui.

Concrétement, tu doit mettre à null toutes les références vers l'objet.

Tu peut aussi appeler manuellement le garbage collector, mais celà n'est pas assuré (en fait, tu lui demande de se lancer, mais c'est pas sûre qu'il le fasse).

Code:

System.gc();

Sinon, depuis JAVA 1.5, il y a un outils de management de la mémoire lancé en permanence dans la JVM standard de Sun, c'est juste une info, je ne me rapelle plus excatement comment le lancer mais il pourrait t'aider à trouver les possible memory leak de ton programmes.
04/12/2006, 15h18
Claythest

Mettre à null les objets dont tu ne te sers plus mais dont les variables te sont toujours visibles t'aidera certainement ;)
04/12/2006, 15h28
Carrel

Merci à tous, j'ai du réaliser un petit changement afin de permettre ce déchargement de mémoire :

Avant

Citation:

List pols = emp.getListPolice();
for (Iterator iP = pols.iterator(); iP.hasNext(); ) {
Police pol = (Police) iP.next();
this.getObjetDonnees().dbBegin();
trtUnePolice(pol, date, dateT, emp, cr);
this.getObjetDonnees().dbCommit();
pol.initObjet();
pol = null;
}

Après

Citation:

Vector pols = (Vector) emp.getListPolice();
for (int i = pols.size(); i > 0; i--) {
Police pol = (Police) pols.elementAt(i-1);
this.getObjetDonnees().dbBegin();
trtUnePolice(pol, date, dateT, emp, cr);
this.getObjetDonnees().dbCommit();
pol.initObjet();
pol = null;
// Afin de permettre la suppression de la mémoire
pols.remove(i-1);
System.gc();
}

Ce qui me libère la place.
04/12/2006, 17h03
thibaut

Fais gaffe avec les

Code:

System.gc();

ça pourrait ralentir de façon significative ton appli. Normalement si tu libères correctement les références vers tes objets, tu n'à pas besoin de faire System.gc();

Code:

1
2
3
4
5
6
for(...) {
    Police pol = (Police) iP.next();
    ...
    pol.initObjet();
    pol = null; // totalement inutile
}

Le pol=null est totalement inutile...

05/12/2006, 10h17
woodwai

Effectivement, pol=null est totalement inutile car pol est une variable locale à ta boucle. Donc à chaque début de tour de boucle, tu recréé pol donc tu libère la mémoire.

Par contre, enlever pol de ton Vector libère effectivement de la mémoire. Mais pourquoi avoir remplacé ta List par un Vector qui est beaucoup moins performant (car synchronisé)?

Sinon, l'appel à System.gc() doit effectivement être fait avec beaucoup de prudence car il est, potentiellement (car même si on appel le garbage collector on n'est jamais assuré qu'il va réellement passer) lourd en terme de performance. Si tout se passe bien, inutile de l'appeler, sinon, tu peut l'appeler mais évite de le faire trop fréquemment.
05/12/2006, 10h39
Carrel

Je vous remercie à tous pour vos réponses.

J'ai fait un test : (cela dure 25 minutes) et je n'ai plus de problème mémoire.

Deux points en resort :
- 1) System.gc(); Je l'ai laissé, parce que cela libère au fur et à mesure ma place mémoire et je n'ai pas remarqué une dépréciation notable du temps. Par contre, si je ne le fais pas, je prends plus de mémoire puisque c'est le système qui va le déclencher.

- 2) Le changement de Vector à List. C'est une méconnaissance de ma part. Je me suis pas penché sur la problématique de suppression une à une dans List et comme je connaissais avec Vector, je l'ai fait ou je connais le mieux. Je vais me renseigner et trouver la solution par moi-même pour le remettre sur List.

Merci encore à tous.
05/12/2006, 10h51
thibaut

Citation:

Envoyé par Carrel

Par contre, si je ne le fais pas, je prends plus de mémoire puisque c'est le système qui va le déclencher.

Ben parceque dans tout les cas c'est le système qui le déclenche.