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.

[Baboulinet_]

Ok, merci pour tout thelvin

Donc en fait pour lire le fichier xls, je suis obligé de passer par un buffer qui va me permettre de récupérer les données par lots afin de les retranscrire dans mon fichier créé à cet effet, c'est bien ça ?
Le buffer est une sorte de siège du télésiège si je comprends bien et là on peut mettre 1024 personnes sur ce siège (en schématisant hein ) ?

Du coup le nbread correspond au nombre des sièges qu'il va falloir pour emmener ces personnes là de l'autre côté et lorsqu'on en lit un, ce nombre est décrémenté ?

[thelvin]

Donc en fait pour lire le fichier xls, je suis obligé de passer par un buffer qui va me permettre de récupérer les données par lots afin de les retranscrire dans mon fichier créé à cet effet, c'est bien ça ?

Pas du tout. Tu peux le lire un octet à la fois comme indiqué dans mon premier programme. Par contre, ce sera bien plus lent que de le lire par buffers, et ça ne sert à rien...

Le buffer est une sorte de siège du télésiège si je comprends bien et là on peut mettre 1024 personnes sur ce siège (en schématisant hein ) ?

... Je suppose qu'on peut le voir comme ça. De fait, la file avance bien plus vite avec des télésièges larges que des télésièges monoplaces.

Du coup le nbread correspond au nombre des sièges qu'il va falloir pour emmener ces personnes là de l'autre côté et lorsqu'on en lit un, ce nombre est décrémenté ?

Pas du tout. Ça correspond au nombre de personnes qui se sont avancées pour monter sur le siège. Il n'y en a pas forcément eu 1024, et le siège n'attend pas que tout le monde soit là. Et puis peut-être que la file est finie, qu'il n'y a plus personne et que le siège n'est donc pas complet.

Supposons que seulement 823 personnes soient montées sur le siège. À l'autre bout, il ne faut pas chercher à déposer 1024 personnes, mais seulement 823. Parce que le reste, c'est des sièges vides et ça va poser problème...

[Baboulinet_]

Ok merci de tes éclaircissements

A bientôt et merci de toute l'aide que tu apportes au sein de ce forum (je t'ai vu sur de nombreux sujet )