VPF::Динамическое распределение памяти

Salatovec

Дата 22.3.2012, 21:56 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 213
Регистрация: 9.1.2008

Репутация: нет
Всего: -1

Доброго всем времени суток. Имеется следующий код:

Код


class CVector3D
{
public:
    double x, y, z;
};

class MyClass
{
public:
    ~MyClass(void)
    {
        delete vector;
    }

    virtual void init(void) = 0;
protected:
    CVector3D *vector;
};

class Class1 : public MyClass
{
public:
    virtual void init(void)
    {
        int n = 10;
        vector = new CVector3D[n];
    }
};

class Class2 : public MyClass
{
public:
    virtual void init(void)
    {
        int n = 20;
        vector = new CVector3D[n];
    }
};

MyClass *myclass[2];

void main()
{
    for(int i = 0; i < 2; i++)
    {
        myclass[i]->init();

        delete myclass[i];
    }
}

Так вот суть в том, что вроде бы как всё написанно правильно, а память не освобождается. Я что-то не так сделал?

Это сообщение отредактировал(а) Salatovec - 22.3.2012, 21:56

mes

Дата 22.3.2012, 22:06 (ссылка)

(нет голосов)

Загрузка ...

любитель

Профиль
Группа: Участник Клуба
Сообщений: 7954
Регистрация: 14.1.2006

Репутация: 79
Всего: 250

Цитата(Salatovec @ 22.3.2012, 20:56

)

а память не освобождается

с беглого взгляда : у вас деструктор не виртуальный..

--------------------

http://opendots.net

Result

Дата 23.3.2012, 07:29 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 52
Регистрация: 15.5.2011

Репутация: 3
Всего: 5

Цитата(Salatovec @ 22.3.2012, 21:56)

Код


    ~MyClass(void)
    {
        delete vector;
    }
    virtual void init(void)
    {
        int n = 10;
        vector = new CVector3D[n];
    }
    virtual void init(void)
    {
        int n = 20;
        vector = new CVector3D[n];
    }

По мимо не виртуального деструктора, память выделяется под массив а удаляется простым делетом.
Попробуй -

Код


delete [] vector;

borisbn

Дата 23.3.2012, 08:11 (ссылка)

(голосов:1)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 4875
Регистрация: 6.2.2010
Где: Ростов-на-Дону

Репутация: 21
Всего: 135

Ещё память под myclass[i] нигде не выделяется... это не должно работать вообще

Цитата

MyClass *myclass[2];
void main()
{
    for(int i = 0; i < 2; i++)
    {
        myclass[i]->init(); // <-------------- где выделение памяти под myclass[i] ???
        delete myclass[i];
    }
}

Это сообщение отредактировал(а) borisbn - 23.3.2012, 08:12

--------------------

Женщины отличаются от программистов тем, что у них чары состоят из стрингов

Salatovec

Дата 23.3.2012, 16:06 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 213
Регистрация: 9.1.2008

Репутация: нет
Всего: -1

Цитата(borisbn @ 23.3.2012, 08:11)

Ещё память под myclass[i] нигде не выделяется... это не должно работать вообще

Цитата

Ой да я просто не свой код копировал конкретно и забыл дописать этот момент:

Код


MyClass *myclass[2];
void main()
{
        myclass[0] = new Class1;
        myclass[1] = new Class2;

        myclass[0]->init(); 
        myclass[1]->init(); 

        delete myclass[0];
        delete myclass[1];
}

Это сообщение отредактировал(а) Salatovec - 23.3.2012, 16:07

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Для новичков \| Следующая тема »