объекты класса и динамические массивы

[alex_alpha]

Здравствуйте.
Столкнулся с такой проблемой:
Мне необходимо сложить два объекта класса, содержащих динамические массивы. Реализовал я это таким образом:

matrix operator+(const matrix &Obj)
{
	matrix Temp;
	for(int i=0;i<s;i++)
	{
		for(int j=0;j<h;j++)
			Temp.mt[i][j]=mt[i][j]+Obj.mt[i][j];
	}
	return Temp;
}

Вот только при возврате объекта Temp деструктор уничтожает его раньше, и в итоге выскакивает ошибка
Как сделать правильно?

[ozo]

Эммм ? покажите класс чтоли ) Ну или методы его с полями )

[Carbon]

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

Вот только при возврате объекта Temp деструктор уничтожает его раньше, и в итоге выскакивает ошибка

Конструктор копирования покажите.

[ozo]

если у нас нет динамических данных, то он не нужен<экстрасенс>, хотя наверное они есть =)</экстрасенс>

[Carbon]

Наверняка mt - это указатель на указатель, а конструктор копирования отсутствует.

[alex_alpha]

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

Наверняка mt - это указатель на указатель, а конструктор копирования отсутствует.

Да, конструктор копирования я не сделал. Спасибо за подсказку.

[alex_alpha]

Вот мой код полностью:

#include<iostream>
#include<cstdlib>
#include<ctime>
using namespace std;
class matrix{
	int **mt;
	int s, h;
public:
	matrix();
	~matrix();
	matrix operator +(const matrix &);
	void Show_();
};
matrix::matrix()
{
	srand(time(NULL));
	s=5, h=6;
	mt=new int *[s];
	for(int i=0;i<s;i++)
		mt[i]=new int[h];
	for(int i=0;i<s;i++)
	{
		for(int j=0;j<h;j++)
			mt[i][j]=rand()%9+1;
	}
}
matrix::~matrix()
{
	for(int i=0;i<s;i++)
		delete []mt[i];
	delete []mt;
}
matrix matrix::operator+(const matrix &Obj)
{
	matrix Temp;
	for(int i=0;i<s;i++)
	{
		for(int j=0;j<h;j++)
			Temp.mt[i][j]=mt[i][j]+Obj.mt[i][j];
	}
	return Temp; //здесь объект сначала уничтожается
}
void matrix::Show_()
{
	static int count=1;
	cout<<"\nMatrix "<<count<<"\n";
	for(int i=0;i<s;i++)
	{
		for(int j=0;j<h;j++)
			cout<<mt[i][j]<<" ";
		cout<<"\n";
	}
	count++;
}
void main()
{
	matrix M1,M2,M3;
	M1.Show_();
	M2.Show_();
	M3=M1+M2;
	M3.Show_();
}

[kutnyuk]

Такая же проблема,реализовал конструктор копирования,но все равно ошибка.помогите кто может.
//конструктор копирования(матрица размером size X size):

matrix::matrix(const matrix &a)
{
	size=a.size;
	arr=new int*[size];
	for(int i=0;i<size;i++)
		arr[i]=new int[size];
	for(int i=0;i<size;i++)
		for(int j=0;j<size;j++)
			arr[i][j]=a.arr[i][j];	
}

[Ozerich]

kutnyuk, size и arr объявлены как public?
И что за ошибка?

[kutnyuk]

Вот код всей программы:

#include<iostream>
using namespace std;

class matrix
{
	int size;
	int **arr;
public:
	matrix(int a);
	~matrix();
	matrix(const matrix& a);
	matrix& operator+ (const matrix& a);	
	void show();	
};

matrix::matrix(int a=0)
{
	size=5;
	arr=new int*[size];
	for(int i=0;i<size;i++)
		arr[i]=new int[size];
	for(int i=0;i<size;i++)
		for(int j=0;j<size;j++)
			arr[i][j]=a;
}
matrix::~matrix()
{
	for(int i=0;i<size;i++)
		delete []arr[i];
	delete []arr;	
}
matrix::matrix(const matrix &a)
{
	size=a.size;
	arr=new int*[size];
	for(int i=0;i<size;i++)
		arr[i]=new int[size];
	for(int i=0;i<size;i++)
		for(int j=0;j<size;j++)
			arr[i][j]=a.arr[i][j];	
}
matrix& matrix::operator+ (const matrix& a)
{	
	matrix temp=*this;
	for(int i=0;i<size;i++){
		for(int j=0;j<size;j++){
			temp.arr[i][j]=arr[i][j]+a.arr[i][j];
		}
	}
	
	return temp;
}
void matrix::show()
{
	for(int i=0;i<size;i++){
		cout<<"\n";
		for(int j=0;j<size;j++)
			cout<<arr[i][j]<<"  ";
	}
}
void main()
{

	matrix obj1(1);
	cout<<"\nobj1:";
	obj1.show();
	matrix obj2(2);
	cout<<"\nobj2:";
	obj2.show();
	matrix obj3(4);
	cout<<"\nobj3:";
	obj3.show();
	obj3=obj1+obj2;
	cout<<"\nobj3=obj1+obj2:";
	obj3.show();
}

Как мне кажется, проблема в то что при работе перегруженного + деструктор срабатывает раньше,чем возвращается объект.В итоге программа виснет.
Если закоментировать деструктор - то все работает.
Выше писалось,что нужен конструктор копирования - я сделал, но все равно нифига не выходит