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

@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 не проверив.

Новые блоги и статьи Все статьи Все блоги /
Access VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .	Фото: Daniel Greenwood kumehtar 13.11.2025

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

Решение

Решение

@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 Супер-модератор 33187 / 21484 / 8232 Регистрация: 22.10.2011 Сообщений: 36,861 Записей в блоге: 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