0

別の質問で見つけたコードを使用して、double の配列を圧縮し、サイズをテストしています。私はこれを膨大な回数行います。このコードをより効率的にすることはできますか?

public static byte[] Compress(byte[] bytData)
    {
        try
        {
            MemoryStream ms = new MemoryStream();
            Stream s = new GZipStream(ms, CompressionMode.Compress);
            s.Write(bytData, 0, bytData.Length);
            s.Close();
            byte[] compressedData = ms.ToArray();
            return compressedData;
        }
        catch
        {
            return null;
        }
    }

static void Main(string[] args)
{
    public List<double> B;
    while (true)
    {
       datum = getNewDatum();
       B.Insert(0, datum);
       if (B.Count > 500)
                B.RemoveAt(B.Count - 1);
       byte[] byteArray = B.SelectMany(BitConverter.GetBytes).ToArray();
       byte[] compressedData = Compress(byteArray);
       //Console.WriteLine(compressedData.Length);
    }
}

編集1

2 つの領域を高速化できる可能性があります。

  1. 圧縮方法。
  2. Main で byteArray を作成する方法。おそらく、double のリストとして保存し、SelectMany を使用するのはあまり効率的ではないでしょうか?
4

1 に答える 1

0

最適化に関しては、決定論的破壊を提案でき、以下のように書き換えることができます。

 public static byte[] Compress(byte[] bytData)
    {
        try
        {
            using (var ms = new MemoryStream())
            {
                using (var s = new GZipStream(ms, CompressionMode.Compress))
                {
                    s.Write(bytData, 0, bytData.Length);
                    s.Close();
                    byte[] compressedData = ms.ToArray();
                    return compressedData;
                }
            }
        }
        catch
        {
            return null;
        }
    }
于 2013-06-19T10:58:10.220 に答える