C# Image loader memory leak

Question

Я реализую небольшой просмотрщик изображений, к сожалению, я сталкиваюсь с утечкой памяти. Ниже приведен порядок загрузки.

public BitmapSource getImage(string fileName, double width, double height) 
    { 
     FileStream s = File.Open(fileName, FileMode.Open); 
     Image i = Image.FromStream(s, false, false); 
     double iWidth = i.Width; 
     double iHeight = i.Height; 
     i.Dispose(); 
     s.Close(); 

     BitmapImage tmpImage = new BitmapImage(); 
     tmpImage.BeginInit(); 
     tmpImage.CacheOption = BitmapCacheOption.OnLoad; 
     tmpImage.UriSource = new Uri(fileName); 
     if (iWidth > iHeight) 
     { 
      tmpImage.DecodePixelWidth = (int)width; 
     } 
     else 
     { 
      tmpImage.DecodePixelHeight = (int)height; 
     } 
     tmpImage.EndInit(); 
     return tmpImage; 
    } 

Это, как я называю Загрузчик

private void whenArrowKeyPressed(int index) 
{ 
    CurrentImage = fh.getImage(fileList[index], 1920, 1080); 
} 

CurrentImage это свойство, которое связано с WPF ViewBox.

Любые идеи? Я также попытался прочитать StreamSource с тем же эффектом.

Answer 1

Только проблема, которую я мог видеть, это то, что вы не используете FileStream. BitmapImage не одноразовый, и он освободит свою память, если нет ссылок на нее.

Как вы обнаружили, что происходит утечка памяти? Рекомендуется использовать инструмент профилирования. Сборщик мусора не выпускает память, когда что-то выходит из сферы действия, она ожидает, что использование памяти превысит определенные пороговые значения (обычно, когда Gen 0 будет превышать порог). Таким образом, вы увидите некоторое увеличение памяти, и оно освободит память только после того, как сборщик мусора выполнит.

Видимо, вы не используете инструмент для профилирования. В этом случае, если вы хотите проверить, есть ли утечка памяти, вы можете вручную выполнить GC.Collect и дождаться завершения, прежде чем вы получите чтение памяти. Имейте в виду, что вам не нужно вручную выполнять GC.Collect, поскольку это будет происходить автоматически оптимизированным образом, когда это необходимо.

public BitmapSource getImage(string fileName, double width, double height) 
    { 
     using(FileStream s = File.Open(fileName, FileMode.Open)) 
     using(Image i = Image.FromStream(s, false, false)) 
     { 
      double iWidth = i.Width; 
      double iHeight = i.Height;     
     } 

     BitmapImage tmpImage = new BitmapImage(); 
     tmpImage.BeginInit(); 
     tmpImage.CacheOption = BitmapCacheOption.OnLoad; 
     tmpImage.UriSource = new Uri(fileName); 
     if (iWidth > iHeight) 
     { 
      tmpImage.DecodePixelWidth = (int)width; 
     } 
     else 
     { 
      tmpImage.DecodePixelHeight = (int)height; 
     } 
     tmpImage.EndInit(); 
     return tmpImage; 
    } 

private void whenArrowKeyPressed(int index) 
{ 
    CurrentImage = fh.getImage(fileList[index], 1920, 1080); 

    // Remove this once you finish testing. 
    GC.Collect(); 
    GC.WaitForPendingFinalizers(); 
}