Erreur : OutOfMemoryError

Bonjour,

J'ai un souci dans mon application car lorsque je compile un certain nombre de fois j'ai un message du type java.lang.OutOfMemeryError.

Au lieu de réallouer de la mémoire j'aimerai savoir d'ou vien le problème.

Je pense que c'est à cause de la déclration d'un tableau.

Dans ma classe je déclare un tableau ( variable globale) du type
static String sTab[][][] = new String [50][500][50];

puis j'ai une fonction qui lit un fichier et qui appelle un fonction qui traite les lignes de ce fichier.

Dans cette fonction j'affecte de valeurs à mon tableau.

Ce que je ne comprend pas c'est que lorsue j'ai lut tout mon fichier je réinitialise mon tableau comme ceci :

sTab = new String [50][500][50];

Ce que je fait est-il correct ?

Ou alors mon problème viens-il d'ailleur ?

Pour réinitialiser ton tableau, tu devrais plutôt faire un:
"sTab = null;" => cela indique clairement au ramasse-miettes qu'il peut détruire l'objet pour récupérer sa mémoire.
Tu ne réinstancieras ton tableau que lorsque tu en auras besoin.

Ensuite, si tu ne fais que de la lecture de données dans ton fichiers, et que tu as plusieurs chaînes de caractères identiques (ce qui est certainement le cas vu le volume des données), tu devrais appeler la méthode "intern()" sur tes String avant de les stocker dans ton tableau. Tu gagnerais énormément de place en occupation mémoire (j'ai déjà résolu ainsi une problématique similaire)

en fait dans ma fonction de traitement de ligne je récupère des informations que je stock dans mon tableau puis lorsque cela marchera comme il faut (car pour le moment je rencontre pas mal de problèmes liés à mes indices de chaine (je travaille avec charAt et indexOf) et de tableau cla galère ^^.

Tu as vraiment besoin d'un tableau à 3 dimensions de cette taille?

Ton tableau fait 50 * 50 * 500 soit 1250000 de chaînes de caractères...
Donc:

1) il n'est pas étonnant que tu aies un OutOfMemory (le méthode "intern()" que je t'ai proposée ci-dessus devrait grandement t'aider: elle te permet d'utiliser le même objet pour les chaînes de caractères identiques, et donc d'éviter de dupliquer inutilement des données volumineuses en mémoire)

2) tu as certainement un problème de conception à la base de ton application...

Lorsque tu fais un sTab = new String[50][500][50], un nouvel espace mémoire est alloué. L'autre espace n'a plus de référence, mais le temps que le gc passe derrière, l'espace mémoire. Avant de ré-initialiser ton tableau, fais un sTab = null. Tu peux explicitement appeler le gc par Runtime.getRuntime().gc(), mais cette méthode n'est à utiliser que dans les cas critiques.

A toi de voir si tu as vraiment besoin de stocker autant de chaînes de caractères. Si tu sur-dimensionnes ton tableau pour être sûr d'avoir assez de place, cela alloue quand même de l'espace mémoire même si ton tableau est vide. Utilise plutôt des collections dynamiques comme des ArrayList.

ben en fait dans les fichier que je lis il y en a qui font + de 2000 lignes et dans mon tableau va contenir certaines parties de ces lignes.


Il est clair que j'aurais des chaines qui seront identiques mais ce que je veut c'est stocker dans mon tableau chacune des information des lignes qui correspondent à certaines critères donc en fait chaque nouvelle ligne qui correspond à ce que je recherche dans mon fichier va être affecté une nouvelle ligne de mon tableau.

Moi et les tableaux ca n'à jamais été mon fort et je galère ^^

Et dire qu'une fois que j'aurais tout bien rangé dans mon tableau va falloir l'afficher ... bon c'est moins chiant ca c'est clair :p.

Sinon je ne sais pas si avec une ArrayList ca pourrais marcher.

Bon alors je pense avoir trouvé la solution il s'agissait effectivement d'une erreur au niveau de mon programme, j'incrémentais un indice même si je n'avais rien trouvé donc ce qui fesait que je devais déclarer un tableau relativement gros.

pour le moment je l'ai réduit à [200][50][10] ce qui me parait plus raisonnable :).

Merci en tout cas de votre aide je vais voir si ca coince toujours au niveau de la mèmoire maintenant :p.

Utilises des listes, avec des tableaux à taille fixe tu finiras surement par outrepasser la limite...
Donc source de bugs

bon j'ai un peu de mal à passrer mon tableau en paramètre a une fonction vous pouriez m'indiquez comment je pourrais faire ?

Pour info j'ai un tableau de ce type :

String sTab[400][10][10]
avec comme indices
p,q,r ( tous des int)

j'aimerai le passer en paramètre à une fonction du genre

public static String Affiche (String sTab[][][])

C'est la que je ne suis pas trop sur de moi en ce qui conercne l 'argument de ma fonction.

Le problème c'est qu'apres pour mes boules je ne vois aps trop comment faire afficher els différentes valeurs de mon tableaux :(


Ahala moi et les tableaux :X


Merci

Code:

---

1 2 3 4 5 6 7 8 9 10 11

public static String affiche(String[][][] sTab){ String res = ""; for(int i = 0; i < sTab.length; i++){ for(int j = 0; j < sTab[i].length; j++){ for(int k = 0; k < sTab[i][j].length; k++){ res += sTab[i][j][k] + "\n"; } } return res; }

---

Par exemple.