0 / 0 / 0
Регистрация: 28.11.2011
Сообщений: 7
1

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

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

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

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;
        }
    }
 
}
0
Programming
Эксперт
94731 / 64177 / 26122
Регистрация: 12.04.2006
Сообщений: 116,782
24.09.2013, 11:48
Ответы с готовыми решениями:

Удаление элемента из двусвязного списка
Надо удалить элемент из двусвязного списка, если выполняется условие (2013-(current-&gt;god)&lt;3). Я...

Удаление элемента из двусвязного списка
Доброго всем времени суток. Нужна помощь. Есть двусвязный список. Функции передаем какое то...

Удаление элемента из двусвязного списка
Собственно, в коде все рабочее, кроме удаления. После ввода элемента выдает ошибку 0xC0000005....

Удаление элемента из двусвязного списка по значению
Есть такой код, который позволяет добавлять элементы в список и выводить, а мне нужно доделать еще...

2
147 / 82 / 10
Регистрация: 04.09.2013
Сообщений: 261
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 минут
более того, если я не ошибаюсь то из за натыканых бреаков (что не есть гуд) фактического освобождения памяти не происходит, т.к. при любом исходе бреак выкидывает из цикла до команды делит.
И более того у вас двунаправленный список. А при удалении вы не корректируете ссылку на предыдущий элемент, т.о. при любом удалении по сути получается каша а не список
1
0 / 0 / 0
Регистрация: 28.11.2011
Сообщений: 7
24.09.2013, 18:49  [ТС] 3
Спасибо огромное!
0
24.09.2013, 18:49
IT_Exp
Эксперт
87844 / 49110 / 22898
Регистрация: 17.06.2006
Сообщений: 92,604
24.09.2013, 18:49
Помогаю со студенческими работами здесь

Удаление первого и последнего элемента двусвязного списка с++
Удаление первого и последнего элемента двусвязного списка с++ namespace vlados { ...

"Сортировка двусвязного списка путем исключения элемента с минимальным значением и включения его в начало нового списка
Здравствуйте! Возникла проблема с программой. Тема: &quot;Сортировка двусвязного списка путем исключения...

Удаление из двусвязного циклического списка
Начал реализовывать структуру данных - Фибоначчиевы кучи. Столкнулся с проблемой при написании...

Удаление структуры из двусвязного списка
Доброго времени суток! Пытаюсь создать функцию удаления из списка структуры, не выполняется цикл...


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

Или воспользуйтесь поиском по форуму:
3
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2024, CyberForum.ru