Перегрузка оператора ==, в классе Матрица

@serginhold · Регистрация: 20.10.2011

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

Code
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
#ifndef MATRIX_H
#define MATRIX_H
 
template <class T>
class Matrix
{
public:
   Matrix(int Rows, int Cols);
   ~Matrix();
   int RowsCount()const;
   int ColsCount()const;
   T* operator [] (int i);
   Matrix<T> operator = (const Matrix *a);
   bool operator == (const Matrix<T> *a);
   void output(int n, int m);
   void Random(int a, int b);
private:
   T** _data; // матрица
   int Rows; int Cols; // кол-во строк и столбцов
};
 
 
/* Часть реализации: */
 
template <class T>
T* Matrix<T>::operator [] (int i)
{
    // перегружаем оператор [], возвращаем строку матрицы
    return this->_data[i];
}
 
template <class T>
Matrix<T> Matrix<T>::operator = (const Matrix *a)
{
    // перегружаем оператор =, делаем копию матрицы `a`
    this->Rows = a->RowsCount(); this->Cols = a->ColsCount();
    this->~Matrix();
    
    this->_data = new T *[this->Rows];
    for (int i=0; i<this->Rows; i++) this->_data[i] = new T [this->Cols];
    
    for (int i=0; i<this->Rows; i++)
        for (int j=0; j<this->Cols; j++)
            this->_data[i][j] = a[i][j];
}
 
template <class T>
bool Matrix<T>::operator == (const Matrix<T> *a)
{
    // перегружаем оператор ==, сравнием матрицы по элементно
    if ((this->Rows != a->RowsCount()) || (this->Cols != a->ColsCount()))
        return false;
    for (int i=0; i<this->Rows; i++)
        for (int j=0; j<this->Cols; j++)
            if (this->_data[i][j] != a[i][j]) return false;
    return true;
}
 
#endif // MATRIX_H

Вот последнее не работает при таком вызове:

Code
1
if (matr1 == matr2) cout << "равны";

Выдает такие ошибки:

Code
1
2
3
main.cpp:31: ошибка: no match for 'operator==' in 'matr1 == matr2'
 
matrix.h:96: candidates are: bool Matrix<T>::operator==(const Matrix<T>*) [with T = float]

Как заставить это работать?

silent_1991 · 21.10.2011, 23:58

serginhold, вы должны матрицу в оператор передавать не по указателю, а по ссылке. Или писать так:

C++
1
matr1 == (&matr2);

что не есть тру.

@serginhold · 22.10.2011, 00:13 **[ТС]**

Переписал вот так:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
// Добавил функцию get(i,j), получение элемента, сейчас ниже объясню почему.
// Изменил указатель на ссылку:
template <class T>
bool Matrix<T>::operator == (const Matrix<T> &a)
{
    // перегружаем оператор ==, сравнием матрицы по элементно
    if ((this->Rows != a.RowsCount()) || (this->Cols != a.ColsCount()))
        return false;
    for (int i=0; i<this->Rows; i++)
        for (int j=0; j<this->Cols; j++)
            if (this->_data[i][j] != a.get(i,j)) return false;   /* ! внимание на эту строку */
    return true;
}

Как видите я комментарием отметил интересующую меня строку
Если я ее запишу вот таким образом:

C++
1
if (this->_data[i][j] != a[i][j]) return false;

то по вылетают ошибки, поэтому пришлось создать функцию "get(int i, int j)const;" (просто попробовал)

Если я пишу сравнение через "get()" то все компилируется нормально, но при таком коде выдает всегда true:

C++
1
2
3
Matrix <float> m = matr;
m[1][1] = -67.1; // правлю элемент на "левый"
if (m == matr) cout << "\n равны"; else cout << "\n нет";

silent_1991 · 22.10.2011, 00:17

serginhold, опять же, у вас operator[] возвращает указатель, а должен возвращать ссылку.

Добавлено через 59 секунд
Теоретически можно было написать так:

C++
1
if (this->_data[i][j] != a._data[i][j]) return false;

тогда ошибок не будет, но operator[] так и останется неверно перегруженным.

@serginhold · 22.10.2011, 00:32 **[ТС]**

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

опять же, у вас operator[] возвращает указатель, а должен возвращать ссылку.

вот это вообще не въезжаю, все таки там где оператор [] я возвращаю массив _data[i], ну да ладно, забьем пока на это, самое интересное вот что:
перетащил я _data из private в public, переписал как раз вот так:

C++
1
if (this->_data[i][j] != a._data[i][j]) return false;

И все равно при вот таком коде(ниже), возвращает true :

C++
1
2
3
4
5
6
7
8
9
10
11
    Matrix <float> matr(2,2);
    matr[0][0] = 0; matr[0][1] = 1; matr[1][0] = 2; matr[1][1] = 3;
 
    matr.output(8,2);
 
    Matrix <float> m = matr;
    cout << "\n\n";
    m[1][1] = -67.1;
    m.output(8,2);
 
    if (m == matr) cout << "\n равны"; else cout << "\n нет";

silent_1991 · 22.10.2011, 00:59

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

там где оператор [] я возвращаю массив _data[i]

Да, действительно, не заметил этого.

Такой код работает верно. И в паблик ничего переносить не нужно.

C++
1
2
3
4
5
6
7
8
9
10
11
template <class T>
bool Matrix<T>::operator == (const Matrix<T> &a)
{
    // перегружаем оператор ==, сравнием матрицы по элементно
    if ((this->Rows != a.RowsCount()) || (this->Cols != a.ColsCount()))
        return false;
    for (int i=0; i<this->Rows; i++)
        for (int j=0; j<this->Cols; j++)
            if (this->_data[i][j] != a._data[i][j]) return false;
    return true;
}

@serginhold · 22.10.2011, 01:16 **[ТС]**

да, правда, все работает, немного по другому объявил матрицы и вот что вышло:

если так написать, то сравнивает нормально:

C++
1
2
3
4
5
6
7
8
9
10
11
    Matrix <float> matr(2,2);
    matr[0][0] = 0; matr[0][1] = 1; matr[1][0] = 2; matr[1][1] = 3;
 
    matr.output(8,2);
 
    Matrix <float> m(2,2); /* ! */
    cout << "\n\n";
    m[0][0] = 0; m[0][1] = 9; m[1][0] = 2; m[1][1] = 3;
    m.output(8,2);
 
    if (m == matr) cout << "\n ok"; else cout << "\n no";

А если вот так, то что-то его глючит..

C++
1
2
3
4
5
6
7
8
9
10
11
    Matrix <float> matr(2,2);
    matr[0][0] = 0; matr[0][1] = 1; matr[1][0] = 2; matr[1][1] = 3;
 
    matr.output(8,2);
 
    Matrix <float> m = matr; /* ! */
    cout << "\n\n";
    m[0][0] = 0; m[0][1] = 9; m[1][0] = 2; m[1][1] = 3;
    m.output(8,2);
 
    if (m == matr) cout << "\n ok"; else cout << "\n no";

Я так понял, так делать не правильно, или что?

@Net_Wanderer · 22.10.2011, 01:24

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

Я так понял, так делать не правильно, или что?

C++
1
Matrix <float> m = matr;

компилятор преобразовывает в

C++
1
Matrix <float> m(matr);

по этому проблема наверно в конструкторе копирования, я догадываюсь почему
У вас не определен copy constructor, по этому компилятор генерирует свой, который просто копирует все указатели, и получается что _data в m и в matr указывает на одну и туже область памяти; по этому когда вы изменяете m, изменения касаются и matr, и в конце сравниваются две одинаковые матрицы

@serginhold · 22.10.2011, 01:28 **[ТС]**

ясно, именно такого конструктора нет, будем знать)) Спасибо! =)

Новые блоги и статьи Все статьи Все блоги /
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .

@serginhold 2 / 2 / 0 Регистрация: 20.10.2011 Сообщений: 15
	22.10.2011, 01:28 [ТС]
	ясно, именно такого конструктора нет, будем знать)) Спасибо! =) 0