Среда разработки завершает работу

@triatri3 · Регистрация: 16.11.2016

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

Программа должна создавать дерево поиска, и считать кол-во листьев. Но вместо этого она завершает работу, что не так?

C++

#include <iostream>
using namespace std;
struct Tree
{
    int a;
    Tree* right;
    Tree* left;
};
Tree* root;
Tree* cur;
Tree* prev;
int k=0;
int Pods(Tree* root);
Tree *AddTree(Tree *root, float a);
int main()
{
    Tree *AddTree(Tree *root, float a);
    cout<<Pods(root)<<" list'ev"<<endl;
    return 0;
}
 
int Pods(Tree* root)
{
    if ((root->left==NULL) && (root->right==NULL))
    {
        return 1;
    }
    else
    {
        return Pods(root->left)+Pods(root->right);
    }
};
Tree *AddTree(Tree *root, float a)
{
    if (!root)
    {
        root=new Tree;
        root->a=a;
        root->left=0;
        root->right=0;
        return 0;
    }
    else
    {
        Tree *cur=root;
        Tree* prev;
        while (!cur)
        {
            if (cur->a==a)
            {
                return root;
            }
            else
            if (cur->a==a)
            {
                prev=cur;
                cur=cur->right;
            }
            else
            {
                prev=cur;
                cur=cur->left;
            }
        }
        Tree *Add=new Tree;
        Add->a=a;
        Add->right=0;
        Add->left=0;
        if (prev->a<a)
        {
            prev->right=Add;
        }
        else
        {
            prev->left=Add;
            return root;
        }
    }
}

Добавлено через 4 минуты
Всё, разобрался, программа считывает один (первый) элемент, и после считывает кол-во листьев в дереве. Как реализовать создание дерева побольше?

C++

#include<iostream>
struct Tree
{
    int data;
    Tree *left, *right, *parent;
};
Tree *root=0;
Tree *AddTree (Tree *root, int a);
int Pods(Tree* root);
using namespace std;
int main()
{
    int a;
    cin>>a;
    root=AddTree(root,a);
    cout<<Pods(root)<<" list'ev"<<endl;
    return 0;
}
Tree *AddTree (Tree *root, int a)
{
    if (!root)
    {
        root=new Tree;
        root->data=a;
        root->left=NULL;
        root->right=NULL;
        return root;
    }
    else
    {
        Tree *cur=root;
        Tree *prev=0;
        while (cur)
        {
            if(cur->data==a)
            return root;
            else if (cur->data<a)
            {
                prev=cur;
                cur=cur->right;
            }
            else
            {
                prev=cur;
                cur=cur->left;
            }
        }
        Tree *Add=new Tree;
        Add->data=a;
        Add->left=NULL;
        Add->right=NULL;
        if (prev->data<a)
        {
            prev->right=Add;
        }
        else
        {
            prev->left=Add;
        }
        return root;
    }
};
int Pods(Tree* root)
{
    if ((root->left==NULL) && (root->right==NULL))
    {
        return 1;
    }
    else
    {
        return Pods(root->left)+Pods(root->right);
    }
};

MrGluck · 02.05.2017, 17:23

У вас в функцию AddTree передаётся копия указателя root, то есть выделение памяти идёт для локальной копии root (а локальные переменные с теми же названиями перекрывают глобальные). При выходе из функции AddTree, локальная копия root уничтожается, память утекает, оригинальный root содержит NULL, следующей инструкцией вы пытаетесь этот NULL разыменовать и, как итог, падаете под весёлое посвистывание и улюлюкание компилятора.

@triatri3 · 02.05.2017, 18:14 **[ТС]**

MrGluck, слоооожно,как исправить? изменить локальную переменную на глобальную?

MrGluck · 02.05.2017, 18:17

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

MrGluck, слоооожно,как исправить? изменить локальную переменную на глобальную?

Передавать указатель по ссылке или по указателю, как это происходит и с обычными переменными.
От глобального указателя имеет смысл отказаться в любом случае.

@triatri3 · 03.05.2017, 11:26 **[ТС]**

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

У вас в функцию AddTree передаётся копия указателя root, то есть выделение памяти идёт для локальной копии root (а локальные переменные с теми же названиями перекрывают глобальные). При выходе из функции AddTree, локальная копия root уничтожается, память утекает, оригинальный root содержит NULL, следующей инструкцией вы пытаетесь этот NULL разыменовать и, как итог, падаете под весёлое посвистывание и улюлюкание компилятора.

Понял о чём, спасибо огромное, многое прояснило. Для 1 элемента(корня) всё отлично работает, но не подскажите ли как реализовать создание полноценного дерева? Возможно ли через for (int i=0; i<кол-ва элементов; i++)? Не будет ли при таком случае создание лишь одной ветки? И если это не верно, то как реализовать верно?

Добавлено через 31 минуту
Понял в чём проблема. Суть в том, что если есть элемент, который иметт только 1 ссылку из 2-х(например имеет потомка только справа(root->left==NULL) &&(root->right)) то программа прост озавершает работу, т.к. я не придусмотрел это, попытался решить это, вот код. Но при Корректировках среда разработки всё равно завершает работу. В чём ошибка?

C++

#include<iostream>
struct Tree
{
    int data;
    Tree *left, *right;
};
Tree *AddTree (Tree *koren, int a);
int Pods(Tree* root);
using namespace std;
int main()
{
    Tree *root=0;
    Tree *cur;
    Tree *prev;
    int a,number;
    cout<<"Enter number elements"<<endl;
    cin>>number;
    cin>>a;
    root=AddTree(root,a);
    for (int i=0; i<number-1;i++)
    {
        cin>>a;
        prev=cur;
        cur=AddTree(root,a);
    }
    cout<<Pods(root)<<" list'ev"<<endl;
    return 0;
}
Tree *AddTree (Tree *koren, int a)
{
    if (!koren)
    {
        koren=new Tree;
        koren->data=a;
        koren->left=NULL;
        koren->right=NULL;
        return koren;
    }
    else
    {
        Tree *cur=koren;
        Tree *prev=0;
        while (cur)
        {
            if(cur->data==a)
            return koren;
            else if (cur->data<a)
            {
                prev=cur;
                cur=cur->right;
            }
            else
            {
                prev=cur;
                cur=cur->left;
            }
        }
        Tree *Add=new Tree;
        Add->data=a;
        Add->left=NULL;
        Add->right=NULL;
        if (prev->data<a)
        {
            prev->right=Add;
        }
        else
        {
            prev->left=Add;
        }
        return koren;
    }
};
int Pods(Tree* koren)
{
    if ((koren->left==NULL) && (koren->right==NULL))
    {
        return 1;
    }
    else if (koren=NULL)
    {
        return 0;
    }
    else
    {
        if ((koren->left) && (koren->right))
        {
        return Pods(koren->left)+Pods(koren->right);
        }
        else if ((koren->left) && (koren->right==NULL))
        {
            return Pods(koren->left);
        }
        else if ((koren->left==NULL) && (koren->right))
        {
            return Pods(koren->right);
        }
    }
};

Добавлено через 9 минут

C++

#include<iostream>
struct Tree
{
    int data;
    Tree *left, *right;
};
Tree *AddTree (Tree *koren, int a);
int Pods(Tree* root);
using namespace std;
int main()
{
    Tree *root=0;
    Tree *cur;
    Tree *prev;
    int a,number;
    cout<<"Enter number elements"<<endl;
    cin>>number;
    cin>>a;
    root=AddTree(root,a);
    for (int i=0; i<number-1;i++)
    {
        cin>>a;
        prev=cur;
        cur=AddTree(root,a);
    }
    cout<<Pods(root)<<" list'ev"<<endl;
    return 0;
}
Tree *AddTree (Tree *koren, int a)
{
    if (!koren)
    {
        koren=new Tree;
        koren->data=a;
        koren->left=NULL;
        koren->right=NULL;
        return koren;
    }
    else
    {
        Tree *cur=koren;
        Tree *prev=0;
        while (cur)
        {
            if(cur->data==a)
            return koren;
            else if (cur->data<a)
            {
                prev=cur;
                cur=cur->right;
            }
            else
            {
                prev=cur;
                cur=cur->left;
            }
        }
        Tree *Add=new Tree;
        Add->data=a;
        Add->left=NULL;
        Add->right=NULL;
        if (prev->data<a)
        {
            prev->right=Add;
        }
        else
        {
            prev->left=Add;
        }
        return koren;
    }
};
int Pods(Tree* koren)
{
    if ((koren->left==NULL) && (koren->right==NULL))
    {
        return 1;
    }
    else if ((koren->left==NULL) && (koren->right!=0))
    {
        return Pods(koren->right);
    }
    else if ((koren->left!=0) && (koren->right==NULL))
    {
        return Pods(koren->left);
    }
    else
    {
        return Pods(koren->left)+Pods(koren->right);
    }
};

MrGluck Форумчанин 8216 / 5046 / 1437 Регистрация: 29.11.2010 Сообщений: 13,453
	02.05.2017, 17:23	2
	Сообщение было отмечено triatri3 как решение Решение У вас в функцию AddTree передаётся копия указателя root, то есть выделение памяти идёт для локальной копии root (а локальные переменные с теми же названиями перекрывают глобальные). При выходе из функции AddTree, локальная копия root уничтожается, память утекает, оригинальный root содержит NULL, следующей инструкцией вы пытаетесь этот NULL разыменовать и, как итог, падаете под весёлое посвистывание и улюлюкание компилятора. 1

@triatri3 11 / 12 / 8 Регистрация: 16.11.2016 Сообщений: 892
	02.05.2017, 18:14 [ТС]	3
	MrGluck, слоооожно,как исправить? изменить локальную переменную на глобальную? 0

MrGluck Форумчанин 8216 / 5046 / 1437 Регистрация: 29.11.2010 Сообщений: 13,453
	02.05.2017, 18:17	4
	Сообщение от triatri3 MrGluck, слоооожно,как исправить? изменить локальную переменную на глобальную? Передавать указатель по ссылке или по указателю, как это происходит и с обычными переменными. От глобального указателя имеет смысл отказаться в любом случае. 1

	03.05.2017, 11:26

Среда разработки завершает работу

Решение