класс — конструктор C ++ по умолчанию, инициализирующий указатель новым объектом

У меня есть следующая проблема:
В myClass я хочу по умолчанию инициализировать указатель на yourClass с новым адресом yourClass.
К сожалению, если я хочу удалить указатель в любой момент, я получаю (дамп ядра).

class myClass
{
protected:
yourClass * yc;

public:
myClass() { yc = new yourClass(); }

myClass(yourClass * tyc ) { delete yc; yc = tyc; }

~myClass() { delete yc; yc = NULL; }

void setMyClass (yourClass * tyc) { delete yc; yc = tyc; }

void print () { yc->print(); }
};

int main()
{
yourClass b (//parameter);
myClass * a = new myClass();
a->print();
a->setMyClass(&b)
a->print();

delete a;
return 0;
}

Функция print () объекта a должна давать два разных отпечатка, зависящих от // параметров.

Я считал ваш класс yc; вместо yourClass * yc, но я хочу знать, возможно ли это.

РЕДАКТИРОВАТЬ:
Я переработал код следующим образом, и он работает. Все еще выглядит сложно, умные указатели кажутся многообещающими, и я до сих пор не применил «Правило трех».
Здесь код. Спасибо всем.

class myClass
{
protected:
yourClass * yc;
bool dynamic;

public:
myClass() { dynamic = true; yc = new yourClass (); }
myClass (yourClass * tyc )
{
// dynamic init (like default)
if (tyc == NULL ) { dynamic = true; yc = new yourClass (); }
// static use of yc
else { dynamic = false; yc = tyc; }
}
// because only if dynamic is true, we need to erase
~blu () { if (dynamic) { delete yc; dynamic = false; } }

void setMyClass(yourClass* tyc)
{
// leaving unchanged if new-stuff is NULL or like old-stuff
if ( tyc == yc || tyc == NULL ) return;
else // treating dynamic and static differently
{
if (dynamic) // if flag is set, must be deleted
{
delete yc; yc = tyc; dynamic = false;
}
else // must not be deleted, dynamic is still false
{
yc = tyc;
}
}
}
void print () { yc->print(); }
};

6

Решение

Это потому, что вы пытаетесь удалить слишком много:

  • Вы удаляете нераспределенный объект во втором конструкторе (удалить delete yc;)
  • Вы пытаетесь удалить объект, выделенный из стека, b. delete a; попытается удалить указатель на b, который является объектом в стеке; что происходит, зависит от вашей ОС (я ожидаю исключения / дамп ядра / что угодно)

РЕДАКТИРОВАТЬ: еще одна проблема, которую я заметил ..
a->setMyClass(NULL)

Я бы предложил:

  • эта почта на умных указателях
  • этот блог на РАИИ
  • любой праймер C / C ++, объясняющий распределение стека и кучи (статический или динамический?)
7

Другие решения

Вы нарушаете правило трех.

Что такое правило трех?

Также это рецепт катастрофы:

 myClass(yourClass * tyc ) { delete yc; yc = tyc; }

Что будет, если tyc==yc? Правильно. Не красиво 🙂

 myClass(yourClass * tyc ) { if (yc!=tyc) { delete yc; yc = tyc; } }
1

По вопросам рекламы [email protected]