После перегрузки операторов не работает cout

@stv · Регистрация: 05.03.2010

Студворк — интернет-сервис помощи студентам

Здравствуйте. После использования перегруженного оператора + перестает работать оператор вывода. Почему такое возможно? Или у меня ошибка?

C++

#include <iostream.h>
#include <conio.h>
#include <stdlib.h>
#include <time.h>
 
class Matrix
{
    private:
        long int **Matr;
        long int row;
        long int col;
 
        void Create()
        {
            Matr = new long int*[row];
            for (int i=0; i < row; i++)
                Matr[i] = new long int[col];
        }
 
    public:
        // constructors and destructor
        Matrix(): row(5), col(5) { Create();}
        Matrix(long int i): row(i), col(i) { Create(); }
        Matrix(long int i, long int j): row(i), col(j) { Create(); }
        ~Matrix()
        {
            for (int i=0; i < col; i++)
                delete[] Matr[i];
            delete[] Matr;
        }
 
        // methods
        long int& element(long int i, long int j)
        {
            if (i < row && j < col)
                return Matr[i][j];
            else
                cout << "Error: 1";
        }
 
        void MultiplyBy(long int x)
        {
            for(int i=0; i<row; i++)
                for(int j=0; j<col; j++)
                    Matr[i][j] *= x;
        }
 
        void Display()
        {
            for(int i=0; i<row; i++)
            {
                for(int j=0; j<col; j++)
                {
                    cout.width(4);
                    cout << Matr[i][j];
                }
                cout << endl;
            }
        }
        void SetRandom()
        {
            srand(time(NULL));
            for(int i=0; i < row; i++)
            for(int j=0; j < col; j++)
                Matr[i][j] = rand() % 1000;
        }
        
        Matrix operator+ (Matrix &m)
        {
            if (m.row != row || m.col != col) 
            {
                cout << "Error! operator + \n";
                return (*this);
            }
 
            for(int i=0; i < m.row; i++)
            for(int j=0; j < m.col; j++)
                (*this).element(i,j) += m.element(i, j);
            return (*this);
        }
        Matrix operator- (Matrix &m)
        {
            if (m.row != row || m.col != col) 
            {
                cout << "Error! operator + \n";
                return (*this);
            }
 
            for(int i=0; i < m.row; i++)
            for(int j=0; j < m.col; j++)
                (*this).element(i,j) -= m.element(i, j);
            return (*this);
        }   
        Matrix operator= (Matrix &m)
        {
            if (m.row != row || m.col != col) 
            {
                cout << "Error! operator + \n";
                return (*this);
            }
            for(int i=0; i < m.row; i++)
            for(int j=0; j < m.col; j++)
                (*this).element(i,j) = m.element(i, j);
            return (*this);
        }
};
 
 
 
int main()
{
    Matrix a(5), b(5), c(5);
    a.SetRandom();
    b.SetRandom();
    a.Display();
    cout << endl;
    b.Display();
    cout << "test" << endl;
    c = a + b;
    c.Display();
    return 0;
}

Последнее, что выводится на экране - "test". Помогите, пожалуйста.

@alex_x_x · 18.05.2010, 18:37

C++

#include <iostream>
#include <stdlib.h>
#include <time.h>
 
using namespace std;
 
class Matrix
{
    private:
        long int **Matr;
        long int row;
        long int col;
        long int nError;
 
        void Create()
        {
            Matr = new long int*[row];
            for (int i=0; i < row; i++)
                Matr[i] = new long int[col];
        }
 
    public:
        // constructors and destructor
        Matrix(): row(5), col(5), nError(-100500) { Create();}
        Matrix(long int i): row(i), col(i) { Create(); }
        Matrix(long int i, long int j): row(i), col(j) { Create(); }
        ~Matrix()
        {
            for (int i=0; i < col; i++)
                delete[] Matr[i];
            delete[] Matr;
        }
 
        // methods
        long int& element(long int i, long int j)
        {
            if (i < row && j < col)
                return Matr[i][j];
            else
                cout << "Error: 1";
            return nError;
        }
 
        void MultiplyBy(long int x)
        {
            for(int i=0; i<row; i++)
                for(int j=0; j<col; j++)
                    Matr[i][j] *= x;
        }
 
        void Display()
        {
            for(int i=0; i<row; i++)
            {
                for(int j=0; j<col; j++)
                {
                    cout.width(4);
                    cout << Matr[i][j];
                }
                cout << endl;
            }
        }
                void SetRandom()
                {
                        srand(time(NULL));
                        for(int i=0; i < row; i++)
                        for(int j=0; j < col; j++)
                                Matr[i][j] = rand() % 1000;
                }
                
                Matrix& operator+ (Matrix &m)
                {
                        if (m.row != row || m.col != col) 
                        {
                                cout << "Error! operator + \n";
                                return (*this);
                        }
 
                        for(int i=0; i < m.row; i++)
                        for(int j=0; j < m.col; j++)
                                (*this).element(i,j) += m.element(i, j);
                        return (*this);
                }
                Matrix& operator- (Matrix &m)
                {
                        if (m.row != row || m.col != col) 
                        {
                                cout << "Error! operator + \n";
                                return (*this);
                        }
 
                        for(int i=0; i < m.row; i++)
                        for(int j=0; j < m.col; j++)
                                (*this).element(i,j) -= m.element(i, j);
                        return (*this);
                }       
                Matrix& operator= (Matrix &m)
                {
                        if (m.row != row || m.col != col) 
                        {
                                cout << "Error! operator + \n";
                                return (*this);
                        }
                        for(int i=0; i < m.row; i++)
                        for(int j=0; j < m.col; j++)
                                (*this).element(i,j) = m.element(i, j);
                        return (*this);
                }
};
 
 
 
int main()
{
        Matrix a(5), b(5), c(5);
        a.SetRandom();
        b.SetRandom();
        a.Display();
        cout << endl;
        b.Display();
        cout << "test" << endl;
        c = a + b;
        c.Display();
    return 0;
}

когда вы возвращаете return (*this); вы можете:
1)вернуть ссылку на себя (те например результатом Matrix& operator+(Matrix &m) будет сам объект)
2)вернуть свою копию (Matrix operator+(Matrix &m)), те возвращаем копию, и на ее основе конструируем новый объект, однако для этого надо реализовать конструктор копий вида Matrix(const Matrix &m)

@stv · 18.05.2010, 19:20 **[ТС]**

Спасибо большое

Добавлено через 1 минуту

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

конструктор копий вида Matrix(const Matrix &m)

Конструктор-копировщик?

Добавлено через 5 минут
Если завести третью переменную и возвращать ее, то амперсант перед operator не надо?

@Ignat · 18.05.2010, 21:07

Может пригодиться:
Класс Матрица

@stv 26 / 26 / 3 Регистрация: 05.03.2010 Сообщений: 170
	18.05.2010, 19:20 [ТС]	3
	Спасибо большое Добавлено через 1 минуту Сообщение от alex_x_x конструктор копий вида Matrix(const Matrix &m) Конструктор-копировщик? Добавлено через 5 минут Если завести третью переменную и возвращать ее, то амперсант перед operator не надо? 0

@Ignat 1260 / 798 / 108 Регистрация: 16.09.2009 Сообщений: 2,010
	18.05.2010, 21:07	4
	Может пригодиться: Класс Матрица 0

Опции темы