Форум программистов, компьютерный форум CyberForum.ru

Дерево. Утечка памяти, как обнаружить? - C++

Восстановить пароль Регистрация
 
Рейтинг: Рейтинг темы: голосов - 54, средняя оценка - 4.94
Battle_Hamster
 Аватар для Battle_Hamster
1 / 1 / 0
Регистрация: 27.12.2013
Сообщений: 90
20.02.2014, 21:14     Дерево. Утечка памяти, как обнаружить? #1
Приветствую!
Помогите найти ошибку в коде?
Точнее ошибку знаю, строка : element->ojciec->tab_dzieci[element->ojciec->terazn_liczba_synow]= element;
Только не знаю как это исправить? Объект не находит таблицу детей отца этого объекта. Соответственно я не могу приписать ему указатель на ребёнка.
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
#include <iostream>
#include <queue>
 
using namespace std;
 
struct Pracownik
{
    string imie;
    int liczba_synow;
    int terazn_liczba_synow;
    Pracownik *ojciec;
    Pracownik **tab_dzieci;
    Pracownik()
    {
 
        ojciec=NULL;
        liczba_synow=0;
        terazn_liczba_synow=0;
        //tab_dzieci=NULL;
    }
};
 
queue <Pracownik*> kolejka;
 
class Drzewo
{
 
private:
    Pracownik * korzen;
    Pracownik *curr;
 
 
public:
    Drzewo();
    ~Drzewo();
    void dodaj_wezel(int liczba_synow, string imie);
    void postorder(Pracownik* element);
    Pracownik *wierzcholek();
 
};
 
Drzewo::Drzewo()
{
    this->korzen=new Pracownik;
    this->korzen->ojciec=NULL;
    this->korzen=NULL;
    this->curr=NULL;
 
}
 
Drzewo::~Drzewo()
{
 
}
 
void Drzewo::dodaj_wezel(int liczba_synow, string imie)
{
    if (korzen==NULL)  //
    {
 
        Pracownik * element = new Pracownik;
        element->liczba_synow = liczba_synow; //przepisujemy dane do wezla
        element->imie=imie;
        element->tab_dzieci = new Pracownik*[element->liczba_synow];
        for (int i=0; i<liczba_synow;i++)
        {
            element->tab_dzieci[i]=NULL;
            cout<<"tworze tblice dzieci"<<endl;
        }
        element->tab_dzieci=NULL;
        element->ojciec=NULL;
        element->terazn_liczba_synow=0;
        kolejka.push(element);
        this->korzen=element;
        curr=korzen;
 
    }// if
    else if (curr->terazn_liczba_synow<kolejka.front()->liczba_synow)
    {
        Pracownik * element  = new Pracownik;
        element->liczba_synow=liczba_synow;
        element->imie=imie;
        element->tab_dzieci = new Pracownik *[element->liczba_synow];
        cout<<"1"<<endl;
        for (int i=0; i<element->liczba_synow;i++)
        {
            element->tab_dzieci[i]=NULL;
        }
        cout<<"2"<<endl;
        element->ojciec=curr;
        cout<<"3"<<endl;
        element->ojciec->tab_dzieci[element->ojciec->terazn_liczba_synow]= element;
        cout<<"4"<<endl;
        curr->terazn_liczba_synow++;
 
        kolejka.push(element);
        //curr=element;
 
    }
    else
    {
        kolejka.pop();
 
        Pracownik * element  = new Pracownik;
        element->liczba_synow=liczba_synow;
        element->imie=imie;
        element->tab_dzieci = new Pracownik*[element->liczba_synow];
        cout<<"5"<<endl;
        for (int i=0; i<element->liczba_synow;i++)
        {
            element->tab_dzieci[i]=NULL;
        }
        cout<<"6"<<endl;
        //element->ojciec=curr;
        cout<<"7"<<endl;
 
        cout<<"8"<<endl;
        curr->tab_dzieci[curr->terazn_liczba_synow]= element;
        curr->terazn_liczba_synow++;
        cout<<"9"<<endl;
        kolejka.push(element);
        curr=element;
    }
 
 
 
}
 
void Drzewo::postorder (Pracownik * element)
{
    int i;
    for(i=0;i<element->liczba_synow;i++)
       { if (element->tab_dzieci==NULL)
            break;
        postorder(element->tab_dzieci[i]);
}
            cout<<element->imie<<endl;
}
 
Pracownik* Drzewo::wierzcholek()
{
    return korzen;
}
 
 
int main()
{
 
 
    int liczba_pracownikow;
    cin>>liczba_pracownikow;
 
 
    string imie;
    int liczba_synow;
 
 
    Drzewo pracownicy; // utworzono drzewo
 
    for (int i=0; i<liczba_pracownikow;i++)
    {
        cin>>liczba_synow;
        getline(cin,imie);
        getline(cin,imie);
 
        Pracownik *pomocniczy = new Pracownik;
        pomocniczy->imie=imie;
        pomocniczy->liczba_synow=liczba_synow;
 
        pracownicy.dodaj_wezel( liczba_synow,  imie);
        // kolejka.push(pomocniczy);
 
    }
    pracownicy.postorder(pracownicy.wierzcholek());
 
 
 
 
    return 0;
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
20.02.2014, 21:14     Дерево. Утечка памяти, как обнаружить?
Посмотрите здесь:

Утечка памяти C++
Утечка памяти?! C++
C++ Утечка памяти
Утечка памяти. Как ее побороть? C++
C++ Утечка памяти в
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
DrOffset
6419 / 3793 / 877
Регистрация: 30.01.2014
Сообщений: 6,581
20.02.2014, 21:30     Дерево. Утечка памяти, как обнаружить? #2
Особо не разбирался, но вот то, что сразу понятно:
curr->tab_dzieci в том месте кода, где падает - равно NULL
Battle_Hamster
 Аватар для Battle_Hamster
1 / 1 / 0
Регистрация: 27.12.2013
Сообщений: 90
20.02.2014, 21:41  [ТС]     Дерево. Утечка памяти, как обнаружить? #3
изначально эта таблица имеет значение NULL, а в том месте где падает, я хочу приписать этой ячейке таблицы указатель на только что созданный узел дерева.
И оно падает. ибо этой таблицы вообще не видит.
Сейчас перепроверю ещё раз, но раньше была такая проверка if (curr->tab_dzieci), то он дальше не шёл. т.е. не пытался приписать ячейке какое-либо значение.
Battle_Hamster
 Аватар для Battle_Hamster
1 / 1 / 0
Регистрация: 27.12.2013
Сообщений: 90
20.02.2014, 21:50  [ТС]     Дерево. Утечка памяти, как обнаружить? #4
Выше глупость была насчёт if (curr->tab_ddzieci) извиняюсь.

вот. что вещает отладчик в момент падения:
Миниатюры
Дерево. Утечка памяти, как обнаружить?  
DrOffset
6419 / 3793 / 877
Регистрация: 30.01.2014
Сообщений: 6,581
20.02.2014, 22:08     Дерево. Утечка памяти, как обнаружить? #5
Цитата Сообщение от Battle_Hamster Посмотреть сообщение
вот. что вещает отладчик в момент падения:
Так я как раз об этом и сказал. tab_dzieci - NULL.
Battle_Hamster
 Аватар для Battle_Hamster
1 / 1 / 0
Регистрация: 27.12.2013
Сообщений: 90
20.02.2014, 22:33  [ТС]     Дерево. Утечка памяти, как обнаружить? #6
ну так я в этом месте хочу прицепить указатель на объект. чем NULL мешает?
DrOffset
6419 / 3793 / 877
Регистрация: 30.01.2014
Сообщений: 6,581
20.02.2014, 22:42     Дерево. Утечка памяти, как обнаружить? #7
Цитата Сообщение от Battle_Hamster Посмотреть сообщение
ну так я в этом месте хочу прицепить указатель на объект. чем NULL мешает?
Тем, что указатель на массив в который ты хочешь записать - NULL. Чтобы записать что-то по какому-то адресу, адрес должен указывать на объект. В данном случае его (объекта, массива) нет. Проверка там поэтому и стояла раньше.
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
21.02.2014, 00:51     Дерево. Утечка памяти, как обнаружить?
Еще ссылки по теме:

Утечка памяти. new/delete. Всё как доктор прописал, но не работает C++
C++ утечка памяти
Утечка памяти C++

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

Или воспользуйтесь поиском по форуму:
Battle_Hamster
 Аватар для Battle_Hamster
1 / 1 / 0
Регистрация: 27.12.2013
Сообщений: 90
21.02.2014, 00:51  [ТС]     Дерево. Утечка памяти, как обнаружить? #8
Собственно первый косяк найден.
Функция добавления узла в дерево должна выглядеть так (удалена только одна строчка):

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
 void Drzewo::dodaj_wezel(int liczba_synow, string imie)
{
    if (korzen==NULL)  //
    {
 
 
        Pracownik * element = new Pracownik;
        element->liczba_synow = liczba_synow; //przepisujemy dane do wezla
        element->imie=imie;
        element->tab_dzieci = new Pracownik*[element->liczba_synow];
        for (int i=0; i<liczba_synow;i++)
        {
            element->tab_dzieci[i]=NULL;
            cout<<"tworze tblice dzieci"<<endl;
        }
 
        element->ojciec=NULL;
        element->terazn_liczba_synow=0;
        kolejka.push(element);
        this->korzen=element;
        this->curr=korzen;
 
    }// if
    else if (curr->terazn_liczba_synow<curr->liczba_synow)
    {
        ;
        Pracownik * element  = new Pracownik;
        element->liczba_synow=liczba_synow;
        element->imie=imie;
        element->tab_dzieci = new Pracownik *[element->liczba_synow];
        for (int i=0; i<element->liczba_synow;i++)
        {
            element->tab_dzieci[i]=NULL;
        }
 
        element->ojciec=this->curr;
      //  kolejka.push(element);
        element->ojciec->tab_dzieci[element->ojciec->terazn_liczba_synow]= element;
        element->ojciec->terazn_liczba_synow++;
 
 
    }
    else
    {
       // kolejka.pop();
 
        Pracownik * element  = new Pracownik;
        element->liczba_synow=liczba_synow;
        element->imie=imie;
        element->tab_dzieci = new Pracownik*[element->liczba_synow];
 
        for (int i=0; i<element->liczba_synow;i++)
        {
            element->tab_dzieci[i]=NULL;
        }
        element->ojciec=curr;
       // kolejka.push(element);
        curr->tab_dzieci[curr->terazn_liczba_synow]= element;
        curr->terazn_liczba_synow++;
        curr=element;
    }
 
 
 
}

Но дебаггер всё равно выплёвывает утечку памяти
Yandex
Объявления
21.02.2014, 00:51     Дерево. Утечка памяти, как обнаружить?
Ответ Создать тему
Опции темы

Текущее время: 17:01. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru