Форум программистов, компьютерный форум, киберфорум
Наши страницы

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
Рейтинг: Рейтинг темы: голосов - 14, средняя оценка - 4.86
qwer_11
2 / 2 / 0
Регистрация: 30.10.2011
Сообщений: 16
#1

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

21.12.2011, 17:13. Просмотров 2482. Ответов 5
Метки нет (Все метки)

так вот,что-то глючит конструктор копирования, хотя ошибок компиляции не дает((
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];
         }
     }
 }
2
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
21.12.2011, 17:13
Я подобрал для вас темы с готовыми решениями и ответами на вопрос Класс матрица (не работает конструктор копирования) (C++):

Не могу сделать чтобы класс содержал основной конструктор и конструктор копирования - C++
Разработать класс ThreeAngle для работы с плоскими треугольниками. В качестве членов-данных задаются длины трех сторон треугольника. Класс...

Создать класс "Вектор" и реализовать конструктор по умолчанию, конструктор копирования и деструктор - C++
Всем доброго времени суток! нужна ваша помощь! нужно создать класс вектор и реализовать конструктор по умолчанию, копирования и...

Класс, конструктор-копирования - C++
Здравствуйте, я плохо знаю С++ и поэтому у меня возникла проблема с одной задачей, звучит она так: Разработать класс «Множество (целых...

Не работает конструктор копирования - C++
Пишу свой класс матрица. В конструкторе копирования вылезла ошибка, решить которую мне не удается. Отладчиком проходился не один раз,...

Не работает конструктор копирования - C++
Всем доброго времени суток. Помогите пожалуйста, уже битиый день не могу побороть почему не работает конструктор копирования :cry: ...

Не работает конструктор копирования - C++
#include &lt;iostream.h&gt; #include &lt;conio.h&gt; #include &lt;stdio.h&gt; class my {int x1,x2,y1,y2,col; public: my(int _x1=20, int _y1=5,...

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

Добавлено через 3 минуты
у меня даже не хочет запускаться. invalid aalocation size сразу после запуска
0
uglyPinokkio
326 / 229 / 41
Регистрация: 30.05.2014
Сообщений: 682
11.06.2014, 08:06 #3
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];
Что тут имелось в виду?
0
Tsyklop
5 / 5 / 3
Регистрация: 01.01.2014
Сообщений: 356
11.06.2014, 08:41 #4
Заметил ошибку.

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];
0
uglyPinokkio
326 / 229 / 41
Регистрация: 30.05.2014
Сообщений: 682
11.06.2014, 08:43 #5
Цитата Сообщение от Tsyklop Посмотреть сообщение
должно быть так.
Тогда деструктор нужно править - сначала освободить память, отведенную под строки, потом массив указателей.
Сейчас память течет.

Цитата Сообщение от Tsyklop Посмотреть сообщение
я просто не оч могу понять что за ob и зачем он пошел таким путем.
Это конструктор копирования

http://subscribe.ru/archive/comp.sof...04015941.html/

Цитата Сообщение от Tsyklop Посмотреть сообщение
но у меня во время вызывания функции Rand программа крашиться. В чём баян?
хз. на первый взгляд должно работать, если до вечера ТС не разберется, посмотрю. Сейчас некогда.
0
Tsyklop
5 / 5 / 3
Регистрация: 01.01.2014
Сообщений: 356
11.06.2014, 09:08 #6
хочу сделать сложение матрицы с числом, вот что я набрал, но при выполнеии выходит ошибка.
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;
}
0
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
11.06.2014, 09:08
Привет! Вот еще темы с ответами:

Конструктор копирования не работает - C++
Прошу вас помочь разобраться с конструктором копирования. Проблема в следующем при вызове конструктора копирование вылетает ошибка не...

Конструктор копирования для класса с указателем на абстрактный класс - C++
Всем привет. Как реализовать конструктор копирования для класса Foo? class Base { public: ... ...

Как работает конструктор копирования - C++
дан класс я не могу понять как работает конструктор копирования arr.h#pragma once #include &lt;iostream&gt; #include &lt;fstream&gt; #include...

Не работает конструктор копирования почему-то... - C++
#include &quot;stdlib.h&quot; #include &quot;stdio.h&quot; #include &lt;iostream&gt; #include &lt;cstddef&gt; using namespace std; class Top { public: ...


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

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

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