Discussion :

[HELKARROUMI]

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 :

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

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

[HELKARROUMI]

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

Code :

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

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

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

[adiGuba]

Salut,

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

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