Утечка памяти при вызове функции

@QVO · Регистрация: 26.10.2010

Студворк — интернет-сервис помощи студентам

Здравствуйте.
Объясните пожалуйста почему возникает утечка памяти при вызове функции get_msg_error.

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
    int get_msg_error( const UnicodeString &Source )
    {
        TJSONObject* JSONObject;
        TJSONObject* msg_object;
        TJSONArray* msg;
 
        TJSONValue* JSONValue = TJSONObject::ParseJSONValue( Source );
        if ( JSONValue != NULL )
        {
            JSONObject = static_cast<TJSONObject*>( JSONValue );
            TJSONPair* JSONPair = JSONObject->Get("msg");
            if( JSONPair != NULL )
            {
                msg = static_cast<TJSONArray*>( JSONPair->JsonValue );
                JSONValue = msg->Get(0);
                if ( JSONValue != NULL )
                {
                    msg_object = static_cast<TJSONObject*>( JSONPair->JsonValue );
                    return msg_object->Get("id")->JsonValue->Value().ToInt();
                }else
                    return -3;
            }else
                return -2;
        }else
            return -1;
    }

C++
1
2
3
4
5
6
7
8
9
10
    for ( unsigned int i = 0; i < 100; ++i )
    {
        String response = IdHTTP1->Post("http://server.com/", PostData);
        
        if ( get_msg_error(response) > 0 )
        {
            ListBox1->Items->Add( "["+TimeToStr(Time())+"] Ошибка сервера: " + IntToStr(get_msg_error(response)));
            return;
        }
    }

@BRcr · 07.05.2014, 18:02

Я еще в прошлый раз, когда обсуждали парсинг JSON, все криво смотрел на получение указателей на объекты JSON, так как не мог понять механику управления памятью. В справке глядел, там про очистку памяти ни слова не нашел. Зато встречал примеры парсинга, где первый из созданных объектов( здесь это TJSONValue* JSONValue = TJSONObject::ParseJSONValue( Source ); ) в итоге явно освобождался оператором delete.
Сдается мне, первый из объектов сам разрушает все, что создает в процессе работы с ним, а вот его самого надо освобождать вручную - это было бы логично, по крайней мере. Однако, не уверен.

@QVO · 07.05.2014, 18:07 **[ТС]**

BRcr,

C++
1
2
3
4
5
    // ...
    msg_object = static_cast<TJSONObject*>( JSONPair->JsonValue );
    delete JSONValue;
    return msg_object->Get("id")->JsonValue->Value().ToInt();
    // ...

не помогло. Утечка серьезная происходит за пол часа до 1гб ОЗУ занимает.
Я думаю, может не верно передаю строку в функцию?

@volvo · 07.05.2014, 18:21

JSONValue -то при чем? Удалять надо тот указатель, что возвращается из TJSONObject::ParseJSONValue (а не то, что ты уже потом получаешь из Get(0)), сделай для него отдельную переменную, и потом, перед выходом, удали...

@QVO · 07.05.2014, 18:22 **[ТС]**

UI, попробовал утечка осталась.

@BRcr · 07.05.2014, 18:30

Черт его знает, может, тогда уже все получаемые указатели уничтожать вручную? Звучит довольно бредово.

Что это вообще за подход, когда статик-методы возвращают указатели на неизвестно как и где создаваемые объекты? Самое свинство - в справке это даже не упомянуто...

@QVO · 07.05.2014, 18:32 **[ТС]**

Моя вина, сервер ошибки не возвращает нужно было после else добавить освобождение памяти.

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
31
32
33
34
35
36
37
38
39
40
    int get_msg_error( const UnicodeString &Source )
    {
        TJSONObject* JSONObject;
        TJSONObject* msg_object;
        TJSONArray* msg;
 
        TJSONValue* JSONValue = TJSONObject::ParseJSONValue( Source );
        if ( JSONValue != NULL )
        {
            JSONObject = static_cast<TJSONObject*>( JSONValue );
            TJSONPair* JSONPair = JSONObject->Get("msg");
            if( JSONPair != NULL )
            {
                msg = static_cast<TJSONArray*>( JSONPair->JsonValue );
                TJSONValue* JSONValue2 = msg->Get(0);
                if ( JSONValue2 != NULL )
                {
                    msg_object = static_cast<TJSONObject*>( JSONValue2 );
                    delete JSONValue;
                    JSONValue = NULL;
                    return msg_object->Get("id")->JsonValue->Value().ToInt();
                }else
                {
                    delete JSONValue;
                    JSONValue = NULL;
                    return -3;
                }
            }else
            {
                delete JSONValue;
                JSONValue = NULL;
                return -2;
            }
        }else
        {
            delete JSONValue;
            JSONValue = NULL;
            return -1;
        }
    }

@Maluda · 07.05.2014, 20:35

QVO, у тебя наглядный пример того как не надо делать. В таких случаях просто обязательно необходимо использовать smart ptr

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
#include <memory>
 
int get_msg_error(const UnicodeString &Source) {
    std::auto_ptr<TJSONValue>AJSONValuePtr(TJSONObject::ParseJSONValue(Source));
    if (AJSONValuePtr.get()) {
        TJSONObject* AJSONObject = static_cast<TJSONObject*>(AJSONValuePtr);
        TJSONPair* AJSONPair = AJSONObject->Get("msg");
        if (AJSONPair) {
            TJSONArray* AJSONArrayMsg = static_cast<TJSONArray*>(AJSONPair->JsonValue);
            TJSONValue* AJSONValue2 = AJSONArrayMsg->Get(0);
            if (AJSONValue2) {
                TJSONObject *AJSONObjectMsg = static_cast<TJSONObject*>(AJSONValue2);
                return AJSONObjectMsg->Get("id")->JsonValue->Value().ToInt();
            }
            else
                return -3;
        }
        else
            return -2;
    }
    else
        return -1;
}

Добавлено через 2 минуты
П.С. К тому же абсолютно нелогичная функция, которая смешивает значения парсинга и возможной ошибки. По-моему необходимо разделить эти вещи

@volvo · 07.05.2014, 20:38

Опять же, непонятно, зачем вызывать get_msg_error(response) дважды, а нельзя запомнить значение, возвращаемое им, и использовать его потом?

C++
1
2
3
4
5
        if ( (res = get_msg_error(response)) > 0 )
        {
            ListBox1->Items->Add( "["+TimeToStr(Time())+"] Ошибка сервера: " + IntToStr(res));
            return;
        }

@Maluda · 07.05.2014, 20:45

Предлагаю примерно вот так

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
31
32
33
34
35
36
int get_parse_result(const UnicodeString &Source) {
    std::auto_ptr<TJSONValue>AJSONValuePtr(TJSONObject::ParseJSONValue(Source));
    if (!AJSONValuePtr.get())
        throw Exception("Can not parse source <" + Source + ">");
 
    TJSONObject* AJSONObject = dynamic_cast<TJSONObject*>(AJSONValuePtr);
    TJSONPair* AJSONPair = AJSONObject->Get("msg");
    if (!AJSONPair)
        throw Exception("Node <msg> is not found!");
 
    TJSONArray* AJSONArrayMsg = static_cast<TJSONArray*>(AJSONPair->JsonValue);
    TJSONValue* AJSONValue2 = AJSONArrayMsg->Get(0);
    if (!AJSONValue2)
        throw Exception("Node with id <0> is not found");
 
    TJSONObject *AJSONObjectMsg = static_cast<TJSONObject*>(AJSONValue2);
    return AJSONObjectMsg->Get("id")->JsonValue->Value().ToInt();
}
 
//---------------------------------------------------------------------------
void __fastcall TFormMain::FormClick(TObject *Sender) {
 
    UnicodeString sJSONScript = /* здесь данные скрипта */;
 
    int i_parse_result = -1;
    try {
        i_parse_result = get_parse_result(sJSONScript);
    }
    catch(Exception * E) {
        LogError(E->Message);
    }
 
    if (i_parse_result != -1) {
        // работаем дальше с i_parse_result
    }
}

@QVO · 07.05.2014, 21:21 **[ТС]**

UI, это все крутится в таймере нужно быть уверенным, что ошибку не вернул.

Добавлено через 5 минут
Maluda, жаль раньше твоей реализации не увидел, в куче таких же функций искал утечку памяти методом тыка.

@Avazart · 07.05.2014, 21:36

C++
1
2
std::auto_ptr<TJSONValue>AJSONValuePtr(TJSONObject::ParseJSONValue(Source));
if (AJSONValuePtr)

Не хорошо, вероятно лучше воспользоваться __finally либо передавать указатель после его проверки.

@Maluda · 08.05.2014, 09:51

Сообщение от Avazart

Не хорошо

Обоснуй, в чем же нехорошо

@Maluda · 08.05.2014, 09:58

Avazart, вот как это описано в Руководстве разработчика C++ Builder.
Прочитай последний абзац

@Maluda · 08.05.2014, 10:02

И самое главное, обрати внимание, как я проверяю

Сообщение от Maluda

if (AJSONValuePtr.get()) {

а не так

Сообщение от Avazart

if (AJSONValuePtr)

@QVO · 08.05.2014, 11:40 **[ТС]**

Maluda, почему ты используешь в некоторых местах dynamic_cast, а в других static_cast? Что лучше использовать в данном случае?

@QVO · 08.05.2014, 12:03 **[ТС]**

Maluda, попробовал вариант с этого поста

JavaScript
1
{"msg":[{"id":"107"}]}

Выдает ошибку на изображение.

@BRcr · 08.05.2014, 12:57

C++
1
TJSONObject* AJSONObject = dynamic_cast< TJSONObject * >( AJSONValuePtr.get( ) );

@QVO · 08.05.2014, 13:04 **[ТС]**

BRcr, а по-другому никак. Такая запись лишь позволить скопилировать проект.

@Avazart · 08.05.2014, 13:05

QVO, А почитать :
http://ru.wikipedia.org/wiki/Dynamic_cast
http://ru.wikipedia.org/wiki/Static_cast

Сообщение от Maluda

Avazart, вот как это описано в Руководстве разработчика C++ Builder.
Прочитай последний абзац

А что там читать ?
Я про то что нет смысла создавать auto_ptr<> и передавать в него NULL не проверив.

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Программа принимает математическое выражение в виде строки и выдаёт его производную в виде строки и вычисляет значение производной при заданном х Логарифм записывается как: (x-2)log(x^2+2) -. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .

Утечка памяти при вызове функции

Решение

Решение

@BRcr 4043 / 2333 / 292 Регистрация: 03.02.2011 Сообщений: 5,066 Записей в блоге: 10
	07.05.2014, 18:02
	Сообщение было отмечено QVO как решение Решение Я еще в прошлый раз, когда обсуждали парсинг JSON, все криво смотрел на получение указателей на объекты JSON, так как не мог понять механику управления памятью. В справке глядел, там про очистку памяти ни слова не нашел. Зато встречал примеры парсинга, где первый из созданных объектов( здесь это *TJSONValue JSONValue = TJSONObject::ParseJSONValue( Source );** ) в итоге явно освобождался оператором delete. Сдается мне, первый из объектов сам разрушает все, что создает в процессе работы с ним, а вот его самого надо освобождать вручную - это было бы логично, по крайней мере. Однако, не уверен. 1

@volvo Супер-модератор 33393 / 21503 / 8236 Регистрация: 22.10.2011 Сообщений: 36,899 Записей в блоге: 12
	07.05.2014, 18:21
	JSONValue -то при чем? Удалять надо тот указатель, что возвращается из TJSONObject::ParseJSONValue (а не то, что ты уже потом получаешь из Get(0)), сделай для него отдельную переменную, и потом, перед выходом, удали... 1

@QVO 652 / 462 / 80 Регистрация: 26.10.2010 Сообщений: 1,263 Записей в блоге: 4
	07.05.2014, 18:22 [ТС]
	UI, попробовал утечка осталась. 0

@BRcr 4043 / 2333 / 292 Регистрация: 03.02.2011 Сообщений: 5,066 Записей в блоге: 10
	07.05.2014, 18:30
	Черт его знает, может, тогда уже все получаемые указатели уничтожать вручную? Звучит довольно бредово. Что это вообще за подход, когда статик-методы возвращают указатели на неизвестно как и где создаваемые объекты? Самое свинство - в справке это даже не упомянуто... 1

@QVO 652 / 462 / 80 Регистрация: 26.10.2010 Сообщений: 1,263 Записей в блоге: 4
	07.05.2014, 21:21 [ТС]
	UI, это все крутится в таймере нужно быть уверенным, что ошибку не вернул. Добавлено через 5 минут Maluda, жаль раньше твоей реализации не увидел, в куче таких же функций искал утечку памяти методом тыка. 0

@Maluda 1280 / 598 / 116 Регистрация: 18.08.2009 Сообщений: 832
	08.05.2014, 09:58
	Avazart, вот как это описано в Руководстве разработчика C++ Builder. Прочитай последний абзац Миниатюры 0

@QVO 652 / 462 / 80 Регистрация: 26.10.2010 Сообщений: 1,263 Записей в блоге: 4
	08.05.2014, 11:40 [ТС]
	Maluda, почему ты используешь в некоторых местах dynamic_cast, а в других static_cast? Что лучше использовать в данном случае? 0

@QVO 652 / 462 / 80 Регистрация: 26.10.2010 Сообщений: 1,263 Записей в блоге: 4
	08.05.2014, 13:04 [ТС]
	BRcr, а по-другому никак. Такая запись лишь позволить скопилировать проект. 0