Форум программистов, компьютерный форум, киберфорум
Наши страницы
С++ для начинающих
Войти
Регистрация
Восстановить пароль
 
Рейтинг 4.83/6: Рейтинг темы: голосов - 6, средняя оценка - 4.83
HardMorg
3 / 26 / 9
Регистрация: 29.08.2010
Сообщений: 204
1

Ошибка при освобождении памяти

06.04.2012, 16:09. Просмотров 1161. Ответов 9
Метки нет (Все метки)

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
#include <iostream>
 
using namespace std;
 
template<class T>
class pvector  {
 
    T **p;
    int sz;
public:
    pvector(int size) {
 
        p = new T*[size];
 
        for(int i  = 0; i < size; i++)
            p[i] = new T();
 
        sz = size;
    }
    
    ~pvector() {
 
        for(int i = 0; i < sz; i++)
            delete[] p[i];
 
        delete[] p;
    }
};
 
 
class My {
 
public:
    My() {
        cout << "My() " << endl;
    }
    
    ~My() {
 
        cout << "~My()" << endl;
    }
    
};
 
int main() {
 
    pvector<My> m(10);
 
}
выбивает ошибку при освобождении памяти ( но если я уберу деструктор класса My не выводит ошибку)
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
06.04.2012, 16:09
Ответы с готовыми решениями:

Ошибка при освобождении памяти
#include &lt;iostream&gt; using namespace std; #include &lt;stdlib.h&gt; #include...

Ошибка при освобождении памяти
Добрый День! Написал простую программу для понимания сути. Проблема в том, что...

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

Ошибка при освобождении памяти
Задача: Из стандартного входного потока вводятся сроки. Признаком завершения...

Ошибка при освобождении памяти
Здравствуйте, программа завершает работу в этом коде #include &lt;iostream&gt;...

9
igorrr37
1867 / 1483 / 751
Регистрация: 21.12.2010
Сообщений: 2,473
Записей в блоге: 11
06.04.2012, 18:10 2
строка 25
delete p[i];
1
HardMorg
3 / 26 / 9
Регистрация: 29.08.2010
Сообщений: 204
06.04.2012, 20:58  [ТС] 3
igorrr37, можете дать ссылку почитать по этому поводу?
а то я раньше читал где то на просторах интернета что освобождение происходит
C++
1
2
3
4
for(int i = 0; i < sz; i++)
            delete[] p[i];
 
        delete[] p;
интерес даже в том что для простых типов это работает( не знаю на сколько хорошо(если рассматривать из нутри)), и что вообще мне интересно мне если обьявить конструктор My виртуальный, тоже освобождает...

Добавлено через 1 час 2 минуты
Объясните кто все так знает, как верно?
0
retmas
Жарю без масла
865 / 747 / 225
Регистрация: 13.01.2012
Сообщений: 1,702
06.04.2012, 21:26 4
вам же написали, как верно
1
Kuzia domovenok
2320 / 2069 / 480
Регистрация: 25.03.2012
Сообщений: 7,372
Записей в блоге: 1
06.04.2012, 22:33 5
Цитата Сообщение от HardMorg Посмотреть сообщение
Объясните кто все так знает, как верно?
то что выделено new - освобождается с помощью delete
то что выделено new[] - освобождается с помощью delete[]
разве не логично?
1
HardMorg
3 / 26 / 9
Регистрация: 29.08.2010
Сообщений: 204
06.04.2012, 22:41  [ТС] 6
retmas,
Kuzia domovenok,
Ребята вы уж определитель, один утверждает одно, второй совсем другое....

Kuzia domovenok, ты бы хоть прочитал все мои посты... а то вопрос один ответ совсем другой

Добавлено через 2 минуты
Понятно что логично ... но я же написал что у меня вылетает так когда использую пользовательскихе классы, хотя если удалить деструктор или объявить его как виртуальный то не вылетало
0
Toshkarik
1149 / 866 / 90
Регистрация: 03.08.2011
Сообщений: 2,404
Завершенные тесты: 1
06.04.2012, 22:56 7
Почему в классе вашего вектора используется указатель на указатель? Вот так все будет работать
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
#include <iostream>
 
using namespace std;
 
template<class T>
class pvector  {
 
    T *p;
    int sz;
public:
    pvector(int size) {
 
        p = new T[size];
 
 
 
        sz = size;
    }
 
    ~pvector() {
 
        delete[] p;
    }
};
 
 
class My {
 
public:
    My() {
        cout << "My() " << endl;
    }
 
    ~My() {
 
        cout << "~My()" << endl;
    }
 
};
 
int main() {
 
    pvector<My> m(10);
 
    return 0;
}
0
HardMorg
3 / 26 / 9
Регистрация: 29.08.2010
Сообщений: 204
06.04.2012, 23:16  [ТС] 8
Toshkarik, мм... ну суть не в том что и как.. просто хочу всякие аспекты понять, это так сказать в научных целях, понятно что использовать лучше std::vector.
Мне так и не сказали то что я спрашивал
0
Toshkarik
1149 / 866 / 90
Регистрация: 03.08.2011
Сообщений: 2,404
Завершенные тесты: 1
06.04.2012, 23:42 9
Дык дело в том что указатель на указатель можно представить как двумерный массив. А вам нужен одномерный, как и в стандартном векторе. **p означает: массив указателей на массив указателей на тип T. В вашем же случае нужен массив указателей на тип Т. То есть *p. Если уж нужно сделать двумерный массив, то делайте так же как это делается с std::vector, например
C++
1
std::vector< std::vector<int> > b;
Хотя так вроде не советуют делать...

Добавлено через 15 минут
И да как вам уже сказали,
C++
1
delete p[ i ];
в 25 строчке, если вы хотите чтоб ваш код был рабочий. И Kuzia domovenok, верно сказал, как и все кто выше отписался. Вы выделяли память для объектов с помощью new, а не new [] ( строки 16 - 17 ), так и удаляйте их используя delete а не delete []. Или измените в 17 строчке на
C++
1
p[ i ] = new T[ size ];
я думаю что это и имелось ввиду. Иначе у вас создается массив указателей на массив указателей из одного элемента на Т.
0
HardMorg
3 / 26 / 9
Регистрация: 29.08.2010
Сообщений: 204
06.04.2012, 23:53  [ТС] 10
Toshkarik, я же снова повторюсь это в научных целях... просто мне нужно реализовать класс(pvector) он должен содержать указатели на обьекты и каждый обьект должен уничтожатся его деструктором.
я понимаю что вы скажите юзай std::vector<shared_ptr<type>> но мне нужно просто реализовать то что просят, понятно что во первых нигде не будет использоваться а во вторых вообще гавнокод.
И меня интересовала конструкция по освобождению памяти я делал for(...) delete []ptr[i]; delete[]ptr

но прикол в чем когда я использую вектор для стандартных типов то все норм, но когда для других то при освобождении выкидало ошибку... когда же я убирал дестктор объекта которого храню в векторе, то тогда ошибок не выкидало или объявлял деструктор как вирутальны... и вот мне этот момент интересен

Добавлено через 3 минуты
Toshkarik,
Kuzia domovenok,
retmas,
прошу извинения, я просто не понял вас да я был не прав, я понял в чем я ошибся, так протупить
0
06.04.2012, 23:53
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
06.04.2012, 23:53

Ошибка при освобождении памяти
Все идет замечательно, но вот когда происходит удаление массива программа...

Ошибка при освобождении памяти
Здравствуйте! Пишу на MS Visual C++ 2010, windows 7. При вызове функции...

Ошибка при освобождении памяти?
Доброго времени суток! Я только-только взялся за изучение c++. Дошел до...


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

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

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