Форум программистов, компьютерный форум CyberForum.ru

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

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

Не удаляется 2-х связный список C++
Переменная удаляется после присваивания C++
C++ free
C++ не удаляется элемент из массива
C++ Почему удаляется объект?
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Croessmah
Модератор
Эксперт С++
 Аватар для Croessmah
11814 / 6793 / 769
Регистрация: 27.09.2012
Сообщений: 16,861
Записей в блоге: 2
Завершенные тесты: 1
29.03.2014, 15:59     Free - не удаляется память #2
Цитата Сообщение от Van111 Посмотреть сообщение
память под этот элемент точно выделена, NumberItemDelete точно не выходит за границу !
ну тогда free скорее всего работает
Цитата Сообщение от Van111 Посмотреть сообщение
в чём проблема?
а мы откуда знаем что у Вас и как там?
Van111
кодер с++
208 / 187 / 4
Регистрация: 03.08.2011
Сообщений: 2,585
Записей в блоге: 12
30.03.2014, 22:35  [ТС]     Free - не удаляется память #3
Цитата Сообщение от Croessmah Посмотреть сообщение
а мы откуда знаем что у Вас и как там?
если я вам пришлю 378 строк плюс один бинарный файл данных - вы посмотрите ?
Croessmah
Модератор
Эксперт С++
 Аватар для Croessmah
11814 / 6793 / 769
Регистрация: 27.09.2012
Сообщений: 16,861
Записей в блоге: 2
Завершенные тесты: 1
30.03.2014, 22:36     Free - не удаляется память #4
Цитата Сообщение от Van111 Посмотреть сообщение
если я вам пришлю 378 строк плюс один бинарный файл данных - вы посмотрите ?
Давайте, посмотрю
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
30.03.2014, 22:52     Free - не удаляется память #5
Цитата Сообщение от Van111 Посмотреть сообщение
в чём проблема?
С помощью отладчика пробовали выяснить?
Van111
кодер с++
208 / 187 / 4
Регистрация: 03.08.2011
Сообщений: 2,585
Записей в блоге: 12
30.03.2014, 23:02  [ТС]     Free - не удаляется память #6
в функции DeleteItem
Вложения
Тип файла: txt data.txt (32 байт, 3 просмотров)
Van111
кодер с++
208 / 187 / 4
Регистрация: 03.08.2011
Сообщений: 2,585
Записей в блоге: 12
30.03.2014, 23:19  [ТС]     Free - не удаляется память #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 строка - не даёт удалить память строки последнего элемента
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
30.03.2014, 23:23     Free - не удаляется память #8
Цитата Сообщение от Van111 Посмотреть сообщение
234 строка - не даёт удалить память строки последнего элемента
У меня, до 234 строки, код не доходит:
Миниатюры
Free - не удаляется память  
Van111
кодер с++
208 / 187 / 4
Регистрация: 03.08.2011
Сообщений: 2,585
Записей в блоге: 12
30.03.2014, 23:27  [ТС]     Free - не удаляется память #9
alsav22, а файл переименовали?
recoder
 Аватар для recoder
129 / 129 / 28
Регистрация: 13.09.2013
Сообщений: 260
Записей в блоге: 2
30.03.2014, 23:32     Free - не удаляется память #10
56 строка, надо файл то закрывать
C++
1
2
3
4
5
6
if(size < 0)
    {
        ItemSize = 0;
        fclose(fileInit);
        return 0;
    }
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
31.03.2014, 14:39     Free - не удаляется память
Еще ссылки по теме:

И снова динамическая память, malloc и free в конструкторе и деструкторе C++
C++ Динамический массив структур (не происходит передачи и некорректно удаляется)
C++ Объект удаляется до возвращения его из функции

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

Или воспользуйтесь поиском по форуму:
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
31.03.2014, 14:39     Free - не удаляется память #11
Сообщение было отмечено автором темы, экспертом или модератором как ответ
С файлом, который выложен, код, который выложен, исключения не выбрасывает, в отладчике видно, что память под последним элементом (под указателем info, последнего элемента) освобождается.
Yandex
Объявления
31.03.2014, 14:39     Free - не удаляется память
Ответ Создать тему
Опции темы

Текущее время: 13:51. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru