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

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

17.09.2017, 21:50. Показов 1554. Ответов 18
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Помогите, товарищи, пожалуйста. пытаюсь перегрузить оператор для того, чтобы можно было складывать матрицы. Не могу понять, где в программе идет сбой. В конце вылазит ошибка "Access violation reading....."
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
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
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
#include <iostream>
#include <cstdlib>
#include <conio.h>
#include <Windows.h>
#include <locale>
//const int ERR_EXIT = -1;
using namespace std;
 
class Matrix {
private:
    int **Mptr;
    int m;
    int n;
 
    void Create()
    {cout<<"Srabotal constructor!"<<endl;
        Mptr = new int*[m];
        for (int z=0; z<m;z++)
            Mptr[z] = new int[n];
    }
 
public:
    //constructors and destructor
 
    Matrix(): m(5),n(5) { Create();}//or Matrix() { this->Mptr=NULL;this->n=0;this->m=0;}
    Matrix(int i): m(i),n(i) { Create();}
    Matrix(int i, int j): m(i),n(j) {Create();}
    ~Matrix()
    {
        cout<<"Srabotal destructor!"<<endl;
        for (int z=0;z<m;z++)
            delete [] Mptr[z];
        delete [] Mptr;
    }
 
    Matrix (const Matrix &ob) 
    {
        cout<<"Srabotal constructor copirovanija"<<endl;
        n = ob.n;
        m = ob.m;
        Mptr = new int*[m];
        for ( int i = 0; i < m; ++i)
            Mptr[i] = new int[n];
 
        for (int i=0;i<ob.m;i++)
        {
            for (int j=0;j<ob.n;j++)
                Mptr[i][j]=ob.Mptr[i][j];
        }
    } 
 
 
    Matrix operator+ (Matrix &A);//перегрузка оператора сложения матриц
 
    int& Element(long int i, long int j)
    {
        if (i<m && j<n)
            return Mptr[i][j];
        else
            cout << "Error: 1";
    }
    void Display()
    {
        for(int i=0; i<m; i++)
        {
            for(int j=0; j<n; j++)
            {
                cout.width(4);
                cout << Mptr[i][j];
            }
            cout << endl;
        }
    }
};
Matrix Matrix::operator+(Matrix &A)
{
 
    int i,j;
    if ( n != A.n || m != A.m )
        cout << "Матрица не совместима с операцией +";
    //exit(ERR_EXIT);
    else 
    {
        Matrix Result(m,n);
        for (i = 0; i < m; i++)
        {
            for (j = 0; j < n; j++)
                Result.Mptr[i][j] = A.Mptr[i][j] + Mptr[i][j];
        }
        return Result;
    }
}
 
 
 
 
int main()
{
    SetConsoleCP(1251); 
    SetConsoleOutputCP(1251);
    setlocale(LC_ALL,"Russian");
 
    Matrix M1(3,4);//realize cin matr
    for (int i=0; i<3; i++)
    {
        for (int j=0; j<4; j++)
            M1.Element(i, j) = i+j;
    }
    system("pause");
    Matrix M2(M1);
    M2.Display();
    system("pause");
 
    Matrix C(3,4);
    system("pause");
    Matrix L(3,4);
    L = M1 + C;
    //M1.Display();
    _getch();
    L.Display();
    system("PAUSE");
    return 0;
}
Добавлено через 9 минут
У меня есть предположение, что проблема в функции display. По отладчику я дошел до туда. Видимо нужно что-то сделать с указателем this, т.к. я понял, что указатель вытворяет какую-то дичь
0
Лучшие ответы (1)
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
17.09.2017, 21:50
Ответы с готовыми решениями:

Перегрузка оператора +
Добрый день)Почему то не работает перегрузка оператора сложения) #include &lt;iostream&gt; #include &lt;string&gt; #include...

Перегрузка оператора +
не получается в своем классе перегрузить оператор + :( пишет &quot;error C2804: бинарный &quot;operator +&quot; имеет слишком много...

Перегрузка оператора +=
Есть два класса: Vec4(вектор из 4х float) и Mat4(вектор из 4х Vec4). Проблема в том что при перегрузке в Mat4 операции += я получаю ошибку...

18
nd2
3438 / 2817 / 1249
Регистрация: 29.01.2016
Сообщений: 9,427
17.09.2017, 21:55
Оператор присваивания нужен.
0
322 / 174 / 78
Регистрация: 09.10.2014
Сообщений: 809
17.09.2017, 22:00
del
0
1 / 1 / 0
Регистрация: 27.08.2016
Сообщений: 31
17.09.2017, 22:02  [ТС]
lArtl, Можно немного по конкретнее? Направьте в правильное русло
0
322 / 174 / 78
Регистрация: 09.10.2014
Сообщений: 809
17.09.2017, 22:03
Цитата Сообщение от timashenka Посмотреть сообщение
lArtl, Можно немного по конкретнее? Направьте в правильное русло
Я не досмотрел.
0
1 / 1 / 0
Регистрация: 27.08.2016
Сообщений: 31
17.09.2017, 22:11  [ТС]
nd2, уточни.
0
nd2
3438 / 2817 / 1249
Регистрация: 29.01.2016
Сообщений: 9,427
17.09.2017, 22:35
Цитата Сообщение от timashenka Посмотреть сообщение
nd2, уточни.
Уточняю:
Цитата Сообщение от nd2 Посмотреть сообщение
Оператор присваивания нужен.
0
1 / 1 / 0
Регистрация: 27.08.2016
Сообщений: 31
17.09.2017, 22:57  [ТС]
nd2, Добавил. Не помогло
Code
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
Matrix Matrix::operator=(const Matrix &A)
{
    int i,j;
    for (i = 0; i < n; i++)
        delete (Mptr[i]);
        delete (Mptr);
    m = A.m;
    n = A.n;
      for (j = 0; j < A.m; j++)
    {
        for (i = 0; i < A.n; i++)
        {
            Mptr[i][j] = A.Mptr[i][j];
        }
    }
 
    return *this;
}
0
nd2
3438 / 2817 / 1249
Регистрация: 29.01.2016
Сообщений: 9,427
17.09.2017, 23:04
Цитата Сообщение от timashenka Посмотреть сообщение
C++
1
for (i = 0; i < n; i++)
У тебя разве первая размерность n ? Где выделение новой памяти?
0
1 / 1 / 0
Регистрация: 27.08.2016
Сообщений: 31
17.09.2017, 23:09  [ТС]
nd2, Изменил. Теперь выбивает на строке
Code
1
Mptr[i][j] = A.Mptr[i][j];
в
Code
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
Matrix Matrix::operator=(const Matrix &A)
{
    int i,j;
    for (i = 0; i < m; i++)
        delete (Mptr[i]);
        delete (Mptr);
    m = A.m;
    n = A.n;
      for (j = 0; j < A.m; j++)
    {
        for (i = 0; i < A.n; i++)
        {
            Mptr[i][j] = A.Mptr[i][j];
        }
    }
 
    return *this;
}
Хелп(
0
nd2
3438 / 2817 / 1249
Регистрация: 29.01.2016
Сообщений: 9,427
17.09.2017, 23:13
Цитата Сообщение от timashenka Посмотреть сообщение
C++
1
2
delete (Mptr[i]);
 delete (Mptr);
Если память выделялась через new [], то и освобождать нужно через delete [].
Цитата Сообщение от timashenka Посмотреть сообщение
Изменил
???
Цитата Сообщение от nd2 Посмотреть сообщение
Где выделение новой памяти?
0
1 / 1 / 0
Регистрация: 27.08.2016
Сообщений: 31
17.09.2017, 23:17  [ТС]
Цитата Сообщение от nd2 Посмотреть сообщение
Где выделение новой памяти?
Code
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
Matrix Matrix::operator=(const Matrix &A)
{
    int i,j;
    for (i = 0; i < m; i++)
        delete [] Mptr[i];
        delete [] Mptr;
    m = A.m;
    n = A.n;
    Mptr = new int*[m];
        for (int z=0; z<m;z++)
            Mptr[z] = new int[n];
      for (j = 0; j < A.m; j++)
    {
        for (i = 0; i < A.n; i++)
        {
            Mptr[i][j] = A.Mptr[i][j];
        }
    }
 
    return *this;
}
Обьясни немного, для чего здесь нужно перегружать оператор присваивания. И что не так
0
322 / 174 / 78
Регистрация: 09.10.2014
Сообщений: 809
17.09.2017, 23:23
Лучший ответ Сообщение было отмечено timashenka как решение

Решение

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
    Matrix& operator=(const Matrix& ob)
    {
        n = ob.n;
        m = ob.m;
        Mptr = new int*[m];
        for (int i = 0; i < m; ++i)
            Mptr[i] = new int[n];
 
        for (int i = 0; i<ob.m; i++)
        {
            for (int j = 0; j<ob.n; j++)
                Mptr[i][j] = ob.Mptr[i][j];
        }
        return *this;
    }
Цитата Сообщение от timashenka Посмотреть сообщение
Обьясни немного, для чего здесь нужно перегружать оператор присваивания
C++
1
2
3
4
A a; //конструктор по умолчанию
A a1(a);//конструктор копирования
 
a = a1; //Оператор =
0
1 / 1 / 0
Регистрация: 27.08.2016
Сообщений: 31
17.09.2017, 23:23  [ТС]
HALLELLUJAH! lArtl,Готов целовать твои ноги.
1
nd2
3438 / 2817 / 1249
Регистрация: 29.01.2016
Сообщений: 9,427
17.09.2017, 23:25
lArtl, старую память нужно освобождать.
Цитата Сообщение от timashenka Посмотреть сообщение
Обьясни немного, для чего здесь нужно перегружать оператор присваивания.
Чтобы работало это:
Цитата Сообщение от timashenka Посмотреть сообщение
C++
1
L = M1 + C;
1
1 / 1 / 0
Регистрация: 27.08.2016
Сообщений: 31
17.09.2017, 23:30  [ТС]
Цитата Сообщение от nd2 Посмотреть сообщение
Чтобы работало это:
Т.е. изначально я создал объект L и у него был определенный адрес. А после сложения я этому объекту присваиваю память того объекта, что получился при сложении(в данном случае адрес матрицы Result?). Ну и плюс старую память освобождаю
0
nd2
3438 / 2817 / 1249
Регистрация: 29.01.2016
Сообщений: 9,427
17.09.2017, 23:35
Цитата Сообщение от timashenka Посмотреть сообщение
Т.е. изначально я создал объект L и у него был определенный адрес. А после сложения я этому объекту присваиваю память того объекта, что получился при сложении(в данном случае адрес матрицы Result?). Ну и плюс старую память освобождаю
Адреса объектов (и память под ними) тут не при чём. Дело в копировании данных, адреса которых находятся в указателях в объектах.
0
322 / 174 / 78
Регистрация: 09.10.2014
Сообщений: 809
17.09.2017, 23:37
Цитата Сообщение от nd2 Посмотреть сообщение
старую память нужно освобождать.
Или новую не выделять.
0
nd2
3438 / 2817 / 1249
Регистрация: 29.01.2016
Сообщений: 9,427
17.09.2017, 23:39
Цитата Сообщение от lArtl Посмотреть сообщение
Или новую не выделять.
Тогда нужно проверять размер старой, и пр. Но, вообще, присваивание предполагает удаление старых данных.
1
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
17.09.2017, 23:39
Помогаю со студенческими работами здесь

Перегрузка оператора ++
выводит не то значение привет. нужна Ваша подсказка. листинг из учебника: (тема: перегрузка оператора ++) #include &lt;iostream&gt; ...

Перегрузка оператора []
Есть: class any{ char arr }

Перегрузка оператора
Добрый день! Мне нужно перегрузить в классе двумерного массива оператор ++, чтобы она возвращал максимальный элемент массива. класс...

Перегрузка оператора <<
Как вот в таком классе перегрузить оператор &lt;&lt;? #ifndef MATRIXX_H #define MATRIXX_H #include &lt;iostream&gt; using...

Перегрузка оператора[]
Пример взят из книги Лафоре, тема перегрузка операций. Выдает ошибки: 1.Требуется идентификатор. sa1 = i * 10; 2.Использование...


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

Или воспользуйтесь поиском по форуму:
19
Ответ Создать тему
Новые блоги и статьи
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
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
Ломающие изменения в C#.NStar Alpha
Etyuhibosecyu 20.11.2025
Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .
Мысли в слух
kumehtar 18.11.2025
Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .
Создание Single Page Application на фреймах
krapotkin 16.11.2025
Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru