Discussion :

[thedep]

Bonjour à tous.
Dans le programme que je suis entrain de corriger, je dois, à un moment donné, télécharger un fichier sur internet et l'enregistrer sur le disque de la machine.
(Env : Windows 7, Java7).
Ce sont des fichiers doc et pdf.
Le problème 'est que j'ai des erreurs qui apparaissent dans le "code" du fichier récupéré qui le rendent illisible. Le poids du fichier est proche de la taille réelle. (à quelques octets pres). Et quand je l'ouvre avec un lecteur de pdf, par exemple, ça m'affiche une page blanche.

Voici le morceau de 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
 
                                                try
						{
 
							System.out.println(cvDL);
							InputStream in = new 														ByteArrayInputStream(cvDL.getBytes());
							FileOutputStream writenFile = new FileOutputStream(
									fichier);
 
							byte[] buffer = new byte[1024];
							int read;
 
							while ((read = in.read(buffer)) > 0)
							      writenFile.write(buffer, 0, read);
 
							writenFile.flush();
							writenFile.close();
						}
						catch (Exception e)
						{
							e.printStackTrace();
						}

cvDL est du type String et contient le fichier récupéré par une requête HTTP ici :

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
 
				HttpPost httppost = new HttpPost(cv.getPathDownloadFile());
				HttpResponse response = execute(httpclient, httppost, nameValuePairs);
 
				String cvDL = "";
				try
				{
				BufferedReader reader = new BufferedReader(new InputStreamReader(response
						.getEntity().getContent()));
				String ligne = "";
				while ((ligne = reader.readLine()) != null)
				{
					cvDL += ligne + "\n";
				}
				reader.close();
				}
				catch (IllegalStateException e)
				{
				e.printStackTrace();
				}
				catch (IOException e)
				{
				e.printStackTrace();
				}

Je n'arrive pas à trouver d'où ces erreurs de copie peuvent venir.
Problème de buffer ? d'encodage ?

[thelvin]

Il ne faut pas utiliser de Reader ni de String pour copier des fichiers. Uniquement des InputStream et des OutputStream. C'est la différence entre texte et binaire.

Tu dois modifier cet extrait de code pour qu'il n'y ait aucun Reader nulle part. Normalement, il n'y aura plus de String.getBytes() non plus.

[thedep]

Merci pour la réponse c'était bien ça.

Voici le code corrigé :

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
			httppost = new HttpPost(pathDownloadFile);
			response = execute(httpclient, httppost, nameValuePairs);
 
			DataInputStream cvDL = null;
			try
			{
				cvDL = new DataInputStream(response
						.getEntity().getContent());
			}
			catch (IllegalStateException e)
			{
			e.printStackTrace();
			}
			catch (IOException e)
			{
			e.printStackTrace();
			}

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
 
						try
						{
 
							FileOutputStream writenFile = new FileOutputStream(fichier);
 
							byte[] buffer = new byte[1024];
							int read;
 
							while ((read = cvDL.read(buffer)) > 0)
							     writenFile.write(buffer, 0, read);
 
							writenFile.flush();
							writenFile.close();
						}
						catch (Exception e)
						{
							e.printStackTrace();
						}

Je ne sais pas si c'est ce qu'il y a de mieux, mais ça marche bien.

[thelvin]

Ça va plutôt bien. Quelques remarques :

- Le close() doit être dans le finally, pas dans le try !!

En fait, depuis Java 7, le mieux est d'utiliser un try-with-resources, genre :

Code :

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

1
2
3
4
5
6
7
8
try(OutputStream writtenFile = new FileOutputStream(fichier)) {
  byte[] buffer = new byte[1024];
  int read;
 
  while ((read = cvDL.read(buffer)) > 0) {
    writtenFile.write(buffer, 0, read);
  }
}

writtenFile étant déclaré comme une ressource du try, il sera fermé automatiquement à la sortie du try, en cas d'échec comme de réussite.

- Il ne faut pas oublier de fermer le flux en lecture. Lui aussi avec un try-with-resource, de préférence.

- Pourquoi faire un catch sur l'Exception ? Si ça n'a pas marché ça n'a pas marché, il vaut mieux remonter l'erreur à l'appelant.

- flush() ne sert à rien avant un close(). Et de manière générale, il n'est utile que quand on sait exactement pourquoi on le fait.

- DataInputStream n'était pas utile. Il suffisait de garder le InputStream donné par getContent()

[adiGuba]

Salut,

- flush() ne sert à rien avant un close(). Et de manière générale, il n'est utile que quand on sait exactement pourquoi on le fait.

+1 pour tous le reste, mais perso j'aurais tendance à laisser le flush() à la fin du traitement.


Et justement sans connaitre l'implémentation concrête du stream, ca ne coûte rien de le forcer et çà évite de mauvaise surprise...


a++

[thelvin]

Ma foi, je n'ai jamais eu de problème. Et la JavaDoc de close() est parfaitement claire : s'il y a du traitement à finir avant de fermer, close() doit s'en occuper.

Cas typique d'un flux base64, ou zippé, ou chiffré, où il y a des données pending à la fin que flush() ne peut pas envoyer, parce que le traitement n'est pas le même si ces données sont les dernières ou s'il y en aura d'autres après.

Un résultat différent entre flush() puis close(), ou juste close(), démontre un bug dans l'implémentation du stream. Bon les bugs, ça existe, certes.

[thedep]

Merci beaucoup pour les précisions que je viens d'intégrer.