Исключение - нарушение доступа для чтения

@PavelRos · Регистрация: 03.10.2017

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

Добрый вечер, столкнулся с проблемой нарушения доступа для чтения 0х1110112. Задача - написать программу, которая будет умножать матрицу на число, перемножать матрицы и возводить их в квадрат, используя объекты.Матрицы как и их размерности считываются из файлов. Я еще только учусь, возможно много лишнего или косяков. Пожалуйста помогите.

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
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
#include "stdafx.h"
#include <iostream>
#include <fstream>
#define e endl;
 
using namespace std;
 
class Matrice_two;
 
class Matrice_one
{
public:
    int n;
    int m;
    int **arr;
 
public:
    Matrice_one() : n(0), m(0)
    { }
 
    Matrice_one(int x, int y)
    {
        n = x;
        m = y;
        int **arr = new int *[n];
        cout << "Adress" << &arr << arr << e;
 
        for (int i = 0; i < m; i++)
        {
            arr[i] = new int[m];
            cout << "Adress arr[i]" << " " << arr[i] << " " << e;
 
        }
    }
 
    int  **arr_init(int f, int g)
    {
        fstream fin("Matrice_one.txt", ios_base::in);
        for (int i = 0; i < f; i++)
            for (int j = 0; j < g; j++)
            {
                fin >> arr[i][j];
            }
        fin.close();
        return arr;
    }
 
    void arr_out(int h, int j)
    {
        ofstream fout("Matrice_res.txt", ios_base::out);
        for (int l = 0; l < h; l++)
        {
            for (int q = 0; q < j; q++)
            {
                fout << " " << arr[l][q] << " ";
            }
            fout << e;
        }
        fout.close();
    } //Печать матрицы в файл  
 
    friend void Matr_mult(Matrice_one &obj, Matrice_two &ob, int n, int m, int n1, int m1);// перемножение матриц
 
    void Matr_pow(int x, int y);//возведение матрицы в квадрат
 
    void Matr_mult_scal(int c)
    {
        for (int i = 0; i < n; i++)
            for (int j = 0; j < m; j++)
            {
                arr[i][j] = arr[i][j] * c;
            }
        arr_out(n, m);
    }// Умножение матрицы на скаляр
 
    friend Matrice_one operator * (Matrice_one &obj, Matrice_two &ob);
 
    ~Matrice_one()
    {
        cout << "Вызван деструктор" << e;
        for (int i = 0; i < n; i++)
        {
            delete[](arr[i]);
        }
        delete[] arr;
    }; //деструктор
 
};
 
void Matrice_one::Matr_pow(int x, int y)
{
    for (int i = 0; i < x; i++)
    {
        for (int k = 0; k < y; k++)
        {
            arr[i][k] = 0;
            for (int f = 0; f<y; f++)
            {
                arr[i][k] += arr[i][f] * arr[f][k];
            }
        }
    }
    arr_out(x, y);
}
 
class Matrice_two
{
public:
    int n1;
    int m1;
    int **arr1;
public:
    Matrice_two() :n1(0), m1(0)
    { }
 
    Matrice_two(int x, int y)
    {
        n1 = x;
        m1 = y;
        int **arr1 = new int *[n1];
        for (int i = 0; i < m1; i++)
            arr1[i] = new int[m1];
    }
 
    int  **arr_init1(int f, int g)
    {
        fstream fin("Matrice_two.txt", ios_base::in);
        for (int i = 0; i < f; i++)
            for (int j = 0; j <g; j++)
            {
                fin >> arr1[i][j];
            }
        fin.close();
        return arr1;
    }
 
    friend void Matr_mult(Matrice_one &obj, Matrice_two &ob, int n, int m, int n1, int m1);
 
    friend Matrice_one operator * (Matrice_one &obj, Matrice_two &ob);
 
    ~Matrice_two()
    {
        cout << "Вызван деструктор второй матрицы" << e;
        for (int i = 0; i < n1; i++)
        {
            delete[](arr1[i]);
        }
        delete[] arr1;
    }
};
 
void Matr_mult(Matrice_one &obj, Matrice_two &ob, int n, int m, int n1, int m1)
{
    Matrice_one ob1(n, m1);
    obj = obj*ob;
    obj.arr_out(n, m1);
}
 
Matrice_one operator * (Matrice_one &obj, Matrice_two &ob)
{
    Matrice_one temp2(obj.n, ob.m1);
    for (int i = 0; i < obj.n; i++)
    {
        for (int k = 0; k < ob.m1; k++)
        {
            temp2.arr[i][k] = 0;
            for (int f = 0; f<obj.m; f++)
            {
                temp2.arr[i][k] += obj.arr[i][f] * ob.arr1[f][k];
            }
        }
    }
    return temp2;
}
 
int main()
{
    setlocale(LC_ALL, "");
    int x, y;
    fstream fin("Matrise_one_size.txt", ios_base::in);
    if (fin.is_open())
    {
        cout << "Файл открылся" << e;
    }
    else
    {
        cout << "Файл не открылся" << e;
    }
    fin >> x >> y;
    fin.close();
    Matrice_one object(x, y);
    object.arr = object.arr_init(x, y);
    int a;
    cout << "Выберите операцию. 1- Возведение в квадрат, 2- Умножение матрицы на скаляр 3- Перемножение матриц" << e;
    cin >> a;
    switch (a)
    {
    case 1://возведение в квадрат
    {
        object.Matr_pow(x, y);
        object.arr_out(x, y);
        break;
    }
    case 2://умножение на скаляр
    {
        int multipe = 0;
        cout << "Введите множитель" << e;
        cin >> multipe;
        object.Matr_mult_scal(multipe);
        object.arr_out(x, y);
        break;
    }
    case 3://перемножение двух матриц
    {
 
        int x1, y1;
        fstream fin("Matrice_two_size.txt", ios_base::in);
        if (fin.is_open())
            cout << "Файл открылся" << endl;
        else
            cout << "Файл не открылся" << e;
        fin >> x1 >> y1;
        fin.close();
        Matrice_two object1(x1, y1);
        object1.arr1 = object1.arr_init1(x1, y1);
        Matr_mult(object, object1, x, y, x1, y1);
        break;
    }
    }
 
    return 0;
}

@nmcf · 07.11.2018, 21:41

C++
1
2
    Matrice_one() : n(0), m(0), arr(nullptr)
    { }

У тебя неправильная реализация. Размер хранится в классе, его не нужно везде передавать. Кроме конструктора.
И размер можно сразу в файл с числами записать и считывать в arr_init(), создавая массив нужного размера.

@PavelRos · 07.11.2018, 22:49 **[ТС]**

Я внес корректировки в текст программы, убрал передачу размеров из всех функций, где можно обойтись без него, добавил в конструкторы arr(nullptr), нарушение чтения все равно есть и останавливается на 42 строчке.

@nmcf · 07.11.2018, 23:35

Актуальный вариант показывай.

@PavelRos · 07.11.2018, 23:41 **[ТС]**

Вот последний.

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
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
#include "stdafx.h"
#include <iostream>
#include <fstream>
#define e endl;
 
using namespace std;
 
class Matrice_two;
 
class Matrice_one
{
public:
    int n;
    int m;
    int **arr;
 
public:
    Matrice_one() : n(0), m(0), arr(nullptr)
    { }
 
    Matrice_one(int x, int y)
    {
        n = x;
        m = y;
        int **arr = new int *[n];
        cout << "Adress" << &arr << arr << e;
 
        for (int i = 0; i < m; i++)
        {
            arr[i] = new int[m];
            cout << "Adress arr[i]" << " " << arr[i] << " " << e;
 
        }
    }
 
    int  **arr_init()
    {
        ifstream fin("Matrice_one.txt", ios_base::in);
        for (int i = 0; i < n; i++)
            for (int j = 0; j < m; j++)
            {
                fin >> arr[i][j];
            }
        fin.close();
        return arr;
    }
 
    void arr_out()
    {
        ofstream fout("Matrice_res.txt", ios_base::out);
        for (int l = 0; l < n; l++)
        {
            for (int q = 0; q < m; q++)
            {
                fout << " " << arr[l][q] << " ";
            }
            fout << e;
        }
        fout.close();
    } //Печать матрицы в файл  
 
    friend void Matr_mult(Matrice_one &obj, Matrice_two &ob, int n, int m, int n1, int m1);// перемножение матриц
 
    void Matr_pow();//возведение матрицы в квадрат
 
    void Matr_mult_scal(int c)
    {
        for (int i = 0; i < n; i++)
            for (int j = 0; j < m; j++)
            {
                arr[i][j] = arr[i][j] * c;
            }
        arr_out();
    }// Умножение матрицы на скаляр
 
    friend Matrice_one operator * (Matrice_one &obj, Matrice_two &ob);
 
    ~Matrice_one()
    {
        cout << "Вызван деструктор" << e;
        for (int i = 0; i < n; i++)
        {
            delete[](arr[i]);
        }
        delete[] arr;
    }; //деструктор
 
};
 
void Matrice_one::Matr_pow()
{
    for (int i = 0; i < n; i++)
    {
        for (int k = 0; k < m; k++)
        {
            arr[i][k] = 0;
            for (int f = 0; f< m; f++)
            {
                arr[i][k] += arr[i][f] * arr[f][k];
            }
        }
    }
    arr_out();
}
 
class Matrice_two
{
public:
    int n1;
    int m1;
    int **arr1;
public:
    Matrice_two() :n1(0), m1(0), arr1(nullptr)
    { }
 
    Matrice_two(int x, int y)
    {
        n1 = x;
        m1 = y;
        int **arr1 = new int *[n1];
        for (int i = 0; i < m1; i++)
            arr1[i] = new int[m1];
    }
 
    int  **arr_init1()
    {
        ifstream fin("Matrice_two.txt", ios_base::in);
        for (int i = 0; i < n1; i++)
            for (int j = 0; j < m1; j++)
            {
                fin >> arr1[i][j];
            }
        fin.close();
        return arr1;
    }
 
    friend void Matr_mult(Matrice_one &obj, Matrice_two &ob, int n, int m, int n1, int m1);
 
    friend Matrice_one operator * (Matrice_one &obj, Matrice_two &ob);
 
    ~Matrice_two()
    {
        cout << "Вызван деструктор второй матрицы" << e;
        for (int i = 0; i < n1; i++)
        {
            delete[](arr1[i]);
        }
        delete[] arr1;
    }
};
 
void Matr_mult(Matrice_one &obj, Matrice_two &ob, int n, int m, int n1, int m1)
{
    Matrice_one ob1(n, m1);
    ob1 = obj*ob;
    ob1.arr_out();
}
 
 
Matrice_one operator * (Matrice_one &obj, Matrice_two &ob)
{
    Matrice_one temp2(obj.n, ob.m1);
    for (int i = 0; i < obj.n; i++)
    {
        for (int k = 0; k < ob.m1; k++)
        {
            temp2.arr[i][k] = 0;
            for (int f = 0; f<obj.m; f++)
            {
                temp2.arr[i][k] += obj.arr[i][f] * ob.arr1[f][k];
            }
        }
    }
    return temp2;
}
 
int main()
{
    setlocale(LC_ALL, "");
    int x, y;
    ifstream fin("Matrise_one_size.txt", ios_base::in);
    if (fin.is_open())
    {
        cout << "Файл открылся" << e;
    }
    else
    {
        cout << "Файл не открылся" << e;
    }
 
    fin >> x >> y;
    fin.close();
    Matrice_one object(x, y);
    object.arr = object.arr_init();
    int a;
    cout << "Выберите операцию. 1- Возведение в квадрат, 2- Умножение матрицы на скаляр 3- Перемножение матриц" << e;
    cin >> a;
    switch (a)
    {
    case 1://возведение в квадрат
    {
        object.Matr_pow();
        object.arr_out();
        break;
    }
    case 2://умножение на скаляр
    {
        int multipe = 0;
        cout << "Введите множитель" << e;
        cin >> multipe;
        object.Matr_mult_scal(multipe);
        object.arr_out();
        break;
    }
    case 3://перемножение двух матриц
    {
 
        int x1, y1;
        ifstream fin("Matrice_two_size.txt", ios_base::in);
        if (fin.is_open())
            cout << "Файл открылся" << endl;
        else
            cout << "Файл не открылся" << e;
        fin >> x1 >> y1;
        fin.close();
        Matrice_two object1(x1, y1);
        object1.arr1 = object1.arr_init1();
        Matr_mult(object, object1, x, y, x1, y1);
        break;
    }
    }
 
    return 0;
}

@valen10 · 08.11.2018, 00:43

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

C++
1
2
3
4
5
int **arr = new int *[n];
        cout << "Adress" << &arr << arr << e;
 
        for (int i = 0; i < m; i++)
        {

Выделена память для указателей на n строк, но в цикле заполняются только m указателей строк, что является ошибкой. Если m < n, то не для всех строк будет выделена память, и вместо указателей строк с номерами >= m будет мусор. Если m > n, тогда произойдет выход за границы массива указателей arr.

Исправьте на:

C++
1
for (int i = 0; i < n; i++)

В классе Matrice_two та же ошибка. Кстати, зачем два абсолютно одинаковых класса?

@PavelRos · 08.11.2018, 13:39 **[ТС]**

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

Добавлено через 41 секунду
Два класса реализовано исходя из условия задания. Можно было бы обойтись и одним, да.

@valen10 · 08.11.2018, 16:07

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

Но возникли проблемы с высвобождением памяти в деструкторе , поскольку массив двумерный

Какие проблемы? Освобождение у вас правильно написано.

Новые блоги и статьи Все статьи Все блоги /
Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .	Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .
SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .	Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .

@PavelRos 0 / 0 / 0 Регистрация: 03.10.2017 Сообщений: 8
	07.11.2018, 22:49 [ТС]
	Я внес корректировки в текст программы, убрал передачу размеров из всех функций, где можно обойтись без него, добавил в конструкторы arr(nullptr), нарушение чтения все равно есть и останавливается на 42 строчке. 0

@nmcf 7804 / 6568 / 2988 Регистрация: 14.04.2014 Сообщений: 28,705
	07.11.2018, 23:35
	Актуальный вариант показывай. 0

@PavelRos 0 / 0 / 0 Регистрация: 03.10.2017 Сообщений: 8
	08.11.2018, 13:39 [ТС]
	Я исправил ошибку, неправильно выделена память в конструкторе. Но возникли проблемы с высвобождением памяти в деструкторе , поскольку массив двумерный. Подскажите пожалуйста как ее правильно освободить. Без деструктора программа работает как и должна. Добавлено через 41 секунду Два класса реализовано исходя из условия задания. Можно было бы обойтись и одним, да. 0