Discussion :

[mavina]

Bonjour,

J'ai une fonction dans laquelle je lis un fichier caractere par caractere pour ensuite l'afficher dans ma classe qui etend JEditorPane...
voici mon code :

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 void ouvrir(String s)
{
  try
  {
    FileReader f = new FileReader(new File(s));
    String s1="";
    int n=f.read();
    while(n!=-1) // tant qu'on n'est pas a la fin du stream
    {
        s1+=(char)n;
        n=f.read();
    }
    setText(s1);
   }
   catch(Exception e){}
   {
       System.out.println("erreur");
   }
}

Probleme : Le read() ne renvoit jamais -1 mais semble boucler le fichier, ce qui fait que mon programme n'affiche jamais mon ficher dans mon JEditorPane mais gele...
Bug ?

merci

mavina

[Pollux]

Tu as une erreur ou bien la boucle est infinie?

[scifire]

Je te propose plutot ca:

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
25
public void ouvrir(final String fName){
        String line;
        final StringBuffer buffer = new StringBuffer();
        BufferedReader reader = null;
        try {
            reader = new BufferedReader(new FileReader(fName));
            while ((line = reader.readLine()) != null) {
                buffer.append(line);
            }
 
            // setText(buffer.toString());
        } catch (FileNotFoundException e) {
            e.printStackTrace();
        } catch (IOException e) {
            e.printStackTrace();
        } finally {
            try {
                if (reader != null) {
                    reader.close();
                }
            } catch (Exception e) {
 
            }
        }
    }

Surtout il faut eviter

Code :

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

 s1+=(char)n;

parce que les chaines de caracteres sont non-mutables et donc chaque fois tu crees une nouvelle String

[mavina]

Ok, Ca marche a une exception pres : il ne prend pas en compte les retours chariot si ?
Est-il possible de prendre en compte les retours chariot dans ton cas ?

merci

mavina

[sinok]

Au pire tu peux les rajouter mùanuellement vu qu'un readLine correspond à la lecture d'une ligne, donc tu peux toujours ajouter un \n au StringBuffer apres y avoir rentré le contenu de la ligne....

[adiGuba]

Salut,

Ok, Ca marche a une exception pres : il ne prend pas en compte les retours chariot si ?

En effet la méthode readLine() du BufferedReader supprime les fins de ligne. Au pire tu peux les rajouter toi-même avec le code suivant :

Code java :

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

1
2
3
4
String endl = System.getProperty("line.separator");
 
// puis dans le while :
buffer.append(line).append(endl);

Est-il possible de prendre en compte les retours chariot dans ton cas ?

Tu peux également te passer du BufferedReader et effectuer toi même la lecture bufférisé :

Code java :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
    public void ouvrir(final String fName) throws IOException, FileNotFoundException {
        final StringBuffer buffer = new StringBuffer();
        Reader reader = new FileReader(fName);
        char[] cBuf = new char[1024];
        int len;
        try {
            while ((len = reader.read(cBuf)) > 0) {
                buffer.append(cBuf, 0, len);
            }
        } finally {
            reader.close();
        }
    }

a++

[mavina]

adiGuba mise a part ton stringbuffer a la place de mon string, c'est le meme code que mon premier post
donc on va avoir le meme probleme, as tu testé ton code ?

Merci, j'ai ajouté un .append(endl) ca marche sans aucun probleme !

[bulbo]

adiGuba mise a part ton stringbuffer a la place de mon string, c'est le meme code que mon premier post
donc on va avoir le meme probleme, as tu testé ton code ?

Merci, j'ai ajouté un .append(endl) ca marche sans aucun probleme !

AdiGuba utilises un buffer pour lire le fichier au lieu de le lire caractere par caractere comme dans ton code.
Il sera beaucoup plus rapide que toi pour lire le fichier.

J'ai teste ton code et il marche donc ce n'est pas le probleme du -1.

Je vois que tu fais un setText, es-tu sur de le faire depuis l'Event Dispatch Thread ?

Si ce n'est pas le cas utilise SwingUtilities.invokeAndWait() pour faire ton setText ..

Bulbo

[adiGuba]

adiGuba mise a part ton stringbuffer a la place de mon string, c'est le meme code que mon premier post

Justement ton problème venait de là ! En utilisant la concaténation de String avec + tu te retrouve à allouer des tonnes de mémoire pour rien... A chaque concatenation de String tu crées une nouvelle String, et donc tu alloues/desalloues de la mémoire... ce qui devient vite coûteux...

Par exemple pour lire un fichier de 200 Ko (ce qui n'est pas énorme en soit), avec ton code tu vas allouer plus de 20 Go de mémoire (et même le double puisque les char sont codé sur 2 octets...).

Pour preuve ce petit code :

Code java :

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

1
2
3
4
5
6
7
8
9
10
11
12
        long size = 200000;    // taille du fichier à lire (200 Ko)
        long arraySize = 0; // taille de notre String (à 0 au début)
        long totalMemory = 0; // Taille total de la mémoire allouer
 
        // Tant que notre tableau ne fait pas la même taille que le fichier :
        while (arraySize<size) {
            // on aggrandit le tableau de 1
            arraySize++;
            // ce qui équivaut à créer un nouveau tableau et copier le contenu
            totalMemory += arraySize;
        }
        System.out.println("Pour un fichier de " + size + ", " + totalMemory + " 'char' seront utilisés...");

Qui donne :

Pour un fichier de 200000, 20000100000 'char' seront utilisés...

C'est pour cela que ton application semble bloqué... le Garbage Collector travaille comme un malade...

donc on va avoir le meme probleme, as tu testé ton code ?

Oui et je peux t'assurer qu'il marche... Et je te retourne la question : as tu testé mon code ?
Et j'en profite pour te rappeller que tu avais oublier de fermer le flux


Enfin je n'ai pas dit que le code de scifire ne marchait pas (au contraire), mais il a toutefois un désavantage (mais assez minime) car il crée une nouvelle String à chaque ligne. Tu te retrouve donc a avoir alloué un StringBuffer pour la totalité du texte, plus une String par ligne (qui sera désalloué par le garbage collector). Ce qui implique une consommation de mémoire en plus (mais avec un fichier de 200Ko la différence ne se sent pas).

Mais selon le format de ton fichier et sa taille tu peux ressentir une différence sur la consommation de mémoire...


Enfin dernière remarque, la modification de ton composant graphique devrait se faire dans le Thread de l'EDT via SwingUtilities.invokeLater()

a++