Ошибка в деструкторе

@epitxx · Регистрация: 18.01.2018

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

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

Debug Assertion Failed
Expression: __CrtisValidHeapPointer(block)

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

В общем прошу помочь кто-чем может.

Спасибо.

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
template<class Type> class DStringList 
{
private:
    int element_count = 0;
    struct DString
    {
        Type* string = NULL;
        DString* next_DString = NULL;
    };
 
    DString* FirstDString = NULL;
    DString* LastDString = NULL;
    
public:
    DStringList();
    ~DStringList();
 
    void add_string(Type* string);
    int get_elements_count(void) { return element_count; }
 
 
    Type *operator[](int index)
    {
        int i = 0;
        DString* start = NULL;
        for (start = FirstDString; start != NULL; start = start->next_DString)
        {
            if (i == index) break;
            i++;
        }
 
        return start->string;
    }
    
};
 
template<class Type> DStringList<Type>::DStringList()
{
}
 
template<class Type> DStringList<Type>::~DStringList() 
{
    DString* destructor_ptr;
 
    for (destructor_ptr = FirstDString; destructor_ptr != NULL; destructor_ptr = destructor_ptr->next_DString)
        delete[] destructor_ptr->string;
 
    for (destructor_ptr = FirstDString->next_DString; destructor_ptr != NULL; destructor_ptr = destructor_ptr->next_DString)
    {
        delete FirstDString;
        FirstDString = destructor_ptr;
    }
}
 
template<class Type> void DStringList<Type>::add_string(Type * string)
{
    int string_len = 0;
    for (string_len = 0; string[string_len] != '\0'; string_len++);
 
 
    DString* new_DString = new DString;
    if (FirstDString == NULL) FirstDString = new_DString;
 
    new_DString->string = new Type[string_len + 1];
 
    for (int i = 0; i < string_len; i++)
        new_DString->string[i] = string[i];
    new_DString->string[string_len] = '\0';
 
    if (LastDString != NULL)
        LastDString->next_DString = new_DString;
        
    LastDString = new_DString;
 
    element_count++;
}
 
 
 
 
 
int main(int argc, char** argv) {
    
    DStringList<WCHAR> list;
 
    list.add_string(L"Hello World");
    list.add_string(L"World Hello");
 
    wprintf(L"%d\n", list.get_elements_count());
    wprintf(L"%s\n", list[0]);
    wprintf(L"%s\n", list[1]);
    
    list.~DStringList();
    
 
    getchar();
}

@КулХацкеръ · 11.02.2018, 10:48

epitxx, с конца удаляй:

C++
1
2
3
4
5
6
7
8
9
10
11
12
template<class Type> DStringList<Type>::~DStringList() 
{
    DString* destructor_ptr;
    if(FirstDString!=NULL){
        while (FirstDString!=LastDString){
            for (destructor_ptr=FirstDString; destructor_ptr->next_DString!=LastDString; destructor_ptr=destructor_ptr->next_DString);
            delete LastDString;
            LastDString=destructor_ptr;
        }
        delete LastDString;
    }
}

nd2 · 11.02.2018, 13:28

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

C++
1
list.~DStringList();

Не нужно этого делать. Деструктор автоматически вызывается при выходе из области, где был создан list (main()).

@epitxx · 11.02.2018, 16:05 **[ТС]**

По поводу автоматического вызова деструктора верно, но возникает еще одна проблема.

Вот такой код:

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
WCHAR* l;
 
void Test()
{
    DStringList<WCHAR> list;
 
    list.add_string(L"Hello World");
    list.add_string(L"World Hello");
 
    wprintf(L"%d\n", list.get_elements_count());
    wprintf(L"%s\n", list[0]);
    wprintf(L"%s\n", list[1]);
 
    l = list[0];
    
    wprintf(L"Test l:%s\n", l);
    //list.~DStringList();
    wprintf(L"Test2 l:%s\n", l);
}
 
 
 
int main(int argc, char** argv) {
    
    Test();
 
    wprintf(L"Main l:%s\n", l);
 
    l[0] = 'A';
    l[1] = '\0';
 
    wprintf(L"Main l:%s\n", l);
 
    getchar();
}

При выходе из Test() деструктор действительно срабатывает, но я все еще могу обратиться к строке в структуре DString.

Вот результат работы:
2
Hello World
World Hello
Test l:Hello World
Test2 l:Hello World
Main l:??????????????
Main l:A

nd2 · 11.02.2018, 16:41

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

При выходе из Test() деструктор действительно срабатывает, но я все еще могу обратиться к строке в структуре DString.

И что? delete, как известно, не удаляет, а ОСВОБОЖДАЕТ память: память помечается как свободная. Что там находится после этого - зависит от разного, и к самому освобождению памяти отношения не имеет.

@epitxx · 11.02.2018, 17:38 **[ТС]**

Тогда все прекрасно, большое спасибо за ответ

Новые блоги и статьи Все статьи Все блоги /
Перемещение выделенных строк ТЧ из одного документа в другой Maks 30.03.2026 Реализация из решения ниже выполнена на примере нетипового документа "ВыдачаОборудованияНаСпецтехнику" с единственной табличной частью "ОборудованиеИКомплектующие" разработанного в конфигурации КА2. . . .	Functional First Web Framework Suave DevAlt 30.03.2026 Sauve. IO Апнулись до NET10. Из зависимостей один пакет, работает одинаково хорошо как в режиме проекта так и в интерактивном режиме. из сложностей - чисто функциональный подход. Решил. . .	Автоматическое создание документа при проведении другого документа Maks 29.03.2026 Реализация из решения ниже выполнена на нетиповых документах, разработанных в конфигурации КА2. Есть нетиповой документ "ЗаявкаНаРемонтСпецтехники" и нетиповой документ "ПланированиеСпецтехники". В. . .	Настройка движения справочника по регистру сведений Maks 29.03.2026 Решение ниже реализовано на примере нетипового справочника "ТарифыМобильнойСвязи" разработанного в конфигурации КА2, с целью учета корпоративной мобильной связи в коммерческом предприятии. . . .
Автозаполнение реквизита при выборе элемента справочника 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. В качестве источника данных. . .

@epitxx 0 / 0 / 0 Регистрация: 18.01.2018 Сообщений: 10
	11.02.2018, 17:38 [ТС]
	Тогда все прекрасно, большое спасибо за ответ 0