web-dev-qa-db-ja.com

System.IO.Compressionで作成した後の無効なZipファイル

1つ以上のファイルを含むZipファイルを作成しようとしています。
。NET Framework 4.5、より具体的にはSystem.IO.Compression名前空間を使用しています。
目的は、ユーザーがASP.NETMVCアプリケーションを介してZipファイルをダウンロードできるようにすることです。
Zipファイルが生成されてクライアントに送信されていますが、それをダブルクリックして開こうとすると、次のエラーが発生します。
Windowsはフォルダを開くことができません。圧縮（zip形式）フォルダー...が無効です。
これが私のコードです：

[HttpGet]
public FileResult Download()
{
    var fileOne = CreateFile(VegieType.POTATO);
    var fileTwo = CreateFile(VegieType.ONION);
    var fileThree = CreateFile(VegieType.CARROT);

    IEnumerable<FileContentResult> files = new List<FileContentResult>() { fileOne, fileTwo, fileThree };
    var Zip = CreateZip(files);

    return Zip;
}

private FileContentResult CreateFile(VegieType vType)
{
    string fileName = string.Empty;
    string fileContent = string.Empty;

    switch (vType)
    {
        case VegieType.BATATA:
            fileName = "batata.csv";
            fileContent = "THIS,IS,A,POTATO";
            break;
        case VegieType.CEBOLA:
            fileName = "cebola.csv";
            fileContent = "THIS,IS,AN,ONION";
            break;
        case VegieType.CENOURA:
            fileName = "cenoura.csv";
            fileContent = "THIS,IS,A,CARROT";
            break;
        default:
            break;
    }

    var fileBytes = Encoding.GetEncoding(1252).GetBytes(fileContent);
    return File(fileBytes, MediaTypeNames.Application.Octet, fileName);
}

private FileResult CreateZip(IEnumerable<FileContentResult> files)
{
    byte[] retVal = null;

    if (files.Any())
    {
        using (MemoryStream zipStream = new MemoryStream())
        {
            using (ZipArchive archive = new ZipArchive(zipStream, ZipArchiveMode.Create, false))
            {
                foreach (var f in files)
                {
                    var entry = archive.CreateEntry(f.FileDownloadName, CompressionLevel.Fastest);
                    using (var entryStream = entry.Open())
                    {
                        entryStream.Write(f.FileContents, 0, f.FileContents.Length);
                        entryStream.Close();
                    }
                }

                zipStream.Position = 0;
                retVal = zipStream.ToArray();
            }
        }
    }

    return File(retVal, MediaTypeNames.Application.Zip, "horta.Zip");
}

誰かが、私がそれをダブルクリックしたときに私のZipファイルが無効であるとウィンドウが言っている理由にいくつかの光を当てることができますか？.
最後の考慮事項として、7-Zipを使用して開くことができます。

c#asp.net-mvcsystem.io.compression
16
César Lourenço

ZipArchiveオブジェクトが破棄された後、ToArrayを介してMemoryStreamバッファを取得する必要があります。そうしないと、アーカイブが破損してしまいます。

また、エントリを追加するときにZipArchiveコンストラクターのパラメーターを開いたままにするように変更したことに注意してください。

ZipArchiveが破棄されているときにチェックサムが実行されているため、以前にMemoryStreamを読んだ場合、それはまだ不完全です。

    private FileResult CreateZip(IEnumerable<FileContentResult> files)
    {
        byte[] retVal = null;

        if (files.Any())
        {
            using (MemoryStream zipStream = new MemoryStream())
            {
                using (ZipArchive archive = new ZipArchive(zipStream, ZipArchiveMode.Create, true))
                {
                    foreach (var f in files)
                    {
                        var entry = archive.CreateEntry(f.FileDownloadName, CompressionLevel.Fastest);
                        using (BinaryWriter writer = new BinaryWriter(entry.Open()))
                        {                                   
                            writer.Write(f.FileContents, 0, f.FileContents.Length);
                            writer.Close();
                        }
                    }

                    zipStream.Position = 0;
                }
                retVal = zipStream.ToArray();
            }
        }

        return File(retVal, MediaTypeNames.Application.Zip, "horta.Zip");
    }
24
Michal Hainc

ストリームを返すだけです...

private ActionResult CreateZip(IEnumerable files)
{
    if (files.Any())
    {
        MemoryStream zipStream = new MemoryStream();
        using (ZipArchive archive = new ZipArchive(zipStream, ZipArchiveMode.Create, false))
        {
            foreach (var f in files)
            {
               var entry = archive.CreateEntry(f.FileDownloadName, CompressionLevel.Fastest);
               using (var entryStream = entry.Open())
               {
                   entryStream.Write(f.FileContents, 0, f.FileContents.Length);
                   entryStream.Close();
               }
           }

        }

        zipStream.Position = 0;
        return File(zipStream, MediaTypeNames.Application.Zip, "horta.Zip");
    }

    return new EmptyResult();
}
3
Dave

変更してみてください

using (ZipArchive archive = new ZipArchive(zipStream, ZipArchiveMode.Create, false))

に

using (ZipArchive archive = new ZipArchive(zipStream, ZipArchiveMode.Create, true))

この使用法では、アーカイブは閉じられたときにストリームへの書き込みを強制されます。ただし、コンストラクターのleaveOpen引数がfalseに設定されている場合は、基になるストリームも閉じます。

0
Phil

例のようにエントリに間違った名前を追加したとき

var fileToZip = "/abc.txt";
ZipArchiveEntry zipFileEntry = zipArchive.CreateEntry(fileToZip);

同じエラーが発生しました。ファイル名を修正したら、問題ありません。

0
ozgurozkanakdemirci