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

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

Войти
Регистрация
Восстановить пароль
 
riggs
0 / 0 / 0
Регистрация: 28.11.2011
Сообщений: 7
#1

Удаление элемента из двусвязного списка - C++

24.09.2013, 11:48. Просмотров 558. Ответов 2
Метки нет (Все метки)

Помогите пожалуйста найти где я ошибся. Элементы в список добавляются, вывод на экран тоже работает, но после удаления любого элемента: добавление в конпец списка не работает, попытка удалить элемент вызывает аварийное завершение? В чем напутал? Заранее спасибо.

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
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
#include<iostream>
#include<conio.h>
#include<string>
#include<cstdlib>
#include<new>
#include<iomanip>
 
using namespace std;
 
class list
{
private:
struct store
{
    string name;
    int amount;
    double price;
    store *next;
    store *prev;
    store()
    {
        next=NULL;
        prev=NULL;
    }
};
    store *head;
    store *tail;
public:
    
    list()
    {
        head=NULL;
        tail=NULL;
    };
    void menu();
    void add();
    void del();
    void print();
};
 
int main()
{
    setlocale(LC_ALL, "russian");
    cout << "Добро пожаловать в программу" << endl;
    cout << endl;
    cout << "Для создания списка нажмите (1)" << endl;
    char c;
    c=getch();
    if(c=='1')
    {
        class list myList;
        cout << "список создан\n" << endl;
        myList.menu();
    }
    else
    {
        cout << "Произошла ошибка выбора. Программа будет закрыта" << endl;
        getch();
    }
 
    return 0;
}
 
void list::add()
{
    setlocale(LC_ALL, "russian");
    store *temp=new store;
    temp->next=NULL;
    cout << "введите название товара:" << endl;
    cin.ignore();
    getline(cin, temp->name);
    cout << "введите количество товара:" << endl;
    cin >> temp->amount;
    cout << "введите цену товара:" << endl;
    cin >> temp->price;
    if(!head)
    {
        temp->prev=NULL;
        head=tail=temp;
        temp->next=NULL;
    }
    else
    {
        temp->prev=tail;
        tail->next=temp;
        tail=temp;
    }
    cout << endl;
}
 
void list::del()
{
    setlocale(LC_ALL, "russian");
    cout << "введите названия искомого товара:" << endl;
    cin.ignore();
    string example;
    getline(cin, example);
    bool result=0;
    store *tmp= new store;
    tmp=head;
    while(tmp!=NULL)
    {
        
        if(example.compare(tmp->name)==0)
        {
            if(head->next==NULL)// если единственный элемент 
            {
                head=NULL;
                result=1;
                break;  
            }
            else
            {
                tmp->prev->next=tmp->next;
                result=1;
                break;
            }
                delete tmp;
                break;
        }
        if(!result)
        tmp=tmp->next;
        else
            break;
    }
    if(result)
        cout << "Удаление прошло успешно" << endl;
    else
        cout << "Произошла ошибка. Искомый товар не найден" << endl;
    cout << endl;
}
 
void list::print()
{
    store *temporary=new store;
    temporary=head;
    cout << setw(15) << "товар:" << setw(10) << "кол-во:" << setw(10) << "цена:" << endl;
    while(temporary)
    {
        cout << setw(15) << temporary->name << setw(10) << temporary->amount << setw(10) << temporary->price << endl;
        cout << endl;
        temporary=temporary->next;
    }
    delete temporary;
    
}
void list::menu()
{
    while(true)
    {
        cout << "меню:\n" << endl;
        cout << "(1) просмотр списка" << endl;
        cout << "(2) добавление в конец нового элемента" << endl;
        cout << "(3) корректировка(удаление элемента)" << endl;
        cout << "(4) выход и закрытие программы" << endl;
        short int choise;
        cin >> choise;
        switch (choise)
        {
        case 1:
            print();
            break;
        case 2:
            add();
            break;
        case 3:
            del();
            break;
        case 4:
            exit(0);
            break;
        default:
            cout << " Произошла ошибка, проверьте правельность ввода" << endl;
            break;
        }
    }
 
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
24.09.2013, 11:48     Удаление элемента из двусвязного списка
Посмотрите здесь:

Удаление элемента из двусвязного списка C++
C++ Поменять местами два элемента двусвязного списка
Удаление элемента из двусвязного списка C++
C++ Удаление элемента со списка
Добавление и удаление элементов из двусвязного списка C++
C++ Удаление структуры из двусвязного списка
C++ Удаление элемента из двусвязного списка по значению
C++ Удаление из двусвязного циклического списка
C++ Поменять местами два элемента двусвязного списка и удалить из него указанный элемент
Удаление элемента из списка C++
C++ Написать функцию стирания элемента двусвязного списка
C++ Удаление элемента из списка

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

Или воспользуйтесь поиском по форуму:
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
HedgehogLu
 Аватар для HedgehogLu
147 / 68 / 1
Регистрация: 04.09.2013
Сообщений: 254
24.09.2013, 12:44     Удаление элемента из двусвязного списка #2
ну во первых при удалении нет корректировки удаления первого не единственного элемента. т.е. как только ты удаляешь верхушку со списком будет полная амба.
вот исправленный кусок кода
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
while(tmp!=NULL)
{
    if(example.compare(tmp->name)==0)
        {   
        if(tmp->next)
        {
            tmp->next->prev=tmp->prev;
        }
        else 
        {
            tail=tmp->prev;
            if (tail) tail->next=NULL;
        }   
        if (tmp->prev)
        {
            tmp->prev->next=tmp->next;
        }
        else
        {
            head=tmp->next;
            if (head) head->prev=NULL;
        }
    
    delete(tmp);
    tmp=NULL;
        result=1;
    }
        else
    {
        tmp=tmp->next;
    }
 
}
по идее так

Добавлено через 8 минут
более того, если я не ошибаюсь то из за натыканых бреаков (что не есть гуд) фактического освобождения памяти не происходит, т.к. при любом исходе бреак выкидывает из цикла до команды делит.
И более того у вас двунаправленный список. А при удалении вы не корректируете ссылку на предыдущий элемент, т.о. при любом удалении по сути получается каша а не список
riggs
0 / 0 / 0
Регистрация: 28.11.2011
Сообщений: 7
24.09.2013, 18:49  [ТС]     Удаление элемента из двусвязного списка #3
Спасибо огромное!
Yandex
Объявления
24.09.2013, 18:49     Удаление элемента из двусвязного списка
Ответ Создать тему
Опции темы

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