Класс и динамические двумерные матрицы

[Rekky]

Добрый день
вот мой чудо-код

сlass Matrix 
{
public:
	int size;
	int **Matr;
	Matrix(int isize)
	{
		size = isize;
	        Matr = new int*[size];
		for (int i=0; i< size; i++) Matr[i] = new int[size];
	}
	void Add(int i, int  j, int value)
	{
		Matr[i][j] = value;
	}
	Matrix Plus (Matrix A, Matrix B)
	{
		for (int i=0;i<size;i++)
		{
			for (int j=0;j<size;j++)
			{
				Matr[i][j] = B.Matr[i][j]+A.Matr[i][j];
			}
		}
		return this;
	}
};

ругается: error C2664: 'Matrix::Matrix(int)' : cannot convert parameter 1 from 'Matrix *const ' to 'int'
где return this, если убрать строку то все запускается..но насчет корректности не уверена. правильно ли будет..?!
я вроде и понимаю ошибку. но как сделать правильно не знаю..мне нужно чтобы я в программе основной могла задать любой размер матрицы..вызывать методы сложения и умножения для них..

[artush1984]

return this;

Вы возвращаете указатель на матрицу в место самой матрицы

надо return *this;

[Stilet]

ругается: error C2664: 'Matrix::Matrix(int)' : cannot convert parameter 1 from 'Matrix *const ' to 'int'

Ты цутьцуть не то показываешь. Не ту ошибку. Ругается он на

2 IntelliSense: не существует подходящего конструктора для преобразования из "Matrix *" в "Matrix"

[onewho]

тогда уж надо возвращать ссылку Matrix& либо создавать временный объект и его возвращать

[Rekky]

и правда со * работает и вроде даже правильно
Спасибо

[f.hump]

на мой взгляд, какая-то мрачноватая операция сложения. (что мешает оператор перегрузить?)

но вы это или референс возвращайте

Matrix & Plus (Matrix A, Matrix B)

или конструктор копирования напишите.

кстати, а что будет если А и В разного размера?

[Stilet]

на мой взгляд, какая-то мрачноватая операция сложения.

По-моему вполне норм. Объект вбирающий в себя суммы элементов двух матриц...

а что будет если А и В разного размера?

А вот это уже правильный вопрос )

[Rekky]

должны быть одинакового размера я не стала усложнять себе итак жуткую задачу..
перегруз делала сначала...но тоже были ошибки, переписала так..хоть работает) на перегруз тоже менять не оч хочется

[f.hump]

Matrix & Matrix::operator +=(const Matrix & src) {
	for (int i(0);i<size;i++)
		for (int j(0);j<size;j++)

		Matr[i][j] += src.Matr[i][j];

	return *this;
}
Matrix Matrix::operator +(const Matrix & src) const {
	Matrix result(*this);
	
	result += src;
	return result;
}

это если забыть про проверку размера, и нужен конструктор копирования.

[Rekky]

f.hump, я не понимаю, чем хуже мой код, и зачем нужен будет конструктор копирования..ни разу с таким не работала
Можете объяснить?)