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

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

Восстановить пароль Регистрация
 
riggs
0 / 0 / 0
Регистрация: 28.11.2011
Сообщений: 7
24.09.2013, 11:48     Удаление элемента из двусвязного списка #1
Помогите пожалуйста найти где я ошибся. Элементы в список добавляются, вывод на экран тоже работает, но после удаления любого элемента: добавление в конпец списка не работает, попытка удалить элемент вызывает аварийное завершение? В чем напутал? Заранее спасибо.

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++ Удаление из двусвязного циклического списка

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

Или воспользуйтесь поиском по форуму:
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
HedgehogLu
 Аватар для HedgehogLu
146 / 67 / 1
Регистрация: 04.09.2013
Сообщений: 250
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     Удаление элемента из двусвязного списка
Ответ Создать тему
Опции темы

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