Проблема со стандартным копировщиком

[RAFA91]

Всем доброго дня !

Столкнулся с пакостным явлением.

Почему стандартный конструктор копирования не работает с данными

класса хранящимися в динамической памяти ?

_________________

cуть проблемы такова :

в функцию function_1() передаю обьект как значение.

обьект D класса Vita имеет тот-же адрес в указателе х , что и

обьект А класса Vita функции main().

при выходе из функции function_1() ячейка по адресу в х освобождается.

при выходе из функции main() ячейка по адресу в х повторно

освобождается, что вызывает ошибку.

#include <stdio.h>
#include <conio.h>

class Vita

{

private:

	int *x;

public:

	Vita() : x(new int(0)) {printf("\nKONSTRUKTOR 1 %p x = %p\n",this,x);}

	Vita(int a) : x(new int(a)) {printf("\nKONSTRUKTOR 2 %p x = %p\n",this,x);}

	~Vita() 
	
	{
		printf("\nDESTRUKTOR %p x = %p\n",this,x);

		delete x;
	}

	int fvf() const {printf("\nfvf() %p x = %p\n",this,x); return *x;}

	void hgh(int a) const {printf("\nhgh(int a) %p x = %p\n",this,x); *x = a;}
};

void function_1(Vita);


int main()

{
	Vita A(600);

	printf("\nA = %p\n",&A);

	printf("\nA = %d\n",A.fvf());
	
	function_1(A);

	printf("\nA = %d\n",A.fvf());

	getch();

	return 0;
}



void function_1(Vita D)

{
	printf("\nfunction_1() D = %d %p\n",D.fvf(),&D);

	D.hgh(23);
}

[p51x]

Почему стандартный конструктор копирования не работает с данными
класса хранящимися в динамической памяти ?

А почему должен? Он по стандарту простейший побайтовый копировальщик. Есть значение указателя - он его скопировал, а что там и где не его дело.

[RAFA91]

Сообщение от p51x

А почему должен? Он по стандарту простейший побайтовый копировальщик. Есть значение указателя - он его скопировал, а что там и где не его дело.

так что - единственный вариант , это писать свой копировщик и

оператор = ?

[waleri]

Сообщение от RAFA91

единственный вариант

Да.

<десять символов>

[RAFA91]

написал оператор =

что лучше посоветуете применять

*x = rhs.fvf();

или

*x = *(rhs.x);

???

Vita & operator = (const Vita &rhs)

	{
               if (this == &rhs) return *this;

		*x = rhs.fvf();

		//*x = *(rhs.x);

		return *this;
	}

[crazy horse]

*x = rhs.fvf();

Нифига не читабельно.

               if (this == &rhs) return *this;

		*x = rhs.fvf();

		//*x = *(rhs.x);

		return *this;

Я бы не советовал плодить return'ы на ровном месте.

if (this != &rhs)

и дальше по тексту.

[RAFA91]

Сообщение от crazy horse

*x = rhs.fvf();

Нифига не читабельно.

[

что по Вашему тогда читабельно ?

[crazy horse]

*x = *(rhs.x);

[RAFA91]

значит так и напишу.

[Krok27]

Я бы не советовал плодить return'ы на ровном месте.

Как тогда вернуть из функции:

Vita & operator = (const Vita &rhs)