System.Drawing.Image.FromStream()上的Out of Memory异常

Jer*_*rry 5 c# image image-processing out-of-memory

我有一个应用程序,它处理和重新调整图像大小和偶尔在长迭代期间我得到OutOfMemoryException.

我将我的图像作为文件流存储在数据库中,在处理过程中我需要将它们保存到临时物理位置.

我的模特:

[Table("Car")]
public class Car
{
   [... some fields ...]
   public virtual ICollection<CarPhoto> CarPhotos { get; set; }
}

[Table("CarPhoto")]
public class CarPhoto
{
   [... some fields ...]
   public Guid Key { get; set; }

   [Column(TypeName = "image")]
   public byte[] Binary { get; set; }
}
Run Code Online (Sandbox Code Playgroud)

处理看起来大致如下:

foreach (var car in cars)
{
    foreach (var photo in car.CarPhotos)
    {
        using (var memoryStream = new MemoryStream(photo.Binary))
        {
            using (var image = Image.FromStream(memoryStream)) // this is where the exception is thrown
            {
                var ratioX = 600.00 / image.Width;

                var newWidth = (int)(image.Width * ratioX);
                var newHeight = (int)(image.Height * ratioX);

                using (var bitmap = new Bitmap(newWidth, newHeight))
                {
                    Graphics.FromImage(bitmap).DrawImage(image, 0, 0, newWidth, newHeight);
                    bitmap.Save(directory + filePath);
                }
            }
        }
    }
}
Run Code Online (Sandbox Code Playgroud)

我看过这个类似的帖子,但没有一个答案似乎适用于我的情况.

其中一个答案建议使用Image.FromStream(),但它仍然是我正在做的事情.

我非常有信心我的所有图像都是有效的.异常似乎是随机发生的,最常见的是处理较大的文件,但也适用于较小的文件.有时一个图像会失败,但下次处理时会很好.

据我所知,我正在处理所有事情(内存流,图像和位图).

Hangfire正在努力完成这项工作.这可能会导致问题吗?

Han*_*ant 6

你的程序占用了大量内存,获得OOM肯定不会出乎意料.它究竟会死在哪里是不可预测的.但是,是的,创建位图是它可能首先死亡的地方.按顺序解决最可能的原因:

   foreach (var car in cars)
Run Code Online (Sandbox Code Playgroud)

您处理的汽车数量没有明显的上限.每辆车都有多个图像,您似乎将这些图像存储在内存中(photos.Binary).或者换句话说,这个程序是保证死早晚,只是因为它需要处理日益多个高增长的汽车数量.获得成功的唯一方法是通过串行处理汽车而不是将它们全部存储在内存中.可能是不愉快的建议,强烈建议在64位模式下运行此代码.

   using (var memoryStream = new MemoryStream(photo.Binary))
Run Code Online (Sandbox Code Playgroud)

该内存流是一个大问题,它的底层缓冲区很可能存储在大对象堆中.LOH没有被压缩,重复分配MemoryStream会使这个堆可能碎片化.迟早你会跑出一个足够大的洞,以适应下一张照片的缓冲区.超出随机,确切地说,何时发生取决于您之前处理的照片类型.您将需要重复使用该对象,而不是一遍又一遍地重新分配它.只需在进入循环之前创建一次,将容量设置为一个很好的大数字.

    Graphics.FromImage(bitmap).DrawImage(image, 0, 0, newWidth, newHeight);
Run Code Online (Sandbox Code Playgroud)

需要处理Graphics对象.像使用其他对象一样使用using语句.


总的来说,你的程序的真正问题在于,它只是不能扩展,并且在需要处理不断增加的数据集时总是会出现问题.修复这可能是一个非常重要的重写,你几乎肯定想要在其上翻转忽略位并利用64位进程中的可用地址空间.硬件救援,今天很容易获得.