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

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

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

Author24 — интернет-сервис помощи студентам
Привет
Столкнулся со следующей проблемой. Написал простенькое приложение: читаем 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
Ответы с готовыми решениями:

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

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

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

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

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

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

Цитата Сообщение от rigorMortis Посмотреть сообщение
Не из-за этого могут быть мои проблемы?
Могут быть из-за этого.
0
7803 / 6567 / 2988
Регистрация: 14.04.2014
Сообщений: 28,705
14.03.2015, 19:25 3
Надо освобождать.
0
0 / 0 / 0
Регистрация: 01.05.2014
Сообщений: 74
Записей в блоге: 1
14.03.2015, 21:32  [ТС] 4
Но тогда возникают вопросы - когда освобождать? Проследить где в коде последний раз используется указатель на эту память -str - и вызывать free()? Но код может может дорабатываться и переменная после удаления может начать использоваться. В сложном коде это будет чрезвычайно сложно проследить. И потом функция может использоваться другим разработчиком который может не иметь понятия о том что внутри неё. Реализовывать собственный сборщик мусора наподобие .NET?
Как принято делать?
0
Неэпический
18112 / 10699 / 2062
Регистрация: 27.09.2012
Сообщений: 26,945
Записей в блоге: 1
14.03.2015, 21:51 5
Цитата Сообщение от rigorMortis Посмотреть сообщение
Реализовывать собственный сборщик мусора наподобие .NET?
Идиома RAII. Используйте умные указатели.
0
37 / 37 / 18
Регистрация: 15.05.2013
Сообщений: 236
14.03.2015, 22:16 6
rigirMortis, тут как бы ветка С++, а ты кидаешь код на С.
В этой задачи можно обойтись без ручного выделения памяти.
Скинь весь код. Посмотрим.
0
0 / 0 / 0
Регистрация: 01.05.2014
Сообщений: 74
Записей в блоге: 1
15.03.2015, 01:51  [ТС] 7
Нашёл в общем. Там в цикле вызывается такая функция:

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
19409 / 10028 / 2443
Регистрация: 30.01.2014
Сообщений: 17,678
15.03.2015, 04:34 8
Цитата Сообщение от 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  [ТС] 9
Да. Оставил на ночь работать. Потребляемая память выросла с 760 кб до 6786 кб.

Добавлено через 5 часов 23 минуты
Цитата Сообщение от DrOffset Посмотреть сообщение
В данном случае уместнее всего просто заменить голый wchar_t* на std::wstring.
А проблем не будет с переводом из std::wstring в WCHAR? У меня там вызовы winapi, а там многие методы WCHAR принимают или LPWSTR.
0
19409 / 10028 / 2443
Регистрация: 30.01.2014
Сообщений: 17,678
15.03.2015, 15:42 10
Цитата Сообщение от 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  [ТС] 11
Спасибо. А такой вопрос, приложение, работающее на уровне ядра проблем со string иметь не будет?
0
19409 / 10028 / 2443
Регистрация: 30.01.2014
Сообщений: 17,678
15.03.2015, 19:19 12
Цитата Сообщение от rigorMortis Посмотреть сообщение
А такой вопрос, приложение, работающее на уровне ядра проблем со string иметь не будет?
Будет из-за аллокатора (но если его поменять на другой, то нет). Но и с обычным malloc тоже будут проблемы. На уровне ядра для распределения памяти нужно использовать соответствующие средства (читай доки к Windows DDK), средства из userspace не подойдут.
0
15.03.2015, 19:19
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
15.03.2015, 19:19
Помогаю со студенческими работами здесь

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

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

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

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


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

Или воспользуйтесь поиском по форуму:
12
Ответ Создать тему
Новые блоги и статьи
Как написать микросервис на Go/Golang
InfoMaster 14.01.2025
Определение микросервиса, преимущества использования Go/ Golang Микросервис – это архитектурный подход к разработке программного обеспечения, при котором приложение состоит из небольших, независимо. . .
Как написать микросервис с нуля на C#
InfoMaster 14.01.2025
В современном мире разработки программного обеспечения микросервисная архитектура стала стандартом де-факто для создания масштабируемых и гибких приложений. Этот архитектурный подход предполагает. . .
Как создать интернет-магазин на PHP и JavaScript
InfoMaster 14.01.2025
В современном мире электронная коммерция стала неотъемлемой частью бизнеса. Создание собственного интернет-магазина открывает широкие возможности для предпринимателей, позволяя достичь большей. . .
Как написать Тетрис на Ассемблере
InfoMaster 14.01.2025
Тетрис – одна из самых узнаваемых и популярных компьютерных игр, созданная в 1984 году советским программистом Алексеем Пажитновым. За прошедшие десятилетия она завоевала симпатии миллионы людей по. . .
Как создать игру "Танчики" на Unity3d и C#
InfoMaster 14.01.2025
Разработка игр – это увлекательный процесс, сочетающий в себе творчество и технические навыки. В этой статье мы рассмотрим создание классической игры "Танчики" с использованием Unity3D и языка. . .
Организую платный онлайн микро-курс по доработке Android-клиента Telegram
_Ivana 14.01.2025
Официальная версия и распространенные форки не полностью устраивают? Сделай свою кастомную версию клиента! 4 занятия по 2 часа (2 недели пн, ср 19:00-21:00 по Москве). Первое вводное занятие. . .
Как создать приложение для фитнеса для iOS/iPhone на Kotlin
InfoMaster 14.01.2025
Создание собственного фитнес-приложения — это не только захватывающий, но и полезный процесс, ведь оно может стать вашим верным помощником на пути к здоровому и активному образу жизни. В современных. . .
Как создать приложение магазина для iOS/iPhone на Swift
InfoMaster 14.01.2025
Введение в разработку iOS-приложений Разработка приложений для iPhone и других устройств на базе iOS открывает огромные возможности для создания инновационных мобильных решений. В данной статье мы. . .
Это работает. Скорость асинхронной логики велика. Вопрос видимо останется в стабильности. Плата - огонь!
Hrethgir 13.01.2025
По прошлому проекту в Logisim Evolution https:/ / www. cyberforum. ru/ blogs/ 223907/ blog8781. html прилагаю файл архива проекта в Gowin Eda. Восьмибитный счётчик из сумматора+ генератор сигнала. . .
UserScript для подсветки кнопок языков программировани­­­­я в зависимости от текущего раздела
volvo 13.01.2025
В результате работы этого скрипта подсвечиваются нужные кнопки не только в форме быстрого ответа, но и при редактировании сообщения: / / ==UserScript== / / @name CF_DefaultLangSelect / / . . .
Введение в модели и алгоритмы машинного обучения
InfoMaster 12.01.2025
Машинное обучение представляет собой одну из наиболее динамично развивающихся областей искусственного интеллекта, которая фокусируется на разработке алгоритмов и методов, позволяющих компьютерам. . .
Как на Python создать нейросеть для решения задач
InfoMaster 12.01.2025
В контексте стремительного развития современных технологий особое внимание уделяется таким инструментам, как нейросети. Эти структуры, вдохновленные биологическими нейронными сетями, используются для. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru