Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.71/7: Рейтинг темы: голосов - 7, средняя оценка - 4.71
2 / 2 / 1
Регистрация: 26.06.2015
Сообщений: 56

Не работает перегруженный оператор + для класса динамической матрицы

13.01.2016, 22:44. Показов 1417. Ответов 12
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Подскажите, почему не работает перегруженный + для складывания двух матриц одного размера.

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
#include <iostream>
#include <iomanip>
using namespace std;
 
class Matrix
{
    int** matrix;
    size_t rows, cols;
 
public:
    Matrix();
    Matrix(size_t, size_t);
    ~Matrix();
    void Show();
    int* operator[](size_t);
    Matrix operator+(const Matrix&);
    Matrix& operator=(const Matrix&);
    Matrix& operator+=(const Matrix&);
};
 
Matrix::Matrix()
{
    matrix = nullptr;
    rows = cols = 0;
}
 
Matrix::Matrix(size_t r, size_t c)
{
    rows = r;
    cols = c;
    matrix = new int*[rows];
    for (size_t i = 0; i < rows; i++)
    {
        matrix[i] = new int[cols];
        for (size_t j = 0; j < cols; j++)
        {
            matrix[i][j] = rand() % 25;
        }
    }
}
 
Matrix::~Matrix()
{
    if (matrix)
    {
        for (size_t i = 0; i < rows; i++)
        {
            delete[] matrix[i];
        }
        delete[]matrix;
        matrix = nullptr;
    }
}
 
void Matrix::Show()
{
    for (size_t i = 0; i < rows; i++)
    {
        for (size_t j = 0; j < cols; j++)
        {
            cout << setw(4) << matrix[i][j];
        }
        cout << endl;
    }
    cout << endl;
}
Хочу сделать например, так:
C++
1
2
3
4
5
6
7
8
9
10
11
12
void main()
{
    Matrix A(3, 3);
    A.Show();
    Matrix B(3, 3);
    B.Show();
 
    
    Matrix C(3, 3);
    C = A + B;
    C.Show();
}
И вот на складывании вылетает.
0
Лучшие ответы (1)
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
13.01.2016, 22:44
Ответы с готовыми решениями:

Как для класса написать перегруженный оператор new?
пробую таким способом, но ошибка class X { public: X(int x) :mX(x) { } void SetX(int x) { mX=x; } int GetX() const ...

Перегруженный оператор сдвига '<<' не функции-члена класса для объекта ostream
Добрый день. В учебнике попалось на первый взгляд простое задание: #include &lt;iostream&gt; int main() { std::cout &lt;&lt;...

Определить перегруженный бинарный и унарный оператор* для класса "Комплексное число"
Как можно реализовать перегруженный бинарный и унарный оператор* для класса &quot;Комплексное число&quot;?

12
31 / 34 / 18
Регистрация: 30.09.2011
Сообщений: 202
13.01.2016, 22:57
Где код методов???
C++
1
2
3
4
int* operator[](size_t);
Matrix operator+(const Matrix&);
Matrix& operator=(const Matrix&);
Matrix& operator+=(const Matrix&);
0
2 / 2 / 1
Регистрация: 26.06.2015
Сообщений: 56
13.01.2016, 23:07  [ТС]
упс..

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
Matrix Matrix::operator+(const Matrix &obj)
{
    if (rows == obj.rows && cols == obj.cols)
    {
        Matrix temp(rows, cols);
                
        for (size_t i = 0; i < rows; i++)
        {
            for (size_t j = 0; j < cols; j++)
            {
                temp.matrix[i][j] = matrix[i][j] + obj.matrix[i][j];
            }
        }
        return temp;
    }
    else return Matrix();
}
 
Matrix & Matrix::operator=(const Matrix &obj)
{
    if (this == &obj)
        return *this;
 
    for (size_t i = 0; i < rows; i++)
    {
        delete[] matrix[i];
    }
    delete[]matrix;
 
    if (obj.rows && obj.cols)
    {
        rows = obj.rows;
        cols = obj.cols;
    }
    else rows = cols = 0;
 
    if (obj.matrix)
    {
        matrix = new int*[rows];
        for (size_t i = 0; i < rows; i++)
        {
            matrix[i] = new int[cols];
            for (size_t j = 0; j < cols; j++)
            {
                matrix[i][j] = obj.matrix[i][j];
            }
        }
    }
    else matrix = nullptr;
 
    return *this;
}
 
Matrix & Matrix::operator+=(const Matrix &obj)
{
    if (rows == obj.rows && cols == obj.cols)
    {
        *this = *this + obj;
    }   
    return *this;
}
Добавлено через 7 минут
мне почему-то кажется, что проблема в конструкторе с параметрами или не?
0
31 / 34 / 18
Регистрация: 30.09.2011
Сообщений: 202
13.01.2016, 23:13
нет, проблема в операторе присваивания...
0
2 / 2 / 1
Регистрация: 26.06.2015
Сообщений: 56
13.01.2016, 23:15  [ТС]
он работает, а если я в мэйне пишу A + B; тогда проблема
0
31 / 34 / 18
Регистрация: 30.09.2011
Сообщений: 202
13.01.2016, 23:19
operator + у тебя в порядке, а вот оператор присваивания барахлит... дай мне пару минут, я пытаюсь разобраться в чем беда)
1
19497 / 10102 / 2461
Регистрация: 30.01.2014
Сообщений: 17,808
13.01.2016, 23:27
Лучший ответ Сообщение было отмечено petuz как решение

Решение

petuz, конструктор копирования наверное Пушкин напишет?
1
31 / 34 / 18
Регистрация: 30.09.2011
Сообщений: 202
13.01.2016, 23:29
Да дело не в конструкторе копирования... вообщем деструктор в топку и все работает...
P.S: правда я пока не понял почему именно из-за него беда))))
0
2 / 2 / 1
Регистрация: 26.06.2015
Сообщений: 56
13.01.2016, 23:33  [ТС]
DrOffset, не было необходимости пока, да и задание еще в процессе...
0
265 / 165 / 56
Регистрация: 25.02.2015
Сообщений: 435
13.01.2016, 23:36
Matrix operator+(const Matrix&);
т.е. возвращается объект. этот объект конструируется за счет вызова конструктора копирования. т.к. кастомного конструктора копирования нет - используется автогенеренный, который все в конце и ломает. точнее все ломается в деструкторе, т.к. получается двойное удаление одного и того же.
2
19497 / 10102 / 2461
Регистрация: 30.01.2014
Сообщений: 17,808
13.01.2016, 23:38
Цитата Сообщение от petuz Посмотреть сообщение
не было необходимости пока
Теперь есть. Т.к. именно он в твоем коде очень нужен и именно из-за его отсутствия (вернее присутствия конструктора копии, сгенерированного компилятором, который копирует указатели и размерности, а не содержимое матрицы) у тебя и не работает. Если не очень отдаляться от твоего стиля кода, то так:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
Matrix::Matrix(Matrix const & x)
    : matrix(nullptr), rows(x.rows), cols(x.cols)
{
    if(rows && cols)
    {
        matrix = new int*[rows];
        for (size_t i = 0; i < rows; i++)
        {
            matrix[i] = new int[cols];
            for (size_t j = 0; j < cols; j++)
            {
                matrix[i][j] = x.matrix[i][j];
            }
        }
    }
}
Собственно, вот, онлайн пример: http://rextester.com/PKITR98936
1
2 / 2 / 1
Регистрация: 26.06.2015
Сообщений: 56
13.01.2016, 23:40  [ТС]
Perfilov, а в каком моменте происходит копирование? временный объект создается же вроде с помощью конструктора с параметрами.

ЗЫ. уже получил ответ.

Всем спасибо!!!
0
19497 / 10102 / 2461
Регистрация: 30.01.2014
Сообщений: 17,808
13.01.2016, 23:40
Цитата Сообщение от petuz Посмотреть сообщение
а в каком моменте происходит копирование?
На возврате из оператора +
1
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
13.01.2016, 23:40
Помогаю со студенческими работами здесь

Не работает перегруженный оператор
Привет, у меня есть класс такой: class Sterling { private: long pounds = 0; int shillings = 0; int pennys = 0; public:...

Дружественный перегруженный оператор не может обратиться к приватной переменной класса
#include&lt;iostream&gt; #include&lt;cstring&gt; #include&lt;fstream&gt; using namespace std; class phonebook { char name; char area; ...

Неправильно работает перегруженный оператор
#include &lt;iostream&gt; using namespace std; class PhoneNumber { friend ostream &amp;operator &lt;&lt; (ostream &amp;, const PhoneNumber &amp;); ...

Не работает перегруженный оператор присваивания
#include &lt;iostream&gt; #include &lt;string&gt; #include &lt;Windows.h&gt; class OutOfBoundsException {}; class MyString { private: unsigned...

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


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

Или воспользуйтесь поиском по форуму:
13
Ответ Создать тему
Новые блоги и статьи
Философия технологии
iceja 01.02.2026
На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .
SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf
8Observer8 01.02.2026
Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .
SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли
8Observer8 30.01.2026
Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .
SDL3 для Web (WebAssembly): Установка Emscripten SDK (emsdk) и CMake для сборки C и C++ приложений в Wasm
8Observer8 30.01.2026
Содержание блога Для того чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. . . .
SDL3 для Android: Подключение Box2D v3, физика и отрисовка коллайдеров
8Observer8 29.01.2026
Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .
Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT
bedvit 28.01.2026
Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .
SDL3 для Android: Загрузка PNG с альфа-каналом с помощью SDL_LoadPNG (без SDL3_image)
8Observer8 28.01.2026
Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .
SDL3 для Android: Загрузка PNG с альфа-каналом с помощью SDL3_image
8Observer8 27.01.2026
Содержание блога SDL3_image - это библиотека для загрузки и работы с изображениями. Эта пошаговая инструкция покажет, как загрузить и вывести на экран смартфона картинку с альфа-каналом, то есть с. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru