Optimiser le stockage d'un fichier dans un String

Bonjour,

J'ai besoin de stocker un fichier texte dans un String, voici le code que j'utilise:

Code:

---

1 2 3 4 5

String monString = new String(); Scanner sc = new Scanner(new File("monFichier")); while(sc.hasNext()){ monString = monString + sc.nextLine() + "\n"; }

---

Mais le souci c'est que le fichier que je dois traiter fait plusieurs dizaines de milliers de lignes. La compilation L'execution du code est donc très longue!

Connaissez vous un moyen d'optimiser cela?

Merci d'avance

Citation:

---

Envoyé par tupac25

La compilation du code est donc très longue!

---

L’exécution, si c'est vraiment la compilation, le problème ne vient certainement pas de là.
Sinon peut-être en faisant un truc de ce style :

Code:

---

1 2 3 4 5 6 7 8 9

byte[] buffer = new byte[(int) new File(filePath).length()]; BufferedInputStream f = null; try { f = new BufferedInputStream(new FileInputStream(filePath)); f.read(buffer); } finally { if (f != null) try { f.close(); } catch (IOException ignored) { } } return new String(buffer);

---

Il y a aussi des choses pour faire ça dans apache commons je crois (mais je n'ai jamais testé).

Salut,


Ce n'est pas la compilation qui est longue... mais l'exécution !


De plus pourquoi lire un fichier ligne par ligne alors que tu veux le récupérer en entier.

Enfin ton erreur vient de la concaténation de chaine dans une boucle via l'opérateur +, qui implique la création d'une nouvelle chaine à chaque iteration ce qui implique la création de plusieurs objets temporaires...


:arrow: Il est préférable de passer par une lecture par bloc via un Reader, en spécifiant le charset de préférence.

Par exemple :

Code:

---

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

public static String read(File file) throws IOException { return read(file, Charset.defaultCharset()); }   public static String read(File file, Charset charset) throws IOException { InputStreamReader reader = new InputStreamReader(new FileInputStream(file), charset); try { StringBuilder sb = new StringBuilder((int)file.length()); char[] cbuf = new char[8192]; int len;   while ( (len=reader.read(cbuf)) >= 0 ) { sb.append(cbuf, 0, len); } return sb.toString(); } finally { reader.close(); } }

---

a++

@gailuris : Attention car rien ne garantie que le read() lise la totalité du buffer ! Il est impératif de vérifier son retour et de passer par une boucle.

a++

Citation:

---

Envoyé par adiGuba

@gailuris : Attention car rien ne garantie que le read() lise la totalité du buffer ! Il est impératif de vérifier son retour et de passer par une boucle.

---

Bien vu, merci (c'est un exemple trouvé sur le net et copié un peu trop vite)

J'ai corrigé mon erreur, en effet c'est bien de l'exécution que je parle.

Tout d'abord merci de m'avoir répondu. Mais j'ai oublié de préciser que je réalise des traitements sur certaines lignes, ce qui m'oblige à lire le fichier ligne par ligne.

Grace à votre aide j'ai trouvé la solution à mon problème! Merci!

Voici mon code:

Code:

---

1 2 3 4 5 6 7

String monString = new String(); StringBuilder sb = new StringBuilder(); Scanner sc = new Scanner(new File("monFichier")); while(sc.hasNext()){ sb.append(sc.nextLine() + "\n"); } monString = sb.toString();

---

Citation:

---

Envoyé par tupac25

Mais j'ai oublié de préciser que je réalise des traitements sur certaines lignes, ce qui m'oblige à lire le fichier ligne par ligne.

---

Il n'y a aucun traitements sur ces lignes dans ton code. C'est normal ?

Et il manque le try/finally pour le close...

a++

Je n'ai pas mis le traitement que j'effectue sur les lignes dans mon post pour éviter de surcharger le code et d'aller directement à l'essentiel.

@adiGuba: que veux tu dire par:

Citation:

---

Et il manque le try/finally pour le close...

---

Citation:

---

Envoyé par tupac25

Je n'ai pas mis le traitement que j'effectue sur les lignes dans mon post pour éviter de surcharger le code et d'aller directement à l'essentiel.

---

Disons que si on ne sait pas ce que tu veux faire exactement, on risque de te donner de mauvaises pistes...


Sinon pour le try/finally voir mon code ou la FAQ : Comment libérer proprement les ressources (ou comment utiliser proprement les bloc try/finally) ?


a++

Merci pour votre aide!