Discussion :

[seroa]

Bonjour à tous, j'utilise une méthode en java qui permet de m'extraire du code html qui pour un planning.

Si mon planning n'a pas beaucoup de ligne tout va bien mais à partir d'une dizaine de ligne j'ai un message disant que mon agent utilise beaucoup de mémoire.

Je pense que cela vient de la :

Code :

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

1
2
3
4
5
6
 
for (int i = 0; i < persons.size(); i++) {
				PlanningPerson pP = (PlanningPerson) persons.elementAt(i);
				sb.append(pP.lineHTML());
 
			}

persons est un vecteur qui contient des objets PlanningPerson
Je parcours ce vecteur et charge à chaque fois ses objets, je pense que c'est cela qui prend beaucoup de mémoire.
Il faudrait que je puisse libérer pP à chaque fois.

Comment je pourrais faire svp ?

Merci.

[CheryBen]

Bonjour, je te conseille de jeter un oeil à cette page de la FAQ : Pourquoi ne faut-il pas employer la classe Vector ?

Ormis cela, il est difficile de voir d'où vient le problème sans connaitre l'exception ni avoir plus de code.

[FAMSK]

Bonjour ,

Je vais te donner une proposition , mais franchement je ne suis pas sur que c'est ça la raison ,

Au lieu de ça : for (int i = 0; i < persons.size(); i++) {

Essaye de faire
for(iterator it = persons.iterator ; it.hasNext() ; ) {


PlanningPerson pP = (PlanningPerson) it.next();;
sb.append(pP.lineHTML());

}

[seroa]

Merci pour vos réponses.

J'ai remplacé les Vector par des ArrayList, mais je n'ai pas trop gagné en perf, je reste à 13% (Agent memory usage (13) percentage of total)

J'utilise également l'iterator.

En fait je n'ai pas d'exception, c'est juste un avertissement du serveur ou est exécuté l'agent java.

Je peux peut être vous envoyer les deux classes si vous le voulez.

Merci.

[FAMSK]

Rebonjour ,

Oui tu peux envoyer les classes si tu veux , pour qu'on puisse savoir la cause du ce problème.

[Marco46]

Commence par éviter les déclarations de références dans une boucle.

[herve91]

Commence par éviter les déclarations de références dans une boucle.

Et pourquoi donc ?

[herve91]

Code :

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

1
2
3
4
for (int i = 0; i < persons.size(); i++) {
				PlanningPerson pP = (PlanningPerson) persons.elementAt(i);
				sb.append(pP.lineHTML());
							}

Il faudrait que je puisse libérer pP à chaque fois.
Comment je pourrais faire svp ?

La variable pP est modifiée à chaque tour de boucle, tu n'as pas à libérer quoique ce soit, ce n'est qu'une référence vers un des objets contenus dans le vecteur persons. Ce qui peut être gros, c'est le code HTML renvoyée par lineHTML() et/ou le contenu du StringBuilder résultat de la concaténation des codes HTML

[Marco46]

La variable pP est modifiée à chaque tour de boucle, tu n'as pas à libérer quoique ce soit, ce n'est qu'une référence vers un des objets contenus dans le vecteur persons. Ce qui peut être gros, c'est le code HTML renvoyée par lineHTML() et/ou le contenu du StringBuilder résultat de la concaténation des codes HTML

Effectivement j'ai dit une connerie. Autant pour moi !

[tchize_]

je gagerais, à vue de nez, que ton problème de perfs (qu'on a pas identifié d'ailleurs, tu parle de 13% , mais 13% de quoi??!), si il existe, viens de PlanningPerson.lineHTML(). La boucle en elle-même n'a aucun soucis (sauf si chaque ligne HTML fait 10M, mais j'en doute)

[seroa]

Bonjour à tous et merci pour vos réponses.

J'ai toujours ce problème de mémoire. En fait c'est la console du serveur domino qui m'envoie ce message lorsque mon agent java est lancé.

Je pense aussi que le problème vient de

Code :

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

sb.append(pP.lineHTML());

, c'est assez long ce que renvoi pP.lineHTML().

Mais pourquoi il affiche un message qu'à partir d'un certain nombre de ligne ?

J'ai le message seulement après une modification, lorsque je relance le planning et que celui-ci n'a pas été modifié, il n'affiche pas le message, comme s'il l'avait gardé en cache.

Merci.

[tchize_]

on a pas le code de ton lineHTML, donc impossible de te dire pourquoi il prend de l'espace en mémoire!