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

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
Рейтинг: Рейтинг темы: голосов - 54, средняя оценка - 4.94
Battle_Hamster
1 / 1 / 0
Регистрация: 27.12.2013
Сообщений: 91
#1

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

20.02.2014, 21:14. Просмотров 6539. Ответов 7
Метки нет (Все метки)

Приветствую!
Помогите найти ошибку в коде?
Точнее ошибку знаю, строка : 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;
}
0
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
20.02.2014, 21:14
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Дерево. Утечка памяти, как обнаружить? (C++):

Утечка памяти. Как ее побороть? - C++
void Player::add() { Ship *bombers; bombers = new Bomber; for(int i=0; i&lt;num_bombers; i++) { Bomber temp(7, 40, 1, 1);...

Есть ли утечка памяти в проекте? И как можно это перепроверять? - C++
#include &quot;stdafx.h&quot; #include &lt;iostream&gt; using namespace std; class MyArray { int *ptrarray; //массив ...

Утечка памяти. new/delete. Всё как доктор прописал, но не работает - C++
Всем привет! Начну сразу с кода: #include &quot;stdafx.h&quot; #include &lt;conio.h&gt; #include &lt;iostream&gt; using namespace std; class...

Утечка памяти - C++
#include &quot;stdafx.h&quot; #include &lt;iostream&gt; #include &quot;ArrayOperation.h&quot; using namespace std; int main() { int...

Утечка памяти - C++
Либо я себе мозг запудрила, либо помогите мне :) есть у меня вектор vector&lt;char*&gt; names_variable; дополняю его таким образом ...

Утечка памяти - C++
Взял листинг кода из книги Стивен Прата Язык программирования С++. Меня терзают смутные сомнения нет ли здесь утечки памяти, указатель ...

Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
DrOffset
7152 / 4293 / 968
Регистрация: 30.01.2014
Сообщений: 7,093
20.02.2014, 21:30 #2
Особо не разбирался, но вот то, что сразу понятно:
curr->tab_dzieci в том месте кода, где падает - равно NULL
0
Battle_Hamster
1 / 1 / 0
Регистрация: 27.12.2013
Сообщений: 91
20.02.2014, 21:41  [ТС] #3
изначально эта таблица имеет значение NULL, а в том месте где падает, я хочу приписать этой ячейке таблицы указатель на только что созданный узел дерева.
И оно падает. ибо этой таблицы вообще не видит.
Сейчас перепроверю ещё раз, но раньше была такая проверка if (curr->tab_dzieci), то он дальше не шёл. т.е. не пытался приписать ячейке какое-либо значение.
0
Battle_Hamster
1 / 1 / 0
Регистрация: 27.12.2013
Сообщений: 91
20.02.2014, 21:50  [ТС] #4
Выше глупость была насчёт if (curr->tab_ddzieci) извиняюсь.

вот. что вещает отладчик в момент падения:
0
Миниатюры
Дерево. Утечка памяти, как обнаружить?  
DrOffset
7152 / 4293 / 968
Регистрация: 30.01.2014
Сообщений: 7,093
20.02.2014, 22:08 #5
Цитата Сообщение от Battle_Hamster Посмотреть сообщение
вот. что вещает отладчик в момент падения:
Так я как раз об этом и сказал. tab_dzieci - NULL.
0
Battle_Hamster
1 / 1 / 0
Регистрация: 27.12.2013
Сообщений: 91
20.02.2014, 22:33  [ТС] #6
ну так я в этом месте хочу прицепить указатель на объект. чем NULL мешает?
0
DrOffset
7152 / 4293 / 968
Регистрация: 30.01.2014
Сообщений: 7,093
20.02.2014, 22:42 #7
Цитата Сообщение от Battle_Hamster Посмотреть сообщение
ну так я в этом месте хочу прицепить указатель на объект. чем NULL мешает?
Тем, что указатель на массив в который ты хочешь записать - NULL. Чтобы записать что-то по какому-то адресу, адрес должен указывать на объект. В данном случае его (объекта, массива) нет. Проверка там поэтому и стояла раньше.
1
Battle_Hamster
1 / 1 / 0
Регистрация: 27.12.2013
Сообщений: 91
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;
    }
 
 
 
}

Но дебаггер всё равно выплёвывает утечку памяти
0
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
21.02.2014, 00:51
Привет! Вот еще темы с ответами:

утечка памяти - C++
доброго времени суток Уважаемые форумчане, помогите пожалуйста разобраться с утечками памяти есть массив с геометрическими фигурами. ...

Утечка памяти - C++
Доброго времени суток! Столкнулся с проблемой утечки памяти! Будь у меня маленькая программка, выловил бы легко, но в 2000 строк кода все...

Утечка памяти?! - C++
Джесс Либерти и Дэвид Хорват &quot;Освой самостоятельно С++ за 24 часа&quot;, вырезка из листинга 15.4 (стр 233): Строка 26 - Утечка памяти?...

Утечка памяти - C++
Не могу понять как избежать утечки памяти в своей программе... привожу кусок в одном из мест где на мой взгляд она происходит... Array...


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

Или воспользуйтесь поиском по форуму:
Yandex
Объявления
21.02.2014, 00:51
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru