Форум программистов, компьютерный форум, киберфорум
C# .NET
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.83/6: Рейтинг темы: голосов - 6, средняя оценка - 4.83
0 / 0 / 0
Регистрация: 04.09.2013
Сообщений: 3

CA2000 Dispose objects before losing scope

04.09.2013, 15:40. Показов 1339. Ответов 5
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Доброго времени суток.
Есть вот такой кусочек кода (сам кусочек пока не делает всего что нужно, но это не важно - суть в нем есть некая работа с Bitmap которую потом надо вернуть в Frame'е):

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
 private Frame MakeFrameFromChildNode(XmlNode node, ref string log)
        {
 
            Bitmap tempFrame = null;
 
            try
            {
                tempFrame = new Bitmap(_resolutionX, _resolutionY);
 
                if (node.Name == "frame")
                {                      
                    foreach (XmlNode frameChildNode in node.ChildNodes)
                    {
                        PlacePictureFromTagOnBitmap(frameChildNode);
                    }             
                }
                else
                {
                    log += string.Format("{0} Обнаружен неизвестный тэг {1}. Его содержимое игнорируется.\n", DateTime.Now.ToString("[hh:mm:ss]"),node.Name);
                }
 
                var result = tempFrame;
                tempFrame = null;
                return new Frame(result, 0);
            }
            finally
            {
                if (tempFrame != null) tempFrame.Dispose();
            }            
        }
Где Frame это класс для передачи аргументов:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
using System.Drawing;
 
namespace BitmapBuilder
{
    public sealed class Frame
    {
        private readonly Bitmap _bitmap;
        private readonly double _duration;
 
        public Frame(Bitmap bitmap, double duration)
        {
            _duration = duration;
            _bitmap = bitmap;
        }
 
        public Bitmap Bitmap
        {
            get { return _bitmap; }
        }
 
        public double Duration
        {
            get { return _duration; }
        }
    }
}
Суть проблемы: компилятор выдает варнинг на tempFrame
CA2000 Dispose objects before losing scope In method 'XmlToBitmapConverter.MakeFrameFromChild Node(XmlNode, ref string)', call System.IDisposable.Dispose on object 'tempFrame' before all references to it are out of scope.
В случае если функция MakeFrameFromChildNode возвращает тупо Bitmap проблем не возникает, а как быть если мне нужно вернуть именно экземпляр моего класса? Возвращать Bitmap не вариант т.к. это используется и в других местах, в том числе для передачи через Wcf.

Помогите пофиксить. Спасибо.
0
Programming
Эксперт
39485 / 9562 / 3019
Регистрация: 12.04.2006
Сообщений: 41,671
Блог
04.09.2013, 15:40
Ответы с готовыми решениями:

Request scope или Session scope
Всем привет Столкнулся с проблемой хранения данных. У меня есть страница которая отображает список пользователей, но перед прогрузкой...

Предупреждение CA2000 с объектом OleDbConnection
Доброго времени суток! При использовании анализа кода выскакивает порядка 70 предупреждений типа: Я пытался вызывать Dispoce для...

литература shared objects & dynamic shared objects
Привет, товариСЧи. Подкиньте пожалуйста пару тройку книженций по теме инглиш тоже пойдет, но лучше рус. Добавлено через 42 минуты ...

5
25 / 25 / 3
Регистрация: 10.06.2009
Сообщений: 134
04.09.2013, 15:45
Заключить тело метода в
C#
1
2
3
using(Bitmap btpam= new Bitmap(){
    //тело
}
и убрать try..finaly вообще.
0
0 / 0 / 0
Регистрация: 04.09.2013
Сообщений: 3
04.09.2013, 15:52  [ТС]
Цитата Сообщение от Nickopoll Посмотреть сообщение
Заключить тело метода в
C#
1
2
3
using(Bitmap btpam= new Bitmap(){
    //тело
}
и убрать try..finaly вообще.
Я боюсь что так ничего не выйдет. Вот смотрите еще один кусочек - там возвращается тупо Bitmap:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
private Bitmap ProcessBitmap(Bitmap sourceBitmap)
        {
            Contract.Requires(sourceBitmap != null);
 
            Bitmap temp = null;
 
            try
            {
                temp = new Bitmap(sourceBitmap);
 
 
                for (var pixelIndexX = 0; pixelIndexX < temp.Width; pixelIndexX++)
                    for (var pixelIndexY = 0; pixelIndexY < temp.Height; pixelIndexY++)
                    {
                        var bestColor = CalculateMatchingColor(pixelIndexX, pixelIndexY, temp);
                        SaveNewPixel(pixelIndexX, pixelIndexY, bestColor, temp);
                    }
 
                var result = temp;
                temp = null;
                return result;
            }
            finally
            {
                if (temp != null)
                {
                    temp.Dispose();
                }
            }
        }
При замене try..finally на using функция которая работает с результатом, например:
C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 private static BitmapSource GetSourceFromBitmap(Bitmap source)
        {
            var ip = source.GetHbitmap();
            BitmapSource bs;
            int result;
            try
            {
                bs = System.Windows.Interop.Imaging.CreateBitmapSourceFromHBitmap(ip,
                   IntPtr.Zero, Int32Rect.Empty,
                   BitmapSizeOptions.FromEmptyOptions());
            }
            finally
            {
                result = NativeMethods.DeleteObject(ip);
            }
            if (result == 0)
                throw new InvalidOperationException("NativeMethods.DeleteObject returns 0 (operation failed)");
 
            return bs;
        }
Вывалится с ошибкой в процессе выполнения. Собственно поэтому я и использовал данный пример с try..finally с msdn. Или я чего то не понимаю и в первом случае все будет хорошо?
0
25 / 25 / 3
Регистрация: 10.06.2009
Сообщений: 134
04.09.2013, 19:21
Ваше предупреждение CA2000 вываливается из-за того, что в методе, где есть выделение памяти для объекта ссылочного типа (в данном случае, объекта класса Bitmap), НЕ в любом случае возможно "корректное" ее (памяти) освобождение. То есть, присваивание объекту значения null не выделяет и не освобождает память, а лишь устанавливает указатель объекта в... ну... в "ничто" - точнее выразиться не получается. Если, перед тем, как объекту было присвоено null, память под него была выделена (явно, с помощью слова new, или неявно, с помощью присваивания другого объекта того же типа), то после присвоения null память так и останется занятой до тех пор, пока сборщик мусора .NET сам ее не очистит, что может быть весьма не скоро и вы можете наблюдать, как ваша прога все больше и больше жрет памяти.

Поэтому, когда объектом вы больше не пользуетесь, вызываете явно метод Dispose() явно (btmap.Dispose()), или неявно (using(Bitmap btmap = new Bitmap()) { //код }) . Но лишь в том случае, если объект ссылочного типа и имеет в своей цепочке наследования интерфейс IDisposable - если не имеет, то Visual Studio попросту не покажет метод Dispose() в выпадающем списке после установки точки на объекте, а в случае using - выдаст ошибку компиляции.

Короче говоря, если мало что из выше сказанного понятно, делайте так, как советовал в первом посте.

Добавлено через 4 минуты
C#
1
2
3
4
                if (temp != null)
                {
                    temp.Dispose();
                }
Если temp не равен null, то память освобождена не будет, а если равен, тогда что освобождать, если temp указывает на пустое место?
Вот это и генерирует предупреждение.
Не хотите юзать using - уберите if вообще оставьте в finaly только temp.Dispose(); .
1
0 / 0 / 0
Регистрация: 04.09.2013
Сообщений: 3
05.09.2013, 12:18  [ТС]
Спасибо. Я все понимаю, просто немного запутался А сейчас вот перечитал все еще раз и разложил себе по полочкам.
0
Master of Orion
Эксперт .NET
 Аватар для Psilon
6102 / 4958 / 905
Регистрация: 10.07.2011
Сообщений: 14,522
Записей в блоге: 5
05.09.2013, 23:06
Nickopoll, он же этот BitMap возвращает. Если он его будет диспозить, то функция всегда будет возвращать битую ссылку. Так что на этот ворнинг можно забить, если все будет нормально, то нормальный битмап уйдет наружу, если же ненормально, то память не выделится и беспокоиться не о чем.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
inter-admin
Эксперт
29715 / 6470 / 2152
Регистрация: 06.03.2009
Сообщений: 28,500
Блог
05.09.2013, 23:06
Помогаю со студенческими работами здесь

Можно ли в yii соединить (смержить) scope текущего класса со scope класса родителя?
Добрый день, Например есть: class ActiveRecord extends CActiveRecord { ... return , ];

'arr' was not declared in this scope 'sum' was not declared in this scope такие вот ошибки.(
#include &lt;iostream&gt; #include &lt;cstdlib&gt; #include &lt;ctime&gt; using namespace std; int main () { setlocale(LC_ALL, &quot;rus&quot;); ...

Linq to objects
Даны последовательности A и C, включающие следующие поля: A: &lt;Код потребителя&gt; &lt;Улица проживания&gt; &lt;Год рождения&gt; C:...

Shared Objects
подскажите пожалуйста как в Shared Objects (.so) прописать её версию, и как грамотнее её создать если несколько заголовков и исполняемых...

Invalid objects.
Всем привет. Очень нужно точное определение понятия &quot;инвалидные объекты&quot;. В инете найти описание и определение не удалось,в том числе...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
6
Ответ Создать тему
Новые блоги и статьи
Автоматическое создание документа при проведении другого документа
Maks 29.03.2026
Реализация из решения ниже выполнена на нетиповых документах, разработанных в конфигурации КА2. Есть нетиповой документ "ЗаявкаНаРемонтСпецтехники" и нетиповой документ "ПланированиеСпецтехники". В. . .
Настройка движения справочника по регистру сведений
Maks 29.03.2026
Решение ниже реализовано на примере нетипового справочника "ТарифыМобильнойСвязи" разработанного в конфигурации КА2, с целью учета корпоративной мобильной связи в коммерческом предприятии. . . .
Автозаполнение реквизита при выборе элемента справочника
Maks 27.03.2026
Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. При выборе "Спецтехники" (Тип Справочник. Спецтехника), заполняется. . .
Сумматор с применением элементов трёх состояний.
Hrethgir 26.03.2026
Тут. https:/ / fips. ru/ EGD/ ab3c85c8-836d-4866-871b-c2f0c5d77fbc Первый документ красиво выглядит, но без схемы. Это конечно не даёт никаких плюсов автору, но тем не менее. . . всё может быть. . .
Автозаполнение реквизитов при создании документа
Maks 26.03.2026
Программный код из решения ниже размещается в модуле объекта документа, в процедуре "ПриСозданииНаСервере". Алгоритм проверки заполнения реализован для исключения перезаписи значения реквизита,. . .
Команды формы и диалоговое окно
Maks 26.03.2026
1. Команда формы "ЗаполнитьЗапчасти". Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. В качестве источника данных. . .
Кому нужен AOT?
DevAlt 26.03.2026
Решил сделать простой ланчер Написал заготовку: dotnet new console --aot -o UrlHandler var items = args. Split(":"); var tag = items; var id = items; var executable = args;. . .
Отправка уведомления на почту при создании или изменении элементов справочника
Maks 24.03.2026
Программная отправка письма электронной почты на примере типового справочника "Склады" в конфигурации БП3. Перед реализацией необходимо выполнить настройку системной учетной записи электронной. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru