Имеется класс, типа такого

Код

#include <iostream.h> typedef struct{ int x; int y; int h; }SSTRUC, *PSSTRUC; class C{ public: C(){pSt = new SSTRUC;} C(const C &st){pSt = new SSTRUC; pSt = st.pSt;} C* SetC(PSSTRUC st){pSt = st; return this;} int GetCx(){return pSt->x;} ~C(){delete pSt;} private: PSSTRUC pSt; }; void main(){ PSSTRUC st = new SSTRUC; C* cl1 = new C; C* cl2 = new C; st->x = 10; cl1->SetC(st); cl2 = cl1; cout << cl2->GetCx(); st->x = 20; cl1->SetC(st); cout << cl2->GetCx(); /*delete cl1; delete cl2;*/ }

в результате будет 1020, что я не так сделал?

Ошибки

1) возможны утечки памяти из-за неправильно реализованного SetC
Лучше так
C* С::SetC(PSSTRUC st)
{
if (pSt)
delete pSt;
pSt = st;
return this;
}
или так, если нужна копия структуры
C* С::SetC(PSSTRUC st)
{
*pSt = *st;
return this;
}


2) возможна утечка паямяти и попытка двойного освобождения pSt, если использован конструктор копирования.
из-за неправильно реализованного конструктора копирования
Надо
C(const C &st){ pSt = new SSTRUC; *pSt = *st.pSt;}

3) Конструктор копирования у тебя не используется, даже не задействован. Они бы работал, если ты бы написал что-то вроде этого
C cl3 =*cl1;
А при cl2 = cl1; копируются указатель. В этом случае в cl2 храниться указатель на объект, на который также указывает сд1 и кодом
delete cl1;
delete cl2;
ты пытаешь удалить дважды один и тот же объект. Второй объект у тебя просто теряется
Если ты хотел выполнить копирования
тебе надо было выполнить
*cl2 = *cl1;
Но при этом надо определить оператор копирования


const C& C::operator=( const C& other)
{
if ( this != &other )
{
*pSt = *other.pSt;
}
return *this;
}

Вроде все. Кажется больше ничего не забыл.

спасибо, все работает, надо было просто перегрузить =

Если остальное не исправишь, то тоже нарвешься либо на утечки памяти, либо на исключение