Discussion :

[Carrel]

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

[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 :

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

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.

[Claythest]

Mettre à null les objets dont tu ne te sers plus mais dont les variables te sont toujours visibles t'aidera certainement

[Carrel]

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

Avant

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

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.

[thibaut]

Fais gaffe avec les

Code :

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

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();

[®om]

Code :

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

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

Le pol=null est totalement inutile...

[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.