Discussion :

[pierrelm]

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 :

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 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.

[fumiste972]

Slt,

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

Code :

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

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.

[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 ...

[NeptuS]

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

[le y@m's]

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 .

[pierrelm]

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

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
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 "";
        }

[in]

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

C'était une bonne idée je pense

[pierrelm]

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

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 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 "";
        }
    }

[sinok]

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

Code :

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

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());

[adiGuba]

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 :

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

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 :

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

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++

[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 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)

[adiGuba]

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...

Je crois qu'on peut définitivement me ranger au rang 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 !!!


a++

[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.