Конструктор копии и операторы

@JamesBlack · Регистрация: 28.07.2009

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

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
list::list(list const& l)
{
    l.H();
    if (l.head==NULL) {head=present=NULL; return;}
    head=present=new element(l.head);
    while(l.present->next)
    {
        l.Next();
        present->next = new element(l.present->number);
        Next();
    }
}
 
list& operator = (list const& a)
{
    Clear();
    a.H();
    if (a.head==NULL) return *this;
    head = present = new element(a.head);
    while(a.present->next)
    {
        a.Next();
        present->next = new element(a.present->number);
        Next();
    }
    return *this;
}
 
list& operator + (list const& a)
{
    list *result = new list(*this);
    result->Add(a);
    return *result; 
}
 
void main()
{
list a(...), b(...), c(...);
...
a = b + c;
}

Мне надо перегрузить операции + и = для моего класса list. Только приходится в + создавать временный объект класса в куче. А если так, то получается, что он там и остается - я его нигде не удаляю. Как быть с этой проблемой? Или можно как-то сделать все через автоматическую память?

Добавлено через 1 час 42 минуты
мда... все спят.

@M128K145 · 23.11.2009, 04:29

JamesBlack, выложил бы весь код класса что ли...

@JamesBlack · 23.11.2009, 05:49 **[ТС]**

Ты уверен, что оно надо? Я выложил только 3 метода, где как раз у меня возникате проблема. Но если так хотите...

просьба не писать вопросы типа "что за названия и комменты?" - прога девушки. Я точнее пишу ей прогу, она ее "переделывает под себя".

class.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
39
40
41
42
43
44
45
46
47
48
49
50
51
#ifndef _CLASS_H_
#define _CLASS_H_
 
#include <stdio.h>
 
class list
{
    struct element
    {
        int number;
        element *next;
        element ()  {   number=0;   next=NULL; }
        element (int num) {     number=num;     next=NULL; }
        element (int num, element* _next) { number=num; next=_next; }
    };
 
    element *head;      // голова
    element *present;  // текущий
public:
    list ();                  //пустой конструктор
    list (int *mas, int l);   //конструктор с массивом
    list::list (list const&);
    ~list ();                 //деструктор
    list& Next ();             //перейти к следующему
    list& H();                 //перейти в голову
    list& Insert (int n, int num);//номер позиции с которой нужно вставляем число и само число
    list& InsertMas (int, int*, int); //номер позиции с которой вставляем массив, сам массив и его длина
    list& Del (int);               //удаление элемента
    int GetNum (int) const;             //получить значение конкретного элемента
    int GetValue() const;
    list& Del ();                  //удаление элемента
    list& Clear();
    list& SetNum (int, int);       //установить значение конкретного элемента
    list& Add (list&);           //+
    list& Sub (list&);               //-
    int Compare (list const& )const;
    list& Out() const;
 
    list& operator = (list const&);
    list& operator + (list const&);
    list& operator + (int);
    list& operator - (list const&);
    list& operator - (int);
    int operator == (list const&);
    int operator != (list const&);
    int operator < (list const&);
    int operator > (list const&);
    friend ostream& operator << (ostream&, list const&);
    friend istream& operator >> (ostream&, list&);
};
#endif

class.cpp

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
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
#include "class.h"
#include <conio.h>
#include <iostream.h>
#include <stdio.h>
 
 
list::list()
{
    head=present=new element();
}
//_____________________________
 
list::list (int *mas, int l)
{
    int i;
    element *prev=NULL, *temp=NULL;
    head=NULL;
    present=NULL;
    for (i=0; i<l; i++)
    {
        temp=new element(mas[i]);
        if (i==0)
            head=temp;
        if (prev)
            prev->next=temp;
        present = temp;
        prev =temp;
    }
    prev->next=NULL;
}
//_____________________________
 
list::list(list const& l)
{
    l.H();
    if (l.head==NULL) {head=present=NULL; return;}
    head=present=new element(l.head);
    while(l.present->next)
    {
        l.Next();
        present->next = new element(l.present->number);
        Next();
    }
}
//_____________________________
 
list::~list ()
{
    present=head;
    while(present)
    {
        head=present;
        present=head->next;
        delete head;
    }
}
//_____________________________
 
list& list::Next()
{
    present=present->next;
    return *this;
};
//_____________________________
 
list& list::H()
{
    present=head;
    return *this;
};
//_____________________________
 
list& list::Insert(int n, int num)
{
    element *temp;
    H();
    if (!n)                                              //Ґб«Ё ўбв*ў«пҐ¬ б 0 н«Ґ¬Ґ*в*
    {
        head = new element(num, present);
        return *this;
    }
    for (int i=1; i<n; i++)
        if (present->next)            //Ґб«Ё б«Ґ¤ н«Ґ¬Ґ*в бгйҐбвўгҐв, в® ¤ўЁЈ*Ґ¬бп ¤*«миҐ
            Next();
        else                                //Ґб«Ё *Ґв, в® ®бв***ў«Ёў*Ґ¬бп
            break;
    //ўбв*ў«пҐ¬ зЁб«®
    temp = present->next;
    present->next = new element(num, temp);
    return *this;
};
//_____________________________
 
list& list::InsertMas(int n, int *mas, int l)
{
    element *temp;
    H();
    if (!n)                                                                        //Ґб«Ё ўбв*ў«пҐ¬ б 0 н«Ґ¬Ґ*в*
    {
        temp = present;
        present = head = new element(mas[0], temp);
        Next();
        for (int j=1; j<l-1; j++)
        {
            present->next = new element(mas[j]);
            Next();
        }
        present->next = new element(mas[l-1], temp);
        return *this;
    }
    for (int i=1; i<n; i++)
        if (present->next)   //Ґб«Ё б«Ґ¤ н«Ґ¬Ґ*в бгйҐбвўгҐв, в® ¤ўЁЈ*Ґ¬бп ¤*«миҐ
            Next();
        else                                                                    //Ґб«Ё *Ґв, в® ®бв***ў«Ёў*Ґ¬бп
            break;
    //ўбв*ў«пҐ¬ ¬*ббЁў
    temp = present->next;
    for (int j=0; j<l-1; j++)
    {
        present->next = new element(mas[j]);
        Next();
    }
    present->next = new element(mas[l-1], temp);
    return *this;
};
//_____________________________
 
list& list::Del(int n)
{
    element *temp;
    H();
    if (!n)
    {
        Next();
        delete head;
        head = present;
        return *this;
    }
    for(int i=1; i<n; i++)
         if (present->next)
                Next ();
         else
                return this;
    temp = present->next->next;
    delete present->next;
    present->next = temp;
    return *this;
};
//_____________________________
 
int list::GetNum(int n) const
{
    H();
    for (int i=0; i<n; i++)
        if (present->next)
            Next();
        else
            return -1;
    return present->number;
};
//_____________________________
 
int list::GetValue () const
{
    return present->number;
};
 
//____________________________
 
list& list::Del ()
{
    element *temp=present;
    H();
    if (temp==head)
    {
        Next();
        delete head;
        head = present;
        return *this;
    }
    while (present->next!=temp)
         if (present->next)
                Next ();
         else
                return *this;
    temp = temp->next;
    delete present->next;
    present->next = temp;
    return *this;
};
//____________________________
 
list& list::Clear()
{
    element *temp=head;
    H();
    while (present)
    {
        temp=present->next;
        delete present;
        present=temp;
    }
    head = present = NULL;
    return *this;
};
//_____________________________
 
list& list::SetNum(int n, int num) //¬Ґ*пҐ¬ §**зҐ*ЁҐ ў пзҐ©ЄҐ бЇЁбЄ*
{
    H();
    for(int i=0; i<n; i++)
        if (present->next)
            Next();
        else
            return *this;
    present->number=num;
    return *this;
};
//_____________________________
 
list& list::Add (list& a)
{
    while (present->next)
        Next();
    a.H();
    while (a.present)
    {
        present->next= new element (a.GetValue());
        Next();
        a.Next();
    }
    present->next=NULL;
    return this;
};
 
//____________________________
 
list& list::Sub (list& a)
{
    a.H();
    int temp;
    while (a.present)
    {
         temp = a.GetValue();
         H();
         while (present)
         {
                if (GetValue()==temp)
                    Del();
                    Next();
         }
         a.Next();
    }
    return *this;
};
//____________________________
 
list& list::Out() const
{
    H();
    while (present)
    {
        cout << GetValue() <<" ";
        Next();
    }
    cout << "\n";
    return *this;
};
//___________________________
 
list& operator + (list const& a)
{
        list *result = new list(*this);
        result->Add(a);
        return *result;
}
 
list& operator = (list const& a)
{
        Clear();
        a.H();
        if (a.head==NULL) return *this;
        head = present = new element(a.head);
        while(a.present->next)
        {
                a.Next();
                present->next = new element(a.present->number);
                Next();
        }
        return *this;
}

Добавлено через 1 минуту
Черт, еще и кодировка сбилась.

Добавлено через 45 минут
Черт, ну подскажите кто-нить, через 2 часа в универе надо быть, а если я буду там без лабы, то получу каблуком в очень мягкое место от той самой девушки

@M128K145 · 23.11.2009, 09:22

JamesBlack, если я не ошибаюсь, то result будет удален при выходе из метода и за выделяемую память не стоит переживать

@JamesBlack · 24.11.2009, 13:09 **[ТС]**

Да нет, я в кучу записывал, там ничего не удаляется.

Спс всем, я разобрался - там надо в операциях возвращать не ссылку, а сам объект, чтобы вызывался конструктор копии. И тогда все создавать в автоматической памяти и оно само будет удаляться.

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .
Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .

@M128K145 8384 / 3617 / 419 Регистрация: 03.07.2009 Сообщений: 10,709
	23.11.2009, 04:29
	JamesBlack, выложил бы весь код класса что ли... 0

@M128K145 8384 / 3617 / 419 Регистрация: 03.07.2009 Сообщений: 10,709
	23.11.2009, 09:22
	JamesBlack, если я не ошибаюсь, то result будет удален при выходе из метода и за выделяемую память не стоит переживать 0

@JamesBlack 46 / 46 / 5 Регистрация: 28.07.2009 Сообщений: 144
	24.11.2009, 13:09 [ТС]
	Да нет, я в кучу записывал, там ничего не удаляется. Спс всем, я разобрался - там надо в операциях возвращать не ссылку, а сам объект, чтобы вызывался конструктор копии. И тогда все создавать в автоматической памяти и оно само будет удаляться. 0