Шаблоны классов

@Invisibletime · Регистрация: 14.10.2018

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

Добрый день. Начали изучать шаблоны классов, для примера дали программу, но в ней есть. Не могли бы объяснить как их исправить .

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
template<class type>
 
class CListTemplate
 
{
 
struct SNode
 
{
 
type *data;
 
SNode *next;
 
};
 
SNode *head,*p;
 
public:
 
CListTemplate() {head=p=0;}
 
void AddElem(type *d);
 
bool RemoveElem(type *d);
 
void DisplayList();
 
void clear();
 
~CListTemplate(){clear();}
 
};
 
template<class type>
 
void CListTemplate<type>::AddElem(type *d)
 
{
 
if(head)
 
{
 
p->next = (SNode*)malloc(sizeof(SNode));
 
p = p->next;
 
}
 
else
 
{
 
head = (SNode*)malloc(sizeof(SNode));
 
p=head;
 
}
 
p->data = d;
 
p->next = 0;
 
}
 
template<class type>
 
bool CListTemplate<type>::RemoveElem(type *d)
 
{
 
SNode *curr = head;
 
SNode *p1;
 
if (*(head->data)==*d)
 
{
 
p1=head;
 
head=head->next;
 
delete p1;
 
return true;
 
}
 
while(curr->next && *(curr->next->data)!=*d)
 
curr=curr->next;
 
if (curr->next)
 
{
 
p1=curr->next;
 
curr->next=curr->next->next;
 
delete p1;
 
return true;
 
}
 
return false;
 
}
 
template<class type>
 
void CListTemplate<type>::DisplayList()
 
{
 
SNode *curr=head;
 
while (curr)
 
{
 
cout<<*(curr->data)<<" ";
 
curr=curr->next;
 
}
 
cout<<endl;
 
}
 
template<class type>
 
void CListTemplate<type>::clear()
 
{
 
SNode *curr=head;
 
while(head)
 
{
 
curr=head;
 
head=head->next;
 
delete curr;
 
}
 
}
 
int main()
 
{
 
CListTemplate<Complex> cl;
 
CListTemplate<int> c_int;
 
for (int i=0;i<10; i++)
 
{
 
cl.AddElem(new Complex(10.0f*i,10.0f+i));
 
c_int.AddElem(new int(10*i));
 
}
 
cout<<endl<<endl<<endl;
 
cl.DisplayList();
 
Complex c_ob(40.0f,14.0f);
 
cl.RemoveElem(&c_ob);
 
cl.DisplayList();
 
cout<<endl<<endl<<endl;
 
c_int.DisplayList();
 
int i=90;
 
c_int.RemoveElem(&i);
 
c_int.DisplayList();
 
return 0;
 
}

@L0M · 23.11.2019, 14:21

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

для примера дали программу, но в ней есть.

Действительно есть. Странненькая программа, особенно в качестве образца.
Я там подправил:

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
#include <iostream>
#include <complex>
 
using namespace std;
 
template<class type>
class CListTemplate
{
    struct SNode
    {
        type* data;
        SNode* next;
    };
    SNode* head, * p;
public:
    CListTemplate() { head = p = 0; }
    void AddElem(type* d);
    bool RemoveElem(type* d);
    void DisplayList();
    void clear();
    ~CListTemplate() { clear(); }
};
template<class type>
void CListTemplate<type>::AddElem(type* d)
{
    if (head)
    {
        p->next = new SNode;
        p = p->next;
    }
    else
    {
        head = new SNode;
        p = head;
    }
    p->data = d;
    p->next = 0;
}
template<class type>
bool CListTemplate<type>::RemoveElem(type* d)
{
    SNode* curr = head;
    SNode* p1;
    if (*(head->data) == *d)
    {
        p1 = head;
        head = head->next;
        if (p1->data)
            delete p1->data;
        delete p1;
        return true;
    }
    while (curr->next && *(curr->next->data) != *d)
        curr = curr->next;
    if (curr->next)
    {
        p1 = curr->next;
        curr->next = curr->next->next;
        if (p1->data)
            delete p1->data;
        delete p1;
        return true;
    }
    return false;
}
template<class type>
void CListTemplate<type>::DisplayList()
{
    SNode* curr = head;
    while (curr)
    {
        cout << *(curr->data) << " ";
        curr = curr->next;
    }
    cout << endl;
}
template<class type>
void CListTemplate<type>::clear()
{
    SNode* curr = head;
    while (head)
    {
        curr = head;
        head = head->next;
        if (curr->data)
            delete curr->data;
        delete curr;
    }
}
int main()
{
    CListTemplate<complex<float>> cl;
    CListTemplate<int> c_int;
    for (int i = 0; i < 10; i++)
    {
        cl.AddElem(new complex<float>(10.0f * i, 10.0f + i));
        c_int.AddElem(new int(10 * i));
    }
    cout << endl << endl << endl;
    cl.DisplayList();
    complex<float> c_ob(40.0f, 14.0f);
    cl.RemoveElem(&c_ob);
    cl.DisplayList();
    cout << endl << endl << endl;
    c_int.DisplayList();
    int i = 90;
    c_int.RemoveElem(&i);
    c_int.DisplayList();
    return 0;
}

В оригинальной программе, видимо, есть что-то типа #include "Complex.h" для использования самописного класса комплексных чисел. Но все инклюды вы обрезали, и самописного класса комплексных чисел тоже нет. Я переделал под std::complex из STL. Что б работало.

Кроме того убрал утечки памяти.

@Invisibletime · 23.11.2019, 14:27 **[ТС]**

Огромнейшее Вам спасибо.

@marat_miaki · 23.11.2019, 16:04

L0M,

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

if (curr->data) delete curr->data; delete curr;

Почему два удаления, разве delete curr; недостаточно?

@L0M · 23.11.2019, 17:26

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

Почему два удаления, разве delete curr; недостаточно?

Недостаточно.
delete curr удалит только экземпляр класса SNode. Но SNode.data указывает на данные, которые тоже размещаются в куче: cl.AddElem(new complex<float>(10.0f * i, 10.0f + i)). Для того, что бы не было утечки памяти сначала удаляем данные (delete curr->data), а затем элемент списка (delete curr).

Вообще, в данной реализации класса CListTemplate есть один неприятный момент: void CListTemplate<type>::AddElem(type* d). AddElem() принимает указатель на экземпляр. И впоследствии нет никакой возможности понять на что он указывает: на объект в куче, на объект в стеке или на объект в глобальной памяти. У вас сейчас в main() создаются объекты на куче, в метод передаётся указатель на объект и этот адрес обольше ни где не запоминается. Поэтому считаем, что ответственность за уничтожение объекта несёт CListTemplate. Однако, с таким же успехом в AddElem() можно передать адрес объекта на стеке. Тогда при попытке его удаления случится неприятность.

Лучшим решением было бы

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
template<class type>
class CListTemplate
{
    struct SNode
    {
        type data;
        SNode* next;
    };
    SNode* head, * p;
public:
    CListTemplate() { head = p = 0; }
    void AddElem(type d);
    bool RemoveElem(type d);
    void DisplayList();
    void clear();
    ~CListTemplate() { clear(); }
};

Соответственно, в AddElem() будет простое присваивание фактического параметра метода полю data экземпляра структуры SNode. Т.е. в списке будет находиться копия добавляемого объекта. Тогда, естественно, delete для поля data не нужно.

При этом ни что не мешает создать список указателей на объекты: параметром шаблона будет тип-указатель. Но тогда вызывающий код будет однозначно ответственен как за создание объектов на куче, так и за их удаление. И это правильно.

Новые блоги и статьи Все статьи Все блоги /
Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .	SDL3 для Web (WebAssembly): Сборка библиотек SDL3 и Box2D из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия SDL 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual. . .	SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»

@Invisibletime 2 / 2 / 0 Регистрация: 14.10.2018 Сообщений: 122
	23.11.2019, 14:27 [ТС]
	Огромнейшее Вам спасибо. 0

Шаблоны классов

Решение