Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.60/5: Рейтинг темы: голосов - 5, средняя оценка - 4.60
0 / 0 / 0
Регистрация: 14.09.2016
Сообщений: 153

После работы перегруженного оператора происходит удаление списка

25.12.2018, 11:32. Показов 1091. Ответов 3

Студворк — интернет-сервис помощи студентам
C++ (Qt)
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
// ConsoleApplication6.cpp: определяет точку входа для консольного приложения.
//
 
#include "stdafx.h"
#include <cstring>
#include <iostream>
#include <fstream>
 
using namespace std;
#include "zagolovok.h"
 
int main()
{
    country * obje1 = new country ("United States of America", "Washington, D.C.", "English", 325719178, 9833520);
    country * obje2 = new country(*obje1);
    cout << *obje2;
    country * obje3 = new country;
    *obje3 = *obje1;
    cout << endl;
    cout << *obje3;
    odns obj1;
    obj1 + obje1;
    obj1.print();
    system("pause");
    return 0;
}
Файл zagolovok.h
C++ (Qt)
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
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
class country {
private:
    char * name;
    char * stolica;
    char * sch;
    long lud;
    long s;
    country * next;
public:
    void print(void) {
        cout << "nazvaniye: " << name << " stolica: " << stolica << " yazik: " << sch << " naseleniye: " << lud << " ploshad, km^2: " << s << endl;
    };
    void set_console() {
        cin >> name >> stolica >> sch >> lud >> s;
        next = NULL;
    };
    country operator=(country obj) {
        lud = obj.lud;
        s = obj.s;
        strcpy(name, obj.name);
        strcpy(stolica, obj.stolica);
        strcpy(sch, obj.sch);
        return *this;
    }
    country(void) {
        name = new char[30];
        stolica = new char[20];
        sch = new char[20];
        lud = 0;
        s = 0;
        next = 0;
    };
    country(const country &obj) {
        lud = obj.lud;
        s = obj.s;
        name = new char[30];
        stolica = new char[20];
        sch = new char[20];
        strcpy(name, obj.name);
        strcpy(stolica, obj.stolica);
        strcpy(sch, obj.sch);
    };
    
    country(char* c_name, char * c_stolica, char * c_sch, long c_lud, long c_s) {
        name = new char[30];
        stolica = new char[20];
        sch = new char[20];
        set_main(c_name, c_stolica, c_sch, c_lud, c_s);
    };
    void set_main(char* c_name, char * c_stolica, char * c_sch, long c_lud, long c_s) {
        strcpy(name, c_name);
        strcpy(stolica, c_stolica);
        strcpy(sch, c_sch);
        lud = c_lud;
        s = c_s;
        next = NULL;
    };
    void set_next(country * rex) {
        next = rex;
    };
    country * get_next(void) {
        return next;
    };
    char * get_name(void) {
        return name;
    };
    char * get_stolica(void) {
        return stolica;
    };
    char * get_sch(void) {
        return sch;
    };
    void set_name(const char * c_name) {
        delete[] name;
        name = new char[30];
        strcpy(name, c_name);
    };
    void set_stolica(const char * c_stolica) {
        delete[] stolica;
        stolica = new char[20];
        strcpy(stolica, c_stolica);
    };
    void set_sch(const char * c_sch) {
        delete[] sch;
        sch = new char[20];
        strcpy(sch, c_sch);
    };
    void set_lud(long c_lud) {
        lud = c_lud;
    };
    void set_s(long c_s) {
        s = c_s;
    };
    long get_s(void) {
        return s;
    };
    long get_lud(void) {
        return lud;
    };
    ~country() {
        delete[] name;
        delete[] stolica;
        delete[] sch;
    }
    friend ostream& operator<<(ostream& os, const country& obj);
};
class odns {
private:
    country * eins;
    country * end;
public:
    odns(const odns &obj) {
        eins = obj.eins;
        end = obj.end;
    }
    odns operator=(odns obj) {
        eins = obj.eins;
        end = obj.end;
    }
    odns operator+(country * obj) {
        add(obj);
        return *this;
    }
    void save(const char * fn) {
        ofstream fout(fn);
        country * rex = eins;
        fout << "nazvaniye: " << rex->get_name() << " stolica: " << rex->get_stolica() << " yazik: " << rex->get_sch() << " naseleniye: " << rex->get_lud() << " ploshad, km^2: " << rex->get_s() << endl;
        while ((*rex).get_next() != NULL) {
            rex = rex->get_next();
            fout << "nazvaniye: " << rex->get_name() << " stolica: " << rex->get_stolica() << " yazik: " << rex->get_sch() << " naseleniye: " << rex->get_lud() << " ploshad, km^2: " << rex->get_s() << endl;
        }
        fout.close();
    }
    int load(const char * fn) {
        ifstream fin(fn);
        int i = 1;
        if (!fin.is_open()) {
            cout << "Can't open file" << endl;
            return i;
        }
        else {
            char* name = new char[16];
            char* stolica = new char[16];
            char* sch = new char[16];
            long lud;
            long s;
            while (!fin.eof()) {
                i = 0;
                fin >> name >> stolica >> sch >> lud >> s;
                addel(name, stolica, sch, lud, s);
            }
            fin.close();
        }
        if (i == 1) {
            cout << "file is empty" << endl;
        }
        return i;
    }
    void addel(char* c_name, char * c_stolica, char * c_sch, long c_lud, long c_s) {
        country * rex = new country(c_name, c_stolica, c_sch, c_lud, c_s);
        add(rex);
    }
    odns(void) {
        eins = NULL;
        end = NULL;
    };
    void add(country * rex) {  //вставка элемента rex в конец списка с конечнымэлементом end
        if (end != NULL) {
            end->set_next(rex);
        }
        else {
            eins = rex;
        }
        end = rex;
    }
    void print(void) {
        if (eins != NULL) {
            country * rex = eins;
            rex->print();
            while ((*rex).get_next() != NULL) {
                rex = rex->get_next();
                rex->print();
            }
        }
        else
            cout << "spisok pust" << endl;
    };
    int operacia(double a, double b, char * op) {//функция возвращает результат выполнения операции из строки op над числами a и b
        if (op[0] == '<')
            return (a<b);
        if (op[0] == '>')
            return (a>b);
        if (op[0] == '=')
            return (a == b);
    };
    void insert(country * ctr, int n) { //вставка элемента rex в список с начальным элементом beg под омером n
        country * rex = eins;
        if (n>1) {
            for (int i = 1; i<n - 1; i++) {
                rex = rex->get_next();
            }
 
            country * rexis = (*rex).get_next();
            rex->set_next(ctr);
            ctr->set_next(rexis);
 
        }
        else {
            eins = ctr;
            ctr->set_next(rex);
        }
 
 
    };
    void my_remove(int n) {//удаление элемента под номером n из списка
        country * rex;
        if (n<2) {
            rex = eins->get_next();
            delete (eins);
            eins = rex;
        }
        else {
            rex = eins;
            for (int i = 1; i<n - 1; i++) {
                rex = rex->get_next();
            }
            if (rex->get_next()->get_next() == NULL)
                end = rex;
            country * rexis = rex->get_next();
            rex->set_next(rexis->get_next());
            delete(rexis);
        }
 
    };
    country * get_el(int n) {
        country * rex = eins;
        for (int i = 1; i<n; i++) {
            rex = rex->get_next();
        }
        return rex;
    };
    void set_el(int n, country * ctr) {
        country * rex = eins;
        for (int i = 1; i<n - 1; i++) {
            rex = rex->get_next();
        }
        ctr->set_next(rex->get_next()->get_next());
        country * rexis = rex->get_next();
        rex->set_next(ctr);
        delete (rexis);
    };
    country * find(char*s) { //функция возвращает элемент из списка с названием страны s
        int len = strlen(s);
        country * rex = eins;
        char * h;
        while (rex != NULL) {
            h = rex->get_name();
            int j = 0;
            if (strlen(h) == len) {
                for (int i = 0; i<len; i++) {
                    if (h[i] != s[i]) {
                        j = 1;
                        break;
                    }
                }
            }
            else {
                j = 1;
            }
            if (j == 0) {
                return rex;
            }
            rex = rex->get_next();
        }
        return NULL;
    };
    odns * fltr(char * op, long r) {//функция удаляет из списка элементы, не удвлетворяющие условию
        int j = 1;
        country * rex = eins;
        odns * a = new odns;
        while (rex != NULL) {
            if (operacia(rex->get_s(), r, op))
                a->add(rex);
            rex = rex->get_next();
        }
        return a;
    };
    ~odns(void) {
        country * rex = eins->get_next();
        while (eins != NULL) {
            delete(eins);
            eins = rex;
            rex = (eins != NULL) ? eins->get_next() : NULL;
        }
    };
};
ostream& operator<<(ostream& os, const country& obj) {
    os << "nazvaniye: " << obj.name << " stolica: " << obj.stolica << " yazik: " << obj.sch << " naseleniye: " << obj.lud << " ploshad, km^2: " << obj.s;
    return os;
}
При вызове obj1 + obje1 сначала всё идёт нормально (элемент добавляется в список). Потом почему-то внезапно происходит вызов деструктора.
0
Лучшие ответы (1)
Programming
Эксперт
39485 / 9562 / 3019
Регистрация: 12.04.2006
Сообщений: 41,671
Блог
25.12.2018, 11:32
Ответы с готовыми решениями:

Как преобразовать тип System::String ^ в string? (или что можно сделать ещё для работы перегруженного оператора ostream::operator<<)
Друзья! В обработчик события Click добавил такой код: std::ofstream file_out (&quot;file.txt&quot;); file_out&lt;&lt;...

Неправильная работа оператора присваивания после работы оператора суммирования
Доброго времени суток. У меня есть класс вектор class TVector {//ewde public: TVector(); //Vector(Vector &amp;v); ...

Вызов перегруженного оператора
В классе Ellipse есть перезагрузка, не могу ее вызвать в main вот само задание: Создать класс Ellipse,содержащий следующие...

3
 Аватар для Fulcrum_013
2083 / 1574 / 169
Регистрация: 14.12.2014
Сообщений: 13,614
25.12.2018, 13:10
Цитата Сообщение от Матричный бог Посмотреть сообщение
Потом почему-то внезапно происходит вызов деструктора.
Не внезапно а вполне ожидаемо. Вы возращаете из оператора + объект на который указывает this по значению - т.е. все это копируется во временный объект который уничтожается в конце выражения. А поскольку копирование идет через автоматически созданный конструктор копирования, то копируются в этот временный объект просто указатели, а не данные на которые они указывают.
0
0 / 0 / 0
Регистрация: 14.09.2016
Сообщений: 153
25.12.2018, 15:22  [ТС]
А почему это происходит автоматическое создание конструктора копирования? Я же написал конструктор копирования...
0
 Аватар для Fulcrum_013
2083 / 1574 / 169
Регистрация: 14.12.2014
Сообщений: 13,614
26.12.2018, 01:43
Лучший ответ Сообщение было отмечено Матричный бог как решение

Решение

Цитата Сообщение от Матричный бог Посмотреть сообщение
А почему это происходит автоматическое создание конструктора копирования? Я же написал конструктор копирования...
Не заметил его. Могли бы и не писать. Точно такой же создаст компилятор. Но главная проблема в том что вы возвращаете по значению из оператора + и именно это приводит к созданию копии в которую попадают те же указатели и потом эта копия уничтожается. т.е возвращать нужно ссылку.
А касательно конструкторов - то для объектов владеющих динамически выделенными объектами, либо конструктор копирования должен осуществлять глубокое копирование, либо нужно делать конструктор перемещения а не конструктор копирования. Иначе будет падать с сегментейшн фаулт на каждом шагу.
1
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
inter-admin
Эксперт
29715 / 6470 / 2152
Регистрация: 06.03.2009
Сообщений: 28,500
Блог
26.12.2018, 01:43
Помогаю со студенческими работами здесь

Использование перегруженного оператора <<
#include &lt;iostream&gt; #include &lt;conio.h&gt; #include &lt;vector&gt; #include &lt;string&gt; #include &lt;algorithm&gt; using namespace std; class...

Странный вывод перегруженного оператора
Здравствуйте. Делаю цепочки вывода, но получается вот что: Почему выводится число 0x605508 ? Непечатный символ что-ли какой-то? И,...

Возврат ссылки от функции перегруженного оператора
Здравствуйте, меня интересует вопрос, когда функция-член класса, к примеру перегруженного оператора присваивания возвращает ссылку, зачем...

Указатель this внутри перегруженного оператора сложения
пишу класс Fraction для работы с дробными числами. Почему не допускает использовать this, пишет: &quot;Error: оператор this можно...

Ошибка при вызове в switch перегруженного оператора
есть перегруженный оператор: std::istream&amp; operator&gt;&gt;(istream&amp; is, diagram&amp; dd) { string kod; std::cout&lt;&lt;&quot;Vvedite...


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

Или воспользуйтесь поиском по форуму:
4
Ответ Создать тему
Новые блоги и статьи
Access
VikBal 11.12.2025
Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук
volvo 07.12.2025
Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом
volvo 04.12.2025
Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python
IndentationError 23.11.2025
Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов
Argus19 22.11.2025
Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
Сколько Государство потратило денег на меня, обеспечивая инсулином.
Programma_Boinc 20.11.2025
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
Ломающие изменения в C#.NStar Alpha
Etyuhibosecyu 20.11.2025
Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .
Мысли в слух
kumehtar 18.11.2025
Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .
Создание Single Page Application на фреймах
krapotkin 16.11.2025
Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .
Фото: Daniel Greenwood
kumehtar 13.11.2025
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru