2013-06-19 3 views
0

Я использую код, который я нашел в другом вопросе, чтобы сжать массив из двух пар, а затем проверить размер. Я делаю это огромное количество раз. Могу ли я сделать этот код более эффективным?Ускоренный способ сжатия массива двойных чисел

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); 
    } 
} 

EDIT 1

Две области могли бы быть быстрее:

  1. Метод сжатия.
  2. Как я создаю byteArray в Main. Возможно, сохранение в виде списка double и использование SelectMany не очень эффективно?

ответ

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; 
     } 
    } 
+0

Я не знаком с «помощью» - как вы думаете, это вызовет меньше работы, когда функция неоднократно? – ManInMoon

+0

Оператор using неявно вызывает Dispose, которые по существу освобождают ресурсы, используемые этим объектом. Это лучше всего подходит для управления памятью. http://msdn.microsoft.com/en-us/library/yh598w02(v=vs.80).aspx – Nair

+0

Правильно - но возможно ли это медленнее? Использует ли использование силы «Использовать» Dispose для вызова каждый раз - в то время как более прерывистый предыдущий способ? – ManInMoon

Смежные вопросы