Общая ошибка GDI +: избегается при вызове GC.Collect ()

Question

У меня есть этот фрагмент кода («ruta1» и «ruta2» - это строки, содержащие путь к различным изображениям):

   Bitmap pic;
   Bitmap b = new Bitmap(SomeWidth, SomeHeight);
   g = Graphics.FromImage(b);
   g.FillRectangle(new SolidBrush(Color.White), 0, 0, b.Width, b.Height);

   b.Save(ruta2, ImageFormat.Jpeg);
   g.Dispose();
   b.Dispose();

   b = new Bitmap(OtherWidth, OtherHeight);
   pic = new Bitmap(ruta2);
   g = Graphics.FromImage(b);
   g.FillRectangle(new SolidBrush(Color.White), 0, 0, b.Width, b.Height);
   g.DrawImage(pic, 0, 0, pic.Height, pic.Width); 

   pic.Dispose();
   b.Save(ruta2, ImageFormat.Jpeg);
   g.Dispose();
   b.Dispose();

   pic = new Bitmap(ruta1);
   b = new Bitmap(AnotherWidth, AnotherHeight);
   g = Graphics.FromImage(b);
   g.FillRectangle(new SolidBrush(Color.White), 0, 0, b.Width, b.Height);

   int k = 1;

   // I just draw the "pic" in a pattern on "b"
   for (h = 0; h <= b.Height; h += pic.Height)
       for (w = 0; w <= b.Width; w += pic.Width)
            g.DrawImage(pic, w, h, k * pic.Width, k * pic.Height);

   pic.Dispose();            
   GC.Collect();  // If I comment this line, I get a "generic GDI+ Error" Exception
   b.Save(ruta1, ImageFormat.Jpeg);
   g.Dispose();
   b.Dispose();

Неважно, если я установлю pic = null после его удаления, если я не вызову сборщик мусора, я получу исключение «Generic GDI + Error». Только когда я вызываю сборщик мусора, моя программа всегда работает нормально.

Может кто-нибудь объяснить это поведение? Зависит ли это от версии .Net Framework? Я использую Visual C # Express 2008, с .Net Framework 3.5

Answer 1

Во-первых, было бы неплохо, если бы вы использовали ключевое слово "using", чтобы охватить использование ваших одноразовых объектов (например, Bitmap и Graphics), вместо того, чтобы вызывать Dispose() вручную для каждого. Использование «using» лучше по многим причинам, таким как очистка содержимого при возникновении исключения, но также очень помогает для читабельности кода.

Во-вторых, кисти GDI также являются IDisposable объектами, поэтому вы не должны создавать и забывать их. Сделайте это вместо:

using (var brush = new SolidBrush(Color.White))
{
    g.FillRectangle(brush, 0, 0, width, height)
}

... или, что еще лучше, создайте свои кисти в начале своего приложения и держитесь за них до конца (но не забывайте также утилизировать их). IIRC, создание / утилизация кистей значительно влияет на производительность, если делать это часто.

В-третьих, я считаю, что ваша ошибка во 2-м разделе:

1. Вы открываете изображение "ruta2"
2. Вы создаете новое изображение
3. Вы рисуете содержимое "ruta2" внутри этого нового изображения
4. Вы сохраняете эту новую вещь поверх файла "ruta2", но исходное изображение из шага 1 все еще загружается и, вероятно, имеет некоторый дескриптор файла "ruta2", поэтому вам нужно избавиться от него перед тем, как перезаписать файл.

Если вы реорганизуете второй раздел, как это, он должен работать:

using (var b = new Bitmap(OtherWidth, OtherHeight))
using (var g = Graphics.FromImage(b))
{
     using (var brush = new SolidBrush(Color.Red))
     {
         g.FillRectangle(brush, 0, 0, b.Width, b.Height);
     }
     using (var pic = new Bitmap(ruta2))
     {
         g.DrawImage(pic, 0, 0, pic.Height, pic.Width);
     }
     b.Save(ruta2, ImageFormat.Jpeg);
 }