zipper des fichiers

zipper des fichiers - Java - Programmation

Marsh Posté le 09-07-2009 à 14:10:27    

Bonjour,
je dois recuperer des fichiers sur un serveur, les zipper, puis les envoyer sur la sortie en telechargement (java web pour info).
 
J'ai un code qui compile mais le fichier semble etre corrompu. Il fait la taille du premier fichier que je lui passe seulement et ni windows ni 7zip ne peuvent l'ouvrir.
 
Voici mon code, en esperant que quelqu'un pourra m'aider.

Code :
  1. // These are the files to include in the ZIP file
  2. List<String> filenames = down.getFilenames();
  3. List<String> urls = down.getUrls();
  4. byte data[] = new byte[1024];
  6. try {
  7. HttpClient client = new HttpClient();
  8. // set default retry handler
  10. client.getParams().setParameter(HttpMethodParams.RETRY_HANDLER,
  11. new DefaultHttpMethodRetryHandler());
  13. // Create the ZIP file
  14. String outFilename = "outfile.zip";
  16. FileOutputStream dest = new FileOutputStream(outFilename);
  17. BufferedOutputStream buff = new BufferedOutputStream(dest);
  18. ZipOutputStream out = new ZipOutputStream(buff);
  20. out.setMethod(ZipOutputStream.DEFLATED);
  21. out.setLevel(9);
  23. // Compress the files
  24. for (int i=0; i<filenames.size(); i++) {
  26. //make an HTTP GET
  27. HttpMethod method = new GetMethod(urls.get(i));
  28. client.executeMethod(method);
  30. InputStream is = method.getResponseBodyAsStream();
  32. BufferedInputStream buffi=new BufferedInputStream(is,1024);
  34. ZipEntry ze=new ZipEntry(filenames.get(i));
  36. // Add ZIP entry to output stream.
  37. out.putNextEntry(ze);
  39. int count;
  40. while((count = buffi.read(data, 0, 1024)) != -1) {
  41. out.write(data, 0, count);
  42. }
  44. // Complete the entry
  45. out.closeEntry();
  47. buffi.close();
  49. }
  50. out.finish();
  51. out.close();
  52. String zip=buff.toString();
  54. response.setContentType( "application/zip" );
  55. response.setHeader( "Content-Disposition", "attachment; filename=\""+outFilename+"\"" );
  57. ServletOutputStream op = response.getOutputStream();
  58. op.println(zip);
  59. op.flush();


---------------
Toinou87 ;-p May the force be with you!!
Reply

Marsh Posté le 09-07-2009 à 14:10:27   

Reply

Marsh Posté le 09-07-2009 à 14:51:28    

Precision:
je viens d'ouvrir le zip generer avec notepad, c'est l'envoie sur la sortie qui plante.
Le toString() n'envoie pas les données mais le type de l'objet.
Donc le fichier generer est un fichier texte avec BufferedInputStream.
 
Le probleme est donc:
passer les données de out ou buff à op


---------------
Toinou87 ;-p May the force be with you!!
Reply

Marsh Posté le 09-07-2009 à 15:23:37    

Mais pourquoi diable t'attends-tu à ce que toString() et println() envoient des données zippées ? [:pingouino]

 

T'as du binaire, pas des caractères.


Message édité par Taiche le 09-07-2009 à 15:24:20

---------------
Everyone thinks of changing the world, but no one thinks of changing himself  |  It is the peculiar quality of a fool to perceive the faults of others and to forget his own  |  Early clumsiness is not a verdict, it’s an essential ingredient.
Reply

Marsh Posté le 09-07-2009 à 15:34:35    

bon alors le boulot du println c'est d'envoyer les données sur la sortie. En l'occurence ici ServletOutputStream et donc il me renvoie bien le fichier generer.
Par contre pour le toString, je l'avais pris sur un tuto que je retrouve pas et je pensais qu'il etait overriden pour renvoyer les bytes du bufferedinputstream dans une string.


---------------
Toinou87 ;-p May the force be with you!!
Reply

Marsh Posté le 09-07-2009 à 15:39:04    

Bin non, toString() ça renvoie une String, c'est la signature de la méthode [:spamafote] (c'est même son nom)
Pour println() non, son boulot c'est envoyer des chaînes, pas des tableaux de bytes. Vois plutôt du côté de write() [:petrus75]

 

Sinon, en vrac :
* t'as pas besoin de BufferedOutputStream, tu peux mettre direct ton FileOutputStream, ça t'évitera la création d'un objet.
* une fois que t'as fait close() sur ton ZipOutputStream, ton opération est terminée et tu devrais avoir tes données dans ton fichier... non ?
* pis révise un peu la notion de streams en Java, l'idée même de vouloir faire un toString ou de récupérer des données quelconques sur buff me fait peur [:dawao]


Message édité par Taiche le 09-07-2009 à 15:40:17

---------------
Everyone thinks of changing the world, but no one thinks of changing himself  |  It is the peculiar quality of a fool to perceive the faults of others and to forget his own  |  Early clumsiness is not a verdict, it’s an essential ingredient.
Reply

Marsh Posté le 09-07-2009 à 15:41:18    

trouvé, j'utilise comme constructeur du buffered output stream le servlet output stream et ca marche (bon j'avais trouvé ca y a 30 min mais les fichiers du serveurs etaient perimés et ca me renvoyait des fichier vides).
Il reste peut etre des artefacts d'erreurs.
 

Code :
  1. List<String> filenames = down.getFilenames();
  2.     List<String> urls = down.getUrls();
  3.     byte data[] = new byte[1024];
  4.    
  5.     try {
  6.      HttpClient client = new HttpClient();
  7.   // set default retry handler
  9.   client.getParams().setParameter(HttpMethodParams.RETRY_HANDLER,
  10.     new DefaultHttpMethodRetryHandler());
  12.   BufferedInputStream buffi=null;
  14.         // Create the ZIP file
  15.         String outFilename = "outfile.zip";
  16.        
  18.         response.setContentType( "application/zip" );
  19.         response.setHeader( "Content-Disposition", "attachment; filename=\""+outFilename+"\"" );
  20.         BufferedOutputStream buff = new BufferedOutputStream(response.getOutputStream());
  21.         ZipOutputStream out = new ZipOutputStream(buff);
  22.        
  23.         out.setMethod(ZipOutputStream.DEFLATED);
  24.         out.setLevel(9);
  25.        
  26.         // Compress the files
  27.         for (int i=0; i<filenames.size(); i++) {
  28.        
  29.          //make an HTTP GET
  30.       HttpMethod method = new GetMethod(urls.get(i));
  31.       client.executeMethod(method);
  32.    
  33.       InputStream is = method.getResponseBodyAsStream();
  34.    
  35.       buffi=new BufferedInputStream(is,1024);
  37.       ZipEntry ze=new ZipEntry(filenames.get(i));
  38.            
  39.             // Add ZIP entry to output stream.
  40.             out.putNextEntry(ze);
  41.            
  42.             int count;
  43.             while((count = buffi.read(data, 0, 1024)) != -1) {
  44.                 out.write(data, 0, count);
  45.                 System.out.println(count);
  46.             }
  48.             // Complete the entry
  49.             out.closeEntry();
  51.             buffi.close();
  52.         
  53.         }
  54.         out.finish();
  55.         out.close();
  56.        
  57.        
  58.         out.flush();


---------------
Toinou87 ;-p May the force be with you!!
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed