Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
 
Рейтинг 4.75/4: Рейтинг темы: голосов - 4, средняя оценка - 4.75
0 / 0 / 0
Регистрация: 15.08.2017
Сообщений: 25
1

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

16.09.2017, 08:11. Просмотров 671. Ответов 13

Пытаюсь создать копирующий конструктор для класса матрицы и перегрузить оператор <<
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
    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++
1
2
3
4
5
6
7
8
9
10
11
12
13
    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++
1
2
3
4
    int *Matrix::operator[] (int i)
    {
        return mat[i];
    }
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
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];
Я новичок и не совсем понимаю в чём проблема
До этого работало, но копию не выводила
Теперь даже просто созданную матрицу не выводит
0
Programming
Эксперт
94731 / 64177 / 26122
Регистрация: 12.04.2006
Сообщений: 116,782
16.09.2017, 08:11
Ответы с готовыми решениями:

Для шаблонного класса перегрузить оператор присваивания, copy-конструктор, объекты cin и cout, оператор *
Помогите в следующем: Для класса шаблона следует перегрузить оператор присваивания, конструктор...

Создать конструктор копий и оператор присваивания для класса компьютер и члена класса марка
Создать конструктор копий и оператор присваивания для класса компьютер и члена класса марка. ...

Перегрузить для класса оператор ==
немогу написать продолжение кода... нужно написать : перегрузить для данного класса оператор...

Перегрузить оператор присваивания для класса
Помогите, пожалуйста. Требуется написать оператор присваивания для данного класса class ARRAY{...

13
Каждому свое
530 / 216 / 81
Регистрация: 05.08.2013
Сообщений: 1,613
16.09.2017, 09:14 2
adLer21,
Цитата Сообщение от adLer21 Посмотреть сообщение
После того, как я пытаюсь вывести матрицу std::cout << mat << std::endl; Программа возвращает меня на строку
Что значит "возвращает"? Ошибку выдает? Ну во-первых я бы переписал вот так бы:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
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++
1
2
3
4
5
6
7
8
9
class SomeClass
{
       ...
       SomeClass( const SomeClass& someClass )
       {
              ...      
       }
       ...
};
тоесть:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
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];
      }
}
0
0 / 0 / 0
Регистрация: 15.08.2017
Сообщений: 25
16.09.2017, 09:59  [ТС] 3
iomanip я подключил
Интересно (для меня), что как только я прописываю const перед Matrix &object функции getLine() и getCol() перестают быть доступными и пишет "объект содержит квалификаторы, несовместимые с членом функцию Program::Matrix::getCol()
Также mat
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
    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;
    }
0
Модератор
Эксперт С++
10150 / 8499 / 5165
Регистрация: 18.12.2011
Сообщений: 22,736
16.09.2017, 10:44 4
Цитата Сообщение от adLer21 Посмотреть сообщение
что как только я прописываю const перед Matrix &object функции getLine() и getCol() перестают быть доступными
C++
1
2
    int Matrix::getLine() const{return line;}
    int Matrix::getCol() const{return col;}
з.ы. Слово "object" ключевое слово компилятора,
не советую использовать его в качестве имени переменной.
0
Каждому свое
530 / 216 / 81
Регистрация: 05.08.2013
Сообщений: 1,613
16.09.2017, 12:05 5
zss,
Цитата Сообщение от zss Посмотреть сообщение
Слово "object" ключевое слово компилятора
Разве у компилятора c++ есть ключевое слово object?
0
0 / 0 / 0
Регистрация: 15.08.2017
Сообщений: 25
16.09.2017, 12:37  [ТС] 6
Проблема точно не в этом
0
13707 / 7331 / 1750
Регистрация: 30.01.2014
Сообщений: 12,292
16.09.2017, 12:47 7
Цитата Сообщение от adLer21 Посмотреть сообщение
функции getLine() и getCol() перестают быть доступными и пишет "объект содержит квалификаторы, несовместимые с членом функцию Program::Matrix::getCol()
Так уже указали на ошибку:
Цитата Сообщение от zss Посмотреть сообщение
C++
1
2
    int Matrix::getLine() const{return line;}
* * int Matrix::getCol() const{return col;}
Правда так квалифицировать имя внутри класса нельзя, поэтому правильнее было бы так:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
    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, изменить таким же образом.
0
Каждому свое
530 / 216 / 81
Регистрация: 05.08.2013
Сообщений: 1,613
16.09.2017, 12:48 8
adLer21,
Цитата Сообщение от adLer21 Посмотреть сообщение
Проблема точно не в этом
Ты можешь скинуть полный код, и указать ошибку где и какая? У тебя неоднозначные вопросы, не понятно что ты хочешь
0
0 / 0 / 0
Регистрация: 15.08.2017
Сообщений: 25
16.09.2017, 13:08  [ТС] 9
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
#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++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
#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++
1
2
3
4
5
6
7
8
9
10
11
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@@@st d@@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\S ource.obj 1
Серьезность Код Описание Проект Файл Строка Состояние подавления
Ошибка LNK1120 неразрешенных внешних элементов: 1 labs_1.5_3semestr C:\Users\Vladislav\Documents\Visual Studio 2015\Projects\c++\labs\labs_1.5_3semestr\Debug\labs_1.5_3sem estr.exe 1
0
13707 / 7331 / 1750
Регистрация: 30.01.2014
Сообщений: 12,292
16.09.2017, 13:14 10
Цитата Сообщение от adLer21 Посмотреть сообщение
Теперь даже просто созданную матрицу не выводит
В конструкторе копирования забылась инициализация line и col:
C++
1
2
3
4
5
6
7
8
9
10
11
    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 &);
0
0 / 0 / 0
Регистрация: 15.08.2017
Сообщений: 25
16.09.2017, 13:23  [ТС] 11
Сейчас попробую

Добавлено через 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\S ource.cpp 13
0
13707 / 7331 / 1750
Регистрация: 30.01.2014
Сообщений: 12,292
16.09.2017, 13:27 12
Цитата Сообщение от adLer21 Посмотреть сообщение
А как вот это побороть?
В 18 строке тоже самое. friend-декларация должна соответствовать объявлению.
0
13707 / 7331 / 1750
Регистрация: 30.01.2014
Сообщений: 12,292
16.09.2017, 13:33 13
adLer21, поправил комплексно твой код:
Кликните здесь для просмотра всего текста
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
#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
0
0 / 0 / 0
Регистрация: 15.08.2017
Сообщений: 25
16.09.2017, 13:33  [ТС] 14
Фух, спасибо вам огромное за помощь
Вроде как всё заработало, низкий поклон
0
IT_Exp
Эксперт
87844 / 49110 / 22898
Регистрация: 17.06.2006
Сообщений: 92,604
16.09.2017, 13:33

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

Перегрузить для класса оператор == (много ошибок)
не могу разабраться в чем ошибки при компиляции... помогите пожалуйста код: #include...

Как перегрузить оператор для своего класса?
У меня есть класс, .h class Vertex { public: ~Vertex(); Vertex(); double GetX(); double...

Перегрузить оператор () для пользовательского класса Matrix
Суть, есть класс matrix , есть int **p(матрица) и int row, col; , нужно перегрузить оператор ()...

Как правильно перегрузить оператор для шаблона класса
Имеется такой шаблон: template&lt;int Low = 0, int High = 1000&gt; class Diapazon { }; Внутри...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
14
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2021, vBulletin Solutions, Inc.