Discussion :

[Baboulinet_]

Bonjour tout l'monde,

C'est par un beau matin ensoleillé en Bretagne qu'il m'arrive un petit souci !
Je suis en train de développer une application qui va permettre de récupérer des fichiers xls sur une boîte de messagerie pour les balancer dans un répertoire en dur.

Tout se passe bien, le fichier xls est copié. Mais son contenu est ... bizarre.
En effet, j'ai l'impression que le contenu du fichier n'est pas vraiment copié, mais que ce sont les lignes qui le composent qui sont copiées, ce qui donne au final quelque chose de moche et incompréhensible dans le fichier alors que je devrai avoir des lignes, des graphiques, et plein de trucs jolis concernant des statistiques.

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
21
22
23
24
25
26
27
28
public void DownloadFile(int num)
  {
 
    try
    {
      Multipart mp = (Multipart) myMailBox[num].getContent();
      int n=mp.getCount();
      for (int j=0;j<n;j++) {
          Part part = mp.getBodyPart(j);
          String disposition = part.getDisposition();
          if ((disposition != null)&&((disposition.equals(Part.ATTACHMENT)||(disposition.equals(Part.INLINE))))){
            File save=new File("C:\\temp\\"+part.getFileName());
            FileWriter ecriture=new FileWriter(save);
            InputStreamReader lecture=new InputStreamReader(part.getInputStream());
            int flux;
            while ((flux=lecture.read())!=-1) {
                    ecriture.write(flux);
            }
            lecture.close();
            ecriture.close();
 
          }
      }
    }
    catch(Exception ex)
    {
       System.out.println(ex.getMessage());
    }

Ma variable num correspond au numéro du mail choisi dans la liste.

Merci de toute aide apportée

[thelvin]

Les fichiers XLS sont du binaire, pas du texte. Il ne faut pas les passer dans des Reader ou des Writer.
Il faut utiliser directement le InputStream et écrire dans un FileOutputStream.

[Baboulinet_]

Les fichiers XLS sont du binaire, pas du texte. Il ne faut pas les passer dans des Reader ou des Writer.
Il faut utiliser directement le InputStream et écrire dans un FileOutputStream.

Merci pour m'avoir aiguillé.
Je vois où tu veux en venir, mais je vois pas trop comment ces méthodes sans lecture/écriture. J'ai trop été habitué pour réfléchir sans utiliser ces moyens faciles d'utilisation. Il est grand temps que j'enlève les roulettes d'appui de mon vélo, mais j'ai besoin d'un coup de main

[thelvin]

Transformation de ton code :

Code :

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

1
2
3
4
5
6
OutputStream ecriture = new FileOutputStream(save);
InputStream lecture = part.getInputStream();
int flux;
while ((flux=lecture.read())!=-1) {
  ecriture.write(flux);
}

Pas de Reader ni de Writer, juste des InputStream et OutputStream.

Version plus efficace :

Code :

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

1
2
3
4
5
6
7
8
OutputStream ecriture = new FileOutputStream(save);
InputStream lecture = part.getInputStream();
byte[] buffer = new byte[1024];
int nbRead = lecture.read(buffer);
while(nbRead > 0) {
  ecriture.write(buffer, 0, nbRead);
  nbRead = lecture.read(buffer);
}

Edit : erreur corrigée dans le code du dessus.

[Baboulinet_]

De cette manière ma boucle ne va pas s'arrêter.
J'ai testé, et c'est bien le cas. J'ai dû arrêter l'application quand j'ai vu que mon fichier xls dépassait les 2 Go

L'utilisation du buffer m'était peu connue et je suis bien d'accord sur son utilisation.
En ce qui concerne les byte, pourquoi 1024?

[thelvin]

De cette manière ma boucle ne va pas s'arrêter.
J'ai testé, et c'est bien le cas. J'ai dû arrêter l'application quand j'ai vu que mon fichier xls dépassait les 2 Go

Oui, j'avais oublié de lire un nouveau buffer dans la boucle. J'avais oublié cette ligne :

Code :

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

nbRead = lecture.read(buffer);

J'ai édité mon post pour que le code soit correct.

L'utilisation du buffer m'était peu connue et je suis bien d'accord sur son utilisation.
En ce qui concerne les byte, pourquoi 1024?

Pourquoi pas ?
L'idée est de lire les octets par lots au lieu d'un par un, pour éviter de faire trop d'appels IO.
Après, la taille du buffer, le nombre maximum d'octets qui seront lus à chaque fois, n'est pas très important. Ce pourrait être 512 ou 2048 ou 4096.
Attention quand même, se rappeler que ce buffer occupe de la mémoire, ce n'est donc pas une bonne idée d'en créer trop qui soient trop gros.
De plus, un buffer n'est pas forcément rempli par un appel à read(), et en fait au-delà d'une certaine taille ça ne sert à rien de faire plus grand.