Перегрузка операторов

@kuli4ok · Регистрация: 10.10.2012

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

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
#include <stdio.h>
#include <stdlib.h>
 
class matrix
{
protected:
    int **mass;
    int n;
public:
    matrix()
    {
        n = 5;
        mass = new int*[n];
        for(int i=0; i<n; i++)
        {
            mass[i] = new int[n];
        }
    }
    ~matrix()
    {
        for(int i=n-1; i>=0; i--)
        {
            delete [] mass[i];
        }
    }
    matrix(const matrix &o)
    {
        n = o.n;
        mass = new int*[n];
        for(int i=0; i<n; i++)
        {
            mass[i] = new int[n];
        }
        for(int i=0; i<n; i++)
        {
            for(int j=0; j<n; j++)
            {
                mass[i][j] = o.mass[i][j];
            }
        }
    }
    void show()
    {
        for(int i=0; i<n; i++)
        {
            for(int j=0; j<n; j++)
            {
                printf("%d ", mass[i][j]);
            }
            printf("\n");
        }
    }
    void get_rand()
    {
        for(int i=0; i<n; i++)
        {
            for(int j=0; j<n; j++)
            {
                mass[i][j] = rand()%10;
            }
        }
    }
    matrix operator+(matrix o)
    {
        matrix m;
        for(int i=0; i<n; i++)
        {
            for(int j=0; j<n; j++)
            {
                m.mass[i][j] = mass[i][j] + o.mass[i][j];
            }
        }
        return m;
    }
};
 
int main()
{
    matrix o1, o2;
    o1.get_rand();
    o2.get_rand();
    o1.show();
    printf("\n");
    o2.show();
    matrix o3;
    o3 =o1+o2;
    printf("\n");
    o3.show();
    return 0;
}

Здесь перегружен оператор + для сложения элементов матрицы, но почему то он складывает все элементы за исключением первого столбца.
Пример выходных данных:

3 6 7 5 3
5 6 2 9 1
2 7 0 9 3
6 0 6 2 6
1 8 7 9 2

0 2 3 7 5
9 2 2 8 9
7 3 6 1 2
9 3 1 9 4
7 8 4 5 0

134525712 8 10 12 8
134525736 8 4 17 10
134525760 10 6 10 5
134525784 3 7 11 10
0 16 11 14 2

Подскажите, пожалуйста, в чем проблема?
p.s. только начал изучать перегрузку операторов, сильно не пинайте

Croessmah · 17.11.2012, 17:44

Вы возвращаете временный объект.

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

C++
1
2
3
4
5
6
7
8
9
10
11
    matrix & operator+(matrix & o)
    {
        for(int i=0; i<n; i++)
        {
            for(int j=0; j<n; j++)
            {
                mass[i][j] = mass[i][j] + o.mass[i][j];
            }
        }
        return *this;
    }

попробуйте так(не проверял, просто взбрело в голову).

@kuli4ok · 17.11.2012, 17:46 **[ТС]**

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

Вы возвращаете временный объект.

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

C++
1
2
3
4
5
6
7
8
9
10
11
    matrix & operator+(matrix & o)
    {
        for(int i=0; i<n; i++)
        {
            for(int j=0; j<n; j++)
            {
                mass[i][j] = mass[i][j] + o.mass[i][j];
            }
        }
        return *this;
    }

попробуйте так(не проверял, просто взбрело в голову).

Да, теперь работает как надо, спасибо.

@DU · 17.11.2012, 17:50

проблема не в этом, а в том что у вас нет оператора =.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
matrix o3;
o3 =o1+o2;  // тут работает оператор = а не конструктор копирования. а дефолтный будет приводить к проблемам.
 
// вот это добавте к себе в код:
    matrix& operator = (const matrix &o)
    {
      matrix m(o);
      int** tmpMass = mass;
      int tmpN = n;
      mass = m.mass;
      n = m.n;
      m.mass = tmpMass;
      m.n = tmpN;
      return *this;
    }
 
//и привильная сигнатура operator + должна быть такой:
matrix operator + (const matrix & o) const;
 
matrix & operator+(matrix & o); // это для оператора +=
 
и еще в деструкторе у вас пропущено удаление
delete [] mass;

Croessmah · 17.11.2012, 17:53

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

проблема не в этом, а в том что у вас нет оператора =.

Есть конструктор копий

@kuli4ok · 17.11.2012, 17:54 **[ТС]**

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

проблема не в этом, а в том что у вас нет оператора =.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
matrix o3;
o3 =o1+o2;  // тут работает оператор = а не конструктор копирования. а дефолтный будет приводить к проблемам.
 
// вот это добавте к себе в код:
    matrix& operator = (const matrix &o)
    {
      matrix m(o);
      int** tmpMass = mass;
      int tmpN = n;
      mass = m.mass;
      n = m.n;
      m.mass = tmpMass;
      m.n = tmpN;
      return *this;
    }
 
//и привильная сигнатура operator + должна быть такой:
matrix operator + (const matrix & o) const;
 
matrix & operator+(matrix & o); // это для оператора +=
 
и еще в деструкторе у вас пропущено удаление
delete [] mass;

Ну я же говорю, что только начал перегрузку, а про перегруженный оператор присвоения еще не читал, точнее было написано, что будет рассмотрен далее, но все равно спасибо.
p.s. А вот с деструктором действительно пропустил, сам себе удивлюсь

@DU · 17.11.2012, 18:00

Есть конструктор копий

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

o3 =o1+o2 - что тут происходит.
в результате сложения создается временный объект, в котором результат сложения.
далее для o3 вызывается дефолтный оператор = которому передается ссылка на этот временный объект и оператор просто копирует указатели.
в результате временный объект и o3 указывают на один и тот же массив. потом временный объект удаляется,
и в своем деструкторе подчищает за собой. в результате в o3 переменная mass указывает на то, что только что удалил временный объект.

Croessmah · 17.11.2012, 18:07

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

o3 =o1+o2 - что тут происходит.

я функцию main не смотрел. Думал, что там все корректно.

Новые блоги и статьи Все статьи Все блоги /
сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .	Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .
Модель микоризы: классовый агентный подход 3 anaschu 06.01.2026 aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114	Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ФедосеевПавел 06.01.2026 Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .	Модель микоризы: классовый агентный подход 2 anaschu 06.01.2026 репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .	Расчёт токов в цепи постоянного тока igorrr37 05.01.2026 / * Дана цепь постоянного тока с сопротивлениями и источниками (напряжения, ЭДС и тока). Найти токи и напряжения во всех элементах. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа и. . .