Ошибка с удалением элемента в односвязном списке

@Skier97 · Регистрация: 11.11.2016

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

Здравствуйте! Вроде написал функцию удаления элемента в указанной позиции, но почему то не срабатывает. Где ошибка?Или если есть вариант лучше, предложите свой. Заранее благодарен! Функция Delete - удаления!

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
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
#include "stdafx.h"
#include <conio.h>
#include <iostream>
 
using namespace std;
struct A
{
    int key;
};
 
struct List
{
    int Data;
    A elem;
    List* next;
 
};
 
/*void Print(List* b)
{
    List* print = b;
 
    while (print)
    {
        cout << print->elem.key << "->";
        print = print->next;
    }
    cout << "NULL\n";
}*/
 
void Print(List*begin)
{
    if (begin != NULL)
    {
        cout << begin->Data << "\t";
        Print(begin->next);
    }
    else cout<<"\n";
}
 
/*void Init(List** begin)
{
    *begin = new List;
 
    A elem[5] = { 1,20,35,44,55 };
 
    (*begin)->elem.key = 11;
    (*begin)->next = NULL;
 
    List* end = *begin;
 
    for (int i(0); i < 5; i++)
    {
        end->next = new List;
        end = end->next;
        end->elem = elem[i];
        end->next = NULL;
    }
}*/
 
/*void Insert(List**begin, A &a)
{
    List*ins = new List;
    ins->elem = a;
 
    if (*begin == NULL)
    {
        ins->next = NULL;
        *begin = ins;
        return;
    }
 
    List*first = *begin;
 
    if (first->elem.key > ins->elem.key)
    {
        ins->next = first;
        *begin = ins;
        return;
    }
 
    List*end = first->next;
 
    while (end)
    {
        if (first->elem.key < ins->elem.key && ins->elem.key <= end->elem.key)
        {
            first->next = ins;
            ins->next = end;
            return;
        }
 
        first = end;
        end = end->next;
    }
 
    first->next = ins;
    ins->next = NULL;
}*/
 
void Insert(List *begin, unsigned n, int val) 
{
    unsigned i = 0;
    List *tmp = NULL;
    //Находим нужный элемент. Если вышли за пределы списка, то выходим из цикла,
    //ошибка выбрасываться не будет, произойдёт вставка в конец
    while (i < (n-1) && begin->next) 
    {
        begin = begin->next;
        i++;
    }
    tmp = (List*)malloc(sizeof(List));
    tmp->Data = val;
    //Если это не последний элемент, то next перекидываем на следующий узел
    if (begin->next)
    {
        tmp->next = begin->next;
        //иначе на NULL
    }
    else tmp->next = NULL;
    begin->next = tmp;
}
 
 
void Search(List*begin, int val)
{
    bool f=false;
    int i=0;
    while (begin)
    {
        if (begin->Data == val)
        {
            f = true;
            cout << "Элемент найден,его позиция: "<<i<<endl;
            break;
        }
        else begin = begin->next;
        i++;
    }
    if (f == false) cout << "Элемент не найден";
}
 
void Delete(List**begin, A&a)
{
    if (*begin == NULL)
    {
        return;
    }
 
    List*first = *begin;
    if (first->elem.key == a.key)
    {
        *begin = first->next;
        delete first;
        return;
    }
 
    List*end = first->next;
 
    while (end)
    {
        if (end->elem.key == a.key)
        {
            first->next = end->next;
            delete end;
            return;
        }
        first = end;
        end = end->next;
    }
}
 
void Free(List**begin)
{
    if (*begin == 0)return;
    List*p = *begin;
    List*first;
 
    while (p)
    {
        first = p;
        p = p->next;
        delete first;
    }
    *begin = NULL;
}
 
void Check(List** begin)
{
    if (*begin == NULL)
    {
        cout << "Список пустой!";
    }
    else cout << "Список не пустой!";
}
 
void Init1(int n, List**begin)
{
    if (n > 0)
    {
        (*begin) = new List();
        cout << "Введите значение: ";
        cin >> (*begin)->Data;
        (*begin)->next = NULL;
        Init1(n - 1, &((*begin)->next));
    }
}
 
 
int main()
{
    int n,val,a,vall;
    setlocale(LC_ALL, "Russian"); 
 
    List* begin=NULL;
    A  b ;
    
    cout << "Введите размер n: ";
    cin >> n;
    //Init(&begin);
    Init1(n,&begin);
    Print(begin);
    cout << "Введите позицию, на которую хотите вставить новый элемент: ";
    cin >> a;
    cout << "Введите вставляемое число: ";
    cin >> val;
    Insert(begin, a,val);
    Print(begin);
    cout<<"Введите удаляемый элемент: ";
    cin >> b.key;
    Delete(&begin, b);
    Print(begin);
    cout << "Введите искомое число: ";
    cin >> vall;
    Search(begin, vall);
    //cout << val;
    Free(&begin);
    Print(begin);
    Check(&begin);
    _getch();
    return 0;
}

@Songpool · 13.12.2016, 17:02

Skier97, а ты не хочешь в delete посылать head и key, потом в цикле бежать пока не встретишь нужный элемент? вроде того

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
void Delete(List *head, int key)
{
    List *temp = head;
    List *el = NULL;
    if (head && head->key == key) {
        el = head;
        head = head->next;
    }
    while (temp->next) {
        if (temp->next->key == key) {
            el = temp->next; 
            temp->next = el->next;
            delete el;
            break;
        }
    }
}

@Skier97 · 13.12.2016, 17:21 **[ТС]**

Блин,странно,я похоже удалил это случайно,сейчас попробую. Вроде было

Добавлено через 4 минуты
У меня зависает,видимо бесконечный цикл там

@Songpool · 13.12.2016, 17:29

Skier97, согласен, мой косяк, в конце добавь

C++
1
2
3
4
while (temp->next) {
...
    temp = temp->next;
}

@Skier97 · 13.12.2016, 17:29 **[ТС]**

Я еще убрал структуру А и ее элемент вставил во вторую структуру, ну и соответственно в main поменял типы входных данных

@Skier97 · 13.12.2016, 17:31 **[ТС]**

Сейчас попробую

Добавлено через 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
void Delete(List *head, int key)
{
    List *temp = head;
    List *el = NULL;
    if (head && head->key == key) 
    {
        el = head;
        head = head->next;
    }
    while (temp->next) 
    {
        if (temp->next->key == key) 
        {
            el = temp->next;
            temp->next = el->next;
            delete el;
            break;
        }
        temp = temp->next;
 
    }
 
}

@Songpool · 13.12.2016, 17:34

Skier97, да, но не совсем, я забыл в if где head удалить el и сделать break

@Skier97 · 13.12.2016, 17:47 **[ТС]**

Так ничего не меняется почему то

Добавлено через 5 минут
Вот что вышло. Break там ставить нельзя,ведь там не цикл,а так ничего не меняется почему то

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
void Delete(List *head, int key)
{
    List *temp = head;
    List *el = NULL;
    if (head && head->key == key) 
    {
        el = head;
        head = head->next;
        delete el;
        
    }
    while (temp->next) 
    {
        if (temp->next->key == key) 
        {
            el = temp->next;
            temp->next = el->next;
            delete el;
            break;
        }
        temp = temp->next;
 
    }
 
}

Добавлено через 8 минут
или я может что то не понимаю?

@Songpool · 13.12.2016, 17:49

Skier97, там поставить return; , а не break

@Skier97 · 13.12.2016, 17:51 **[ТС]**

Так?
Я пробовал уже,потом о нем вспомнил,все равно не работает

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
void Delete(List *head, int key)
{
    List *temp = head;
    List *el = NULL;
    if (head && head->key == key) 
    {
        el = head;
        head = head->next;
        delete el;
        return;
    }
    while (temp->next) 
    {
        if (temp->next->key == key) 
        {
            el = temp->next;
            temp->next = el->next;
            delete el;
            break;
        }
        temp = temp->next;
 
    }
 
}

@Songpool · 13.12.2016, 17:59

Skier97, скинь код, который ты запускаешь

@Skier97 · 13.12.2016, 18:00 **[ТС]**

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
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
#include "stdafx.h"
#include <conio.h>
#include <iostream>
 
using namespace std;
/*struct A
{
    int key;
};*/
 
struct List
{
    int Data;
    int key;
    List* next;
 
};
 
/*void Print(List* b)
{
    List* print = b;
 
    while (print)
    {
        cout << print->elem.key << "->";
        print = print->next;
    }
    cout << "NULL\n";
}*/
 
void Print(List*begin)
{
    if (begin != NULL)
    {
        cout << begin->Data << "\t";
        Print(begin->next);
    }
    else cout<<"\n";
}
 
/*void Init(List** begin)
{
    *begin = new List;
 
    A elem[5] = { 1,20,35,44,55 };
 
    (*begin)->elem.key = 11;
    (*begin)->next = NULL;
 
    List* end = *begin;
 
    for (int i(0); i < 5; i++)
    {
        end->next = new List;
        end = end->next;
        end->elem = elem[i];
        end->next = NULL;
    }
}*/
 
/*void Insert(List**begin, A &a)
{
    List*ins = new List;
    ins->elem = a;
 
    if (*begin == NULL)
    {
        ins->next = NULL;
        *begin = ins;
        return;
    }
 
    List*first = *begin;
 
    if (first->elem.key > ins->elem.key)
    {
        ins->next = first;
        *begin = ins;
        return;
    }
 
    List*end = first->next;
 
    while (end)
    {
        if (first->elem.key < ins->elem.key && ins->elem.key <= end->elem.key)
        {
            first->next = ins;
            ins->next = end;
            return;
        }
 
        first = end;
        end = end->next;
    }
 
    first->next = ins;
    ins->next = NULL;
}*/
 
void Insert(List *begin, unsigned n, int val) 
{
    unsigned i = 0;
    List *tmp = NULL;
    //Находим нужный элемент. Если вышли за пределы списка, то выходим из цикла,
    //ошибка выбрасываться не будет, произойдёт вставка в конец
    while (i < (n-1) && begin->next) 
    {
        begin = begin->next;
        i++;
    }
    tmp = (List*)malloc(sizeof(List));
    tmp->Data = val;
    //Если это не последний элемент, то next перекидываем на следующий узел
    if (begin->next)
    {
        tmp->next = begin->next;
        //иначе на NULL
    }
    else tmp->next = NULL;
    begin->next = tmp;
}
 
 
void Search(List*begin, int val)
{
    bool f=false;
    int i=0;
    while (begin)
    {
        if (begin->Data == val)
        {
            f = true;
            cout << "Элемент найден,его позиция: "<<i<<endl;
            break;
        }
        else begin = begin->next;
        i++;
    }
    if (f == false) cout << "Элемент не найден";
}
 
/*void Delete(List**begin, A&a)
{
    if (*begin == NULL)
    {
        return;
    }
 
    List*first = *begin;
    if (first->elem.key == a.key)
    {
        *begin = first->next;
        delete first;
        return;
    }
 
    List*end = first->next;
 
    while (end)
    {
        if (end->elem.key == a.key)
        {
            first->next = end->next;
            delete end;
            return;
        }
        first = end;
        end = end->next;
    }
}*/
 
void Delete(List *head, int key)
{
    List *temp = head;
    List *el = NULL;
    if (head && head->key == key) 
    {
        el = head;
        head = head->next;
        delete el;
        return;
    }
    while (temp->next) 
    {
        if (temp->next->key == key) 
        {
            el = temp->next;
            temp->next = el->next;
            delete el;
            break;
        }
        temp = temp->next;
 
    }
 
}
 
void Free(List**begin)
{
    if (*begin == 0)return;
    List*p = *begin;
    List*first;
 
    while (p)
    {
        first = p;
        p = p->next;
        delete first;
    }
    *begin = NULL;
}
 
void Check(List** begin)
{
    if (*begin == NULL)
    {
        cout << "Список пустой!";
    }
    else cout << "Список не пустой!";
}
 
void Init1(int n, List**begin)
{
    if (n > 0)
    {
        (*begin) = new List();
        cout << "Введите значение: ";
        cin >> (*begin)->Data;
        (*begin)->next = NULL;
        Init1(n - 1, &((*begin)->next));
    }
}
 
 
int main()
{
    int n,val,a,vall,b;
    setlocale(LC_ALL, "Russian"); 
 
    List* begin=NULL;
    //List  b ;
    
    cout << "Введите размер n: ";
    cin >> n;
    //Init(&begin);
    Init1(n,&begin);
    Print(begin);
    cout << "Введите позицию, на которую хотите вставить новый элемент: ";
    cin >> a;
    cout << "Введите вставляемое число: ";
    cin >> val;
    Insert(begin, a,val);
    Print(begin);
    cout<<"Введите удаляемый элемент: ";
    cin >> b;
    Delete(begin, b);
    Print(begin);
    cout << "Введите искомое число: ";
    cin >> vall;
    Search(begin, vall);
    //cout << val;
    Free(&begin);
    Print(begin);
    Check(&begin);
    _getch();
    return 0;
}

@Songpool · 13.12.2016, 18:18

Skier97, конечно, он у тебя не работает, потому что у тебя key всегда 0, сравнивай тогда head->Data и temp->next->Data

Добавлено через 1 минуту
Skier97, и поправь косяки, на счет позиции, выводи не с нуля, а с позиция + 1, пользователь не знает, что у тебя с нуля начинается счет, и в конце у тебя выводиться строка, что список пуст, когда он не пуст

@Skier97 · 13.12.2016, 18:22 **[ТС]**

Насчет позиции и пустого списка я знаю,там были пробные варианты для других функций
Вот так надо чтоли?

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
void Delete(List *head, int key)
{
    List *temp = head;
    List *el = NULL;
    if (head->Data && head->next->Data == key) 
    {
        el = head;
        head = head->next;
        delete el;
        return;
    }
    while (temp->next) 
    {
        if (temp->next->key == key) 
        {
            el = temp->next;
            temp->next = el->next;
            delete el;
            break;
        }
        temp = temp->next;
 
    }
 
}

Просто так он выдает во время работы самой ошибку,выводить чушь какую то.

@Songpool · 13.12.2016, 18:24

Skier97, нет, вот так

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
void Delete(List *head, int data)
{
    List *temp = head;
    List *el = NULL;
    if (head && head->Data == data) 
    {
        el = head;
        head = head->next;
        delete el;
        return;
    }
    while (temp->next) 
    {
        if (temp->next->Data == data) 
        {
            el = temp->next;
            temp->next = el->next;
            delete el;
            break;
        }
        temp = temp->next;
 
    }
}

@Skier97 · 13.12.2016, 18:47 **[ТС]**

Спасибо большое!!!Все заработало

Добавлено через 14 минут
А источник про односвязные списки есть какой нибудь хороший?Ссылка или книга

@Songpool · 13.12.2016, 19:28

Skier97, не знаю на сколько хороший, но описано как можно сделать тут, а книги можно посмотреть Прата и Страуструпа

@Skier97 · 13.12.2016, 20:04 **[ТС]**

спасибо

Новые блоги и статьи Все статьи Все блоги /
Фото всей Земли с борта корабля Orion миссии Artemis II kumehtar 04.04.2026 Это первое подобное фото сделанное человеком за 50 лет. Снимок называют новым вариантом легендарной фотографии «The Blue Marble» 1972 года, сделанной с борта корабля «Аполлон-17». Новое фото. . .	Вывод диалогового окна перед закрытием, если документ не проведён Maks 04.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: реализовать программный контроль на предмет проведения документа. . .	Программный контроль заполнения реквизита табличной части документа Maks 02.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: реализовать контроль заполнения реквизита "ПричинаСписания". . .	wmic не является внутренней или внешней командой Maks 02.04.2026 Решение: DISM / Online / Add-Capability / CapabilityName:WMIC~~~~ Отсюда: https:/ / winitpro. ru/ index. php/ 2025/ 02/ 14/ komanda-wmic-ne-naydena/
Программная установка даты и запрет ее изменения Maks 02.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: при создании документов установить период списания автоматически. . .	Вывод данных в справочнике через динамический список Maks 01.04.2026 Реализация из решения ниже выполнена на примере нетипового справочника "Спецтехника" разработанного в конфигурации КА2. Задача: вывести данные из ТЧ нетипового документа. . .	Программное заполнения текстового поля в реквизите формы документа Maks 01.04.2026 Алгоритм из решения ниже реализован на нетиповом документе "ВыдачаОборудованияНаСпецтехнику" разработанного в конфигурации КА2, в дополнении к предыдущему решению. На форме документа создается. . .	К слову об оптимизации kumehtar 01.04.2026 Вспоминаю начало 2000-х, университет, когда я писал на Delphi. Тогда среди программистов на форумах активно обсуждали аккуратную работу с памятью: нужно было следить за переменными, вовремя. . .

@Skier97 1 / 1 / 0 Регистрация: 11.11.2016 Сообщений: 104
	13.12.2016, 17:21 [ТС]
	Блин,странно,я похоже удалил это случайно,сейчас попробую. Вроде было Добавлено через 4 минуты У меня зависает,видимо бесконечный цикл там 0

@Skier97 1 / 1 / 0 Регистрация: 11.11.2016 Сообщений: 104
	13.12.2016, 17:29 [ТС]
	Я еще убрал структуру А и ее элемент вставил во вторую структуру, ну и соответственно в main поменял типы входных данных 0

@Songpool 53 / 42 / 31 Регистрация: 24.11.2016 Сообщений: 140
	13.12.2016, 17:34
	Skier97, да, но не совсем, я забыл в if где head удалить el и сделать break 0

@Songpool 53 / 42 / 31 Регистрация: 24.11.2016 Сообщений: 140
	13.12.2016, 17:49
	Skier97, там поставить return; , а не break 0

@Songpool 53 / 42 / 31 Регистрация: 24.11.2016 Сообщений: 140
	13.12.2016, 17:59
	Skier97, скинь код, который ты запускаешь 0

@Songpool 53 / 42 / 31 Регистрация: 24.11.2016 Сообщений: 140
	13.12.2016, 18:18
	Skier97, конечно, он у тебя не работает, потому что у тебя key всегда 0, сравнивай тогда head->Data и temp->next->Data Добавлено через 1 минуту Skier97, и поправь косяки, на счет позиции, выводи не с нуля, а с позиция + 1, пользователь не знает, что у тебя с нуля начинается счет, и в конце у тебя выводиться строка, что список пуст, когда он не пуст 0

@Skier97 1 / 1 / 0 Регистрация: 11.11.2016 Сообщений: 104
	13.12.2016, 18:47 [ТС]
	Спасибо большое!!!Все заработало Добавлено через 14 минут А источник про односвязные списки есть какой нибудь хороший?Ссылка или книга 0

@Songpool 53 / 42 / 31 Регистрация: 24.11.2016 Сообщений: 140
	13.12.2016, 19:28
	Skier97, не знаю на сколько хороший, но описано как можно сделать тут, а книги можно посмотреть Прата и Страуструпа 0

@Skier97 1 / 1 / 0 Регистрация: 11.11.2016 Сообщений: 104
	13.12.2016, 20:04 [ТС]
	спасибо 0

Ошибка с удалением элемента в односвязном списке

Решение