Probleme de taille avec ZipInputStream

Marsh Posté le 30-10-2003 à 16:02:27

Salut !

J'ai un problème lors de la création de fichiers zippé en Java (avec ZipOutputStream).
Lors de la lecture du fichier encodé, si le fichier est compressé (DEFLATED), je n'arrive pas a bien le relire car le ZipEntry que je prends ne me renvoie pas de taille (-1).
Je fais pourtant la méthode ZipEntry.setSize() a la création mais je ne peux pas faire de setCompressedSize(), ne connaissant pas la taille une fois compressé.

Comment faire pour compresser les fichiers et récupérer la bonne taille?

Merci!

Reply

Marsh Posté le 30-10-2003 à 16:02:27

Reply

Marsh Posté le 30-10-2003 à 16:03:46

euh c'est un peu brouillon j'ai pas compris. Tu arrives à créer une archive zip ou pas? tu peux l'ouvrir avec un soft annexe (genre winzip)

---------------
Just because you feel good does not make you right

Reply

Marsh Posté le 30-10-2003 à 16:24:16

Il n'y a pas besoin de faire un setSize() pour ecrire ni un getSize() pour lire.
Ca se manipule comme un OutputStream normal.
Si tu veux j'ai un classe qui ecris un fichier ZIP a la volée.

Message édité par nerisson le 30-10-2003 à 16:26:46

---------------
Light is right

Reply

Marsh Posté le 31-10-2003 à 21:26:57

DarkLord : je sais que c'est un peu brouillon, mais oui, je fais des fichiers Zip lisibles.

Je me sert de getSize() pour recréer mon tableau de bytes de bonne taille a la lecture c'est pour ca. Mais c'est bizarre que ca marche quand c'est pas zippé (level 0 - STORED) et pas quand c'est compressé (level 9 - DEFLATED).

Merci, je vais faire autrement pour recréer mon tableau de bytes.

Edit : y'a beau avoir marqué Shaman Yo comme pseudo, c'est bien la meme personne, Yo C Spi

Message édité par Shaman-Yo le 02-11-2003 à 02:46:16

Reply

Marsh Posté le 03-11-2003 à 09:23:29

Bon alors voici un petit exemple d'une classe permettant d'ecrire un fichier ZIP a la volee:

Code :

public final class ZipFileWriter extends FileWriter {
/** Flux de sortie. */
private ZipOutputStream zip;
/** Buffer d'ecriture de 16Ko. */
private static final int BUFFER_SIZE = 16 * 1024;
/**
* Constructeur.
* @param file nom du fichier
* @throws IOException en cas d'erreur
*/
public ZipFileWriter(final File file) throws IOException {
super(file);
initZIP(file.getCanonicalPath());
}//end ZipFileWriter
/**
* Méthode non supportée.
* @param fd descripteur du fichier
* @throws IOException en cas d'erreur
*/
public ZipFileWriter(final FileDescriptor fd) throws IOException {
super("" );
throw new UnsupportedOperationException();
}//end ZipFileWriter
/**
* Méthode non supportée.
* @param fileName nom du fichier
* @param append indique si l'on ajoute les donnees a la fin du fichier
* @throws IOException en cas d'erreur
*/
public ZipFileWriter(final String fileName, boolean append) throws IOException {
super("" );
throw new UnsupportedOperationException();
}//end ZipFileWriter
/**
* Constructeur.
* @param fileName nom du fichier
* @throws IOException en cas d'erreur
*/
public ZipFileWriter(final String fileName) throws IOException {
super(fileName);
initZIP(fileName);
}//end ZipFileWriter
/**
* Initialise le fichier.
* @param compressedFile nom du fichier compressé
* @throws IOException en cas d'erreur
*/
private void initZIP(final String compressedFile) throws IOException {
/* Cree le flux de sortie */
final FileOutputStream fos = new FileOutputStream(compressedFile);
final BufferedOutputStream bos = new BufferedOutputStream(fos, BUFFER_SIZE);
zip = new ZipOutputStream(bos);
/* Prepare le zip */
final String fileName = compressedFile.substring(0, compressedFile.length()-4);
final ZipEntry entry = new ZipEntry(fileName);
entry.setMethod(ZipEntry.DEFLATED);
zip.putNextEntry(entry);
}//end initZIP
/**
* Force l'écriture des données en purgeant le cache.
* @throws IOException en cas d'erreur
*/
public void flush() throws IOException {
zip.flush();
}//end flush
/**
* Ferme le zip.
* @throws IOException en cas d'erreur
*/
public void close() throws IOException {
try {
zip.flush();
} catch(final IOException excpt) {
}//end try
zip.close();
super.close();
}//end close
/**
* Envoie la chaine dans le fichier ZIP.
* @param str la chaine
* @throws IOException en cas d'erreur
*/
public void write(final String str) throws IOException {
zip.write(str.getBytes());
}//end write
/**
* Envoie le tableau de caracteres dans le fichier ZIP.
* @param str le tableau de caracteres
* @throws IOException en cas d'erreur
*/
public void write(final char[] str) throws IOException {
byte[] tab = new byte[str.length];
for(int i=0;i<str.length;i++)
tab[i] = (byte)str[i];
zip.write(tab);
}//end write
/**
* Envoie le caractere dans le fichier ZIP.
* @param c le caractere
* @throws IOException en cas d'erreur
*/
public void write(final int c) throws IOException {
zip.write(c);
}//end write
/**
* Envoie le tableau de caracteres dans le fichier ZIP.
* @param str le tableau de caracteres
* @param off indice de debut des donnees
* @param len longueur des donnees
* @throws IOException en cas d'erreur
*/
public void write(final char[] str, final int off, final int len) throws IOException {
byte[] tab = new byte[len];
final int max = off+len;
for(int i=off;i<max;i++)
tab[i-off] = (byte)str[i-off];
zip.write(tab);
}//end write
/**
* Envoie la chaine dans le fichier ZIP.
* @param str la chaine
* @param off indice de debut des donnees
* @param len longueur des donnees
* @throws IOException en cas d'erreur
*/
public void write(final String str, int off, int len) throws IOException {
final String s = str.substring(off, off+len);
zip.write(s.getBytes());
}//end write
/**
* Destructeur.
* @throws Throwable en cas d'erreur
*/
protected void finalize() throws Throwable {
super.finalize();
}//end finalize
}//end ZipFileWriter

Patapé si le code n'est pa bo !

---------------
Light is right

Reply

Marsh Posté le 03-11-2003 à 09:38:54

ah non il a l'air top au contraire :jap:

Edit: euh on fait comment pour ajouter des fichiers au juste

Message édité par darklord le 03-11-2003 à 09:40:14

---------------
Just because you feel good does not make you right

Reply

Marsh Posté le 03-11-2003 à 17:49:42

Je crois que je me suis mal fait comprendre (en meme temps c'est normal vu les explications que j'ai donné, je pensais que ton code m'aiderai Nerrisson mais non.
Je résume :

Je n'ai aucun problème pour créer un zip, mais voila comment je fais (simplifié) :

Code :

//j'ai un tableau de bytes input
//et un OutputStream out
ZipOutputStream zos = new ZipOutputStream(out);
// Définition de la compression
zos.setMethod(ZipOutputStream.DEFLATED);
zos.setLevel(9);
ZipEntry entry = new ZipEntry("pouet" );
entry.setSize(input.length);
zos.putNextEntry(entry);
zos.write(input, 0, input.length);

Mais c'est pour lire le zip ainsi créé que j'ai des problèmes :

Code :

ZipInputStream zis = new ZipInputStream(in);
ZipEntry entry = zis.getNextEntry()
byte[] output = new Byte[entry.getSize()]; //=> c'est là le bug !!!
.....

La methode getSize() me renvoie '-1' et ce uniquement quand je créé des Zip avec une compression (qui est bien effective, les fichiers sont plus petits que la normale), si les données sont juste stockées dans un Zip, le problème ne se montre pas.

Merci !
(Si vous ne comprenez encore pas, n'hésitez pas a le dire mais je dois faire mes posts sans le code sous les yeux)

Reply

Marsh Posté le 03-11-2003 à 17:59:02

Bon ba j'ai testé ton bout de code pour lire et euh... j'ai pas de problème !
J'ai fait ce truc rapidos en 1.4 et ça marche nickel :

Code :

public class Grabadaou
{
public static void main(String[] args)
{
try {
FileInputStream in = new FileInputStream("E:\\dev.zip" );
ZipInputStream zis = new ZipInputStream(in);
ZipEntry entry = zis.getNextEntry();
System.out.println(entry.getSize());
}
catch(Exception e)
{
e.printStackTrace();
}
}
}

Donc ça doit être ton écriture qui foire [:spamafote]

Message édité par Taiche le 03-11-2003 à 17:59:44

---------------
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 03-11-2003 à 18:12:43

Ah ouais non. Chu en train de tester le merdier, c'est vrai que c'est bien laid et que ça pue. WinZip trouve bien le truc une fois écrit et tout mais pour le lire, ZipInputStream retrouve bien le nom nais pas la taille :??: Alors qu'il y arrive avec un fichier ZIP à moi...
Ca sent mauvais

EDIT : ouais bin j'confirme, c'est lors de l'écriture, le setSize() a pas l'air de faire son taf :??: La taille n'est pas écrite dans le .zip final, donc forcément, le getSize() retourne -1. Y a pu qu'à trouver pourquoi ça se fait pas lors de l'écriture

Message édité par Taiche le 03-11-2003 à 18:24:21

---------------
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 03-11-2003 à 18:33:42

Trouvé \o/
La réponse est là : http://mindprod.com/jgloss/zip.html

Citation :

ZipOutputStream produces a slighly non-standard format. ZipOutputStream puts the compressed and uncompressed size and CRC after the data, instead of in the local header just in front of it. Unfortuately, when you come to read this file with ZipInputStream, when you do an ZipEntry.getSize() you will get 0, because ZipInputStream is a stream, and can't look ahead to find it. There is a second copy of the header put at the end of the file forming an index. ZipFile is able to use this index to randomly access the file to read individual elements.

Moralité : toujours utiliser ZipFile pour lire les fichiers Zip.

EDIT : un pitit bout de code qui marche pour avoir des bons points

Code :

ZipFile zip = new ZipFile("E:\\miaou.zip" );
Enumeration enum = zip.entries();
while(enum.hasMoreElements())
{
ZipEntry entryRead = (ZipEntry)enum.nextElement();
System.out.println(entryRead.getName() + " : " + entryRead.getSize());
}
zip.close();

Message édité par Taiche le 03-11-2003 à 18:38:19

---------------
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 03-11-2003 à 18:33:42

Reply

Marsh Posté le 04-11-2003 à 16:39:50

Ok merci a tous!

2 questions :
1 - Nerisson, pourquoi utilises-tu un BufferedInputStream?
2 - Il y a dans ZipInputStream et ZipEntry le methode setMethod(int).
Laquelle définit le type de compression exactement? J'ai cru comprendre que celle de ZipEntry prédominait sur celle de ZipInputStream. Est-ce exact?

Merci.

Reply

Marsh Posté le 04-11-2003 à 16:44:45

Yo c Spi a écrit :

Ok merci a tous!
2 - Il y a dans ZipInputStream et ZipEntry le methode setMethod(int).
Laquelle définit le type de compression exactement? J'ai cru comprendre que celle de ZipEntry prédominait sur celle de ZipInputStream. Est-ce exact?

C'est dans ZipOutputStream [:aloy]
Bin je dirais que normalement ça s'applique à une ZipEntry mais que si elle n'est pas settée dans l'objet, alors c'est celle du ZipOutputStream qui sera utilisée. C'est d'ailleurs ce qui est dit dans la javadoc :

Citation :

This default will be used whenever the compression method is not specified for an individual ZIP file entry

---------------
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 04-11-2003 à 17:44:15

Yo c Spi a écrit :

Ok merci a tous!

2 questions :
1 - Nerisson, pourquoi utilises-tu un BufferedInputStream?
2 - Il y a dans ZipInputStream et ZipEntry le methode setMethod(int).
Laquelle définit le type de compression exactement? J'ai cru comprendre que celle de ZipEntry prédominait sur celle de ZipInputStream. Est-ce exact?

Merci.

Euh, c'est un BufferedOutputStream.
Ca sert a ameliorer les performances en utilisant un buffer d'ecriture.

---------------
Light is right

Reply

Probleme de taille avec ZipInputStream

Sujets relatifs:

Leave a Replay