Bonjour a tous
 
J'ai crée une classe Fichier dans la laquelle j'ai une methode lire() qui doit lire le fichier complet et me retourner une string.
 
Pour les fichiers court ca va bien, mais j'ai des fichiers jusqu'a 1Mo et la ca ca devient impossible tellement c'est lent !
 
voila le code de la methode lire :
 
public String lire (){
 String maString = "";
 
 try {
  FileInputStream fr = new FileInputStream(this);
         BufferedInputStream br = new BufferedInputStream( fr );
         
  while ( br.available() != 0 ) {
   maString += (char)(br.read()&0xFF);
  }
 
  br.close ();
 
         System.out.println ( "Le fichier lu contient " + maString.length() + " octets" );  
  } catch ( IOException e) {
   System.out.println (e.getMessage ());
   maString = "";
  }
   
  return maString;
 }
 
Je suppose que c'est lent car je lis les octets 1 par 1, et concatener la chaine au fur et a mesure doit devenir de plus en plus lent.
Mais comment faire autrement ?
La string retournée peut donc faire jusqu'a 1Mo et les fichiers peuvent etre binaires.
 
Bonne journée a tous !

Voilà, tu concatène avec une String normale, spabien. La première optimisation qu'on t'apprend à l'école, c'est d'utiliser un StringBuilder, et sa méthode append

Le pb vient probablement du fait que tu utilise une chaine
 
re donc !

Ben le pb c'est justement j'apprends le java par moi meme...
Donc j'ai pas appri le StringBuilder...
 
Le truc c'est que je dois imperativement utiliser une chaine en retour (qui peut effectivement etre grosse, jusqu'a 1Mo)
 
Y'a pas un moyen par exemple de lire tout d'un coup le fichier dans un tableau puis convertir le tableau en string ?
 
C'est dur le java !
Je vais jeter un oeil a StringBuilder pour voir

Bon ben c'est impeccable, voici mon nouveau code et maintenant le temps de lecture/creation chaine = environ 10s pour 512Ko. Donc c'est deja bien mieux !
 
Merci encore
 
   
 
 
 public String lire (){
  //String maString = "";
  StringBuffer maString = new StringBuffer("" );
 
  try {
   FileInputStream fr = new FileInputStream(this);
         BufferedInputStream br = new BufferedInputStream( fr );
         
         System.out.println ( "Lecture du fichier '" + nomCompletFichier + "'" );  
 
   while ( br.available() != 0 ) {
    char c = (char)(br.read()&0xFF);
    maString.append( c );
   }
 
   br.close ();
 
         System.out.println ( "Le fichier lu contient " + maString.length() + " octets" );  
  } catch ( IOException e) {
   System.out.println (e.getMessage ());
   return( "" );
  }
   
  return maString.toString();
 }

pourquoi tu lis char par char ? lis par bloc !

NB : si une exception survient, tu pers un descripteur de fichier (.close() pas appelée)

Oui j'y pensé a lire par bloc, mais je vois pas trop comment faire. COmme je suis vraiement debutant en java j'ai pas encore toutes les astuces.
 
Exact pour le .close() et l'exception, je vais etudier ca
 
Merci

RTFM
http://java.sun.com/j2se/1.5.0/doc [...] eader.html

Et à quoi sert :

Pourquoi tu fais un masque avec &0xFF ?

bah surtout, pourquoi il utilise pas un Reader ...

Oui le masque sert plus a rien car je convertis le int en char.
 
Il est resté d'un autre essai ou je faisais diverses experiences.
 
Pour the real moins moins : Ben parce que je debarque en java et j'y connais que dalle !!
 
A+

rien à voir avec Java. tu fais une lecture char par char sans tester EOF mais en utilisant .available() ... y pas plus lent quelque soit le langage.

tu aurais peut-être dû la lire cette page avant de lui dire de la lire

regarde du coté de l'API nio

Mais ne vous inquietez plus, ca marche bien a present !
 
Je lis effectivement caractere par caractere, mais avec StringBuffer la concatenation est rapide donc ca met maxi 10s a charger un fichier. Donc ca me convient bien.
 
Merci encore pour vos aides
A+

tente avec nio quand meme ca booste bien et la modif est legere

Je verrai ca dans la prochaine release du soft !