Convertir un fichier en String

Version imprimable

Bonjour,
je cherche un moyen rapide de convertir le contenu d'un fichier en String (dans le but de l'inclure dans un fichier XML). Pour l'instant je fais de cette manière :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
public static String affiche(String nomFichier) {
        String retour = "";
 
    	try {
            FileReader reader = new FileReader(nomFichier);
            int lu;
            char c;
            do {
<div style="margin-left:40px">                // on lit un caractère sous forme d'un entier:
                lu = reader.read(); 
                if (lu != -1) { // si on est pas à la fin du fichier
                   <div style="margin-left:40px"> c = (char) lu;
                    retour += c;</div>}</div>
            } while (lu != -1); // tant que l'on est pas à la fin du fichier
            reader.close();
            return retour;
        } catch (Exception e) {
        	return "";
        }
    }

mais cela met 30 secondes pour un fichier de 45ko. Si quelqu'un avait des pistes pour aller plus vite, ca m'aiderait beaucoup.
D'avance merci.

15/06/2006, 12h05
fumiste972
Slt,

pour commencer tu pourrais utiliser la classe BufferedReader pour lire ligne par ligne plutot que par caracteres.
Code:

1 2 3 BufferedReader buf = new BufferedReader(new FileReader(nomFichier)); String line = buf.readLine();
Maintenant si tu veux ke ca aille plus vite je ne pense pas qu'on puisse lire plus vite que ca. Les traitement de fichier restent en general assez longs.
15/06/2006, 12h08
in

je pense que pour concaténer ce que tu lis tu devrais utiliser un StringBuffer (en 1.4 en tout cas), ya un autre type d'objet en 1.5 mais j'ai oublié le nom exact (genre StringBuilder)

Apres tu le transforme en String si tu veux ...
15/06/2006, 13h27
NeptuS

tu peux tout aussi bien utiliser un Vector<String>, très pratique aussi.
15/06/2006, 13h36
le y@m's

Citation:

Envoyé par NeptuS

tu peux tout aussi bien utiliser un Vector<String>, très pratique aussi.

Pour la performance il vaut mieux éviter Vector qui est Thred-safe et plutôt utiliser une ArrayList ;).

StringBuffer

Merci pour vos réponses, avec le StringBuffer c'est quasi instantané !

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
public static String afficheRapide(String nomFichier) {
        StringBuffer retour = new StringBuffer();
 
    	try {
            FileReader reader = new FileReader(nomFichier);
            int lu;
            char c;
            do {
<div style="margin-left:40px">// on lit un caractère sous forme d'un entier:
                lu = reader.read(); 
                if (lu != -1) { // si on est pas à la fin du fichier
                    <div style="margin-left:40px">c = (char) lu;
                    retour.append(c);</div>}</div>                
            } while (lu != -1); // tant que l'on est pas à la fin du fichier
            reader.close();
            return retour.toString();
        } catch (FileNotFoundException e) {
            System.out.println("Impossible de lire le fichier "+nomFichier+" ! " +e);
            return "";
        } catch (IOException e) {
        	System.out.println("Erreur de lecture !" +e);
        	return "";
        }

15/06/2006, 14h27
in

Pourquoi tu ne lis pas ligne par ligne comme te le suggérais fumiste972 ?

C'était une bonne idée je pense

encore plus rapide

Je vous propose une autre solution qui semble être encore plus rapide :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 public static String afficheUltraRapide(String nomFichier) {
    	String retour;
    try{
    	File f = new File(nomFichier);
        byte[] buffer = new byte[(int)f.length()];
        DataInputStream in = new DataInputStream(new FileInputStream(f));
        in.readFully(buffer);
        in.close();
    	retour = new String(buffer);
            return retour;
        } catch (FileNotFoundException e) {
            System.out.println("Impossible de lire le fichier "+nomFichier+" ! " +e);
            return "";
        } catch (IOException e) {
        	System.out.println("Erreur de lecture !" +e);
        	return "";
        }
    }

Ou encore un petit peu plus rapide en utilisant les channels nio:

Code:

1
2
3
4
5
6
7
 
            File f = new File("chemin");
            FileChannel channel = new FileInputStream(f).getChannel();
            ByteBuffer b = ByteBuffer.allocate((int) channel.size());
            channel.read(b);
            channel.close();
            String s = new String(b.array());

Salut,

Juste une remarque : pensez à fermer proprement vos flux !!! Dans les codes que vous donnez le flux ne sera pas fermé en cas d'exception...

Il suffit simplement d'utiliser un try/finally pour obtenir un code plus sécurisé. Par exemple pour les deux derniers codes cela pourrait donner (si on se contente de renvoyer l'exception) :

Code:

1
2
3
4
5
6
7
8
9
10
11
    public static String afficheUltraRapide(String nomFichier) throws IOException {
        DataInputStream in = new DataInputStream(new FileInputStream(nomFichier));
        try{
            File f = new File(nomFichier);
            byte[] buffer = new byte[(int)f.length()];
            in.readFully(buffer);
            return new String(buffer);
        } finally {
            in.close();
        }
    }

Code:

1
2
3
4
5
6
7
8
9
10
    public static String load(File f) throws IOException {
        FileChannel channel = new FileInputStream(f).getChannel();
        try {
            ByteBuffer b = ByteBuffer.allocate((int) channel.size());
            channel.read(b);
            return new String(b.array());
        } finally {        
            channel.close();
        }
    }

a++

16/06/2006, 15h36
sinok

On est d'accord, juste que j'avais la flemme d'ouvrir une usine à gaz pour tout faire....

Je crois qu'on peut définitivement me ranger au rang :boulet: pourrave sur l'étagère des chiottes (sisi celle où l'on ne met que ce qui ne sert à rien et que l'on garde par acquis de conscience)
16/06/2006, 16h37
adiGuba

Citation:

Envoyé par sinok

On est d'accord, juste que j'avais la flemme d'ouvrir une usine à gaz pour tout faire....

Mais je préfère toujours préciser car il se trouve que peu de monde utilise les bloc try/finally... et ce n'est pas vraiment une usine à gaz...

Citation:

Envoyé par sinok

Je crois qu'on peut définitivement me ranger au rang :boulet: pourrave sur l'étagère des chiottes (sisi celle où l'on ne met que ce qui ne sert à rien et que l'on garde par acquis de conscience)

Meuuuh non !!! :calim2:

a++ ;)
16/06/2006, 16h58
sinok

L'usine à gaz c'était éclipse pour le coder.... Puis je fais la grève du codage dans ma boite aujourd'hui de toute façon.