Освобождение динамически созданных переменных (2-ого уровня)

@Izual · Регистрация: 13.11.2012

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

И так, продолжу старую тему про динамику.

На этот раз есть вот что:

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
#include <iostream>
#include <stdio.h>
#include <conio.h>
#include <math.h>
 
using namespace std;
 
typedef enum { BOOL,INT,FLOAT,STR } vtr;
 
struct var
{
    vtr t;//тип переменной
    union
    {
        bool b;
        int i;
        float f;
        char *s;
    }d;//данные переменной
};
var *v;
var *vn;
 
int gi = 0;
bool a = false;
int b = 0;
int DelMas();
int AddVar(int);
int Convert(int);
int Otext(int);
int main (int argc, char* argv[])
{
    int r = 0;
    b = 1;
    v = new var[b]; // 1 элемент
    v[0].t = BOOL;
    v[0].d.b = true;
    a = true;
    while(a == true)
    {
        r = 0;
        cout << "Viberite deictvie" << endl << "1:Read variable" << endl << "2:Add variable" << endl << "3:Exit" << endl;
        cin >> r;
        switch(r)
        {
        case 1:
            {
                if(b > 0)
                {
                    for(int i=0;i<b;i++)
                    {
                        cout << "Num: " << i << "; Type: " << v[i].t << "; Data: ";
                        gi = Otext(i);
                    }
                }
                else
                    cout << "Base Clean, you must add variable" << endl;
            }
            break;
        case 2:
            {
                cout << "Put special number equalent of type variable:" << endl;
                cout << "0.Bool; 1.Int; 2.Float; 3.Char" << endl;
                cin >> gi;
                if((gi < 0)||(gi > 3))
                {
                    cout << "Error number of type" << endl;
                    break;
                }
                gi=AddVar(gi);
                if(gi==0)
                {
                    cout << "Error adding variable" << endl;
                    break;
                }
            }
            break;
        case 3:
            a = false;
            break;
        default:
            cout << "Unknown procedure" << endl;
        }
    }
    gi = DelMas();
    delete []v;
    cout << "Struct destroyed" << endl;
    getch();
    return 0;
}
int Convert(int fi)
{
    switch(v[fi].t)
    {
    case BOOL:
        vn[fi].d.b = v[fi].d.b;
        break;
    case INT:
        vn[fi].d.i = v[fi].d.i;
        break;
    case FLOAT:
        vn[fi].d.f = v[fi].d.f;
        break;
    case STR:
        //strcpy(vn[fi].d.s,v[fi].d.s);
        vn[fi].d.s = v[fi].d.s;
        //delete[] v[fi].d.s;
        break;
    default:
        cout << "Unknown type" << endl;
    }
    return 0;
}
int DelMas()
{
    if(b > 0)
    {
        for(int i=0;i<b;i++)
        {
            if(v[i].t == STR)
            {
                delete[] v[i].d.s;
            }
        }
    }
    return 0;
}
int AddVar(int ft)
{
    char buf[255]="";
    int fRi=0;
    if((ft < 0)||(ft > 3))
    {
        return 0;
    }
    else
    {
        vn = new var[b+1];
        switch(ft)
        {
        case 0:
            {
                vn[b].t = BOOL;
                cout << "Enter Number: 0 - False; 1 - True;" << endl;
                gi=0;
                cin >> gi;
                if((gi < 0)||(gi > 1))
                {
                    cout << "Error, allowed only 1 or 0" << endl;
                    break;
                }
                if(gi==0)
                    vn[b].d.b = false;
                if(gi==1)
                    vn[b].d.b = true;
                fRi=1;
            }
            break;
        case 1:
            {
                vn[b].t = INT;
                cout << "Enter Integer Number" << endl;
                cin >> vn[b].d.i;
                cout << "Integer: " << vn[b].d.i << endl;
                fRi=1;
            }
            break;
        case 2:
            {
                vn[b].t = FLOAT;
                cout << "Enter Float Number" << endl;
                cin >> vn[b].d.f;
                fRi=1;
            }
            break;
        case 3:
            {
                vn[b].t = STR;
                cout << "Enter Letters" << endl;
                cin >> buf;
                vn[b].d.s = new char[strlen(buf)+1];
                strcpy(vn[b].d.s,buf);
                //cout << "Result: " << vn[b].d.s << endl;
                fRi=1;
            }
            break;
        default:
            cout << "Unknown type" << endl;
        }
        cout << "Start convert.." << endl;
        if(fRi==1)
        {
            for(int j=0;j<b;j++)
            {
                vn[j].t = v[j].t;
                gi = Convert(j);
            }
            b++;
            delete []v;
            v = vn;
        }
        else
        {
            delete []vn;
        }
    }
    return fRi;
}
int Otext(int fi)
{
    switch(v[fi].t)
    {
    case BOOL:
        if(v[fi].d.b == false)
        {
            cout << "False" << endl;
        }
        else
        {
            cout << "True" << endl;
        }
        break;
    case INT:
        cout << v[fi].d.i << endl;
        break;
    case FLOAT:
        cout << v[fi].d.f << endl;
        break;
    case STR:
        cout << v[fi].d.s << endl;
        break;
    default:
        cout << "Unknown type" << endl;
    }
    return 0;
}

Как видно из кода, динамически создаётся структура (возможно не правильно выразился, но не суть) в main:

C++
1
v = new var[b];

Я назвал это "первым уровнем".
Далее при добавлении переменной приходится снова обращяться к оператору New:

C++
1
vn = new var[b+1];

Естественно перезаписываются прежние данные в новый экземпляр vn, а старый v - удаляется.
Если новая переменная является STR, то приходится вновь пользоваться new:

C++
1
vn[b].d.s = new char[strlen(buf)+1];

Это второй уровень. ( типа вложенная динамическая переменная в динамическую структуру... ололо

)

Вопрос собственно в том как правильно пользоваться оператором delete[] в данном случае. Я предположил что тут:

C++
1
2
vn[fi].d.s = v[fi].d.s;
delete[] v[fi].d.s;

надо бы удалить старую переменную, однако если разкоментировать эту операцию удаления, то при добавлении более двух раз типа STR первая будет стираться.
В общем поправьте код кто может, и если не затруднит может объясните как было бы правильно (можно и ссылку дать, если где то уже объяснено).
Спасибо.
*Пс: В архиве собственно сам exe, но с закоментированной строкой "//delete[] v[fi].d.s;"

@КОП · 10.08.2014, 01:49

Если я правильно понял суть проблемы, то вы зачем-то хотите освободить ту память, где хранится само значение строки, хотя

C++
1
vn[fi].d.s = v[fi].d.s;

копирует лишь указатель, но не дублирует саму строку. В данном случае вообще нет необходимости в delete[].

А вообще, попробуйте использовать вместо массива вектор, тогда никаких мутных копирований массивов не будет.

@Izual · 10.08.2014, 05:35 **[ТС]**

Сообщение от КОП

В данном случае вообще нет необходимости в delete[].

Но ведь если исходить из логики того, что если переменная была создана через New, то и удалять её надо Delete. В моём случае мне кажется что если я не удалю, то будет утечка памяти.(кстати можно как нибудь проверить?)

Сообщение от КОП

попробуйте использовать вместо массива вектор

НЕ ПРЕДЛАГАТЬ !!! (у меня свой стиль, векторы я принципиально использовать не хочу)

Я тут подумал, а ведь новую переменную vn[fi].d.s я не создаю через new... значит и удалять старую не нужно, хотя если не удалить, то старая останется, что не очень хорошо, ведь доступа к ней не будет. (ну я так думаю, а рез нет доступа, то при функции очищения памяти, вызываемой в конце main я не смогу освободить память этой переменной, т.к. освободится уже новая vn (удаляться то будет конешно v, но ведь после создания переменной v = vn)

Честно говоря я в замешательстве...

Сообщение от КОП

копирует лишь указатель

значит мне надо создать через new новую vn[fi].d.s, потом скопировать само значение и удалить старую? Вообще стоит ли это сделать?

Добавлено через 54 минуты
И кстати, когда я сделаю ещё функцию изменения переменной, то мне точно надо будет иметь возможность удалять старые значения, а значит мне нужен не указатель, а реальное значение...

@КОП · 10.08.2014, 15:19

Проверка утечки памяти с++, гугл, первый запрос.

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

Я тут подумал, а ведь новую переменную vn[fi].d.s я не создаю через new... значит и удалять старую не нужно

Вот на этом и нужно поставить точку в предложении. Кол-во new равно кол-ву delete. Никакой старой переменной не остается, она всегда будет одна, вы лишь копируете указатели на неё.

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

значит мне надо создать через new новую vn[fi].d.s, потом скопировать само значение и удалить старую? Вообще стоит ли это сделать?

Новую создавать не нужно, при работе со строками копируют как раз указатели, что вы и сделали.

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

И кстати, когда я сделаю ещё функцию изменения переменной, то мне точно надо будет иметь возможность удалять старые значения, а значит мне нужен не указатель, а реальное значение...

вот тогда и будете освобождать память, и для этого нужен именно указатель, а реальное значение никакой роли не играет.

И напоследок, прошу не кидаться камнями, переписал программу с использованием векторов. (Сделал это, т.к. сам пару дней назад открыл для себя векторы и не успел потренироваться) Как по мне, код стало гораздо легче читать и вероятность накосячить с той же памятью в разы меньше.

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

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
#include "stdafx.h"
#include <iostream>
#include <conio.h>
#include <vector>
using namespace std;
 
typedef enum { BOOL, INT, FLOAT, STR } vtr;
 
struct var
{
    vtr t;//тип переменной
    union
    {
        bool b;
        int i;
        float f;
        char *s;
    }d;//данные переменной
};
vector<var> v;
 
 
int gi = 0;
bool a = false;
int DelMas();
int AddVar(int);
int Otext(int);
int main(int argc, char* argv[])
{
    var tmp;
    tmp.t = BOOL;
    tmp.d.b = true;
    v.push_back(tmp); // 1 элемент
    
    a = true;
    while (a == true)
    {
        int r = 0;
        cout << "Viberite deictvie" << endl << "1:Read variable" << endl << "2:Add variable" << endl << "3:Exit" << endl;
        cin >> r;
        switch (r)
        {
        case 1:
        {
                  if (!v.empty())
                  {
                      for (int i = 0; i<v.size(); i++)
                      {
                          cout << "Num: " << i << "; Type: " << v[i].t << "; Data: ";
                          gi = Otext(i);
                      }
                  }
                  else
                      cout << "Base Clean, you must add variable" << endl;
        }
            break;
        case 2:
        {
                  cout << "Put special number equalent of type variable:" << endl;
                  cout << "0.Bool; 1.Int; 2.Float; 3.Char" << endl;
                  cin >> gi;
                  if ((gi < 0) || (gi > 3))
                  {
                      cout << "Error number of type" << endl;
                      break;
                  }
                  gi = AddVar(gi);
                  if (gi == 0)
                  {
                      cout << "Error adding variable" << endl;
                      break;
                  }
        }
            break;
        case 3:
            a = false;
            break;
        default:
            cout << "Unknown procedure" << endl;
        }
    }
    gi = DelMas();
    cout << "Struct destroyed" << endl;
    system("pause");
    return 0;
}
int DelMas()
{
    if (!v.empty())
    {
        for (int i = 0; i<v.size(); i++)
        {
            if (v[i].t == STR)
            {
                delete[] v[i].d.s;
            }
        }
    }
    return 0;
}
int AddVar(int ft)
{
    var temp;
    char buf[255] = "";
    int fRi = 0;
    if ((ft < 0) || (ft > 3))
    {
        return 0;
    }
    else
    {
        switch (ft)
        {
        case 0:
        {
                  temp.t = BOOL;
                  cout << "Enter Number: 0 - False; 1 - True;" << endl;
                  cin >> gi;
                  if ((gi < 0) || (gi > 1))
                  {
                      cout << "Error, allowed only 1 or 0" << endl;
                      break;
                  }
                  if (gi == 0)
                      temp.d.b = false;
                  if (gi == 1)
                      temp.d.b = true;
                  fRi = 1;
        }
            break;
        case 1:
        {
                  temp.t = INT;
                  cout << "Enter Integer Number" << endl;
                  cin >> temp.d.i;
                  cout << "Integer: " << temp.d.i << endl;
                  fRi = 1;
        }
            break;
        case 2:
        {
                  temp.t = FLOAT;
                  cout << "Enter Float Number" << endl;
                  cin >> temp.d.f;
                  fRi = 1;
        }
            break;
        case 3:
        {
                  temp.t = STR;
                  cout << "Enter Letters" << endl;
                  cin >> buf;
                  temp.d.s = new char[strlen(buf) + 1];
                  strcpy(temp.d.s, buf);
                  //cout << "Result: " << vn[b].d.s << endl;
                  fRi = 1;
        }
            break;
        default:
            cout << "Unknown type" << endl;
        }
        v.push_back(temp);
    }
    return fRi;
}
int Otext(int fi)
{
    switch (v[fi].t)
    {
    case BOOL:
        cout << std::boolalpha << v[fi].d.b << endl;
        break;
    case INT:
        cout << v[fi].d.i << endl;
        break;
    case FLOAT:
        cout << v[fi].d.f << endl;
        break;
    case STR:
        cout << v[fi].d.s << endl;
        break;
    default:
        cout << "Unknown type" << endl;
    }
    return 0;
}

@Izual · 10.08.2014, 17:43 **[ТС]**

Сообщение от КОП

а реальное значение никакой роли не играет.

Так мне же надо удалить реальное значение, оно же место занимает, а исходя из того как я понимаю, если я в последствии удалю "delete[] v[fi].d.s", то я удалю не само значение, а указатель. Так?

@КОП · 10.08.2014, 17:47

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

удалю "delete[] v[fi].d.s", то я удалю не само значение, а указатель

указатель нельзя удалить, можно лишь освободить память, на которую он указывает, т.е. значение

@Izual · 10.08.2014, 18:35 **[ТС]**

Сообщение от КОП

можно лишь освободить память, на которую он указывает, т.е. значение

Значит первый мною предложенный вариант будет правильно работать? (без delete[]) И значит что если мне надо будет удалить переменную, то указывая на delete[] v[fi].d.s удалится само значение(находящееся фиг знает где, куда как раз таки и указывает этот указатель)?
Ну в таком случае я спокоен, значит всё работает как надо.(без использования stl(векторов))

Новые блоги и статьи Все статьи Все блоги /
Автозаполнение реквизита при выборе элемента справочника Maks 27.03.2026 Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. При выборе "Спецтехники" (Тип Справочник. Спецтехника), заполняется. . .	Сумматор с применением элементов трёх состояний. Hrethgir 26.03.2026 Тут. https:/ / fips. ru/ EGD/ ab3c85c8-836d-4866-871b-c2f0c5d77fbc Первый документ красиво выглядит, но без схемы. Это конечно не даёт никаких плюсов автору, но тем не менее. . . всё может быть. . .	Автозаполнение реквизитов при создании документа Maks 26.03.2026 Программный код из решения ниже размещается в модуле объекта документа, в процедуре "ПриСозданииНаСервере". Алгоритм проверки заполнения реализован для исключения перезаписи значения реквизита,. . .	Команды формы и диалоговое окно Maks 26.03.2026 1. Команда формы "ЗаполнитьЗапчасти". Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. В качестве источника данных. . .
Кому нужен AOT? DevAlt 26.03.2026 Решил сделать простой ланчер Написал заготовку: dotnet new console --aot -o UrlHandler var items = args. Split(":"); var tag = items; var id = items; var executable = args;. . .	Отправка уведомления на почту при изменении наименования справочника Maks 24.03.2026 Программная отправка письма электронной почты на примере изменения наименования типового справочника "Склады" в конфигурации БП3. Перед реализацией необходимо выполнить настройку системной учетной. . .	модель ЗдравоСохранения 5. Меньше увольнений- больше дохода! anaschu 24.03.2026 Теперь система здравосохранения уменьшает количество увольнений. 9TO2GP2bpX4 a42b81fb172ffc12ca589c7898261ccb/ https:/ / rutube. ru/ video/ a42b81fb172ffc12ca589c7898261ccb/ Слева синяя линия -. . .	Midnight Chicago Blues kumehtar 24.03.2026 Такой Midnight Chicago Blues, знаешь?. . Когда вечерние улицы становятся ночными, а ты не можешь уснуть. Ты идёшь в любимый старый бар, и бармен наливает тебе виски. Ты смотришь на пролетающие. . .