Проблема с перегрузкой операторов

[Furchtlos]

Нужно сделать программу для сложения, вычитания, умножения матриц. В конечном счете нужно получить результат x1=2*A*(A+B)-3*A. x1,A,B - объекты. Написал код, но проблема в том, что строка x1=(x2+x3)*2 вызывает "Invalid allocation size". В чем проблема? Причем, если возвращать от оператора не временный класс, а собственный то проблемы вроде нет.

class matrix
	{
	char imya[20];
	int size;
	double **x;
	public:
	matrix()
	{	cout<<"Vvedite imya matrici: ";
		cin>>imya;
		cout<<endl<<"Vverite razmer matrici ";
		cin>>size;
		x=new double *[size];
		for (int i=0;i<size;i++)
		{
			x[i]=new double[size];
		}
		for(int i=0;i<size;i++)
		{for(int j=0;j<size;j++)
			{cout<<"Vvedite X["<<i<<"]["<<j<<"]"<<endl;
			cin>>x[i][j];}
		}
	}
	matrix(int size)
	{	
		x=new double *[size];
		for (int i=0;i<size;i++)
		{
			x[i]=new double[size];
		}
		for(int i=0;i<size;i++)
		{for(int j=0;j<size;j++)
			{x[i][j]=0;}
		}
	}
	void vivod()
	{
		cout<<imya<<endl;
		for(int i=0;i<size;i++)
		{
			for(int j=0;j<size;j++)
			{
				cout<<x[i][j]<<" ";
			}
			cout<<endl;
		}
	}
	matrix operator+(matrix m2)
	{
		matrix temp(size);
		for(int i=0;i<size;i++)
		{
			for(int j=0;j<size;j++)
			{
				temp.x[i][j]=x[i][j]+m2.x[i][j];}
		}
		return temp;
		
		
	}
	matrix operator-(matrix m2)
	{
		matrix temp(size);
		for(int i=0;i<size;i++)
		{
			for(int j=0;j<size;j++)
			{
				temp.x[i][j]=x[i][j];
				temp.x[i][j]=x[i][j]-m2.x[i][j];}
		}
	 	return temp;
	}
	matrix operator=(matrix m2)
	{
		for(int i=0;i<size;i++)
		{
			for(int j=0;j<size;j++)
			{x[i][j]=m2.x[i][j];}
		}
		return *this;
	}
	matrix operator*(matrix m2)
	{
		matrix temp(size);
		for(int i=0;i<size;i++)
		{
			for(int j=0;j<size;j++)
			{
				temp.x[i][j]=x[i][j]*m2.x[i][j];}
		}
		return temp;
	}
	matrix operator*(int m)
	{
		matrix temp(size);
		for(int i=0;i<size;i++)
		{
			for(int j=0;j<size;j++)
			{
				temp.x[i][j]=x[i][j]*m;}
		}
		return temp;
	}
	matrix operator*=(int m2)
	{
		for(int i=0;i<size;i++)
		{
			for(int j=0;j<size;j++)
			{x[i][j]=x[i][j]*m2;}
		}
		return *this;
	}
	};
	matrix(x1);
	matrix(x2);
	matrix(x3);
	x1=(x2+x3)*2;
	x1.vivod();

[_Bers]

Далеко не лазил, на вскидку:

matrix operator*(matrix m2)
	{
		matrix temp(size);
		for(int i=0;i<size;i++)
		{
			for(int j=0;j<size;j++)
			{
				temp.x[i][j]=x[i][j]*m2.x[i][j];}
		}
		return temp;
	}

Возвращем локальные объекты, да?

То есть, мы знаем, что время жизни локального объекта - пока не закончится область видимости блока, который порождает локальный объект, но нам пофегу, да?
Мы щитаем, что так делать правильно? Возвращать указатели на уже убитые объекты?

[Пепел Феникса]

Возвращем локальные объекты, да?

То есть, мы знаем, что время жизни локального объекта - пока не закончится область видимости блока, который порождает локальный объект, но нам пофегу, да?
Мы щитаем, что так делать правильно?

да так делать правильно в данном случае.
тем более мы возвращаем объект который еще жив.

[_Bers]

Сообщение от Пепел Феникса

да так делать правильно в данном случае.
тем более мы возвращаем объект который еще жив.

Поясните момент. Не вижу ни одной надежды, быть ему живым по выходу.

приёмник = источник.operator*(m2); // срабатывает копирующий конструктор. Вот только что он копирует? Данные протухшего объекта?

[Пепел Феникса]

Поясните момент.

иначе придется возвращать обьект созданный через new, и тогда будет утечка памяти.
для операторов возвращающих *this лучше возвращать ссылку.

а вот как аргумент стоит принимать ссылку(кроме базовых типов).

Вот только что он копирует?

вы возвращаем объект, а не ссылку/указатель на него.
итого копируем return-value.

// срабатывает копирующий конструктор.

только при инициализации, иначе operator =.

http://cplusplus.com/reference/string/operator+/
посмотрите здесь типы

[_Bers]

Сообщение от Пепел Феникса

иначе придется возвращать обьект созданный через new, и тогда будет утечка памяти.
для операторов возвращающих *this лучше возвращать ссылку.

а вот как аргумент стоит принимать ссылку(кроме базовых типов).

Данивапрос!

SomeClass& getData( SomeClass& dst){ return dts; }

SomeClass& getData( SomeClass& target  ){  Swap(this, target)  return *this; }

Всякие реализации ограничены только задачами и фантазиями

[Пепел Феникса]

Функция изменяющая обьект и функция возвращающая новый обьект это разные вещи.
да и про такие функции(например оператор *=) я сказал что там правильнее возвращать ссылку.

Даниваврос!

компиляторы оптимизируют этот момент.

[_Bers]

Сообщение от Пепел Феникса

вы возвращаем объект, а не ссылку/указатель на него.
итого копируем return-value.только при инициализации, иначе operator =.

http://cplusplus.com/reference/string/operator+/
посмотрите здесь типы

Не важно как происходит копирование. Важно что копируется, о боже! Вышедший из области видимости объект.

Трупака копируем

[Пепел Феникса]

Трупака копируем

да нет же!
вы что не можете понять код?

int do(int t)
{
    int r=t+10;
    return r;
}

тут то возвращаем же все нормально.

	matrix operator*(int m)
	{
		matrix temp(size);
		for(int i=0;i<size;i++)
		{
			for(int j=0;j<size;j++)
			{
				temp.x[i][j]=x[i][j]*m;}
		}
		return temp;//возвращаем объект, если без оптимизации то будет конструктор копирования.
	}//тут объекта temp уже нету, но результат функции есть(если без оптимизации)

что еще не ясно?

[_Bers]

ааа... да да, понятна