3

По рекомендации Qwertiy из прошлой темы, выделяю сей вопрос в отдельную.

Собственно проблема в указана в заголовке. Есть код и при его работе постепенно увеличивается отъедаемый приложением объем оперативной памяти. Чем больше то изображение, с которым работает код, тем заметней эта утечка. Как исправить ситуацию?

using System;
using System.Drawing;
using System.Drawing.Drawing2D;
using System.Windows.Forms;

namespace ImgRotate
{
    public partial class Form1 : Form
    {
        Bitmap btmpIcn;
        public Form1()
        {
            InitializeComponent();
            timer1.Enabled = true;
            btmpIcn = notifyIcon1.Icon.ToBitmap();
        }

        int an;
        private void timer1_Tick(object sender, EventArgs e)
        {
            if (an >= 350)
                an = 0;
            else
                an += 10;

            notifyIcon1.Icon = Icon.FromHandle(RotateImage(btmpIcn, an).GetHicon());
        }

        /// <summary>
        /// method to rotate an image either clockwise or counter-clockwise
        /// </summary>
        /// <param name="img">the image to be rotated</param>
        /// <param name="rotationAngle">the angle (in degrees).
        /// NOTE: 
        /// Positive values will rotate clockwise
        /// negative values will rotate counter-clockwise
        /// </param>
        /// <returns></returns>
        public static Bitmap RotateImage(Image img, float rotationAngle)
        {
            //create an empty Bitmap image
            Bitmap bmp = new Bitmap(img.Width, img.Height);

            //turn the Bitmap into a Graphics object
            Graphics gfx = Graphics.FromImage(bmp);

            //now we set the rotation point to the center of our image
            gfx.TranslateTransform((float)bmp.Width / 2, (float)bmp.Height / 2);

            //now rotate the image
            gfx.RotateTransform(rotationAngle);

            gfx.TranslateTransform(-(float)bmp.Width / 2, -(float)bmp.Height / 2);

            //set the InterpolationMode to HighQualityBicubic so to ensure a high
            //quality image once it is transformed to the specified size
            gfx.InterpolationMode = InterpolationMode.HighQualityBicubic;

            //now draw our new image onto the graphics object
            gfx.DrawImage(img, new Point(0, 0));

            //dispose of our Graphics object
            gfx.Dispose();

            //return the image
            return bmp;
        }
    }
}
14
  • 1
    то-есть вы на каждый тик порождаете новый битмап и хотите чтоб утечек не было? а диспозить предыдущее состояние не пробовали?
    – Dmitry
    22 янв 2016 в 19:50
  • Недостаточно изучил что автоматически освобождается, а что вручную диспозить нужно... Благодарю за подсказку. Проблема решилась.
    – Raf-9600
    22 янв 2016 в 19:54
  • создайте необходимые изображения, соберите их в коллекцию и используйте их для вывода в tray.
    – Stack
    22 янв 2016 в 19:55
  • @Stack коллекция может например зависеть от входящего сообщения ;)
    – Dmitry
    22 янв 2016 в 19:56
  • "Недостаточно изучил что автоматически освобождается, а что вручную диспозить нужно" -- у всех объектов с IDisposable при ненадобности надо вызывать Dispose()
    – Stack
    22 янв 2016 в 19:58

2 ответа 2

2

Если посмотреть исходники .NET Framework, то видно, что вызов Icon.FromHandle(IntPtr) транслируется в вызов Icon(IntPtr, takeOwnership=false).
Чтобы избежать утечки памяти, надо для каждого IntPtr, который был получен при вызове GetHicon, вызывать DestroyIcon, после того как IntPtr перестал быть нужен.

[DllImport("user32.dll", CharSet = CharSet.Auto)]
extern static bool DestroyIcon(IntPtr handle);

И у Bitmap, который больше не нужен, надо вызывать метод Dispose();

Чтобы каждый раз по таймеру не создавать новые Bitmap, как я уже говорил в комментарии тут, надо создать необходимые изображения, собрать их в коллекцию и использовать их для вывода в tray.


Можно конвертировать Bitmap в Icon не вызывая GetHicon.
см. "Fast and high quality Bitmap to icon converter".


UPDATE

using System.Drawing;
using System.Collections.Generic;
using System.Runtime.InteropServices; 

class Pic : IDisposable {
    [DllImport("user32.dll", CharSet = CharSet.Auto)]
    extern static bool DestroyIcon(IntPtr handle);    // см. ниже @
    public readonly int Angle;
    public readonly Bitmap Image;
    IntPtr Hicon;
    public Icon Icon;
    Bitmap RotateImage(Image prev, int an) { 
        throw new NotImplementedException();  // todo
    }
    public Pic(int angle, Image original) {
        this.Image = RotateImage(original, angle);
        this.Angle = angle;
        this.Hicon = this.Image.GetHicon();
        this.Icon = Icon.FromHandle(this.Hicon);
    }
    public void Dispose() {
        if (this.Hicon == IntPtr.Zero) return;
        this.Icon.Dispose();
        DestroyIcon(this.Hicon);     // см. ниже @
        this.Hicon = IntPtr.Zero;
        this.Image.Dispose();
    }
}

public partial class Form1 : Form {

  Dictionary<int, Pic> pics;
  int an = 0;      

  public Form1() {
    pics = new Dictionary<int, Pic>();
    pics.Add(new Pic(0, Image=notifyIcon1.Icon.ToBitmap()) );
  }

  void timer1_Tick(object sender, EventArgs e) {
     var prev = pics[an]; 
     if (an >= 350) an = 0; 
     else an += 10;
     Pic p;
     if (pics.TryGetValue(an, out p) == false) {
       p = new Pic(an, prev.Image);
       pics.Add(p);
     }
     notifyIcon1.Icon = p.Icon;
  }
}

@ для гарантированного вызова DestroyIcon надо использовать специальную обертку над неуправляемым ресурсом. Пример -- тут.

1
  • Очень-очень полезный ответ!!! Как выяснилось, без DestroyIcon, через 6 минут работы приложения возникала ошибка "В GDI+ возникла ошибка общего вида." и Студия подчёркивала notifyIcon1.Icon = Icon.FromHandle(RotateImage(btmpIcn, an).GetHicon());
    – Raf-9600
    22 янв 2016 в 21:30
1

Это не совсем утечка памяти. Со временем мусор будет чиститься, а память освобождаться. Однако, для объектов, реализующих IDisposable надо вызывать Dispose для освобождения ресурсов (в первую очередь unmanaged-ресурсов), как только объект стал не нужен.

Вместо явного вызова Dispose следует использовать блок using, поскольку в нём Dispose вызывается в finally, т. е. его вызов почти гарантирован. Кроме того, это более красивая конструкция в плане кода.

Что касается приведённого кода, то, во-первых, Bitmap обладает unmanaged-ресурсами, но твой код не пытается освободить их вызовом Dispose - ты уничтожаешь только Graphics.

Однако, я бы действовал другим способом. Ясно, что есть ограниченное количество картинок, поэтому я бы сразу создал массив Bitmap'ов с разными поворотами, а в таймере просто менял иконку. Нет смысла на каждой итерации выполнять поворот заново.

И сомнения вызывает необходимость преобразования Icon в Bitmap.

8
  • вызов GetHicon возвращает IntPtr - это дескриптор unmanaged-ресурса, который надо освобождать с помощью DestroyIcon. а вызов Dispose тут не поможет.
    – Stack
    22 янв 2016 в 21:27
  • Даже с Dispose программа с каждой секундой увеличивает потребляемую память. Конечно, гораздо меньше чем без него, однако всё равно странно. Идея с массивом изображений мне не по душе. Те же браузеры вращают троббер имея лишь одно изображение. Насчёт преобразования Icon в Bitmap - я не понимаю. Переписать сам метод вращения мне знаний не хватает.
    – Raf-9600
    22 янв 2016 в 21:28
  • @Stack, да. Не обратил внимания.
    – Qwertiy
    22 янв 2016 в 21:46
  • @Raf-9600, чем не нравится? Браузер вообще не так вращает - он видеокарту использует.
    – Qwertiy
    22 янв 2016 в 21:48
  • 1
    @Raf-9600, "Чтобы создаваемые функцией RotateImage изображения сохранять в массив, и когда их будет достаточно для анимации (в моём случае 35 штук) то уже брать для notifyIcon1 изображения из массива, а не рисовать их через RotateImage" - вот именно это я и имел в виду. Только при запуске программы все сразу сложить в массив. Но можно и твой вариант - может он и лучше даже.
    – Qwertiy
    22 янв 2016 в 22:17

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.