Discussion :

[projetGTR]

Voila, je suis entrain de développer une application qui me parse un fichier Text d'environ 2Mo, ma fonction d'extraction du contenu du fichier texte est réalisé de cette facon :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
	public static StringBuffer readFile(String fileName){
		StringBuffer recoverTotalLine = new StringBuffer();
		try{
			File file = new File(fileName);
			InputStream ips=new FileInputStream(file); 
			InputStreamReader ipsr=new InputStreamReader(ips);
			BufferedReader br=new BufferedReader(ipsr);
			String ligne;
			while ((ligne=br.readLine())!=null){
				recoverTotalLine.append(ligne).toString();
			}
			br.close(); 
		}
		catch (Exception e){
			System.out.println(e.toString());
		}
 
		return recoverTotalLine;
	}

vous l'aurez compris, je concactène avec mon StringBuffer toutes les lignes trouvées du fichier.
Tout se passe bien pour les fichiers telques 50/150/350/500 ko mais quand je passe à une taille "classique" de l'ordre de 2mo, ca me dit :"Java heap space"; j'ai dans un premiere temps augmenté l'allocation de mémoire dans éclipse (-Xmx512M) mais rien n'y fait ... .
est ce que qqun aurait une petite idée pour allèger ce code??
il faut savoir aussi que l'extraction du contenu du fichier Text est quasiment instantanné sur des fichiers de 100/150/350 ko... je ne comprend pas :'(

[adiGuba]

Salut,


Par curiosité quel version de Java tu utilises ? Car tu ne devrait pas avoir une tel erreur sur ce code !

Par contre dans ta boucle tu fais des appels inutiles à toString() ce qui provoquer à chaque itération une copie de la chaine courante... Bref cela revient à allouer un très grand nombre d'objet et donc à surcharger le GC !

Je comprendrais que ce code soit vraiment très lent (création d'un très grand nombre d'objet temporaire) mais le "Java heap space" est surprenant je trouve...

Penses également à utiliser un try/finally pour fermer ton flux :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
	public static StringBuffer readFile(String fileName){
		StringBuffer recoverTotalLine = new StringBuffer();
		try{
			BufferedReader br = new BufferedReader(new FileReader(fileName));
			try {
				String ligne;
				while ((ligne=br.readLine())!=null){
					recoverTotalLine.append(ligne);
				}
			} finally {
				br.close();
			}
		}
		catch (Exception e){
			System.out.println(e.toString());
		}
		return recoverTotalLine;
	}

a++

[projetGTR]

aaah
Je crois que tout était lié au .toString() en trop parceque la rapidité extraction à considérablement augmenté
Un grand merci à toi

PS : désolé adiGuba j'ai vraiment du mal à trouver les bonnes sections