2

DotNetZipライブラリを使用して簡単なテストをまとめました。このライブラリは.bmpファイルでいっぱいのzipファイルを開き、それらを.jpg形式に変換します。

これ以前は、すべてのファイルをフォルダーに書き込み、変換し、jpgファイルを保存してから、元のbmpファイルを削除していました。

私は最初にそれらをメモリに解凍し、jpgに変換してから保存するつもりはありません。

コードは機能しますが、それほど速くはありません。コードを改善するために私ができることについて誰かが私に何か指針を教えてもらえますか？また、糸脱毛は役に立ちますか？

string zipToUnpack = "c:\\test\\1000.zip";
string unpackDirectory = "c:\\temp\\";

string f = string.Empty;
Bitmap bm;
MemoryStream ms;

using (ZipFile zip = ZipFile.Read(zipToUnpack))
{                 
  foreach (ZipEntry e in zip)
  {
    if (e.FileName.ToLower().IndexOf(".bmp") > 0)
    {
      ms = new MemoryStream();
      e.Extract(ms);
      try
      {
        bm = new Bitmap(ms);                              
        f = unpackDirectory + e.FileName.ToLower().Replace(".bmp", ".jpg");
        bm.Save(f, System.Drawing.Imaging.ImageFormat.Jpeg);
      }
      catch (Exception ex)
      {
        Console.WriteLine("File: " + e.FileName + " " + ex.ToString());
      }
      ms.Dispose();
    }
  }
}

ありがとう

4

1 に答える 1

3

通常、DotNetZip はシングルスレッドです。複数のスレッドで複数のアーカイブを開くことができますが、各アーカイブは 1 つのスレッドでのみ開くことができます。

複数の CPU またはコアを登録する場合は、MemoryStream のデータを jpg に変換する部分で QueueUserWorkItem を呼び出すことをお勧めします。

ZipEntry.Extract() の呼び出しは、すべてのエントリに対して同じスレッドで実行する必要があります。これは、Zipfile がすべての読み取りアクセスに対して単一の FileStream を維持し、エントリを抽出する複数のスレッドがファイル ポインター算術エラーを引き起こすためです。

したがって、次のようなものです。

    public class State
    {
        public string FileName;
        public MemoryStream stream;
    }

    public void Run()
    {
        string unpackDirectory = "c:\\temp\\";
        string zipToUnpack = "c:\\test\\1000.zip";

        var ConvertImage = new WaitCallback( (o) => {
                State s = o as State;
                try
                {
                    var bm = new Bitmap(s.stream);
                    var f = unpackDirectory + s.FileName.ToLower().Replace(".bmp", ".jpg");
                    bm.Save(f, System.Drawing.Imaging.ImageFormat.Jpeg);
                }
                catch (Exception ex)
                {
                    Console.WriteLine("File: " + s.FileName + " " + ex.ToString());
                }
            });


        using (ZipFile zip = ZipFile.Read(zipToUnpack))
        {
            foreach (ZipEntry e in zip)
            {
                if (e.FileName.ToLower().IndexOf(".bmp") > 0)
                {
                    var ms = new MemoryStream();
                    e.Extract(ms);
                    ThreadPool.QueueUserWorkItem ( ConvertImage, 
                                                   new State {
                                                       FileName = e.FileName, stream = ms }
                                                   });                      
                }
            }
        }
    }
于 2011-03-11T02:16:30.620 に答える