Почему массив изменяется?

Question

[Евгений Якушов]

class MatrixClass
{
	private:
		int **Matrix;
		int SizeM;
	public:
		MatrixClass(int);
		MatrixClass(){};
		~MatrixClass();

		int GetSize();
		void PrintMatrix();

		friend MatrixClass* operator+ (MatrixClass A, MatrixClass B);
		friend MatrixClass* operator- (MatrixClass A, MatrixClass B);
		// friend MatrixClass* operator* (MatrixClass A, MatrixClass B);
		// friend MatrixClass* operator/ (MatrixClass A, MatrixClass B);
}

MatrixClass::MatrixClass(int SizeM)
{
	this->SizeM = SizeM;
	Matrix = new int *[SizeM];
    for(int i = 0; i < SizeM; i++)
    {
        Matrix[i] = new int [SizeM];
        for(int j = 0; j < SizeM; j++)
            Matrix[i][j] = rand() % 5;
    }
}

MatrixClass::~MatrixClass()
{
	for(int i = 0; i < SizeM; i++)
	{
	    delete [](Matrix[i]);
	}
	delete [](Matrix);
}

int MatrixClass::GetSize(){return SizeM;}

void MatrixClass::PrintMatrix()
{
	for(int i = 0; i < SizeM; i++){
		cout << "\t| ";
        for(int j = 0; j < SizeM; j++)
            cout << Matrix[i][j] << " ";
        cout << "|" << endl;
	}
};

MatrixClass* operator+ (MatrixClass A, MatrixClass B)
{
	MatrixClass* C = new MatrixClass(A.GetSize());

	for(int i = 0; i < A.GetSize(); i++){
        for(int j = 0; j < A.GetSize(); j++){
            C->Matrix[i][j] = A.Matrix[i][j] + B.Matrix[i][j];
        }
	}

	return C;
}

MatrixClass* operator- (MatrixClass A, MatrixClass B)
{
	MatrixClass* C = new MatrixClass(A.GetSize());

	for(int i = 0; i < A.GetSize(); i++){
        for(int j = 0; j < A.GetSize(); j++){
            C->Matrix[i][j] = A.Matrix[i][j] - B.Matrix[i][j];
        }
	}

	return C;
}

void main()
{
	clrscr();
	
	// 4, 10
	MatrixClass A(3), B(3), *C;

	cout<<"A: "<<endl; A.PrintMatrix(); getch();
	cout<<"B: "<<endl; B.PrintMatrix(); getch();

	C = A + B;
	cout<<"A + B: "<<endl;
	C->PrintMatrix(); getch();

	cout<<"A: "<<endl; A.PrintMatrix(); getch();
	cout<<"B: "<<endl; B.PrintMatrix(); getch();

	C = A - B;
	cout<<"A - B: "<<endl;
	C->PrintMatrix(); getch();

	// C = A * B;
	// cout<<"Overload C = A + B: ";
	// C.PrintMatrix(); getch();

	// C = A / B;
	// cout<<"Overload C = A - B: ";
	// C.PrintMatrix(); getch();
}

Есть такой код, я не понимаю почему сбиваются массивы А и B, хотя я их даже не изменяю. Изменяются 2 первые строчки массива А и B. Подскажите в чем проблема

Comments

[lz961]

Чем это компилировалось?

[Евгений Якушов]

lz961, borland

[lz961]

Евгений Якушов, тогда были специфические представления, о том, что есть C++
ошибка действительно в том, что при передаче в функцию класса происходит вызов конструктора копии (который генерируется по умолчанию) а после выхода из функции происходит вызов деструктора, который очищает память, на которую указывает A.Maxtrix. При компиляции clang ом это приводит к ошибке адресации при вызове A - B

Answer 1

[Vitaly]

Операторы + и - принимают объекты по значению, а не по ссылке. Вероятно проблема в этом.

Comments

[Sazoks]

Хм, не могли бы вы подробней разъяснить, почему именно в этом?
Ведь, как я понял, смысл этой программы в том, что динамически создается новый объект матрицы, размерами A.GetSize() (это при условии, что размеры объекта А и B одинаковы), и в конце по идее должна возвращаться ССЫЛКА на этот объект, которая присваивается в указатель С.
Я тоже новичок, очень интересно разобраться в этом)

[Roman]

Евгений Якушов, Вам нужен копирующий конструктор

и заодно поменяйте

class MatrixClass
{
  private:
    int **Matrix; // <----
    int SizeM; // <----
// ...

на

  private:
    int SizeM; // <----
    int **Matrix; // <----

[Евгений Якушов]

Есть одна теория, что при передаче матрицы в качестве аргумента оператора сложения она копируется, но при этом указатель остаётся тот же самый, а потом эта копия удаляется, вызывается деструктор, он освобождает (с помощью delete[]) память, выделенную на элементы. Тогда нужно как-то копию посылать

[Roman]

Евгений Якушов,
теорий нет или по ссылке как пишет Vitaly, или копия.

int GetSize() const;
// ...
MatrixClass::MatrixClass(const MatrixClass& other)
{
    SizeM = other.GetSize();
    Matrix = new int *[SizeM];
    for(int i = 0; i < SizeM; i++)
    {
        Matrix[i] = new int [SizeM];
        for(int j = 0; j < SizeM; j++)
            Matrix[i][j] = other.Matrix[i][j];
    }
}

Answer 2

[Евгений Якушов]

Было

friend MatrixClass* operator+ (MatrixClass A, MatrixClass B);
//...
MatrixClass* operator+ (MatrixClass A, MatrixClass B)
{
	//...
}

Стало

friend MatrixClass* operator+ (MatrixClass& A, MatrixClass& B);
//...
MatrixClass* operator+ (MatrixClass& A, MatrixClass& B)
{
	//...
}

Профит

Comments

[Vitaly]

По const-ссылке надо. Вам же не надо изменять объекты, которые вы передали.

[Roman]

Профит

"Еще не вечер"

A = A - B;
MatrixClass D = B - A;
MatrixClass E(B + D);