Конструктор копии: при возвращении объекта ошибка памяти

@artart96 · Регистрация: 04.10.2014

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

Реализация класса:

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
#include"Class_Taxi.h"
#include<fstream>
#include<conio.h>
using namespace std;
 
Zakaz::Zakaz()
{
    Number_Zak = 0;
    Kol_Pos = 0;
    Dom = 0;
    Kv = 0;
    Familia_Zak = new char [MAX];
}
 
Zakaz::Zakaz(int n,int k,std::string s,int d,int f,char*m)
{
    this->Number_Zak = n;
    this->Kol_Pos = k;
    this->Dom = d;
    this->Kv = f;
    this->Street = s;
    this->Familia_Zak = m;
}
 
Zakaz::~Zakaz()
{
delete Familia_Zak;
}
 
Zakaz::Zakaz(const Zakaz &ob)
{
    cout<<endl<<"Вызван конструктор копирования"<<endl;
    Familia_Zak = new char[MAX];
    strcpy(Familia_Zak,ob.Familia_Zak);
}
 
void Zakaz::Set_Number(int n)
{
    Number_Zak = n;
}
 
int Zakaz::Get_Number()
{
    return Number_Zak;
}
 
 
void Zakaz::Set_Kol_Pos(int k)
{
    Kol_Pos = k;
}
 
int Zakaz::Get_Kol_Pos()
{
    return Kol_Pos;
}
 
void Zakaz::Set_Street(std::string s)
{
    Street = s;
}
 
std::string Zakaz::Get_Street()
{
    return Street;
}
 
void Zakaz::Set_Dom(int d)
{
    Dom = d;
}
 
int Zakaz::Get_Dom()
{
    return Dom;
}
 
void Zakaz::Set_Kv(int k)
{
    Kv = k;
}
 
int Zakaz::Get_Kv()
{
    return Kv;
}
 
 
void Zakaz::V_Fail()
{
ofstream fout;
fout.open("D:\\Zakaz.txt",ios::app);
fout<<"Номер заказа:"<<Get_Number()<<endl<<"Улица:"<<Get_Street()<<endl<<"Дом:"<<Get_Dom()<<endl<<"Квартира:"<<Get_Kv()<<endl;
cout<<"Сведения о заказах записаны в файл!"<<endl;
fout.close();
_getch();
}
 
void Zakaz::Registration()
{
    cout<<"Регистрация заказа:"<<endl<<"Введите номер заказа:";cin>>Number_Zak;
    cout<<"Введите улицу:";cin>>Street;
    cout<<"Введите дом:";cin>>Dom;
    cout<<"Введите квартиру:";cin>>Kv;
    cout<<"Введите количестов пассажиров:";cin>>Kol_Pos;
    cout<<"Введите фамилию:";this->Familia_Zak = new char[MAX];cin>>this->Familia_Zak;
}
 
 
void Zakaz::Show()
{
    cout<<endl<<"Номер заказа:";cout<<this->Number_Zak;
    cout<<endl<<"Улица:";cout<<this->Street;
    cout<<endl<<"Дом:";cout<<this->Dom;
    cout<<endl<<"Квартира:";cout<<this->Kv;
    cout<<endl<<"Количестов пассажиров:";cout<<this->Kol_Pos;
    cout<<endl<<"Фамилия заказчика:";cout<<this->Familia_Zak<<endl;
}
 
int Vibor();
 
int Fix_Zakaz(Zakaz ob)
{
    switch( Vibor() )
        {  
    case 1:cout<<"Новый номер заказа:";cin>>ob.Number_Zak;break;
    case 2:cout<<"Новая улица:";cin>>ob.Street;break;
    case 3:cout<<"Новый номер дома:";cin>>ob.Dom;break;
    case 4:cout<<"Новый номер квартиры:";cin>>ob.Kv;break;
    case 5:cout<<"Новая фамилия:";cin>>ob.Familia_Zak;break;
        }      
            return 0;   
}
 
 
int Vibor()
{
    int vibor;
    cout<<"[1] - Изменение номера заказа:"<<endl<<"[2] - Изменение улицы:"<<endl<<"[3] - Изменение номера дома:"<<endl<<"[4] - Изменение номера квартиры"<<endl<<"[5] - Изменение фамилии заказчика:"<<endl<<"Ваш выбор:";
    cin>>vibor;
    return vibor;
}
 
void Zakaz::Set_Failia(char*s)
{
    this->Familia_Zak = s;
}
 
char* Zakaz::Get_Familia()
{
    return (this->Familia_Zak);
}
 
Zakaz C_C(Zakaz&ob)
{
    return ob;
}
 
 
 
main:
#include"Class_Taxi.h"
#include<vld.h>
#define MAX 15
using namespace std;
 
int main()
{
    setlocale(LC_ALL,"rus");
    Zakaz ob1;
    ob1.Registration();ob1.Show();ob1.V_Fail();cout<<endl;
    Zakaz ob2;
    cout<<"Введите номер заказа:";int n;cin>>n;ob2.Set_Number(n);
    cout<<"Введите улицу:";string s;cin>>s;ob2.Set_Street(s);
    cout<<"Введите номер дома:";int d;cin>>d;ob2.Set_Dom(d);
    cout<<"Введите номер квартиры:";int kv;cin>>kv;ob2.Set_Kv(kv);
    cout<<"Введите количество пассажиров:";int pos;cin>>pos;ob2.Set_Kol_Pos(pos);
    cout<<"Введите фамилию заказчика:";char*st = new char[MAX];cin>>st;ob2.Set_Failia(st);
    ob2.V_Fail();cout<<endl;
    cout<<"Введенные вами данные заказа:"<<endl;
    cout<<"Номер заказа:"<<ob2.Get_Number()<<endl;
    cout<<"Улица:"<<ob2.Get_Street()<<endl;
    cout<<"Номер дома:"<<ob2.Get_Dom()<<endl;
    cout<<"Номер квартиры:"<<ob2.Get_Kv()<<endl;
    cout<<"Номер квартиры:"<<ob2.Get_Familia()<<endl;
    cout<<"Kоличество пассажиров:"<<ob2.Get_Kol_Pos()<<endl;
 
    
    cout<<endl<<"Введите номер заказа:";int n_3;cin>>n_3;
    cout<<"Введите улицу:";std::string yl;cin>>yl;
    cout<<"Введите дом:";int d_3;cin>>d_3;
    cout<<"Введите квартиру:";int kv_3;cin>>kv_3;
    cout<<"Введите количестов пассажиров:";int k_p;cin>>k_p;
    cout<<"Введите фамилию заказчика:";char*s3 = new char[MAX];cin>>s3;
    Zakaz ob3(n_3,k_p,yl,d_3,kv_3,s3);ob3.V_Fail();
    cout<<endl<<"Введеные вами данные:"<<endl;ob3.Show();cout<<endl;
 
    Fix_Zakaz(ob3);ob3.Show();ob3.Get_Familia();cout<<endl;
 
    Zakaz ob4 = ob3;ob3.Show();ob3.Get_Familia();ob4.Show();cout<<endl;
 
    Zakaz ob_8; ob_8  = C_C(ob1);
    return 0;
}

Проблема с последней функцией - при возвращении объекта.

@zss · 24.09.2015, 12:46

А где "Class_Taxi.h"
По тому, что видно
1.

C++
1
2
3
4
Zakaz::~Zakaz()
{
delete[] Familia_Zak;
}

2. Нет перегрузки

C++
1
Zakaz& Zakaz::operator=(const Zakaz&);

@artart96 · 24.09.2015, 12:58 **[ТС]**

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

А где "Class_Taxi.h"
По тому, что видно

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
#include <iostream>
#include <string>
#define MAX 15
 
class Zakaz{
    int Number_Zak;
    int Kol_Pos;
    std::string Street;
    int Dom;
    int Kv;
    char* Familia_Zak;
public:
    Zakaz();
    Zakaz(int,int,std::string,int,int,char*);
    Zakaz(const Zakaz &ob);
    ~Zakaz();
 
    void Set_Number(int);
    int Get_Number();
    void Set_Kol_Pos(int);
    int Get_Kol_Pos();
    void Set_Street(std::string);
    std::string Get_Street();
    void Set_Dom(int);
    int Get_Dom();
    void Set_Kv(int);
    int Get_Kv();
    void Set_Failia(char*);
    char* Get_Familia();
 
    void V_Fail();
    void Registration();
    void Show();
 
    friend int Fix_Zakaz(Zakaz);
};
 
Zakaz C_C(Zakaz&ob);

@zss · 24.09.2015, 12:59

Ну и где operator= ?

@artart96 · 24.09.2015, 13:00 **[ТС]**

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

2. Нет перегрузки
Код C++Выделить код
1
Zakaz& Zakaz::operator=(const Zakaz&);

В перегрузки же просто сделать приравнивание полей класса объектов вызова и второго объекта?

@zss · 24.09.2015, 13:07

Распространенные ошибки

@artart96 · 24.09.2015, 13:32 **[ТС]**

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

Ну и где operator= ?

перегрузил:

C++
1
2
3
4
5
6
7
8
9
10
Zakaz& Zakaz::operator=(Zakaz &ob)
{
    this->Dom = ob.Dom;
    this->Familia_Zak = ob.Familia_Zak;
    this->Kol_Pos = ob.Kol_Pos;
    this->Kv = ob.Kv;
    this->Street = ob.Street;
    this->Kol_Pos = ob.Kol_Pos;
    return *this;
}

ошибка осталась

@zss · 24.09.2015, 13:34

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

this->Familia_Zak = ob.Familia_Zak;

C++
1
strcpy(Familia_Zak,ob.Familia_Zak);

И в остальных случаях копирования строки сделайте также!

Вообще, как-то глупо Вы поступаете. Почему familia_Zak - это символьный массив, а Street - это string?
Это все равно, что половину поля вспахать трактором, а вторую половину - лопатой.
Ну ладно, трактор могут экспроприировать, но ведь string никто у Вас отбирать не собирается!

@artart96 · 24.09.2015, 13:51 **[ТС]**

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

И в остальных случаях копирования строки сделайте также!
Вообще, как-то глупо Вы поступаете. Почему familia_Zak - это символьный массив, а Street - это string?
Это все равно, что половину поля вспахать трактором, а вторую половину - лопатой.
Ну ладно, трактор могут экспроприировать, но ведь string никто у Вас отбирать не собирается!

Суть заключается в том, что в задании должно быть динамическая инициализация строки, так бы я все string сделал.Полностью согласен с мнением о коду)

Исправил,вылетает при вызове конструктора с параметрам.

Класс:

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
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
#include"Class_Taxi.h"
#include<fstream>
#include<conio.h>
using namespace std;
 
Zakaz::Zakaz()
{
    Number_Zak = 0;
    Kol_Pos = 0;
    Dom = 0;
    Kv = 0;
    Familia_Zak = new char [MAX];
}
 
Zakaz::Zakaz(int n,int k,std::string s,int d,int f,char*m)
{
    this->Number_Zak = n;
    this->Kol_Pos = k;
    this->Dom = d;
    this->Kv = f;
    this->Street = s;
    strcpy(this->Familia_Zak,m);
}
 
Zakaz::~Zakaz()
{
delete Familia_Zak;
}
 
Zakaz::Zakaz(const Zakaz &ob)
{
    cout<<endl<<"Вызван конструктор копирования"<<endl;
    Familia_Zak = new char[MAX];
    strcpy(Familia_Zak,ob.Familia_Zak);
}
 
void Zakaz::Set_Number(int n)
{
    Number_Zak = n;
}
 
int Zakaz::Get_Number()
{
    return Number_Zak;
}
 
 
void Zakaz::Set_Kol_Pos(int k)
{
    Kol_Pos = k;
}
 
int Zakaz::Get_Kol_Pos()
{
    return Kol_Pos;
}
 
void Zakaz::Set_Street(std::string s)
{
    Street = s;
}
 
std::string Zakaz::Get_Street()
{
    return Street;
}
 
void Zakaz::Set_Dom(int d)
{
    Dom = d;
}
 
int Zakaz::Get_Dom()
{
    return Dom;
}
 
void Zakaz::Set_Kv(int k)
{
    Kv = k;
}
 
int Zakaz::Get_Kv()
{
    return Kv;
}
 
 
void Zakaz::V_Fail()
{
ofstream fout;
fout.open("D:\\Zakaz.txt",ios::app);
fout<<"Номер заказа:"<<Get_Number()<<endl<<"Улица:"<<Get_Street()<<endl<<"Дом:"<<Get_Dom()<<endl<<"Квартира:"<<Get_Kv()<<endl;
cout<<"Сведения о заказах записаны в файл!"<<endl;
fout.close();
_getch();
}
 
void Zakaz::Registration()
{
    cout<<"Регистрация заказа:"<<endl<<"Введите номер заказа:";cin>>Number_Zak;
    cout<<"Введите улицу:";cin>>Street;
    cout<<"Введите дом:";cin>>Dom;
    cout<<"Введите квартиру:";cin>>Kv;
    cout<<"Введите количестов пассажиров:";cin>>Kol_Pos;
    cout<<"Введите фамилию:";char *Fam = new char[MAX];cin>>Fam;strcpy(Familia_Zak,Fam);
}
 
 
void Zakaz::Show()
{
    cout<<endl<<"Номер заказа:";cout<<this->Number_Zak;
    cout<<endl<<"Улица:";cout<<this->Street;
    cout<<endl<<"Дом:";cout<<this->Dom;
    cout<<endl<<"Квартира:";cout<<this->Kv;
    cout<<endl<<"Количестов пассажиров:";cout<<this->Kol_Pos;
    cout<<endl<<"Фамилия заказчика:";cout<<this->Familia_Zak<<endl;
}
 
int Vibor();
 
int Fix_Zakaz(Zakaz ob)
{
    switch( Vibor() )
        {  
    case 1:cout<<"Новый номер заказа:";cin>>ob.Number_Zak;break;
    case 2:cout<<"Новая улица:";cin>>ob.Street;break;
    case 3:cout<<"Новый номер дома:";cin>>ob.Dom;break;
    case 4:cout<<"Новый номер квартиры:";cin>>ob.Kv;break;
    case 5:cout<<"Новая фамилия:";cin>>ob.Familia_Zak;break;
        }      
            return 0;   
}
 
 
int Vibor()
{
    int vibor;
    cout<<"[1] - Изменение номера заказа:"<<endl<<"[2] - Изменение улицы:"<<endl<<"[3] - Изменение номера дома:"<<endl<<"[4] - Изменение номера квартиры"<<endl<<"[5] - Изменение фамилии заказчика:"<<endl<<"Ваш выбор:";
    cin>>vibor;
    return vibor;
}
 
void Zakaz::Set_Failia(char*s)
{
    this->Familia_Zak = s;
}
 
char* Zakaz::Get_Familia()
{
    return (this->Familia_Zak);
}
 
Zakaz C_C(Zakaz&ob)
{
    return ob;
}
 
Zakaz& Zakaz::operator=(Zakaz &ob)
{
    this->Dom = ob.Dom;
    strcpy(this->Familia_Zak, ob.Familia_Zak);
    this->Kol_Pos = ob.Kol_Pos;
    this->Kv = ob.Kv;
    this->Street = ob.Street;
    this->Kol_Pos = ob.Kol_Pos;
    return *this;
}
 
main():
 
#include"Class_Taxi.h"
#include<vld.h>
#define MAX 15
using namespace std;
 
int main()
{
    setlocale(LC_ALL,"rus");
    Zakaz ob1;
    ob1.Registration();ob1.Show();ob1.V_Fail();cout<<endl;
    Zakaz ob2;
    cout<<"Введите номер заказа:";int n;cin>>n;ob2.Set_Number(n);
    cout<<"Введите улицу:";string s;cin>>s;ob2.Set_Street(s);
    cout<<"Введите номер дома:";int d;cin>>d;ob2.Set_Dom(d);
    cout<<"Введите номер квартиры:";int kv;cin>>kv;ob2.Set_Kv(kv);
    cout<<"Введите количество пассажиров:";int pos;cin>>pos;ob2.Set_Kol_Pos(pos);
    cout<<"Введите фамилию заказчика:";char*st = new char[MAX];cin>>st;ob2.Set_Failia(st);
    ob2.V_Fail();cout<<endl;
    cout<<"Введенные вами данные заказа:"<<endl;
    cout<<"Номер заказа:"<<ob2.Get_Number()<<endl;
    cout<<"Улица:"<<ob2.Get_Street()<<endl;
    cout<<"Номер дома:"<<ob2.Get_Dom()<<endl;
    cout<<"Номер квартиры:"<<ob2.Get_Kv()<<endl;
    cout<<"Номер квартиры:"<<ob2.Get_Familia()<<endl;
    cout<<"Kоличество пассажиров:"<<ob2.Get_Kol_Pos()<<endl;
 
    
    cout<<endl<<"Введите номер заказа:";int n_3;cin>>n_3;
    cout<<"Введите улицу:";std::string yl;cin>>yl;
    cout<<"Введите дом:";int d_3;cin>>d_3;
    cout<<"Введите квартиру:";int kv_3;cin>>kv_3;
    cout<<"Введите количестов пассажиров:";int k_p;cin>>k_p;
    cout<<"Введите фамилию заказчика:";char*s3 = new char[MAX];cin>>s3;
    Zakaz ob3(n_3,k_p,yl,d_3,kv_3,s3);ob3.V_Fail();
    cout<<endl<<"Введеные вами данные:"<<endl;ob3.Show();cout<<endl;
 
    Fix_Zakaz(ob3);ob3.Show();ob3.Get_Familia();cout<<endl;
 
    Zakaz ob4 = ob3;ob3.Show();ob3.Get_Familia();ob4.Show();cout<<endl;
 
    Zakaz ob_8; ob_8  = C_C(ob1);
    return 0;
}
 
заголовочный:
 
#include <iostream>
#include <string>
#define MAX 15
 
class Zakaz{
    int Number_Zak;
    int Kol_Pos;
    std::string Street;
    int Dom;
    int Kv;
    char* Familia_Zak;
public:
    Zakaz();
    Zakaz(int,int,std::string,int,int,char*);
    Zakaz(const Zakaz &ob);
    ~Zakaz();
 
    void Set_Number(int);
    int Get_Number();
    void Set_Kol_Pos(int);
    int Get_Kol_Pos();
    void Set_Street(std::string);
    std::string Get_Street();
    void Set_Dom(int);
    int Get_Dom();
    void Set_Kv(int);
    int Get_Kv();
    void Set_Failia(char*);
    char* Get_Familia();
 
    void V_Fail();
    void Registration();
    void Show();
 
    Zakaz& operator=(Zakaz&);
 
    friend int Fix_Zakaz(Zakaz);
};
 
Zakaz C_C(Zakaz&ob);

@zss · 24.09.2015, 14:23

C++
1
2
3
4
5
6
7
8
9
10
11
Zakaz::Zakaz()
{
  ....
    Familia_Zak = new char [MAX];
}
 
Zakaz::Zakaz(int n,int k,std::string s,int d,int f,char*m)
{
   ... 
   strcpy(this->Familia_Zak,m);
}

Чувствуете разницу?
Кстати

C++
1
2
3
4
5
6
Zakaz::Zakaz()
{
  ....
    Familia_Zak = new char [MAX];
    strcpy(Familia_Zak,"Ivanov");
}

Новые блоги и статьи Все статьи Все блоги /
http://iceja.net/ математические сервисы iceja 20.01.2026 Обновила свой сайт http:/ / iceja. net/ , приделала Fast Fourier Transform экстраполяцию сигналов. Однако предсказывает далеко не каждый сигнал (см ограничения http:/ / iceja. net/ fourier/ docs ). Также. . .	http://iceja.net/ сервер решения полиномов iceja 18.01.2026 Выкатила http:/ / iceja. net/ сервер решения полиномов (находит действительные корни полиномов методом Штурма). На сайте документация по API, но скажу прямо VPS слабенький и 200 000 полиномов. . .	Расчёт переходных процессов в цепи постоянного тока igorrr37 16.01.2026 / * Дана цепь постоянного тока с R, L, C, k(ключ), U, E, J. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа, решает её и находит переходные токи и напряжения на элементах схемы. . . .	Восстановить юзерскрипты Greasemonkey из бэкапа браузера damix 15.01.2026 Если восстановить из бэкапа профиль Firefox после переустановки винды, то список юзерскриптов в Greasemonkey будет пустым. Но восстановить их можно так. Для этого понадобится консольная утилита. . .
Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .	Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .

@zss Модератор 13771 / 10964 / 6491 Регистрация: 18.12.2011 Сообщений: 29,241
	24.09.2015, 12:59
	Ну и где operator= ? 1

@zss Модератор 13771 / 10964 / 6491 Регистрация: 18.12.2011 Сообщений: 29,241
	24.09.2015, 13:07
	Распространенные ошибки 0

Конструктор копии: при возвращении объекта ошибка памяти

Решение