Discussion :

[alain123]

Bonjour,
je veux récupèrer un fichier texte relativement lourd (7Mo) pour vérifier s'il contient certains termes.
Je fais le code suivant :

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
21
22
23
24
 
BufferedReader reader = null;
        String txt = "";
        try {
            reader = new BufferedReader(new FileReader("mon_fichier.txt"));
            String line;
            int i=0;
            while ((line = reader.readLine()) != null){
                System.out.println(i);
                txt+=line+"\n";
                i++;
            }
        }
        catch (Exception e) {
        }
        finally{
            try {
                if (reader==null){
                    reader.close();
                }
            } catch (IOException e) {}
        }
 
        System.out.println(txt);

Mais c'est très très lent.

Quelqu'un sait comment lire plus vite ?

Alain

[venegan]

Je pense que tu peux deja remplacer le

Code :

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

String txt = "";

par un StringBuffer.

[adiGuba]

Salut,

Quelqu'un sait comment lire plus vite ?

Il y a trois choses que tu pourrais améliorer :

1. Surement le plus important : ne JAMAIS utiliser l'opérateur + dans une boucle comme ceci ! C'est parfait pour tuer les performances et massacrer l'utilisation mémoire !!! utilises un StringBuffer (voir StringBuilder si Java > 5.0). [edit] Pour plus d'info, lire la FAQ : Comment concatener des chaînes de caractères ?
2. Utilises une lecture par buffer de taille fixe plutôt que par ligne. Quel intérêt y a-t-il à lire par ligne si c'est pour tout regrouper dans une même chaine de caractère sans traitement particulier sur les lignes ???
3. Ne surtout pas faire de bloc catch() vide si on ne veux pas passer des heures à chercher un bug mystérieux...

Sinon perso je préfère mettre le finally à l'intérieur d'un try/catch pour éviter le double traitement des exceptions, mais c'est juste un détail.

Exemple :

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
        try {
        	Reader reader = new FileReader("mon_fichier.txt");
        	try {
        		StringBuffer sb = new StringBuffer();
        		char[] cbuf = new char[8192];
        		int len;
 
        		while((len=reader.read(cbuf)) >= 0) {
        			sb.append(cbuf, 0, len);
        		}
        		txt = sb.toString();
        	} finally {
        		reader.close();
        	}
        }
        catch (Exception e) {
        	e.printStackTrace(); // NE SURTOUT PAS IGNORER L'EXCEPTION
        }
        System.out.println(txt);

a++

[alain123]

Merci pour votre aide.
C'est bien un stringbuffer qu'il fallait utiliser.