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

@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 не смотрел. Думал, что там все корректно.

Новые блоги и статьи Все статьи Все блоги /
Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Символьное дифференцирование igorrr37 13.02.2026 / * Программа принимает математическое выражение в виде строки и выдаёт его производную в виде строки и вычисляет значение производной при заданном х Логарифм записывается как: (x-2)log(x^2+2) -. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .