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

@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 29.03.2026 Реализация из решения ниже выполнена на нетиповых документах, разработанных в конфигурации КА2. Есть нетиповой документ "ЗаявкаНаРемонтСпецтехники" и нетиповой документ "ПланированиеСпецтехники". В. . .	Настройка движения справочника по регистру сведений Maks 29.03.2026 Решение ниже реализовано на примере нетипового справочника "ТарифыМобильнойСвязи" разработанного в конфигурации КА2, с целью учета корпоративной мобильной связи в коммерческом предприятии. . . .	Автозаполнение реквизита при выборе элемента справочника Maks 27.03.2026 Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. При выборе "Спецтехники" (Тип Справочник. Спецтехника), заполняется. . .	Сумматор с применением элементов трёх состояний. Hrethgir 26.03.2026 Тут. https:/ / fips. ru/ EGD/ ab3c85c8-836d-4866-871b-c2f0c5d77fbc Первый документ красиво выглядит, но без схемы. Это конечно не даёт никаких плюсов автору, но тем не менее. . . всё может быть. . .
Автозаполнение реквизитов при создании документа Maks 26.03.2026 Программный код из решения ниже размещается в модуле объекта документа, в процедуре "ПриСозданииНаСервере". Алгоритм проверки заполнения реализован для исключения перезаписи значения реквизита,. . .	Команды формы и диалоговое окно Maks 26.03.2026 1. Команда формы "ЗаполнитьЗапчасти". Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. В качестве источника данных. . .	Кому нужен AOT? DevAlt 26.03.2026 Решил сделать простой ланчер Написал заготовку: dotnet new console --aot -o UrlHandler var items = args. Split(":"); var tag = items; var id = items; var executable = args;. . .	Отправка уведомления на почту при создании или изменении элементов справочника Maks 24.03.2026 Программная отправка письма электронной почты на примере типового справочника "Склады" в конфигурации БП3. Перед реализацией необходимо выполнить настройку системной учетной записи электронной. . .

@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