Форум программистов, компьютерный форум, киберфорум
Наши страницы

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
Van111
209 / 188 / 4
Регистрация: 03.08.2011
Сообщений: 2,598
Записей в блоге: 12
#1

Free - не удаляется память - C++

29.03.2014, 14:51. Просмотров 376. Ответов 10
Метки нет (Все метки)

C++
1
free(Items[NumberItemDelete].info);//для последнего элемента память не уудаляется - костыль!!!!!!!!!!!!!!!
делал лабу на СИ, пришлось соответственно использовать malloc И free
Items - глобальный массив структур, каждая структура имеет поле char *info, для всех его info - free срабатывает как надо. Для последнего элемента выкидывает исключение а потом и ошибку, память под этот элемент точно выделена, NumberItemDelete точно не выходит за границу !
MVS 2012
в чём проблема? За ранее спасибо.
0
Лучшие ответы (1)
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
29.03.2014, 14:51
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Free - не удаляется память (C++):

И снова динамическая память, malloc и free в конструкторе и деструкторе - C++
Короче есть, класс, который например хранит строку, я его храню в структуре, после я выдиляю динамическу память пот эту структуру и копирую...

Статическая память,Динамическая память. - C++
a) Статическая память. Двумерный массив. Дан массив целых чисел. В массиве есть отрицательные числа. Определить координаты левого нижнего...

Почему удаляется объект? - C++
Помогите, пожалуйста, понять почему удаляется объект и как это исправить? Идея следующая: В Source.cpp я создаю два объекта, вызывая...

Не удаляется 2-х связный список - C++
Работает все, но сам список не удаляется. Помогите найти проблему. #include <iostream> using namespace std; struct Node { ...

не удаляется элемент из массива - C++
Всем привет. Есть 2 ф-ции одна добавляет в конец массива новый елемент(работает нормально), а вторая должна удалять из массива любой...

free - C++
Выделил память под массив(строку), размером 5 элементов. char *str; str = (char*)calloc(5,sizeof(char)); if (str == NULL) { ...

10
Croessmah
Ушел
Эксперт CЭксперт С++
13554 / 7705 / 872
Регистрация: 27.09.2012
Сообщений: 19,006
Записей в блоге: 3
Завершенные тесты: 1
29.03.2014, 15:59 #2
Цитата Сообщение от Van111 Посмотреть сообщение
память под этот элемент точно выделена, NumberItemDelete точно не выходит за границу !
ну тогда free скорее всего работает
Цитата Сообщение от Van111 Посмотреть сообщение
в чём проблема?
а мы откуда знаем что у Вас и как там?
1
Van111
209 / 188 / 4
Регистрация: 03.08.2011
Сообщений: 2,598
Записей в блоге: 12
30.03.2014, 22:35  [ТС] #3
Цитата Сообщение от Croessmah Посмотреть сообщение
а мы откуда знаем что у Вас и как там?
если я вам пришлю 378 строк плюс один бинарный файл данных - вы посмотрите ?
0
Croessmah
Ушел
Эксперт CЭксперт С++
13554 / 7705 / 872
Регистрация: 27.09.2012
Сообщений: 19,006
Записей в блоге: 3
Завершенные тесты: 1
30.03.2014, 22:36 #4
Цитата Сообщение от Van111 Посмотреть сообщение
если я вам пришлю 378 строк плюс один бинарный файл данных - вы посмотрите ?
Давайте, посмотрю
1
alsav22
5426 / 4821 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
30.03.2014, 22:52 #5
Цитата Сообщение от Van111 Посмотреть сообщение
в чём проблема?
С помощью отладчика пробовали выяснить?
1
Van111
209 / 188 / 4
Регистрация: 03.08.2011
Сообщений: 2,598
Записей в блоге: 12
30.03.2014, 23:02  [ТС] #6
в функции DeleteItem
0
Вложения
Тип файла: txt data.txt (32 байт, 3 просмотров)
Van111
209 / 188 / 4
Регистрация: 03.08.2011
Сообщений: 2,598
Записей в блоге: 12
30.03.2014, 23:19  [ТС] #7
data txt замените на data.bin

Добавлено через 4 минуты
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
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
#define _CRT_SECURE_NO_WARNINGS
#include <string.h>
#include <malloc.h>
#include <stdlib.h>
#include <stdio.h>
#include <locale.h>
//////////////////////////////////////////////////////////////////////////
#define ANSWER_EXIT 0
#define ANSWER_ADD 1
#define ANSWER_DELETE 2
#define ANSWER_PRINT 3
#define ANSWER_FIND 4
#define ANSWER_PRINT_MORE_ITEMS 5
#define ANSWER_EMPTY 6
//////////////////////////////////////////////////////////////////////////
#define true 1
#define false 0
#define bool int
#define MaxSizeItem  100 // максимальный размер списка
#define DefautDataSize  5 // количество начальных эллементов
#define MaxStringSizeData  100 // максимальная длинна строки начальных эллементов
#define FILE_NOT_NAME_SAVE ""
const int MaxFileName = 256;
const int MaxString = 2000; // максимальный размер буфера
//////////////////////////////////////////////////////////////////////////
char DataString[DefautDataSize][MaxStringSizeData] = {"значение1","значение2","значение3","значение4","значение5"};//информаия начальных эллементов
int DataKey[DefautDataSize] = {1,2,3,4,5};//ключи начальных эллементов
char *buf;//буфер
char *g_nameFileOpen;
//////////////////////////////////////////////////////////////////////////
 
 
 
struct Item
{
    int key;
    int length;
    int offset;
    char *info;
}; 
struct Item Items[MaxSizeItem];
int ItemSize;
//////////////////////////////////////////////////////////////////////////
bool Load(char *nameFileData_) //требуется для вычесления нового смещения
{
    int i;
    int size = -1;
 
    FILE * fileInit ;
    if ((fileInit = fopen(nameFileData_, "rb"))==NULL) {
        printf("He удается открыть файл.\n");
        return false;
    }
 
    fread(&size,sizeof(int),1,fileInit);
    if(size < 0)
    {
        ItemSize = 0;
        return true;
    }
    ItemSize = size;// размер списка
    //////////////////////////////////////////////////////////////////////////
    for(i = 0; i < ItemSize; i++)
    {
        fread(&Items[i].offset,sizeof(int),1,fileInit);
        fread(&Items[i].length,sizeof(int),1,fileInit);
        fread(&Items[i].key,sizeof(int),1,fileInit);
        Items[i].info = (char*)malloc(sizeof(char) * 7 );//память для строки
        fread(Items[i].info,sizeof(char),Items[i].length,fileInit);
        Items[i].info[Items[i].length] = 0;
    }
    //////////////////////////////////////////////////////////////////////////
    fclose(fileInit);
    return true;
}
 
void Save(char *nameFileSave)
{
    FILE *fileSave;
    int i ;
 
    fileSave = fopen(nameFileSave,"wb");
 
    fwrite(&ItemSize,sizeof(int),1,fileSave);
    for(i = 0; i < ItemSize; i++)
    {
        fwrite(&Items[i].offset,sizeof(int),1,fileSave);
        fwrite(&Items[i].length,sizeof(int),1,fileSave);
        fwrite(&Items[i].key,sizeof(int),1,fileSave);
        fwrite(Items[i].info,sizeof(char),Items[i].length,fileSave);
 
    }
    fclose(fileSave);
}
//////////////////////////////////////////////////////////////////////////
 
bool init(char *nameFileData_)
{
    
//////////////////////////////////////////////////////////////////////////
    
    buf = (char*)malloc(sizeof(char) * ( MaxString + 1) ); // временный буфер для считывания
    return Load(nameFileData_);
    
}
//////////////////////////////////////////////////////////////////////////
 
int FindOneKey(int key)// ищем ключ, возвращаем его порядковый номер в списке
{
    #define NO_ITEM -1
    int i;
    for(i = 0; i < ItemSize; i++)//обход списка
    {
        if (Items[i].key == key)
            return i;//порядковый номер
    }
    return  NO_ITEM;//неудача
};
//////////////////////////////////////////////////////////////////////////
 
bool CheckKey(int key)// проверяем ключ на существование
{
    int i;
    for(i = 0; i < ItemSize; i++)
    {
        if (Items[i].key == key)
            return true; // true , false и bool не существует в СИ, смотрите директивы
    }
    return false; // true , false и bool не существует в СИ, смотрите директивы
}
//////////////////////////////////////////////////////////////////////////
 
void AddItem()
{
    #define REENTERED 1
    int key;
    //////////////////////////////////////////////////////////////////////////
    if(ItemSize == MaxSizeItem)
    {
        puts("список заполнен");
        return;
    }
    puts("выберите ключ");
    while (!scanf("%i", &key))
    {
        scanf( "%*[^\n]" ); scanf( "%*c" );// не много магии
        puts("не корректный ввод, повторите операцию");
    }
    while(CheckKey(key))// проверяем ввод ключа
    {
        puts("такой ключ уже существует, 1 - для ввода нового ключа, любой другой символ для выхода ");
        if(!scanf("%i",&key)) // введенно не число
            return;
        if(key == REENTERED) //введенна единица
        {
            puts("выберите ключ");
            while (!scanf("%i", &key)) 
            {
                scanf( "%*[^\n]" ); scanf( "%*c" );// не много магии
                puts("не корректный ввод, повторите операцию");
            }   
        }
        else // введенна не единица
        {
            return;
        }
    }
    //////////////////////////////////////////////////////////////////////////
 
    printf("введите информацию, максимальнй размер %i символа/символов \r\n", MaxString );
        scanf( "%*[^\n]" ); scanf( "%*c" );//обязательно прочистим после printf
    gets(buf);// новая информация
    Items[ItemSize].key = key;
    Items[ItemSize].info = (char*)malloc(sizeof(char) * ( strlen(buf) + 1)); 
    Items[ItemSize].length = strlen(buf);
    memcpy(Items[ItemSize].info,buf,Items[ItemSize].length);
    Items[ItemSize].info[Items[ItemSize].length] = 0;
    if(!ItemSize)
    {
        Items[ItemSize].offset = sizeof(int);
    }
    else
    {
        Items[ItemSize].offset = Items[ItemSize-1].offset + sizeof(int) * 3 + Items[ItemSize - 1].length;;
    }
    ItemSize++;
    Save(g_nameFileOpen);
}
//////////////////////////////////////////////////////////////////////////
 
void DeleteItem()
{
    #define REENTERED 1
    #define NO_ITEM -1
    int key;
    int i;
    int NumberItemDelete;
    void *Dest_str; //куда копируем
    void * Source_str;//откуда копируем
    int SizeCopy ;//количество копируемых байт
    int del_offset;
    //////////////////////////////////////////////////////////////////////////
    puts("введите ключ");
    while (!scanf("%i", &key))
    {
        scanf( "%*[^\n]" ); scanf( "%*c" );// не много магии
        puts("не корректный ввод, повторите операцию");
    }
    //////////////////////////////////////////////////////////////////////////
    while(!CheckKey(key))
    {
        puts("такой ключ не существует, 1 - для ввода нового ключа, любой другой символ для выхода ");
        if(!scanf("%i",&key))
            return;
        if(key == REENTERED)
        {
            puts("введите ключ");
            while (!scanf("%i", &key))
            {
                scanf( "%*[^\n]" ); scanf( "%*c" );// не много магии
                puts("не корректный ввод, повторите операцию");
            }   
        }
        else
        {
            return;
        }
    }
    //////////////////////////////////////////////////////////////////////////
    NumberItemDelete = FindOneKey(key);//порядковый номер
    if(NumberItemDelete > NO_ITEM)
    {
        del_offset = sizeof(int) * 3 + Items[NumberItemDelete].length;;
        if(NumberItemDelete != ItemSize)
        {
            free(Items[NumberItemDelete].info);//для последнего элемента память не уудаляется - костыль!!!!!!!!!!!!!!!
            Dest_str = &Items[0] +  NumberItemDelete;
            Source_str = &Items[0] + NumberItemDelete+1; 
            SizeCopy = sizeof(struct Item) * (ItemSize - NumberItemDelete - 1);
            memcpy(Dest_str,Source_str,SizeCopy);//сдвигаем массив , альтернативой может быть запись последнего элемента на место удаляемого
        }
    }
    
    for(i = NumberItemDelete; i < ItemSize;i++)
    {
        Items[i].offset -= del_offset;
    }
    ItemSize--;//уменьшаем размер списка
    //Save(g_nameFileOpen);
 
}
//////////////////////////////////////////////////////////////////////////
 
void PrintItems(struct Item *item_, int size)//печать списка
{
    int i;
    for(i = 0; i < size; i++)
    {
        printf("элемент с ключом %i и смещением %i --- %s \r\n", item_[i].key,item_[i].offset, item_[i].info);
    }
}
//////////////////////////////////////////////////////////////////////////
 
int CountIndexKey (int lower_key,int top_key)//определяет количество элементов с подходящим ключом
{
    int i;
    int answer = 0;
    for(i = 0; i < ItemSize; i++)
    {
        if(Items[i].key <= top_key && Items[i].key >= lower_key)
            answer++;// акамулятор
    }
    return answer;
}
//////////////////////////////////////////////////////////////////////////
 
struct Item *FindItems(int *SizeMoreItems)
{
    struct Item *NewItems;// новый списко/массив
    int lower_key, top_key;
    int TempSize = 0; //порядковый номер в новом списке
    int i; //порядковый номер в основном списке
    void *Dest_str; //куда копируем
    void * Source_str;//откуда копируем
    int SizeCopy ;//количество копируемых байт
    //////////////////////////////////////////////////////////////////////////
    puts("введите нижнюю границу ключа");
    while (!scanf("%i", &lower_key))
    {
        scanf( "%*[^\n]" ); scanf( "%*c" );// не много магии
        puts("не корректный ввод, повторите операцию");
    }
    puts("введите верхнюю границу ключа");
    while (!scanf("%i", &top_key))
    {
        scanf( "%*[^\n]" ); scanf( "%*c" );// не много магии
        puts("не корректный ввод, повторите операцию");
    }
    //////////////////////////////////////////////////////////////////////////
    *SizeMoreItems = CountIndexKey(lower_key,top_key);
    NewItems = (struct Item*)malloc(sizeof(struct Item) * *SizeMoreItems);
    for(i = 0; i < ItemSize; i++)
    {
            if(Items[i].key <= top_key && Items[i].key >= lower_key)
            {
                Dest_str = &NewItems[0] +  TempSize; 
                Source_str = &Items[0] + i;
                SizeCopy = sizeof(struct Item);
                memcpy(Dest_str,Source_str,SizeCopy);
                TempSize++;
            }
    }
    //////////////////////////////////////////////////////////////////////////
    return NewItems;
 
}
//////////////////////////////////////////////////////////////////////////
 
int main()
{
    
    
    
    int answer = ANSWER_EMPTY;
    struct Item *MoreItems = 0;
    int SizeMoreItems;
    //////////////////////////////////////////////////////////////////////////
    setlocale(LC_ALL,"Russian");
    g_nameFileOpen = (char*)malloc(sizeof(char) * ( MaxFileName + 1) ); // временный буфер для считывания
    do
    {
        puts("Введите имя файла для считывания данных(можно вводить локальное имя/путь)");
        //gets(g_nameFileOpen);
        strcpy(g_nameFileOpen,"data.bin");
    }while(!init(g_nameFileOpen));
    
    //////////////////////////////////////////////////////////////////////////
    while (answer)
    {
        puts("выберите команду");
        puts("0 - выход");
        puts("1 - добавить элемент");
        puts("2 - удалить элемент");
        puts("3 - напечатать элементы");
        puts("4 - поиск элементов");
        puts("5 - напечатать дополнительный список");
        if (!scanf("%i", &answer))
        {
            scanf( "%*[^\n]" ); scanf( "%*c" );// не много магии
            puts("не корректный ввод, повторите операцию");
            answer = 5;
            continue;
        }
        switch(answer)
        {
            case 1:
                AddItem();
                break;
            case 2:
                DeleteItem();
                break;
            case 3:
                PrintItems(Items,ItemSize);
                break;
            case 4:
                MoreItems = FindItems(&SizeMoreItems);
                break;
            case 5:
                PrintItems(MoreItems,SizeMoreItems);
                break;
        }
    }
    //////////////////////////////////////////////////////////////////////////
    Save(g_nameFileOpen);
    free(buf);
    free(g_nameFileOpen);
    return 0;
}
Добавлено через 3 минуты
C++
1
 if(NumberItemDelete != ItemSize)
234 строка - не даёт удалить память строки последнего элемента
0
alsav22
5426 / 4821 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
30.03.2014, 23:23 #8
Цитата Сообщение от Van111 Посмотреть сообщение
234 строка - не даёт удалить память строки последнего элемента
У меня, до 234 строки, код не доходит:
0
Миниатюры
Free - не удаляется память  
Van111
209 / 188 / 4
Регистрация: 03.08.2011
Сообщений: 2,598
Записей в блоге: 12
30.03.2014, 23:27  [ТС] #9
alsav22, а файл переименовали?
0
recoder
130 / 130 / 28
Регистрация: 13.09.2013
Сообщений: 260
Записей в блоге: 2
30.03.2014, 23:32 #10
56 строка, надо файл то закрывать
C++
1
2
3
4
5
6
if(size < 0)
    {
        ItemSize = 0;
        fclose(fileInit);
        return 0;
    }
1
alsav22
5426 / 4821 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
31.03.2014, 14:39 #11
Лучший ответ Сообщение было отмечено автором темы, экспертом или модератором как ответ
С файлом, который выложен, код, который выложен, исключения не выбрасывает, в отладчике видно, что память под последним элементом (под указателем info, последнего элемента) освобождается.
1
31.03.2014, 14:39
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
31.03.2014, 14:39
Привет! Вот еще темы с ответами:

Не удаляется последний символ строки - C++
Почему не удаляет последний символ строки, выдает, что такого символа нет. Как это исправить, помогите, пожалуйста. puts(&quot;enter char for...

Переменная удаляется после присваивания - C++
После строки 85 переменная c почему-то удаляется. #include &quot;stdafx.h&quot; #include &lt;iostream&gt; #include &lt;string.h&gt; #include &lt;conio.h&gt; ...

Unique: почему не удаляется Point(1,2)? - C++
Добрый вечер, почему не удаляется Point(1,2) ? %-) #include &lt;algorithm&gt; #include &lt;iostream&gt; #include &lt;vector&gt; #include...

Компилятор C-Free - C++
Начинаю изучать язык программирования C++. Какой вы можете дать отзыв о вышеупомянутом компиляторе? Стоит ли его использовать? Если нет, то...


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

Или воспользуйтесь поиском по форуму:
11
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru