Форум программистов, компьютерный форум, киберфорум
Наши страницы
С++ для начинающих
Войти
Регистрация
Восстановить пароль
 
Pewpewpewpew
4 / 4 / 2
Регистрация: 09.07.2016
Сообщений: 62
#1

Ошибка при сложении матриц - C++

16.02.2017, 17:14. Просмотров 217. Ответов 4

Если убрать деструктор, ошибки нет.
Подскажите, пожалуйста, в чем проблема?
Кликните здесь для просмотра всего текста
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
91
92
93
94
95
96
97
98
99
100
101
102
103
104
#include "stdafx.h"
#include "iostream"
#include "iomanip"
 
using namespace std;
 
class Matrix
{
private:
    int** p;
 
    int rows;
    int cols;
 
public:
 
    Matrix(int r, int c) : rows(r), cols(c)
    {
        p = new int*[r];
        for (int ix = 0; ix < r; ix++)
        {
            p[ix] = new int[c];
            for (int jx = 0; jx < c; jx++)
                p[ix][jx] = ix+jx;
        }
 
    }
 
    ~Matrix()
    {
 
        for (int i = 0; i < rows; i++)
            delete[] p[i];
        delete[] p;
        cout << "Mem is clean" << endl;
    }
    
    class Proxy
    {
    private:
        int* arr;
 
    public:
        Proxy(int* arr) : arr(arr)  {}
        int operator[](int index) {
            //cout << ("1") << endl;
            return arr[index-1];
        }
 
    };
    Proxy operator[] (int index) {
        //cout << ("2") << endl;
        return Proxy(p[index-1]);
    }
    friend ostream& operator<<(ostream& a, Matrix& b)
    {
        for (int ix = 0; ix < b.rows; ix++) 
        {
            for (int jx = 0; jx < b.cols; jx++)
                a << setw(5) << b.p[ix][jx];
            cout << endl;
        }
        a << endl;
        return a;
    }
    Matrix operator* (Matrix m)
    {
        Matrix output_m(cols,m.rows);
        for (int i = 0; i < cols; i++)
            for (int j = 0; j < m.rows; j++)
            {
                //for (int z = 0; z < 4; z++)
                    //output_m[i][j] += m[i][z] * p[z][j];
                
            }
 
    }
    
    Matrix operator+ (const Matrix &m)
    {
        Matrix output_m(m.rows, m.cols);
 
        for (int i = 0; i < m.rows; i++)
            for (int j = 0; j < m.cols; j++)
                output_m.p[i][j] = p[i][j] + m.p[i][j];
        return output_m;
    }
 
};
 
int main()
{
    Matrix m(3, 4);
    Matrix m2(3, 4);
    Matrix m3(3, 4);
    
    cout << m2 << endl;
 
    m3 = m + m2;
    cout << m3 << endl;
 
    system("pause");
    return 0;
}

http://www.cyberforum.ru/cpp-beginners/thread328852.html
0
Миниатюры
Ошибка при сложении матриц  
Лучшие ответы (1)
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
16.02.2017, 17:14
Я подобрал для вас темы с готовыми решениями и ответами на вопрос Ошибка при сложении матриц (C++):

Непонятная ошибка при сложении строк
добрый вечер! Возникла проблема в программе: string str=&quot;D:\\Tab&quot;+n+&quot;.txt&quot;...

Порядок вычисления операндов при сложении
Есть такой пример int I = 2, k; k = (I=10)*(I=20); Чему в этом...

Непонятное значение в переменной при сложении
Всем привет ) не занимался с++ уже два года и всё позабывал ) как можно...

Ошибка при транспонировании матриц
всем привет. Ошибка заключается в том ,что при указании размера любой из 3-х...

Ошибка вывода при перемножении матриц
Здравствуйте! Программирую на C++ совсем недавно, столкнулся с проблемой, при...

4
likehood
930 / 789 / 378
Регистрация: 25.12.2016
Сообщений: 2,627
Завершенные тесты: 3
16.02.2017, 17:59 #2
Не хватает перегрузки оператора присваивания.
0
Pewpewpewpew
4 / 4 / 2
Регистрация: 09.07.2016
Сообщений: 62
16.02.2017, 18:01  [ТС] #3
Цитата Сообщение от likehood Посмотреть сообщение
Не хватает перегрузки оператора присваивания.
Если сделать так:
C++
1
cout << m + m2;
Все равно ошибка, хотя присваивания нет.
0
likehood
930 / 789 / 378
Регистрация: 25.12.2016
Сообщений: 2,627
Завершенные тесты: 3
16.02.2017, 18:33 #4
И ещё не хватает конструктора копирования. Иначе при неявном копировании будет создан объект с тем же указателем p, который в итоге будет удалён дважды.
1
VAN0
58 / 58 / 53
Регистрация: 05.05.2013
Сообщений: 150
Завершенные тесты: 1
16.02.2017, 18:49 #5
Лучший ответ Сообщение было отмечено Pewpewpewpew как решение

Решение

Цитата Сообщение от Pewpewpewpew Посмотреть сообщение
Все равно ошибка, хотя присваивания нет.
Проблема в том, что ты создаешь временный объект в методе оператор+, а при выходе из метода объект уничтожается и его память высвобождается деструктором. Чтобы этого избежать, временный объект нужно выделять в куче (в динамической памяти)

C++
1
2
3
4
5
6
7
8
9
10
Matrix& operator+ (const Matrix &m)
    {
        Matrix* output_m = new Matrix (m.rows, m.cols);
        
 
        for (int i = 0; i < m.rows; i++)
            for (int j = 0; j < m.cols; j++)
                output_m->p[i][j] = p[i][j] + m.p[i][j];
        return *output_m;
    }
1
16.02.2017, 18:49
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
16.02.2017, 18:49
Привет! Вот еще темы с решениями:

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

При сложении переменных типа char выводится код символа
Почему когда говоришь вывести символ &quot;а&quot; выводится &quot;А&quot; а когда просишь вывести...

Написать процедуру для суммирования матриц. Ошибка при передаче массива в функцию
Помогите пожалуйста! Дано задание: Написать процедуру для суммирования матриц....

поиск среди чисел такого, которое при возведении в куб и сложении со всеми делителями является квадратом какого-нибудь числа
Необходимо написать функцию, которая в качестве аргументов принимает промежуток...


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

Или воспользуйтесь поиском по форуму:
5
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2018, vBulletin Solutions, Inc.
Рейтинг@Mail.ru