Décompression de fichier et encodage

Bonjour,

je cherche a faire une petite moulinette qui décompresse des archives dans un dossier spécifique, pour cela j'utilise une library open source (SharpZipLib) qui a l'air pas trop mal...

Cependant même si mon algorithme fonctionne sur des essais où j'ai placé des fichiers texte, cela ne fonctionne pas pour autant avec d'autres fichiers encodés (tels que des jpg, des wav, des mp3...). Après avoir comparé le contenu du fichier décompressé avec mon algo et celui du fichier original j'ai l'impression que cela semble être un problème d'encodage des caractère...

Comment faire pour que le fichier soit correctement encoder?

Cf-> code source
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 ZipInputStream zip = new ZipInputStream(File.OpenRead(@"D:\dev\Test\battlelan.zip")); ZipEntry obj; obj = zip.GetNextEntry(); // lit le fichier en cours while (obj != null) { byte[] buffer = new byte[obj.Size]; StreamWriter writer = new StreamWriter(@"D:\dev\Test\" + obj.Name, false); zip.Read(buffer, 0, buffer.Length); // après vérif le byte[] est correct, c'est au moment de la conversion en string que ca pose problème writer.Write(Encoding.ASCII.GetString(buffer, 0,buffer.Length)); writer.Close(); obj = zip.GetNextEntry(); } MessageBox.Show("Archive décompressée");
Merci d'avance pour vos réponses

Je pense qu'il faut plutot utiliser un BinaryWriter à la place du StreamWriter.
Ainsi que la méthode suivante: http://msdn.microsoft.com/en-us/library/ms143302.aspx

Merci pour le coup de main,

Effectivement le BinaryWriter a l'air plus approprié dans la mesure où ce coup ci en comparant les fichiers les caractères ont l'air identiques, cependant il doit manquer encore un petit truc:

Notepad++ me dis que le fichier original et celui décompréssé font le même nombre d'octets ^^ (environ 1,6Mo) cependant le nombre de caractères de celui décompressé est plus petit (20 000 caractères en moins)

En regardant la fin du fichier on voit que les 20 000 derniers caractères sont remplacés par des caractères "null", un problème de taille de buffer ou bien un truc du genre?

cf-> code
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 // lit le fichier en cours while (obj != null) { FileStream fs = File.Open(@"D:\dev\Test\" + obj.Name, FileMode.OpenOrCreate, FileAccess.ReadWrite); byte[] buffer = new byte[obj.Size]; BinaryWriter writer = new BinaryWriter(fs, Encoding.ASCII); zip.Read(buffer, 0, buffer.Length); writer.Write(buffer, 0, buffer.Length); writer.Close(); obj = zip.GetNextEntry(); }

Citation:
Envoyé par Shadow69
Merci pour le coup de main,

Effectivement le BinaryWriter a l'air plus approprié dans la mesure où ce coup ci en comparant les fichiers les caractères ont l'air identiques, cependant il doit manquer encore un petit truc:

Notepad++ me dis que le fichier original et celui décompréssé font le même nombre d'octets ^^ (environ 1,6Mo) cependant le nombre de caractères de celui décompressé est plus petit (20 000 caractères en moins)

En regardant la fin du fichier on voit que les 20 000 derniers caractères sont remplacés par des caractères "null", un problème de taille de buffer ou bien un truc du genre?

cf-> code
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 // lit le fichier en cours while (obj != null) { FileStream fs = File.Open(@"D:\dev\Test\" + obj.Name, FileMode.OpenOrCreate, FileAccess.ReadWrite); byte[] buffer = new byte[obj.Size]; BinaryWriter writer = new BinaryWriter(fs, Encoding.ASCII); zip.Read(buffer, 0, buffer.Length); writer.Write(buffer, 0, buffer.Length); writer.Close(); obj = zip.GetNextEntry(); }
Et sans définir l'encoding du binarywriter?

N'utilise pas de BinaryWriter (et encore moins de StreamWriter), utilise directement le Stream... Tu ne connais pas le contenu des fichiers, donc à partir de là, si tu commences à te préoccuper de l'encodage, tu es sûr de te planter. L'encodage n'a de sens que pour du texte.

Citation:

Envoyé par tomlev

N'utilise pas de BinaryWriter (et encore moins de StreamWriter), utilise directement le Stream... Tu ne connais pas le contenu des fichiers, donc à partir de là, si tu commences à te préoccuper de l'encodage, tu es sûr de te planter. L'encodage n'a de sens que pour du texte.

Ca prend en compte l'encodage un binarywriter?

possible de me faire un exemple rapide avec un Stream, j'ai jamais utilisé encore et j'vois pas comment le coder (sachant que le contenu du fichier en cours d'extraction sera dans buffer[] (byte[])

merci d'avance

Citation:

Envoyé par Shadow69

possible de me faire un exemple rapide avec un Stream, j'ai jamais utilisé encore et j'vois pas comment le coder (sachant que le contenu du fichier en cours d'extraction sera dans buffer[] (byte[])

merci d'avance
Code:

1 2 3 4 5 6 7 8 9 10 while (obj != null) { using(FileStream fs = File.Open(@"D:\dev\Test\" + obj.Name, FileMode.OpenOrCreate, FileAccess.ReadWrite)) { byte[] buffer = new byte[obj.Size]; zip.Read(buffer, 0, buffer.Length); fs.Write(buffer, 0, buffer.Length); } obj = zip.GetNextEntry(); }
PS: Attention à l'utilisation des streams qui requièrent d'appeller Dispose() (ou de passer par un using)

Citation:

Envoyé par PitMaverick78

Ca prend en compte l'encodage un binarywriter?

Ben apparemment oui, puisqu'il y a un constructeur qui prend un Encoding... c'est probablement juste pour écrire des chaines de caractères.

Citation:

Envoyé par PitMaverick78

Code:

1
2
3
4
5
6
7
8
9
10
while (obj != null)
            {
                using(FileStream fs = File.Open(@"D:\dev\Test\" + obj.Name, FileMode.OpenOrCreate, FileAccess.ReadWrite))
{
                byte[] buffer = new byte[obj.Size];
                zip.Read(buffer, 0, buffer.Length);
                fs.Write(buffer, 0, buffer.Length);
}
                obj = zip.GetNextEntry();
            }

Là tu vas avoir des problèmes si le fichier à décompresser pèse 1Go... il vaudrait mieux lire par petits bouts

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
byte[] buffer = new byte[8192];
while (obj != null)
            {
                using(FileStream fs = File.Open(@"D:\dev\Test\" + obj.Name, FileMode.OpenOrCreate, FileAccess.ReadWrite))
{
                int bytesRead;
                while((bytesRead = zip.Read(buffer, 0, buffer.Length)) > 0)
                {
                    fs.Write(buffer, 0, bytesRead);
                }
}
                obj = zip.GetNextEntry();
            }

Citation:

Envoyé par tomlev

Ben apparemment oui, puisqu'il y a un constructeur qui prend un Encoding... c'est probablement juste pour écrire des chaines de caractères.

Etant curieux, j'ai réflectoré la DLL. Alors effectivement ca utilise l'encoding mais uniquement pour les surcharges Write(Char[]), Write(Char[], int, int) et Write(String).
Write(byte[], int, int) appelle directement Stream.Write(byte[], int, int)
Donc ca marchait quand même :D (m'enfin je chipote)

malheureusement résultat identique, les caractères sont transcrits correctement mais sur la longeur ca marche pas... Au pire je crois que j'vais faire une boucle ou j'vais faire ecrire des plages du buffer :s

Citation:

Envoyé par Shadow69

les caractères sont transcrits correctement

Encore heureux, puisqu'ils ne sont pas transcrits du tout ;)
Le traitement se fait au niveau des données binaires, le fait que ces données représentent des caractères ou autre chose n'a absolument aucune importance

Citation:

Envoyé par Shadow69

mais sur la longeur ca marche pas...

Fais voir la dernière version de ton code, il doit y avoir une erreur toute bête... j'ai peut-être déjà une petite idée sur la question

Tiens je te met tout le snippet de la méthode en question. J'ai beau retourner ca dans ma tête j'arrive toujours pas a trouver la solution (sans doute encore trop de vapeurs d'alcool dans la tête :D)

(au fait bonne année ^^)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
  private void button1_Click(object sender, EventArgs e)
        {
            ZipInputStream zip = new ZipInputStream(File.OpenRead(@"D:\dev\Test\battlelan.zip"));
            ZipEntry obj;
            obj = zip.GetNextEntry();
 
 
            // lit le fichier en cours
            while (obj != null)
            {
                FileStream fs = File.Open(@"D:\dev\Test\" + obj.Name, FileMode.OpenOrCreate, FileAccess.ReadWrite);
                byte[] buffer = new byte[obj.Size];
                zip.Read(buffer, 0, buffer.Length);
                fs.Write(buffer, 0, buffer.Length);
                fs.Close(); 
                obj = zip.GetNextEntry();
            }
 
            MessageBox.Show("Archive décompressée");
        }

PS: pour les fichiers décompressé, dans le cas de cette moulinettes les zip sont composés de quelques jpeg et fichiers textes (en gros une taille maxi de 5Mo par fichier, et maxi 30-40Mo pour toute l'archive...)

Bonne année aussi ! :chin:

Bon, je comprends pas trop pourquoi ça marche pas... je viens de faire un test à peu près identique et les fichiers sont correctement décompressés. Voilà mon code, si ça peut aider :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
    string zipPath = @"D:\tmp\test.zip";
    string outputDir = @"D:\tmp\testOutput";
    using (var fileStream = File.OpenRead(zipPath))
    using (var zipStream = new ZipInputStream(fileStream))
    {
        ZipEntry entry = null;
        while ((entry = zipStream.GetNextEntry()) != null)
        {
            if (!entry.IsFile)
                continue;
            string unzippedFilePath = Path.Combine(outputDir, entry.Name);
            Directory.CreateDirectory(Path.GetDirectoryName(unzippedFilePath));
            using (var unzippedFileStream = File.OpenWrite(unzippedFilePath))
            {
                var buffer = new byte[entry.Size];
                zipStream.Read(buffer, 0, buffer.Length);
                unzippedFileStream.Write(buffer, 0, buffer.Length);
            }
        }
    }