Класс матрица (не работает конструктор копирования)

@qwer_11 · Регистрация: 30.10.2011

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

так вот,что-то глючит конструктор копирования, хотя ошибок компиляции не дает((

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
235
236
237
238
239
240
#include <fstream>
#include <iostream>
#include <cstdlib>
using namespace std;
 
class Matrix 
{
    int i,j;
    int **mas;
public:
    Matrix();
    Matrix(int N, int M);
    Matrix(int N, int M, bool none);
    Matrix (const Matrix&);
    ~Matrix();
    void SetSize();
    void Vivod();
    void Vvod_elements();
    void Vvod_random(int N, int M);
    void SetEl(int N, int M);
    int GetEl(int N, int M);
    int Get_i(Matrix ob);
    int Get_j(Matrix ob);
    friend Matrix operator+(Matrix ob1, Matrix ob);
    Matrix operator-(Matrix ob1);
    Matrix operator*(Matrix ob1);
    friend ostream  &operator<<(ostream &stream, Matrix ob);
    friend bool operator ==(Matrix ob, Matrix ob1);
};
 
int main()
{
    Matrix ob3(4,4, true);
    ob3.Vivod();
    Matrix ob1(ob3);
    ob1.Vivod();
    ob1.~Matrix();
    ob3.~Matrix();
    return 0;
}
 
            //Конструкторы класса
Matrix::Matrix()
{
    SetSize();
     int **mas=new int *[i];
     for (int n=0; n<i; n++)
         mas=new int *[j];
    Vvod_elements();
}//Создает массив вручную с вводом размерности
Matrix::Matrix(int N, int M)
{
    i=N;
    j=M;
     int **mas=new int *[i];
     for (int n=0; n<i; n++)
         mas=new int *[j];
    Vvod_elements();
}              //Создает массив, автомотически заполняющийся, с вводом размерности
Matrix::Matrix(int N, int M, bool none)
{
     int **mas=new int *[i];
     for (int n=0; n<i; n++)
         mas=new int *[j];
     if (none==true)
    {
        i=N;
        j=M;
        Vvod_random(N, M);
    }
}
Matrix::~Matrix()
{
    delete [] mas;
}
int Matrix::Get_i(Matrix ob)
{
    return ob.i;
}// Получение № столбца 
int Matrix::Get_j(Matrix ob)
{
    return ob.j;
}//Ввод размеров матрицы 
void Matrix::SetSize()
{
    cout << "vvodite i"<<endl;
    cin >> i;
    cout << "vvodite j"<<endl;
    cin >> j;
}//Вывод на консоль 
void Matrix::Vivod()
{
    
    for (int n=0;n<i;n++)
    {
        for(int m=0;m<j;m++)
            cout<<mas[n][m]<<"\t";
        cout<< "\n";
 
    }
}// Ручной ввод элементов 
void Matrix::Vvod_elements()
{
    cout <<"vvodite matricy\n";
    for (int t = 0; t < i; t++)
    {
        for (int k = 0; k < j; k++)
        {
            cin >>mas[t][k];
        }
    }
}// Случайный ввод элементов матрицы 
void Matrix::Vvod_random(int N, int M)
{
    cout<<"vvod random"<<endl;;
    for (int t = 0; t < N; t++)
    {
        for (int k = 0; k < M; k++)
        {           
            mas[t][k]=rand()%10+1;
        }
    }
}
                // Функция задает значение mas[N][M] элементу
void Matrix::SetEl(int N, int M)
{
        cout <<"vvedite znachenie zamenyaemogo elementa mas["<<N<<","<<M<<"]";
        cin >>mas[N][M];
}
int Matrix::GetEl(int N, int M)
{
        return mas[N][M];
 
}
// оператор сложения 
 Matrix operator+(Matrix ob1, Matrix ob)
 {
     Matrix Result(ob.i, ob1.j, true);
     if ((ob1.i==ob.i)&&(ob1.j==ob.j))
     {
         cout<<"poluchennaya matrica\n";
         for (int t = 0; t < ob1.i; t++)
         {
             for (int k = 0; k < ob1.j; k++)
             {
                 Result.mas[t][k] = ob.mas[t][k] + ob1.mas[t][k];
             }
         }
     }
     else
     {
         cout << "takie matricy skladyvat nelzya\n";
     }
     return Result;
 }// оператор вычитания 
Matrix Matrix::operator -(Matrix ob1)
        {
            Matrix Result(ob1.i, ob1.j, true);
                for (int t = 0; t < ob1.i; t++)
                    for (int k = 0; k < ob1.j; k++)
                        Result.mas[t][k] = mas[t][k] - ob1.mas[t][k];
            return Result;
        }// оператор умножения 
Matrix Matrix::operator *(Matrix ob1)
        {
             if (ob1.j == i)
            {
                cout <<"takie matricy ne peremnozhayutsa\n";
                Matrix ob0(0,0);
           return ob0;
            }
            Matrix Result(ob1.i, j);
            for (int z=0;z<ob1.i;z++)
            {
                for (int q=0;q<j;q++)
                {
                    for (int n=0;n<ob1.i;n++)
                    {
                        Result.mas[z][q] +=ob1.mas[z][n]*mas[n][q]; 
                    }
                }
            }
            return Result;
            }//оператор вывода 
ostream  &operator<<(ostream &stream, Matrix ob)
{ 
    for (int t = 0; t < ob.i; t++)
    {
        for (int k = 0; k < ob.j; k++)
            stream <<ob.GetEl(t,k)<<" ";
    stream<<"\n";
    }
    return stream;
}
 bool operator==(Matrix ob, Matrix ob1)
 {
     
     if(ob1.i==ob.i && ob.j==ob1.j)
     {
         for(int i=0; i<ob.i; i++)
         {
             for (int j=0; j<ob.j; j++)
             {
                 if (ob.mas[i][j]!=ob1.mas[i][j])
                 {
                     ob.i++;
                 }
             }
         }
     }
     else 
     {
         cout <<"takie matricy sravnivats nelzya\n";
         return 0;
     }
     if (ob.i>ob1.i)
         {
             cout <<"matrycy neravny";
             return 0;
         }
     else
      {
          cout <<"matrycy ravny";
          return 1;
      }
 }// констркутор копирования
 // вроде все правильно написано, но потом выскакивает сообщение invalid aalocation size
 Matrix::Matrix(const Matrix& ob)
 {
     int **mas=new int *[ob.i];
     for (int n=0; n<ob.j; n++)
         mas=new int *[ob.j];
     for (int k=0; k<ob.i; k++)
     {
         for(int m=0; m<ob.j; m++)
         {
             mas[k][m]=ob.mas[k][m];
         }
     }
 }

@Tsyklop · 11.06.2014, 07:41

такой вопрос. int Matrix::Get_i(Matrix ob) что такое ob? и как ты его тут получшь?
И да в main-е ты передаёщь вот так: Matrix ob3(4, 4, true); что за true и почему ob3?

Добавлено через 3 минуты
у меня даже не хочет запускаться. invalid aalocation size сразу после запуска

@uglyPinokkio · 11.06.2014, 08:06

C++
1
2
 Matrix::Matrix(const Matrix& ob)
 {

C++
1
     int **mas=new int *[ob.i];

mas - массив указателей на строки

C++
1
2
     for (int n=0; n<ob.j; n++)
         mas=new int *[ob.j];

mas - указатель на одну строку, видимо потерян индекс.

C++
1
2
3
4
5
6
7
8
     for (int k=0; k<ob.i; k++)
     {
         for(int m=0; m<ob.j; m++)
         {
             mas[k][m]=ob.mas[k][m];
         }
     }
 }

Присвоение мимо выделенной памяти.

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

у меня даже не хочет запускаться. invalid aalocation size сразу после запуска

Та же история в дефолтном конструкторе:

C++
1
2
3
     int **mas=new int *[i];
     for (int n=0; n<i; n++)
         mas=new int *[j];

и всех остальных.

Добавлено через 9 минут
UP - сорри, только сейчас заметил что отводится еще один массив указателей.

C++
1
2
for (int n=0; n<ob.j; n++)
         mas=new int *[ob.j];

Что тут имелось в виду?

@Tsyklop · 11.06.2014, 08:41

Заметил ошибку.

mas[n]=new int [j]; должно быть так.

Добавлено через 1 минуту
я просто не оч могу понять что за ob и зачем он пошел таким путем.

Добавлено через 4 минуты
вот как я переделал, но у меня во время вызывания функции Rand программа крашиться. В чём баян?

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
class Matrix
{
private:
    int i, j;
    int **mas;
public:
    Matrix();
    void Init();
    void Matrix::Rand(int i, int j);
    Matrix(const Matrix&);
    ~Matrix();
    void SetSize();
    void Vivod();
};
Matrix::Matrix()
{
    mas = NULL;
    i = 0;
    j = 0;
}              
void Matrix::Init()
{
    cout << "Сработал конструктор Init\n";
    int **mas = new int *[i];
    for (int n = 0; n < i; n++)
    {
        mas[n] = new int [j];
    }
    Rand(i, j);
}
void Matrix::Rand(int i, int j)   
{
    cout << "Сработал конструктор Rand\n"; //выводиться эта строка и прога вылетает
    for (int n = 0; n < i; n++)
    {
        for (int m = 0; m < j; m++)
        {
            mas[n][m] = rand() % 100;
        }
    }
}
Matrix::~Matrix()
{
    delete[] mas;
}
//Ввод размеров матрицы 
void Matrix::SetSize()
{
    cout << "Введите i\t";
    cin >> i;
    cout << "Введите j\t";
    cin >> j;
}
//Вывод на консоль 
// Случайный ввод элементов матрицы 
void Matrix::Vvod_random()
{
    cout << "Заполнение матрицы случайным образом\n";
    for (int t = 0; t < i; t++)
    {
        for (int k = 0; k <j; k++)
        {
            mas[t][k] = rand() % 100;
        }
    }
}
// констркутор копирования
Matrix::Matrix(const Matrix& ob)
{
    int **mas = new int *[ob.i];
    for (int n = 0; n<ob.j; n++)
        mas[n] = new int [ob.j];
    for (int k = 0; k<ob.i; k++)
    {
        for (int m = 0; m<ob.j; m++)
        {
            mas[k][m] = ob.mas[k][m];
        }
    }
}
 
#include "stdfx.h"
#include "Matrix.h"
#include <time.h>
void main()
{
    setlocale(LC_ALL, "rus");
    srand(time(NULL));
    Matrix A;
    A.SetSize();
    A.Init();
    A.Vivod();
}

Добавлено через 12 минут
решил проблему. при выделении памяти нужно писать вот так: mas = new int *[i];

@uglyPinokkio · 11.06.2014, 08:43

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

должно быть так.

Тогда деструктор нужно править - сначала освободить память, отведенную под строки, потом массив указателей.
Сейчас память течет.

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

я просто не оч могу понять что за ob и зачем он пошел таким путем.

Это конструктор копирования

http://subscribe.ru/archive/co... 5941.html/

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

но у меня во время вызывания функции Rand программа крашиться. В чём баян?

хз. на первый взгляд должно работать, если до вечера ТС не разберется, посмотрю. Сейчас некогда.

@Tsyklop · 11.06.2014, 09:08

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

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
Matrix Matrix::operator+(int x)
{
    mas = new int *[i];
    for (int n = 0; n < i; n++)
    {
        mas[n] = new int[j];
    }
    for (int n = 0; n < i; n++)
    {
        for (int m = 0; m < j; m++)
        {
            mas[n][m] += x;
        }
    }
    return *this;
}

Новые блоги и статьи Все статьи Все блоги /
Учёным и волонтёрам проекта «Einstein@home» удалось обнаружить четыре гамма-лучевых пульсара в джете Млечного Пути Programma_Boinc 01.01.2026 Учёным и волонтёрам проекта «Einstein@home» удалось обнаружить четыре гамма-лучевых пульсара в джете Млечного Пути Сочетание глобально распределённой вычислительной мощности и инновационных. . .	Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .	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/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .

@Tsyklop 5 / 6 / 5 Регистрация: 01.01.2014 Сообщений: 399
	11.06.2014, 07:41
	такой вопрос. int Matrix::Get_i(Matrix ob) что такое ob? и как ты его тут получшь? И да в main-е ты передаёщь вот так: Matrix ob3(4, 4, true); что за true и почему ob3? Добавлено через 3 минуты у меня даже не хочет запускаться. invalid aalocation size сразу после запуска 0