Форум программистов [Powered by Invision Power Board]

Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате

Форум программистов > C/C++: Общие вопросы > Как правильно очистить память?

Автор: 0000 21.11.2007, 23:08

Добрый вечер всем..с C++ у меня тяжко..объясните плз. один момент..
допустим есть класс X.. в нем храним массивчик из элементов того же типа X...
как правильно удалить??
вот код
X.h

Код


class X
{
private: 
    X* x;
    int c;
public:
    X();
    void  Add(const X& x1);
    X& operator = (const X& x1);
public:
    ~X();
};

X.cpp

Код


X::X()
{
    c = 0;
    x = new X[0];
}
X& X::operator =(const X& x1)
{
    c = x1.c;
    x = new X[c];
    for (int i = 0; i < c; i++)
    {
        x[i] = x1.x[i];
    }
    return *this;
}
void X::Add(const X& x1)
{
    x[c] = x1;
    c++;
}
X::~X()
{
    delete[] x;
}

main.cpp

Код


#include "stdafx.h"
#include "X.h"
#include <conio.h>

int _tmain(int argc, _TCHAR* argv[])
{
    X * x1 = new X();
    X * x2 = new X();

    x1->Add(*x2);
    delete x1;

    getch();
    return 0;
}

вот на удалении x1 и происходит ругань...плз, как удалить правильно такую структурку элементов?

спасибо

Автор: DjoNIK 21.11.2007, 23:40

Код


X::X()
{
    c = 0;
    x = new X[0];
}

Ты этим ни байта не резервируешь.

Автор: Greeen 21.11.2007, 23:41

В C++ запрещены массивы из 0 элементов

Автор: DjoNIK 21.11.2007, 23:42

Код


void X::Add(const X& x1)
{
    x[c] = x1;
    c++;
}

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

Автор: UnrealMan 22.11.2007, 01:55

Цитата(Greeen @ 21.11.2007, 23:41

)

В C++ запрещены массивы из 0 элементов

А это обязательно - уверенно говорить о том, в чём сам не разбираешься? Хоть бы "IMHO" добавил что ли.

Автор: 0000 22.11.2007, 02:25

разве в C++ запрещены массивы из 0 элементов?? странно...я просто выделяю память через new, но не заполняю массив...
я больше по .NET, так вот это ассоциируется с созданием пустого листа (коллекции)...выделяешь память, не добавляя элементов....

в общем + 1 для DjoNIK, спасибо, именно это и надо было сделать!!! THX

Автор: UnrealMan 22.11.2007, 02:47

Цитата(0000 @ 22.11.2007, 02:25

)

разве в C++ запрещены массивы из 0 элементов??

Создать массив нулевой длины можно, но только затем, чтобы потом его удалить smile

Цитата(0000 @ 22.11.2007, 02:25

)

выделяешь память, не добавляя элементов....

Это тебе std::vector надо использовать. Либо, если по каким-то причинам vector не подходит, самому выделять сырую память вызовом operator new с последующим размещением в ней объектов с помощью placement new.

Автор: Ln78 22.11.2007, 08:56

Цитата(0000 @ 21.11.2007, 23:08

)

Код


X& X::operator =(const X& x1)
{
    c = x1.c;
    x = new X[c];
    for (int i = 0; i < c; i++)
    {
        x[i] = x1.x[i];
    }
    return *this;

}

В самом начале обычно следует проверять:

Код


if( this != &x1)

И перед тем, как вызывать new следует освободить уже занятую. Можно проверить, если размерности совпадают, то операции освобождения/выделения можно не делать, просто присвоить новые значения.

Цитата(0000 @ 21.11.2007, 23:08

)

Код


class X
{
private: 
    X* x;

Я что-то не соображу, а зачем нужен такой класс? Для списков обычно достаточно одного-двух штук, а здесь целый массив, каждый из которых - тоже массив, и т.д. Можно привести пример, когда такая конструкция целесообразна?

Автор: 0000 22.11.2007, 10:11

Цитата(Ln78 @ 22.11.2007, 08:56)

Цитата(0000 @ 21.11.2007, 23:08

)

Код


class X
{
private: 
    X* x;

например для хранения вершин графа..в вершинах содержится инфа о смежных...
хотя тут действительно может лучше использовать vector

Автор: Greeen 22.11.2007, 12:56

Цитата(UnrealMan @ 22.11.2007, 01:55

)

А это обязательно - уверенно говорить о том, в чём сам не разбираешься? Хоть бы "IMHO" добавил что ли.

У меня просто отложилась фраза Александреску "в C++ массивы нулевой длины запрещены", но там говорилось о массивах создающихся статически. Вот чего то и ступил. Извините, UnrealMan, что задел вас (уж не знаю за что)...

Автор: archimed7592 23.11.2007, 02:25

Цитата(Greeen @ 22.11.2007, 12:56

)

У меня просто отложилась фраза Александреску "в C++ массивы нулевой длины запрещены"

.
У меня тоже отложилась и я тоже как то раз "задел" этим UnrealMan'а smile