Lecture fichier texte - Exception (Result too large)

Version imprimable

Bonjour,

J'aimerai afficher le contenu d'un fichier texte dans une division (paragraphe) de ma page JSF.

Ci joint le code, mais j'ai l'exception suivante : (Result too large)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
public String readFile(String path) throws FileNotFoundException, IOException {
        String res="";
        FileInputStream fstream = new FileInputStream(path);
        DataInputStream in = new DataInputStream(fstream);
        BufferedReader br = new BufferedReader(new InputStreamReader(in));
        String strLine;
        while ((strLine = br.readLine()) != null) {
            res+=strLine+" ";
        }
        in.close();
        return res;
    }

J'ai utilisé encore ce code mais ça donne toujours la même exception.

Code:

1
2
3
4
5
6
7
8
9
10
    public String readFile(String path) throws FileNotFoundException, IOException {
        BufferedReader br = null;
        String res="";
        String sCurrentLine;
        br = new BufferedReader(new FileReader(path));
        if ((sCurrentLine = br.readLine()) != null) {
            res+=sCurrentLine+" ";
        }
        return res;
    }

19/05/2014, 16h31
thelvin

La méthode a l'air correcte, et le message d'erreur méconnu.

On peut voir l'erreur complète, au lieu de juste des bouts piochés au hasard ?
19/05/2014, 16h55
adiGuba

Salut,

Citation:

Envoyé par thelvin

La méthode a l'air correcte, et le message d'erreur méconnu.

Il faudrait surtout avoir le message d'erreur complet (et connaitre la taille du fichier en question).

Sinon la méthode est loin d'être correcte : l'utilisation de l'opérateur + sur des String dans une boucle est à proscrire ! Il faut utiliser StringBuilder à la place.
Sans oublier qu'il faudrait utiliser un try/finally ou mieux un try-with-ressource de Java 7 pour libérer proprement les ressources (surtout dans une application web !)

a++
19/05/2014, 17h07
thelvin

Je voulais dire qu'en principe rien de tout cela ne peut provoquer un message de ce genre. Bien que mauvaise pour les performances et la disponibilité du programme, elle devrait donner le résultat voulu sans erreur, pas provoquer une erreur avec ce message.

La correction de l'algorithme et la libération des ressources, je voyais ça après. Mais oui, c'est vrai qu'il faudrait pas que quelqu'un recopie ce code.