Создать копирующий конструктор для класса матрицы и перегрузить оператор <<

@adLer21 · Регистрация: 15.08.2017

Пытаюсь создать копирующий конструктор для класса матрицы и перегрузить оператор <<

C++

    class Matrix
    {
    protected:
        int **mat;
    private:
        int line;
        int col;
    public:
        int Matrix::getLine();
        int Matrix::getCol();
        friend std::ostream &operator << (std::ostream &, Matrix);
        Matrix(int, int);
        Matrix(Matrix &);
        ~Matrix();
        Matrix Matrix::operator + (Matrix &);
        void Matrix::operator++();
        int *Matrix::operator[] (int);
        
    };
    std::ostream &operator << (std::ostream &, Matrix );

Реализация копирующего конструктора

C++

    Matrix::Matrix(Matrix &object)
    {
        mat = new int*[object.line];
        for (int i = 0; i < object.line; i++)
        {
            mat[i] = new int[object.col];
        }
        for (int i = 0; i < object.line; i++)
        {
            for (int j = 0; j < object.col; j++)
                mat[i][j] = object.mat[i][j];
        }
    }

C++

    int *Matrix::operator[] (int i)
    {
        return mat[i];
    }

C++

std::ostream &operator << (std::ostream &stream, Matrix object)
    {
        for (int i = 0; i < object.line; i++)
        {
            for (int j = 0; j < object.col; j++)
            {
                stream << std::setw(5) <<  object.mat[i][j];
            }
            std::cout << std::endl;
        }
        stream << std::endl;
        return stream;
    }

После того, как я пытаюсь вывести матрицу std::cout << mat << std::endl; Программа возвращает меня на строку

C++

1	stream << std::setw(5) << object.mat[i][j];

Я новичок и не совсем понимаю в чём проблема
До этого работало, но копию не выводила
Теперь даже просто созданную матрицу не выводит

@Bretbas · 16.09.2017, 09:14

adLer21,

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

После того, как я пытаюсь вывести матрицу std::cout << mat << std::endl; Программа возвращает меня на строку

Что значит "возвращает"? Ошибку выдает? Ну во-первых я бы переписал вот так бы:

C++

friend std::ostream &operator << (std::ostream& stream, const Matrix& object )
{
      for (int i = 0; i < object.getLine(); i++)
      {
            for (int j = 0; j < object.getCol(); j++)
            {
                stream << std::setw(5)<< object.mat[i][j];
            }
            stream << std::endl;
       }
       stream << std::endl;
       return stream;   
}

Во-вторых, ты подключил #include <iomanip>?

В-третьих, конструктор копирования лучше писать по такому шаблону:

C++

class SomeClass
{
       ...
       SomeClass( const SomeClass& someClass )
       {
              ...      
       }
       ...
};

тоесть:

C++

Matrix::Matrix(const Matrix& object)
{
      mat = new int*[object.line];
      for (int i = 0; i < object.line; i++)
      {
            mat[i] = new int[object.col];
      }
      for (int i = 0; i < object.line; i++)
      {
            for (int j = 0; j < object.col; j++)
                  mat[i][j] = object.mat[i][j];
      }
}

@adLer21 · 16.09.2017, 09:59 **[ТС]**

iomanip я подключил
Интересно (для меня), что как только я прописываю const перед Matrix &object функции getLine() и getCol() перестают быть доступными и пишет "объект содержит квалификаторы, несовместимые с членом функцию Program::Matrix::getCol()
Также mat

C++

    std::ostream &operator << (std::ostream &stream, const  Matrix &object)
    {
        for (int i = 0; i < object.getLine() ; i++)
        {
            for (int j = 0; j < object.getCol(); j++)
            {
                stream << std::setw(5) <<  object.mat[i][j];
            }
            std::cout << std::endl;
        }
        stream << std::endl;
        return stream;
    }

@zss · 16.09.2017, 10:44

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

что как только я прописываю const перед Matrix &object функции getLine() и getCol() перестают быть доступными

C++

1 2	int Matrix::getLine() const{return line;} int Matrix::getCol() const{return col;}

з.ы. Слово "object" ключевое слово компилятора,
не советую использовать его в качестве имени переменной.

@Bretbas · 16.09.2017, 12:05

zss,

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

Слово "object" ключевое слово компилятора

Разве у компилятора c++ есть ключевое слово object?

@adLer21 · 16.09.2017, 12:37 **[ТС]**

Проблема точно не в этом

@DrOffset · 16.09.2017, 12:47

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

функции getLine() и getCol() перестают быть доступными и пишет "объект содержит квалификаторы, несовместимые с членом функцию Program::Matrix::getCol()

Так уже указали на ошибку:

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

C++

1 2	int Matrix::getLine() const{return line;} * * int Matrix::getCol() const{return col;}

Правда так квалифицировать имя внутри класса нельзя, поэтому правильнее было бы так:

C++

    class Matrix
    {
    protected:
        int **mat;
    private:
        int line;
        int col;
    public:
        int getLine() const; //!!
        int getCol() const; //!!
        friend std::ostream &operator << (std::ostream &, Matrix const &); //!!
        Matrix(int, int);
        Matrix(Matrix const &); //!!
        ~Matrix();
        Matrix operator + (Matrix const &);
        void operator++();
        int const * operator[] (int) const; //!!
        int  * operator[] (int) ;
        
    };
//.........
 
int Matrix::getLine() const //!!
{
    return line;
}
int Matrix::getCol() const //!!
{
    return col;
}

Остальные определения, где требуется const, изменить таким же образом.

@Bretbas · 16.09.2017, 12:48

adLer21,

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

Проблема точно не в этом

Ты можешь скинуть полный код, и указать ошибку где и какая? У тебя неоднозначные вопросы, не понятно что ты хочешь

@adLer21 · 16.09.2017, 13:08 **[ТС]**

C++

#pragma once
#include <iostream>
#include <cstdlib>
#include <ctime>
namespace Program 
{
    
    class Matrix
    {
    protected:
        int **mat;
    private:
        int line;
        int col;
    public:
        int Matrix::getLine() const;
        int Matrix::getCol() const;
        friend std::ostream &operator << (std::ostream &, Matrix);
        Matrix(int, int);
        Matrix(const Matrix &);
        Matrix();
        ~Matrix();
        Matrix Matrix::operator + (const Matrix &);
        void Matrix::operator++();
        void Matrix::show();
        int const *Matrix::operator[] (int) const;
        
    };
    std::ostream &operator << (std::ostream &, Matrix );
}

C++

#include "Matrix.h"
#include <iomanip>
 
namespace Program
{
    Matrix::Matrix(int _line, int _col)
    {
        line = _line;
        col = _col;
        mat = new int*[line];
        for (int i = 0; i < line; i++)
            mat[i] = new int[col];
        for (int i = 0; i < line; i++)
        {
            for (int j = 0; j < col; j++)
            {
                mat[i][j] = rand() & 20;
            }
        }
    }
    Matrix::Matrix(const Matrix &object)
    {
        mat = new int*[object.line];
        for (int i = 0; i < object.line; i++)
        {
            mat[i] = new int[object.col];
        }
        for (int i = 0; i < object.line; i++)
        {
            for (int j = 0; j < object.col; j++)
                mat[i][j] = object.mat[i][j];
        }
    }
    int const *Matrix::operator[] (int i) const
    {
        return mat[i];
    }
    Matrix::Matrix()
    {
 
    }
    Matrix Matrix::operator+(const Matrix &object)
    {
        Matrix newmat(2,3);
        newmat.mat = new int*[object.getLine()];
        for (int i = 0; i < object.getLine() ; i++) 
            newmat.mat[i] = new int[getCol()];
        for (int i = 0; i < getLine(); i++)
        {
            for (int j = 0; j < getCol(); j++)
            {
                newmat.mat[i][j] = this->mat[i][j] + object.mat[i][j];
            }
        }
 
        return newmat;
    }
 
    int Matrix::getLine() const { return line; }
    int Matrix::getCol() const { return col; };
    void Matrix::show()
    {
        for (int i = 0; i < line; i++)
        {
            for (int j = 0; j < col; j++)
            {
                std::cout << std::setw(5) << mat[i][j];
            }
            std::cout << std::endl;
        }
        std::cout << std::endl;
    }
    std::ostream &operator << (std::ostream &stream, const  Matrix &object)
    {
        for (int i = 0; i < object.getLine() ; i++)
        {
            for (int j = 0; j < object.getCol(); j++)
            {
                stream << std::setw(5) <<  object[i][j];
            }
            std::cout << std::endl;
        }
        stream << std::endl;
        return stream;
    }
    Matrix::~Matrix()
    {
        for (int i = 0; i < line; i++)
        {
            delete[] mat[i];
        }
        delete[] mat;
    }
}

#include <iostream>
#include <cstdlib>
#include <ctime>
#include <iomanip>
#include "Matrix.h"
using namespace Program;

C++

int main()
{
    setlocale(LC_ALL, "russian");
    srand(time(NULL));
    Matrix mat(2, 3);
    Matrix mat2(2, 3);
    std::cout << mat << std::endl; //Пытаюсь вывести матрицу - не выходит
    system("pause");
    return 0;
 
}

Через метод show работает, но как только я пишу (mat + mat2).show() - не выводит ничего
Я уже окончательно запутался. Вроде как выражение (mat + mat2) должно возвращать новую матрицу
Ошибка LNK2019 ссылка на неразрешенный внешний символ "class std::basic_ostream<char,struct std::char_traits<char> > & __cdecl Program::operator<<(class std::basic_ostream<char,struct std::char_traits<char> > &,class Program::Matrix)" (??6Program@@YAAAV?$basic_ostream@DU?$char_traits@ D@std@@@std@@AAV12@VMatrix@0@@Z) в функции _main labs_1.5_3semestr C:\Users\Vladislav\Documents\Visual Studio 2015\Projects\c++\labs\labs_1.5_3semestr\labs_1.5_ 3semestr\Source.obj 1
Серьезность Код Описание Проект Файл Строка Состояние подавления
Ошибка LNK1120 неразрешенных внешних элементов: 1 labs_1.5_3semestr C:\Users\Vladislav\Documents\Visual Studio 2015\Projects\c++\labs\labs_1.5_3semestr\Debug\lab s_1.5_3semestr.exe 1

@DrOffset · 16.09.2017, 13:14

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

Теперь даже просто созданную матрицу не выводит

В конструкторе копирования забылась инициализация line и col:

C++

    Matrix::Matrix(Matrix &object)
        : line(object.line), col(object.col)
    {
        mat = new int*[object.line];
        for (int i = 0; i < object.line; i++)
        {
            mat[i] = new int[object.col];
            for (int j = 0; j < object.col; j++)
                mat[i][j] = object.mat[i][j];
        }
    }

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

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

ссылка на неразрешенный внешний символ "class std::basic_ostream<char,struct std::char_traits<char> > & __cdecl Program::operator<<(class std::basic_ostream<char,struct std::char_traits<char> > &,class Program::Matrix)"

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

C++

1	std::ostream &operator << (std::ostream &, Matrix );

29 строка должна быть:

C++

1	std::ostream &operator << (std::ostream &, Matrix const &);

@adLer21 · 16.09.2017, 13:23 **[ТС]**

Сейчас попробую

Добавлено через 6 минут
А как вот это побороть?

C++

1	std::cout <<[COLOR="Red"] mat[/COLOR] << std::endl;

Серьезность Код Описание Проект Файл Строка Состояние подавления
Ошибка (активно) существует несколько операторов "<<", соответствующих этим операндам: labs_1.5_3semestr c:\Users\Vladislav\Documents\Visual Studio 2015\Projects\c++\labs\labs_1.5_3semestr\labs_1.5_ 3semestr\Source.cpp 13

@DrOffset · 16.09.2017, 13:27

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

А как вот это побороть?

В 18 строке тоже самое. friend-декларация должна соответствовать объявлению.

@DrOffset · 16.09.2017, 13:33

adLer21, поправил комплексно твой код:

Кликните здесь для просмотра всего текста

C++

#include <iostream>
#include <iomanip>
#include <cstdlib>
#include <algorithm>
 
    class Matrix
    {
    private:
        int line;
        int col;
    protected:
        int **mat;
    public:
        int getLine() const;
        int getCol() const;
 
        Matrix(int, int);
        Matrix();
        Matrix(Matrix const &);
        ~Matrix();
        
        Matrix operator + (Matrix const &);
        void operator++();
        
        int * operator[](int i);
        int const * operator[](int i) const;
    };
 
    Matrix::Matrix(int l, int c) 
        : line(l), col(c)
    {
        mat = new int*[line];
        for (int i = 0; i < line; i++)
        {
            mat[i] = new int[col];
        }
    }
    Matrix::Matrix(const Matrix &object)
        : line(object.line), col(object.col) //!!!
    {
        mat = new int*[object.line];
        for (int i = 0; i < object.line; i++)
        {
            mat[i] = new int[object.col];
            for (int j = 0; j < object.col; j++)
                mat[i][j] = object.mat[i][j];
        }
    }
    int const *Matrix::operator[] (int i) const
    {
        return mat[i];
    }
    int *Matrix::operator[] (int i)
    {
        return mat[i];
    }
    Matrix::Matrix() 
        : line(), col(), mat() //!!!!
    { }
    Matrix Matrix::operator+(const Matrix &object) // !!!!!!!!!
    {
        Matrix newmat(std::min(getLine(), object.getLine()), std::min(getCol(), object.getCol()));
        
        for (int i = 0; i < newmat.getLine(); i++)
        {
            for (int j = 0; j < newmat.getCol(); j++)
            {
                newmat[i][j] = mat[i][j] + object[i][j];
            }
        }
        return newmat;
    }
 
    int Matrix::getLine() const { return line; }
    int Matrix::getCol() const { return col; };
 
    std::ostream &operator << (std::ostream &stream, const  Matrix &object)
    {
        for (int i = 0; i < object.getLine() ; i++)
        {
            for (int j = 0; j < object.getCol(); j++)
            {
                stream << std::setw(5) <<  object[i][j];
            }
            stream << std::endl; //!!!!
        }
        return stream;
    }
    Matrix::~Matrix()
    {
        for (int i = 0; i < line; i++)
        {
            delete[] mat[i];
        }
        delete[] mat;
    }
 
void input(Matrix & object)
{
    for (int i = 0; i < object.getLine() ; i++)
    {
        for (int j = 0; j < object.getCol(); j++)
        {
            object[i][j] = rand() & 20;
        }
    }    
}
 
int main()
{
    srand(time(NULL));
    
    Matrix mat(2, 3);
    input(mat);
    
    Matrix mat2(2, 3);
    input(mat2);
    
    
    std::cout << mat << std::endl;
    std::cout << mat + mat2 << std::endl;
 
    return 0;
}

Тут онлайн пример: http://rextester.com/IZFI50636

@adLer21 · 16.09.2017, 13:33 **[ТС]**

Фух, спасибо вам огромное за помощь

Вроде как всё заработало, низкий поклон

@Bretbas Каждому свое 532 / 218 / 81 Регистрация: 05.08.2013 Сообщений: 1,614
	16.09.2017, 12:05	5
	zss, Сообщение от zss Слово "object" ключевое слово компилятора Разве у компилятора c++ есть ключевое слово object? 0

@adLer21 0 / 0 / 0 Регистрация: 15.08.2017 Сообщений: 25
	16.09.2017, 12:37 [ТС]	6
	Проблема точно не в этом 0

@Bretbas Каждому свое 532 / 218 / 81 Регистрация: 05.08.2013 Сообщений: 1,614
	16.09.2017, 12:48	8
	adLer21, Сообщение от adLer21 Проблема точно не в этом Ты можешь скинуть полный код, и указать ошибку где и какая? У тебя неоднозначные вопросы, не понятно что ты хочешь 0

@DrOffset 16495 / 8988 / 2205 Регистрация: 30.01.2014 Сообщений: 15,611
	16.09.2017, 13:27	12
	Сообщение от adLer21 А как вот это побороть? В 18 строке тоже самое. friend-декларация должна соответствовать объявлению. 0

@adLer21 0 / 0 / 0 Регистрация: 15.08.2017 Сообщений: 25
	16.09.2017, 13:33 [ТС]	14
	Фух, спасибо вам огромное за помощь Вроде как всё заработало, низкий поклон 0