[debutant]Optimisation de code

Bonjour,

je suis en train de développer une petite IHM en java. J'ai instancié beaucoup (trop ?) d'objets et du coup j'ai eu l'erreur suivante :

Citation:

---

java.lang.OutOfMemoryError: Java heap space

---

J'ai donc rajouté l'argument -Xmx512M dans les arguments sous eclipse car 256M ne suffisait pas...

Mon appli ne devrait pas consommer autant de ressources pour ce qu'elle produit comme résultat :aie:

J'aimerais donc connaître votre avis sur des éventuels optimisation de code. Je joins ma classe au post car je n'ai pas assez de place pour mettre tout mon code.

Merci par avance.

PS : soyez indulgent avec moi je débute :mrgreen:

Pièce jointe 25858

Dans ton cas, je pense qu'il ne s'agit pas d'optimiser mais de verifier le code que tu as écris pour voir s'il n'y a pas une coquille : une IHM ne recquiert en general pas beacoup d'espace en memoire. Ton appli integre elle un algorithme ? Il y a de fortes chances que la memoire occupée le soit par la partie métier et non par la partie presentation de ton appli.
ps: j'avais pas vu que t'avais joint les sources, j'y jette un oeil

Merci pour ta réponse.

En fait, j'ai recommencé à coder mon appli car je voulais stocker tous mes objets dans un tableau (Vector). Avant, aucun souci de mémoire maintenant si et je pense que je m'y prend mal pour stocker mes objets dans mon vecteur.

Salut,
- d'une part il faut que tu dissocies la partie graphique de la partie données. Tu n'affiches pas tous les composants Swing en même temps ? Dans ce cas tu peux créer une classe AllyView qui prend une référence sur une instance d'Ally, récupère les valeurs et modifie tous les composants graphiques, mais les composants n'existent qu'en un seul exemplaire (là je crois que t'en crées au moins 50000 c'est normal que ça rame).

- d'autre part, pourquoi chaque objet Ally contient autant d'autres objets ? Ca donne un côté récursif au modèle qui me semble inadapté. Tu as vu que l'objet objectConstructionT81 par exemple va contenir lui-même 12 planètes, 40 constructions, etc, il me semble que c'est pas le but recherché

Plus d'infos sur le but exacte de la classe aurait été apprécié ^^
A+

Citation:

---

Envoyé par Djakisback

Salut,
Tu as vu que l'objet objectConstructionT81 par exemple va contenir lui-même 12 planètes, 40 constructions, etc, il me semble que c'est pas le but recherché

---

peux-tu m'expliquer comment tu es arrivé à cette conclusion stp car c'est vrai que je n'ai pas l'impression de maîtriser la portée de tous mes objets...

Merci

Bin même si t'appelles pas instanciationObjets() à chaque fois (ah si dans le constructeur), tu déclares dans ta classe Ally une centaine d'autres Ally qui chacun déclare une centaine d'autres Ally.

Code:

---

objetPlaneteL1 = new Ally(planeteL1, 35, 54, 100, 10);

---

Appelle bien juste le constructeur

Code:

---

public Ally(JLabel jlabel, int coorX, int coorY, int width, int height)

---

et pas toute la classe, non ?

Y'a pas à chercher plus loin :

Un objet ally, à sa construction, en crée 300 autres. Or, ces 300, à leur construction, ils en créent au total 300 * 300 = 90.000, et ces 90.000, à leur construction en créent 90.000x300 = 27.000.000 et paf 27 millions x4 bytes = 100 Mo en mémoire. Et là, on n'a pas encore fait 27E6x300 ni la suite :P

Edit : tu appelles le constructeur, donc une nouvelle instance de la classe. Et chaque nouvelle instance de la classe en appelle une autre. Alors bon... tu arrives vite à saturer ta mémoire ;)

Autre edit : Utilise les collections, elles sont là pour t'aider, tu verras ;)

D'ailleurs je me demande comment ça pouvait fonctionner même avec 512M car c'est récursif.

Quand tu fais :

Code:

---

objetPlaneteL1 = new Ally(planeteL1, 35, 54, 100, 10);

---

Tous les membres sotn déclarés (donc tous les Ally), et tous les composants Swing sont déclarés et instanciés car tu fais un new directement dans la déclaration :

Code:

---

JLabel planeteL1 = new JLabel("PM");

---

Si tu nous dis ce que tu veux faire, on peut peut-être te proposer un modèle de classes plus adapté :).

ohoh merci pour l'explication qui pourrait expliquer beaucoup de choses :D

Comment faire alors pour éviter d'instancier tous ces objets ?

Tu découpes logiquement ton application en :

classe Application,
classe Planete,
classe Technologie,
classe Flotte,
...

Tu veux dire comme cela :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

class Application{   }   class Planete{   }   class Technologie{   }   class Flotte{   }

---

?

Mais qu'est ce que ça va changer au fait que les objets seront moins instanciés ?


EDIT : faudrait-il que je mette tous mes objets dans une collection et appeler le construteur 1 fois avec cette collection ?

Oui, puis tu te poses les questions suivantes :

Quelles sont les actions que je peux faire sur Application ?
Quelles sont les actions que je peux faire sur Planete ?
...

et tu crées les méthodes correspondantes dans les classes adéquates.