Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.62/13: Рейтинг темы: голосов - 13, средняя оценка - 4.62
0 / 0 / 0
Регистрация: 01.05.2014
Сообщений: 74
Записей в блоге: 1

Выделение и освобождение памяти для локальных переменных функций

14.03.2015, 19:16. Показов 2689. Ответов 11
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Привет
Столкнулся со следующей проблемой. Написал простенькое приложение: читаем 1 строку из файла на диске, записываем 1 строку в файл на диске. Операция повторяется в цикле каждые 5 сек. Так вот, через несколько циклов чтения/записи приложение повисает (в файл ничего не пишется, логгирование в консоли прекращается). Диспетчер задач винды показывает что моё приложение начинает потреблять 25% процессорного времени. Это воспроизводится если запускать exe-файл отдельно. Запуск приложения в VisualStudio таких проблем не проявляет. А так же не проявляет проблем с памятью, которые обнаружились при запуске exe-файла отдельно.
Может я чего с памятью не так сделал?
В моём приложении есть несколько функций наподобие

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
wchar_t* read_string()
{
    int count = 0;
    char *str = NULL;
    FILE* file = _wfopen(Const::File(), L"r");
 
    if (file != NULL)
    {
        str = (char*)malloc(readBuff);
 
        if (fgets(str, readBuff, file) != NULL)
        {
            ++count;
            while (fgets(str, readBuff, file) != NULL)
                ++count;
 
            if (count <= _previousLinesCount)
                str = NULL;
            else
                _previousLinesCount = count;
        }
        else
            str = NULL;
 
        fclose(file);
    }
    else
    {
        FILE* newFile = _wfopen(Const::CommandFile(), L"w");
        fclose(newFile);
    }
 
    return  TString::Encode(str, CP_UTF8);
}

Вот такое str = (char*)malloc(readBuff); потом где-то надо вручную освобождать? Каждый очередной цикл будет выделяться новая память для этой переменной? Не из-за этого могут быть мои проблемы? Странно, что при отладке в VS всё отлично работает.
0
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
14.03.2015, 19:16
Ответы с готовыми решениями:

Как правильно записать в виде функции выделение памяти для двумерного массива и ее освобождение
Здравствуйте! Подскажите, как правильно записать в виде функции выделение памяти для двумерного массива и ее освобождение. ...

Выделение и освобождение памяти
Доброго времени суток. Пишу класс по ООП, и назрел один вопрос, связанный с моментом выделения и освобождения памяти. class SOK { ...

Выделение и освобождение памяти
Приветствую всех, Возник вопрост при работе с выделением памяти. Есть клас, в котором рисуется некий элемент(Фрейм), в дети к этому...

11
Форумчанин
Эксперт CЭксперт С++
 Аватар для MrGluck
8216 / 5047 / 1437
Регистрация: 29.11.2010
Сообщений: 13,453
14.03.2015, 19:24
Цитата Сообщение от rigorMortis Посмотреть сообщение
Вот такое str = (char*)malloc(readBuff); потом где-то надо вручную освобождать?
Да. free(str)

Цитата Сообщение от rigorMortis Посмотреть сообщение
Каждый очередной цикл будет выделяться новая память для этой переменной?
Да

Цитата Сообщение от rigorMortis Посмотреть сообщение
Не из-за этого могут быть мои проблемы?
Могут быть из-за этого.
0
7804 / 6568 / 2988
Регистрация: 14.04.2014
Сообщений: 28,705
14.03.2015, 19:25
Надо освобождать.
0
0 / 0 / 0
Регистрация: 01.05.2014
Сообщений: 74
Записей в блоге: 1
14.03.2015, 21:32  [ТС]
Но тогда возникают вопросы - когда освобождать? Проследить где в коде последний раз используется указатель на эту память -str - и вызывать free()? Но код может может дорабатываться и переменная после удаления может начать использоваться. В сложном коде это будет чрезвычайно сложно проследить. И потом функция может использоваться другим разработчиком который может не иметь понятия о том что внутри неё. Реализовывать собственный сборщик мусора наподобие .NET?
Как принято делать?
0
Неэпический
 Аватар для Croessmah
18144 / 10728 / 2066
Регистрация: 27.09.2012
Сообщений: 27,026
Записей в блоге: 1
14.03.2015, 21:51
Цитата Сообщение от rigorMortis Посмотреть сообщение
Реализовывать собственный сборщик мусора наподобие .NET?
Идиома RAII. Используйте умные указатели.
0
 Аватар для Kant
37 / 37 / 18
Регистрация: 15.05.2013
Сообщений: 236
14.03.2015, 22:16
rigirMortis, тут как бы ветка С++, а ты кидаешь код на С.
В этой задачи можно обойтись без ручного выделения памяти.
Скинь весь код. Посмотрим.
0
0 / 0 / 0
Регистрация: 01.05.2014
Сообщений: 74
Записей в блоге: 1
15.03.2015, 01:51  [ТС]
Нашёл в общем. Там в цикле вызывается такая функция:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
wchar_t* TString::concat(wchar_t **s, int n)
{
    int length = 0;
    for (int i = 0; i < n; i++)
        length += wcslen(s[i]);
 
    length++;
    wchar_t *string = (wchar_t *)malloc(length);
    wcscpy(string, L"");
 
    for (int i = 0; i < n; i++) 
        wcscat(string, s[i]);
 
    return string;
}
примерно так:

C++
1
2
3
    
     wchar_t* strArr[] = { lt, L":\\", Mark, L"_", FILE };
     return TString::concat(strArr, 5);
После того как поменял на malloc(sizeof(wchar_t)*length) проблема исчезла. Непонятно только почему приложение повисало ровно на 10й вызов и почему в VS ничего подобного не было. И в catch не попало. Такие фокусы вводят в заблуждение.
0
19491 / 10097 / 2460
Регистрация: 30.01.2014
Сообщений: 17,805
15.03.2015, 04:34
Цитата Сообщение от rigorMortis Посмотреть сообщение
Непонятно только почему приложение повисало ровно на 10й вызов и почему в VS ничего подобного не было.
UB. Длина в символах была, а malloc принимает количество байт.
Цитата Сообщение от rigorMortis Посмотреть сообщение
И в catch не попало.
И не должно было.
Цитата Сообщение от rigorMortis Посмотреть сообщение
Нашёл в общем.
Память все равно нужно освобождать. Проблему ты решил только одну, а их там много, судя по всему.
Цитата Сообщение от rigorMortis Посмотреть сообщение
В сложном коде это будет чрезвычайно сложно проследить. И потом функция может использоваться другим разработчиком который может не иметь понятия о том что внутри неё. Реализовывать собственный сборщик мусора наподобие .NET?
Как принято делать?
Реализовать для своих ресурсов (в данном случае память) необходимые политики владения. Ресурс без владения не должен гулять в программе, чтобы его не ловить вручную потом. В частности, класс std::string уже реализует всю необходимую механику выделения, освобождения памяти, т.е. владеет ресурсом памяти для строки. В данном случае уместнее всего просто заменить голый wchar_t* на std::wstring. А вообще да, читай про RAII, умные указатели.
0
0 / 0 / 0
Регистрация: 01.05.2014
Сообщений: 74
Записей в блоге: 1
15.03.2015, 15:26  [ТС]
Да. Оставил на ночь работать. Потребляемая память выросла с 760 кб до 6786 кб.

Добавлено через 5 часов 23 минуты
Цитата Сообщение от DrOffset Посмотреть сообщение
В данном случае уместнее всего просто заменить голый wchar_t* на std::wstring.
А проблем не будет с переводом из std::wstring в WCHAR? У меня там вызовы winapi, а там многие методы WCHAR принимают или LPWSTR.
0
19491 / 10097 / 2460
Регистрация: 30.01.2014
Сообщений: 17,805
15.03.2015, 15:42
Цитата Сообщение от rigorMortis Посмотреть сообщение
а там многие методы WCHAR принимают или LPWSTR.
У std::wstring есть метод c_str(), который возвращает const wchar_t *. Для передачи в сторонние сишные функции (в данном случае WinAPI) вполне подойдет. Изучи вот эту страницу.
0
0 / 0 / 0
Регистрация: 01.05.2014
Сообщений: 74
Записей в блоге: 1
15.03.2015, 19:08  [ТС]
Спасибо. А такой вопрос, приложение, работающее на уровне ядра проблем со string иметь не будет?
0
19491 / 10097 / 2460
Регистрация: 30.01.2014
Сообщений: 17,805
15.03.2015, 19:19
Цитата Сообщение от rigorMortis Посмотреть сообщение
А такой вопрос, приложение, работающее на уровне ядра проблем со string иметь не будет?
Будет из-за аллокатора (но если его поменять на другой, то нет). Но и с обычным malloc тоже будут проблемы. На уровне ядра для распределения памяти нужно использовать соответствующие средства (читай доки к Windows DDK), средства из userspace не подойдут.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
15.03.2015, 19:19
Помогаю со студенческими работами здесь

Выделение и освобождение памяти в c++
Не давно наткнулся на такую замечательность : system(&quot;PAUSE&quot;);//1 раз int *p = new int; // память system(&quot;PAUSE&quot;);//2...

Выделение/освобождение памяти
помогите пожалуйста, в С++ есть конструкции вида xx=AllocMemInt2D(LL, NN); DeAllocMemFloat2D(LL, x); которые выделяют и освобождают...

Освобождение и выделение памяти
Помогите разобраться с программой, где здесь выделение памяти (как я понял это new) и где освобождение памяти. #include...

Выделение/освобождение памяти
По заданию делаю библиотеку &quot;Хеш-таблица бинарных данных BLOB с цепочками&quot; на языке C. Написал. Отладил. Проследил, чтобы не было memory...

Выделение и освобождение памяти
Здравствуйте, помогите пожалуйста разобраться. Скажите является ли следующий код правильным в отношении перевыделения памяти, то есть не...


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

Или воспользуйтесь поиском по форуму:
12
Ответ Создать тему
Новые блоги и статьи
PhpStorm 2025.3: WSL Terminal всегда стартует в ~
and_y87 14.12.2025
PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .
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 . Быстренько разберем подход "на фреймах". Мы делаем одну. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru