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

@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;
}

Новые блоги и статьи Все статьи Все блоги /
Уведомление о неверно выбранном значении справочника Maks 06.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "НарядПутевка", разработанного в конфигурации КА2. Задача: уведомлять пользователя, если в документе выбран неверный склад. . .	Установка Qt Creator для C и C++: ставим среду, CMake и MinGW без фреймворка Qt 8Observer8 05.04.2026 Среду разработки Qt Creator можно установить без фреймворка Qt. Есть отдельный репозиторий для этой среды: https:/ / github. com/ qt-creator/ qt-creator, где можно скачать установщик, на вкладке Releases:. . .	AkelPad-скрипты, структуры, и немного лирики.. testuser2 05.04.2026 Такая программа, как AkelPad существует уже давно, и также давно существуют скрипты под нее. Тем не менее, прога живет, периодически что-то не спеша дополняется, улучшается. Что меня в первую очередь. . .	Отображение реквизитов в документе по условию и контроль их заполнения Maks 04.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеСпецтехники", разработанного в конфигурации КА2. Данный документ берёт данные из другого нетипового документа. . .
Фото всей Земли с борта корабля Orion миссии Artemis II kumehtar 04.04.2026 Это первое подобное фото сделанное человеком за 50 лет. Снимок называют новым вариантом легендарной фотографии «The Blue Marble» 1972 года, сделанной с борта корабля «Аполлон-17». Новое фото. . .	Вывод диалогового окна перед закрытием, если документ не проведён Maks 04.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: реализовать программный контроль на предмет проведения документа. . .	Программный контроль заполнения реквизитов табличной части документа Maks 02.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: 1. Реализовать контроль заполнения реквизита. . .	wmic не является внутренней или внешней командой Maks 02.04.2026 Решение: DISM / Online / Add-Capability / CapabilityName:WMIC~~~~ Отсюда: https:/ / winitpro. ru/ index. php/ 2025/ 02/ 14/ komanda-wmic-ne-naydena/

@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