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

@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);
    }
};

Блоги программистов
Как перейти с Options API на Composition API в Vue.js BasicMan 06.01.2025 Почему переход на Composition API актуален В мире современной веб-разработки фреймворк Vue. js продолжает эволюционировать, предлагая разработчикам все более совершенные инструменты для создания. . .	Архитектура современных процессоров inter-admin 06.01.2025 Процессор (центральный процессор, ЦП) является основным вычислительным устройством компьютера, которое выполняет обработку данных и управляет работой всех остальных компонентов системы. Архитектура. . .	История создания реляционной модели баз данных, правила Кодда Programming 06.01.2025 Предпосылки создания реляционной модели В конце 1960-х годов компьютерная индустрия столкнулась с серьезными проблемами в области управления данными. Существовавшие на тот момент модели данных -. . .	Полезные поделки на Arduino, которые можно сделать самому raxper 06.01.2025 Arduino как платформа для творчества Arduino представляет собой удивительную платформу для технического творчества, которая открывает безграничные возможности для создания уникальных проектов. Эта. . .	Подборка решений задач на Python IT_Exp 06.01.2025 Целью данной подборки является предоставление возможности ознакомиться с различными задачами и их решениями на Python, что может быть полезно как для начинающих, так и для опытных программистов. . . .	С чего начать программировать микроконтроллеры raxper 06.01.2025 Введение в мир микроконтроллеров Микроконтроллеры стали неотъемлемой частью современного мира, окружая нас повсюду: от простых бытовых приборов до сложных промышленных систем. Эти маленькие. . .
Из чего собрать игровой компьютер inter-admin 06.01.2025 Сборка игрового компьютера требует особого внимания к выбору комплектующих и их совместимости. Правильно собранный игровой ПК не только обеспечивает комфортный геймплей в современных играх, но и. . .	Обновление сайта www.historian.by Reglage 05.01.2025 Обещал подвести итоги 2024 года для сайта. Однако начну с того, что изменилось за неделю. Добавил краткий урок по последовательности действий при анализе вредоносных файлов и значительно улучшил урок. . .	Как использовать GraphQL в C# с HotChocolate Programming 05.01.2025 GraphQL — это современный подход к разработке API, который позволяет клиентам запрашивать только те данные, которые им необходимы. Это делает взаимодействие с API более гибким и эффективным по. . .	Модель полного двоичного сумматора с помощью логических операций (python) AlexSky-coder 04.01.2025 def binSum(x:list, y:list): s=^y] p=x and y for i in range(1,len(x)): s. append((x^y)^p) p=(x and y)or(p and (x or y)) return s x=list() y=list()	Это мы не проходили, это нам не задавали...(асихронный счётчик с управляющим сигналом зад Hrethgir 04.01.2025 Асинхронный счётчик на сумматорах (шестиразрядный по числу диодов на плате, но наверное разрядов будет больше - восемь или шестнадцать, а диоды на старшие), так как триггеры прошли тестирование и. . .	Руководство по созданию бота для Телеграм на Python IT_Exp 04.01.2025 Боты для Телеграм представляют собой автоматизированные программы, которые выполняют различные задачи, взаимодействуя с пользователями через интерфейс мессенджера. В данной статье мы рассмотрим,. . .

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

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

Решение