У меня есть следующая проблема:
В 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(); }
};
Это потому, что вы пытаетесь удалить слишком много:
delete yc;
)delete a;
попытается удалить указатель на b, который является объектом в стеке; что происходит, зависит от вашей ОС (я ожидаю исключения / дамп ядра / что угодно)РЕДАКТИРОВАТЬ: еще одна проблема, которую я заметил ..
a->setMyClass(NULL)
Я бы предложил:
Вы нарушаете правило трех.
Также это рецепт катастрофы:
myClass(yourClass * tyc ) { delete yc; yc = tyc; }
Что будет, если tyc==yc
? Правильно. Не красиво 🙂
myClass(yourClass * tyc ) { if (yc!=tyc) { delete yc; yc = tyc; } }