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

@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 **[ТС]**

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

Новые блоги и статьи Все статьи Все блоги /
Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .
Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .	SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .	SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .

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