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

Освобождение памяти - C++

Восстановить пароль Регистрация
 
JFF_by
2 / 2 / 0
Регистрация: 16.09.2013
Сообщений: 39
16.09.2013, 23:05     Освобождение памяти #1
Собственно есть простенький класс
C++ (Qt)
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
class Human
{
    
    public:
        Human ( ); 
        void ptintHuman();
    private:
        static  long offset;
        string name;
        int age;
        string sex;
};
long  Human::offset = 0;
 
Human::Human ( ) 
        {
            FILE *fp;
            fp = fopen ("c://3.txt","rb");
            fseek (fp,offset,SEEK_SET);
            fread (this,sizeof(Human),1,fp);
            offset +=  (long int) sizeof(Human); 
            fclose (fp);
        }
void Human::ptintHuman ()
{
    cout << name << "  " << age << "  " << sex << endl;
}
 
void _tmain()
{
    Human *a = new Human [5];
    for(int i=0; i<5; ++i)
        a[i].ptintHuman();
    system ("pause");
    delete [] a;
}

И всё вроде в нём корректно работает кроме оператора delete, который выдаёт вот это:
Подскажите пожалуйста в чём ошибка и как от неё избавиться?


http://photohosting.by/images/2013/0...tln5rc5dud.jpg[/url][/URL]
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
16.09.2013, 23:05     Освобождение памяти
Посмотрите здесь:

C++ Освобождение памяти
резервирование памяти/освобождение памяти для трехмерного массива C++
Освобождение памяти C++
C++ освобождение памяти
C++ Освобождение памяти в C++
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Убежденный
Системный программист
 Аватар для Убежденный
14183 / 6198 / 984
Регистрация: 02.05.2013
Сообщений: 10,324
Завершенные тесты: 1
16.09.2013, 23:12     Освобождение памяти #2
Ошибка в строке 33.
Вместо
C++
1
for(int i=0; i<5; ++i)
Должно быть
C++
1
for(int i=0; i<4; ++i)
castaway
Эксперт С++
4841 / 2980 / 367
Регистрация: 10.11.2010
Сообщений: 11,012
Записей в блоге: 10
Завершенные тесты: 1
16.09.2013, 23:12     Освобождение памяти #3
Цитата Сообщение от JFF_by Посмотреть сообщение
fread (this,sizeof(Human),1,fp);
Я сильно сомневаюсь что это безопасно..
Убежденный
Системный программист
 Аватар для Убежденный
14183 / 6198 / 984
Регистрация: 02.05.2013
Сообщений: 10,324
Завершенные тесты: 1
16.09.2013, 23:19     Освобождение памяти #4
Цитата Сообщение от Убежденный Посмотреть сообщение
Ошибка в строке 33.
Ой, я написал ерунду. Прошу прощения, что-то мне померещилось под вечер...
JFF_by
2 / 2 / 0
Регистрация: 16.09.2013
Сообщений: 39
16.09.2013, 23:29  [ТС]     Освобождение памяти #5
Цитата Сообщение от castaway Посмотреть сообщение
Я сильно сомневаюсь что это безопасно..
а что смущает? в теории всё должно работать. И как тогда, если не так?
castaway
Эксперт С++
4841 / 2980 / 367
Регистрация: 10.11.2010
Сообщений: 11,012
Записей в блоге: 10
Завершенные тесты: 1
16.09.2013, 23:30     Освобождение памяти #6
Как ты думаешь, чему равен размер string name; и string sex; ?
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
16.09.2013, 23:33     Освобождение памяти #7
Цитата Сообщение от JFF_by Посмотреть сообщение
в теории всё должно работать.
Даже в теории не должно.
JFF_by
2 / 2 / 0
Регистрация: 16.09.2013
Сообщений: 39
16.09.2013, 23:47  [ТС]     Освобождение памяти #8
Цитата Сообщение от castaway Посмотреть сообщение
Как ты думаешь, чему равен размер string name; и string sex; ?
точно, не продумал, спасибо за подсказку.
castaway
Эксперт С++
4841 / 2980 / 367
Регистрация: 10.11.2010
Сообщений: 11,012
Записей в блоге: 10
Завершенные тесты: 1
16.09.2013, 23:51     Освобождение памяти #9
Ну если имя и пол всегда одинакового размера, то можно использовать массив символов, например.
char name[32];
char sex[6];
JFF_by
2 / 2 / 0
Регистрация: 16.09.2013
Сообщений: 39
17.09.2013, 00:04  [ТС]     Освобождение памяти #10
Цитата Сообщение от castaway Посмотреть сообщение
Ну если имя и пол всегда одинакового размера, то можно использовать массив символов, например.
char name[32];
char sex[6];
да, так и подумал, спасибо
Пёс
 Аватар для Пёс
228 / 76 / 4
Регистрация: 03.02.2013
Сообщений: 311
17.09.2013, 01:22     Освобождение памяти #11
Убежденный,

Не по теме:

Отдыхать надо от программирования время от времени =)

gomodril
 Аватар для gomodril
6 / 6 / 0
Регистрация: 10.10.2012
Сообщений: 102
17.09.2013, 04:20     Освобождение памяти #12
Удаление не правильно делаете. У вас по сути массив указателей, а он удаляется по другому, как у Вас удаляется, будет утечка памяти.
C++
1
2
for(i=0:i<4;i++) delete [i] a[i];
delete [] a;
castaway
Эксперт С++
4841 / 2980 / 367
Регистрация: 10.11.2010
Сообщений: 11,012
Записей в блоге: 10
Завершенные тесты: 1
17.09.2013, 08:10     Освобождение памяти #13
Чепухи не говори. Все там правильно удаляется.
gomodril
 Аватар для gomodril
6 / 6 / 0
Регистрация: 10.10.2012
Сообщений: 102
18.09.2013, 08:08     Освобождение памяти #14
Цитата Сообщение от castaway Посмотреть сообщение
Чепухи не говори. Все там правильно удаляется.
дело конечно ваше, но даже логически будет правильнее сперва удалить то что находится по указателям, а потом уже сам контейнер содержащий указатели
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
18.09.2013, 10:42     Освобождение памяти #15
Цитата Сообщение от gomodril Посмотреть сообщение
а потом уже сам контейнер содержащий указатели
А где вы, в коде, увидели: "контейнер содержащий указатели"?
gomodril
 Аватар для gomodril
6 / 6 / 0
Регистрация: 10.10.2012
Сообщений: 102
18.09.2013, 12:01     Освобождение памяти #16
Цитата Сообщение от alsav22 Посмотреть сообщение
А где вы, в коде, увидели: "контейнер содержащий указатели"?
C++
1
Human *a = new Human [5];
Если я правильно читаю, то тут выделяется память под массив (я его назвал контейнером, может и не правильно), а массив удаляется не просто освобождением адреса начала массива, его нужно сперва обнулить, иначе в памяти появится не освобожденный участок, недоступный другим приложениям.
Если я не прав, то объясните тогда как освобождается память от массива.
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
18.09.2013, 12:04     Освобождение памяти #17
Цитата Сообщение от gomodril Посмотреть сообщение
Если я правильно читаю, то тут выделяется память под массив
Да. Но разве это массив указателей?
castaway
Эксперт С++
4841 / 2980 / 367
Регистрация: 10.11.2010
Сообщений: 11,012
Записей в блоге: 10
Завершенные тесты: 1
18.09.2013, 12:07     Освобождение памяти #18
Твоя логика противоречит стандарту.
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
18.09.2013, 12:07     Освобождение памяти
Еще ссылки по теме:

C++ Освобождение памяти в c++
Освобождение памяти C++
C++ Освобождение памяти

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

Или воспользуйтесь поиском по форуму:
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
18.09.2013, 12:07     Освобождение памяти #19
Если бы вот так было:
C++
1
Human **a = new Human* [5];
Тогда нужно было бы освобождать память, как вы написали.
Yandex
Объявления
18.09.2013, 12:07     Освобождение памяти
Ответ Создать тему
Опции темы

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