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

@riggs · Регистрация: 28.11.2011

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

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

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

@HedgehogLu · 24.09.2013, 12:44

ну во первых при удалении нет корректировки удаления первого не единственного элемента. т.е. как только ты удаляешь верхушку со списком будет полная амба.
вот исправленный кусок кода

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 · 24.09.2013, 18:49 **[ТС]**

Спасибо огромное!

Новые блоги и статьи Все статьи Все блоги /
Знаешь почему 90% людей редко бывают счастливыми? kumehtar 14.04.2026 Потому что они ждут. Ждут выходных, ждут отпуска, ждут удачного момента. . . а удачный момент так и не приходит.	Фиксация колонок в отчете СКД Maks 14.04.2026 Фиксация колонок в СКД отчета типа Таблица. Задача: зафиксировать три левых колонки в отчете. Процедура ПриКомпоновкеРезультата(ДокументРезультат, ДанныеРасшифровки, СтандартнаяОбработка) / / . . .	Настройки VS Code Loafer 13.04.2026 { "cmake. configureOnOpen": false, "diffEditor. ignoreTrimWhitespace": true, "editor. guides. bracketPairs": "active", "extensions. ignoreRecommendations": true, . . .	Оптимизация кода на разграничение прав доступа к элементам формы Maks 13.04.2026 Алгоритм из решения ниже реализован на нетиповом документе, разработанного в конфигурации КА2. Задачи, как таковой, поставлено не было, проделанное ниже исключительно моя инициатива. Было так:. . .
Контроль заполнения и очистка дат в зависимости от значения перечислений Maks 12.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеПерсонала", разработанного в конфигурации КА2. Задача: реализовать контроль корректности заполнения дат назначения. . .	Архитектура слоя интернета для сервера-слоя. Hrethgir 11.04.2026 В продолжение https:/ / www. cyberforum. ru/ blogs/ 223907/ 10860. html Знаешь что я подумал? Раз мы все источники пишем в голове ветки, то ничего не мешает добавить в голову такой источник, который сам. . .	Подстановка значения реквизита справочника в табличную часть документа Maks 10.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеПерсонала", разработанного в конфигурации КА2. Задача №1: при указании работ (справочник РаботыПоРемонтуСпецтехники),. . .	Очистка реквизитов документа при копировании Maks 09.04.2026 Алгоритм из решения ниже применим как для типовых, так и для нетиповых документов на самых различных конфигурациях. Задача: при копировании документа очищать определенные реквизиты и табличную. . .

@riggs 0 / 0 / 0 Регистрация: 28.11.2011 Сообщений: 7
	24.09.2013, 18:49 [ТС]
	Спасибо огромное! 0