В общем, есть окно, где есть комбобокс с десятками ссылок на картинку весом ~2 мб и разрешением 5037x3657.
Каждое перелистывание картинок добавляет используемое озу к программе, вплоть до 2 ГБ...
Картинки я нигде не кеширую.
VM у меня при каждом изменении значения в ComboBox запрашивает Drawing.Image из класса:
public ImageInfo CurrentImageInfo
{
get => _currentImageInfo;
set
{
_currentImageInfo = value;
OnPropertyChanged(nameof(CurrentImageInfo));
try
{
using (var bitmap = (Bitmap)CurrentImageInfo.GetMarkupedImage())
{
ImageSrc = BitmapToImageSource(bitmap);
}
}
catch (Exception e)
{
LogMessage += "\n" + e;
}
ImageSource BitmapToImageSource(Bitmap bitmap)
{
return System.Windows.Interop.Imaging.CreateBitmapSourceFromHBitmap(
bitmap.GetHbitmap(),
IntPtr.Zero,
Int32Rect.Empty,
BitmapSizeOptions.FromEmptyOptions());
}
}
}
public Image GetMarkupedImage()
{
Image image;
int x;
int y;
using (var srcImage = GetImageCopy())
{
image = MarkupDrawer.CreateImageWithBorder(srcImage, out x, out y);
}
using (var graphic = Graphics.FromImage(image))
{
foreach (var person in Persons)
{
var newRec = MarkupDrawer.ScaleMarkup(person.Rectangle, x, y);
MarkupDrawer.DrawMarkup(graphic, image, person.ToString(), newRec);
}
}
return image;
}
Image GetImageCopy()
{
var memoryStream = new MemoryStream();
File.OpenRead(FilePath).CopyTo(memoryStream);
return Image.FromStream(memoryStream);
}
Вроде, ничего криминального не вижу в коде.
ImageSource не обладает методом Dispose.
var memoryStream = new MemoryStream();
- вот тут проблема. Вы поток не уничтожаете... Используйтеusing
или вызывайтеDispose()
public ImageInfo CurrentImageInfo
это свойство, то я к сожалению впервые вижу такой синтаксис (и оно у меня ожидаемо не компиллируется); что за синтаксисget => _currentImageInfo;
, разве можно объявлять функции внутриset{...}
? в чём магия?? :)