Функция удаления из списка

@mmd · Регистрация: 17.05.2012

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

помогите разобраться почему не работает функция удаления? плз

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
#include <iostream>
using namespace std;
 
class Node{
public:
    int data;
    Node * next;
    Node *prev;
 
    Node(int x){
        data = x;
        next = NULL;
        prev=NULL;
    }
    Node(int x, Node * y, Node *z){
        data = x;
        next = y;
        prev=z;
    }
};
 
 
class linkedList{
    Node *head;
public:
    linkedList(){
        head = NULL;
    }
    void addNode(int value){
        Node *p;
        if(head == NULL)
            head = new Node (value, NULL, NULL);
        else{
            p=head;
            while(p->next !=NULL)
                p=p->next;
            p->next = new Node (value, NULL, NULL);
        }
    }
    void print(){
        Node * p;
        p = head;
        while(p != NULL){
            cout << p->data << "\n";
            p = p->next;
        }
    }
 
    void search(int k)
    {
        Node* p = list_search(head,k);
 
        if (p) cout << endl << p->data << endl;
        else cout << endl << "no" << endl;
    }
 
    Node* list_search(Node *head,int k)
    {
        Node *p = head;
 
        while(p && p->data != k)
        {
            p = p->next;
        }
        return p;
    }
    void delete_list_elem(int k)
    {
        Node *i=list_search(head, k);
        if(i)
        {
            list_delete(head,i);
        }
    }
 
    void list_delete(Node *head, Node *k)
    {
        if(k->prev)
        {
            (k->prev)->next=k->next;
        }
        else
        {
            head=k->next;
        }
 
        if(k->next)
        {
            (k->next)->prev=k->prev;
        }
    }
};
 
 
int main(void){
    linkedList test;
    test.addNode(4);
    test.addNode(76);
    test.addNode(12);
    test.print();
    test.search(76);
    test.delete_list_elem(76);
    cout << "\n";
    test.print();
    return(0);
}

Croessmah · 13.01.2013, 17:31

Комментарий модератора

5.5 Запрещено размещать тему в нескольких разделах одновременно (кросспостинг), а также дублировать тему в одном разделе.
Правила форума

@3BEPb_TyTa · 13.01.2013, 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
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
#include <iostream>
using namespace std;
 
class Node{
public:
    int data;
    Node * next;
    Node *prev;
 
    Node(int x){
        data = x;
        next = NULL;
        prev=NULL;
    }
    Node(int x, Node * y, Node *z){
        data = x;
        next = y;
        prev=z;
    }
};
 
 
class linkedList{
    Node *head;
public:
    linkedList(){
        head = NULL;
    }
    void addNode(int value){
        Node *p;
        if(head == NULL)
            head = new Node (value, NULL, NULL);
        else{
            p=head;
            while(p->next !=NULL)
                p=p->next;
            p->next = new Node (value, NULL, p);
        }
    }
    void print(){
        Node * p;
        p = head;
        while(p != NULL){
            cout << p->data << "\n";
            p = p->next;
        }
    }
 
    void search(int k)
    {
        Node* p = list_search(head,k);
 
        if (p) cout << endl << p->data << endl;
        else cout << endl << "no" << endl;
    }
 
    Node* list_search(Node *head,int k)
    {
        Node *p = head;
 
        while(p && p->data != k)
        {
            p = p->next;
        }
        return p;
    }
    void delete_list_elem(int k)
    {
        Node *i=list_search(head, k);
        if(i)
        {
            list_delete(i);
        }
    }
 
    void list_delete(Node *k)
    {
        Node* tmp = k->prev;
        if(tmp)
        {
            if(k->next){
                tmp->next = k->next;
                k->next->prev = tmp;
            }
            else
                tmp->next = 0;
            delete k;
        }
        else
        {
            tmp = k->next;
            delete k;
            head = tmp;
            head->prev = 0;
        }
    }
};
 
 
int main(void){
    linkedList test;
    test.addNode(4);
    test.addNode(76);
    test.addNode(12);
    test.print();
    test.search(76);
    test.delete_list_elem(12);
    cout << "\n";
    test.print();
    return(0);
}

@mmd · 13.01.2013, 18:37 **[ТС]**

Сообщение от 3BEPb_TyTa

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
#include <iostream>
using namespace std;
class Node{
public:
 int data;
 Node * next;
 Node *prev;
Node(int x){
 data = x;
 next = NULL;
 prev=NULL;
 }
 Node(int x, Node * y, Node *z){
 data = x;
 next = y;
 prev=z;
 }
};
class linkedList{
 Node *head;
public:
 linkedList(){
 head = NULL;
 }
 void addNode(int value){
 Node *p;
 if(head == NULL)
 head = new Node (value, NULL, NULL);
 else{
 p=head;
 while(p->next !=NULL)
 p=p->next;
 p->next = new Node (value, NULL, p);
 }
 }
 void print(){
 Node * p;
 p = head;
 while(p != NULL){
 cout << p->data << "\n";
 p = p->next;
 }
 }
void search(int k)
 {
 Node* p = list_search(head,k);
if (p) cout << endl << p->data << endl;
 else cout << endl << "no" << endl;
 }
Node* list_search(Node *head,int k)
 {
 Node *p = head;
while(p && p->data != k)
 {
 p = p->next;
 }
 return p;
 }
 void delete_list_elem(int k)
 {
 Node *i=list_search(head, k);
 if(i)
 {
 list_delete(i);
 }
 }
void list_delete(Node *k)
 {
 Node* tmp = k->prev;
 if(tmp)
 {
 if(k->next){
 tmp->next = k->next;
 k->next->prev = tmp;
 }
 else
 tmp->next = 0;
 delete k;
 }
 else
 {
 tmp = k->next;
 delete k;
 head = tmp;
 head->prev = 0;
 }
 }
};
int main(void){
 linkedList test;
 test.addNode(4);
 test.addNode(76);
 test.addNode(12);
 test.print();
 test.search(76);
 test.delete_list_elem(12);
 cout << "\n";
 test.print();
 return(0);
}

все равно странно работает
если передать 76 то удаляет и 4 и 76 , а если 12 то вообще зависает

@3BEPb_TyTa · 13.01.2013, 18:56

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
#include <iostream>
using namespace std;
 
class Node{
public:
    int data;
    Node * next;
    Node *prev;
 
    Node(int x){
        data = x;
        next = NULL;
        prev=NULL;
    }
    Node(int x, Node * y, Node *z){
        data = x;
        next = y;
        prev=z;
    }
};
 
 
class linkedList{
    Node *head;
public:
    linkedList(){
        head = NULL;
    }
    void addNode(int value){
        Node *p;
        if(head == NULL)
            head = new Node (value, NULL, NULL);
        else{
            p=head;
            while(p->next !=NULL)
                p=p->next;
            p->next = new Node (value, NULL, p);
        }
    }
    void print(){
        Node * p;
        p = head;
        while(p != NULL){
            cout << p->data << "\n";
            p = p->next;
        }
    }
 
    void search(int k)
    {
        Node* p = list_search(head,k);
 
        if (p) cout << endl << p->data << endl;
        else cout << endl << "no" << endl;
    }
 
    Node* list_search(Node *head,int k)
    {
        Node *p = head;
 
        while(p && p->data != k)
        {
            p = p->next;
        }
        return p;
    }
    void delete_list_elem(int k)
    {
        Node *i=list_search(head, k);
        if(i)
        {
            list_delete(i);
        }
    }
 
    void list_delete(Node *k)
    {
        Node* tmp = k->prev;
        if(tmp)
        {
            if(k->next){
                tmp->next = k->next;
                k->next->prev = tmp;
            }
            else
                tmp->next = 0;
            delete k;
        }
        else
        {
            tmp = k->next;
            delete k;
            head = tmp;
            if(head)
                head->prev = 0;
        }
    }
};
 
 
int main(void){
    linkedList test;
    test.addNode(4);
    test.addNode(76);
    test.addNode(12);
    test.print();
    test.search(76);
    test.delete_list_elem(76);
    test.delete_list_elem(12);
    test.delete_list_elem(4);
    cout << "\n";
    test.print();
    return(0);
}

чуть подправил. Убедитесь, что правильно всё скопировали, у меня всё работает.

@mmd · 13.01.2013, 19:05 **[ТС]**

Сообщение от 3BEPb_TyTa

чуть подправил. Убедитесь, что правильно всё скопировали, у меня всё работает.

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

@3BEPb_TyTa · 13.01.2013, 19:15

не знаю, как у вас так получается
Вот пример удаления второго элемента. После удаления выводится список - 1 и 3 элементы.
http://liveworkspace.org/code/13ZYuT$3

@mmd · 13.01.2013, 19:28 **[ТС]**

Сообщение от 3BEPb_TyTa

не знаю, как у вас так получается
Вот пример удаления второго элемента. После удаления выводится список - 1 и 3 элементы.
http://liveworkspace.org/code/13ZYuT$3

а ну все норм, теперь работает) я хз что такое
спасибо

@Igor3D · 13.01.2013, 19:29

Если поддерживаете prev (двусвязный список) то корректируйте его при вставке нода
Заведите член tail, незачем пробегать каждый раз в конец
Ну и вообще как-то рыхло, создание/удаление нодов здесь не к месту, т.к. часто надо перебрасывать из одного списка в другой. Я бы делал примерно так

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
template <class Node>
class CLinkedList{
    Node *mHead, *mTail;
public:
    CLinkedList( void );
    void Add2Head( Node * );
    void Add2Tail( Node * );
    void InsBefore( Node * node, Node * before );
    void Unlink( Node *);
    Node * LookupByValue( int val ):
    bool Lookup( const Node * ) const:
    size_t Count( void ) const;
};

Новые блоги и статьи Все статьи Все блоги /
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Символьное дифференцирование igorrr37 13.02.2026 / * Программа принимает математическое выражение в виде строки и выдаёт его производную в виде строки и вычисляет значение производной при заданном х Логарифм записывается как: (x-2)log(x^2+2) -. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.
«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .

@3BEPb_TyTa 18 / 18 / 4 Регистрация: 18.05.2010 Сообщений: 83
	13.01.2013, 19:15
	не знаю, как у вас так получается Вот пример удаления второго элемента. После удаления выводится список - 1 и 3 элементы. http://liveworkspace.org/code/13ZYuT$3 1