Форум программистов, компьютерный форум CyberForum.ru

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

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

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

21.12.2011, 17:13. Просмотров 1876. Ответов 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];
         }
     }
 }
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
21.12.2011, 17:13     Класс матрица (не работает конструктор копирования)
Посмотрите здесь:

Класс, конструктор-копирования C++
C++ Конструктор копирования не работает
Конструктор копирования для класса с указателем на абстрактный класс C++
C++ Не работает конструктор копирования
C++ Неправильно работает конструктор копирования list, почему?
C++ Не работает конструктор копирования почему-то...
Как работает конструктор копирования C++
C++ Не работает конструктор копирования
Классы (объединить мой класс с проверкой на его работу, убрать гото и добавить конструктор копирования) C++
Как работает конструктор копирования? C++
Разработать класс матрица, конструктор по умолчанию, вывод на экран C++
Не работает конструктор копирования C++

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

Или воспользуйтесь поиском по форуму:
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Tsyklop
3 / 3 / 2
Регистрация: 01.01.2014
Сообщений: 280
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 сразу после запуска
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];
Что тут имелось в виду?
Tsyklop
3 / 3 / 2
Регистрация: 01.01.2014
Сообщений: 280
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];
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 программа крашиться. В чём баян?
хз. на первый взгляд должно работать, если до вечера ТС не разберется, посмотрю. Сейчас некогда.
Tsyklop
3 / 3 / 2
Регистрация: 01.01.2014
Сообщений: 280
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;
}
Yandex
Объявления
11.06.2014, 09:08     Класс матрица (не работает конструктор копирования)
Ответ Создать тему
Опции темы

Текущее время: 21:30. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru