Реализация умных указателей

[FIDE]

Всем привет. Учу С++, вот решил реализовать для себя умные указатели.Все работает нормально, утечек памяти нет. Но я хочу все-же что бы мой код проверили и сказали нет ли каких багов в моей реализации? Спасибо.
P.S Прошу не писать наподобие "Да нафиг оно надо? Есть готовое."

#include <iostream>
#include <conio.h>
#include <vld.h>//Visual Leak Detector для поиска утечек памяти

using namespace std;

class Temp
{
	int TEMP;    
public:
	Temp(){TEMP=25;}
	void TempFunction(){cout<<"TEMP = "<<TEMP<<"\n";} 
	void TempSet(int T){TEMP=T;}
};

template <typename T>//Реализация умного указателя
class SmartPointer
{
	T *ptr;
	int copy;
public:
	SmartPointer(T *p)
	{
		ptr=p;
		copy=0;
	}

	SmartPointer(const T &obj)
	{
		ptr=obj.ptr;
		copy=1;
	}

	SmartPointer operator=(const T &obj)
	{
		ptr=obj.ptr;
		copy=1;
	}

	~SmartPointer()
	{
		if (copy==0 && ptr!=NULL) 
		{
			delete ptr;
		}
	}

	T* operator ->()
	{
		return ptr;
	}

	T& operator*()
	{
		return *ptr;
	}

    
};

int main()
{
	SmartPointer <Temp> ptr(new Temp);
	ptr->TempSet(999);
	ptr->TempFunction();
	getch();
	return 0;
}

[Luuzuk]

Почитайте, сравните, сделайте выводы
http://wiki.dieg.info/smart_pointer

[FIDE]

Будете смеяться, но я видел уже точно такой же код. Точь-в-точь. Но поскольку ничего не понял, решил сам написать умный указатель.

#include <iostream>
using namespace std;
class Temp
{
	int TEMP;	
	public:
		//конструктор
		Temp(){TEMP=25;}
		//функция показа на экран
		void TempFunction(){
			cout<<"TEMP = "<<TEMP<<"\n\n";
		} 
		//функция установки значения
		void TempSet(int T){
			TEMP=T;
		}
};
// Класс, реализующий умный указатель
class SmartPointer 
{
	// Инкапсулированный указатель
	Temp*ptr;
	//счётчик копий 
	int count_copy;

public:
	//конструктор
	SmartPointer (Temp*p=NULL){
		//записываем 0 при создании копий нет
		count_copy=0;
		ptr=p;
	}
	// конструктор копирования
	SmartPointer (const SmartPointer&obj){
		//создается копия - увеличиваем счётчик
		ptr=obj.ptr;
		count_copy++;		
	}
	//перегрузка оператора равно
	SmartPointer operator=(const SmartPointer&obj){
		//создается копия - увеличиваем счётчик
		ptr=obj.ptr;
		ptr=obj.ptr;
		count_copy++;
		//возвращаем сам объект для ситуации a=b=c
		return *this;
	}
	// уничтожение объекта
	~SmartPointer(){
		//если объект есть и копий нет
		if(ptr!=NULL&&count_copy==0){
			cout<<"\n~Delete Object\n";
			//уничтожаем объект
			delete[]ptr;
		}
		//в противном случае(уничтожается копия)
		else{
			//уменьшаем счётчик
			count_copy--;
			cout<<"\n~Delete Copy\n";
		}
			
	} 
	//старая добрая перегрузка селектора
	Temp* operator->(){
		return ptr;
	}

};

void main(){
	//создаем объект
	Temp*main_ptr=new Temp;
	//инициализируем этим объектом умный указатель
	SmartPointer PTR(main_ptr);
	//проверяем работу умного указателя
	PTR->TempSet(100);
	PTR->TempFunction();
	// создаем копию (работа конструктора копирования)
	SmartPointer PTR2=PTR;
}

Результат работы программы:
// работа с объектом через умный указатель
TEMP = 100
//уничтожение копии
~Delete Copy
//уничтожение самого объекта
~Delete Object

Непонятно как деструктор работает...Почему delete[] а не просто delete?В общем принцип работы вообще тут непонятен мне...В перегруженном операторе присвоения одну строку 2 раза повторили - видимо ошиблись...

[FIDE]

Хотя в своем коде я вижу уже одну ошибку. Оператор = ничего не возвращает. Надо

SmartPointer operator=(const T &obj)
	{
		ptr=obj.ptr;
		copy=1;
                return *this;//На случай a=b=c
	}

[Luuzuk]

А то, что ваш код количество копий вообще не считает, в глаза не бросилось?

Почему delete[] а не просто delete

Ну представьте себе умный указатель на массив, и последствия освобождения его с помощью просто delete

[FIDE]

Хм...А разве это обязательно? Задача сделать указатель который сам освободит память...

[p51x]

Для delete не нужна проверка на null.

Хм...А разве это обязательно? Задача сделать указатель который сам освободит память...

Почитайте, чем отличаются и для чего делались - auto_ptr, unique_ptr, shared_ptr, weak_ptr. Начать можете с http://rsdn.ru/article/cpp/smartptr.xml

[Luuzuk]

Задача сделать указатель который сам освободит память

и когда он должен это сделать?

[_Bers]

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

я хочу все-же что бы мой код проверили и сказали нет ли каких багов в моей реализации?

он у вас не может расшариваться. Значит счетчик копий вам не нужен.
Был бы нужен - нужно было бы использовать size_t, а не инт

[_Bers]

SmartPointer(const T &obj)
	{
		ptr=obj.ptr;  //<--- WTF ???
		copy=1;
	}

код на выброс.

все не правильно.