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

@ajaxlord · Регистрация: 19.01.2019

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

Не работают методы поиска и удаления, сам алгоритм написан правильно, не могу разобраться где ошибка в коде. Помогите.

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
#include <iostream>
using namespace std;
 
// Связанный список
 
class List
{
public:
    List();
    ~List();
 
    void push_back(int data);
    bool remove(int data);
    void show_nodes();
    int find(int data);
    int GetSize() {return Size;}
 
private:
    class Node
    {
    public:
        Node *pNext;
        int data;
 
        Node(int data, Node *pNext = nullptr) // default parameter
        {
            this->data = data;
            this->pNext = pNext;
        }
    };
 
    int Size;
    Node *head;
    Node *tail;
};
 
List::List()
{
    Size = 0;
    head = nullptr;
    tail = nullptr;
}
 
List::~List()
{
 
}
 
void List::push_back(int data)
{
    if (this->head == nullptr)
        this->head = new Node(data);
    else
    {
        this->tail->pNext = new Node(data);
        // Node<X> *current  = this->head;
        // while (current->pNext != nullptr)
        // {
        //     current = current->pNext;
        // }
        
        // current->pNext = new Node<X>(data); //last elem
    }
 
    this->tail = new Node(data);
    cout << this->tail->data << endl;
    Size++;
}
 
void List::show_nodes()
{
    Node *current = this->tail;
 
    //while (current != nullptr)
    //{
        cout << current->data << endl;
        //current = current->pNext;
    //}
}
 
int List::find(int data)
{
    Node *current = this->head;
 
    while (current->pNext != nullptr)
    {
        if (current->data == data)
            return current->data;
 
        current = current->pNext;
    }
 
    return 0;
}
 
bool List::remove(int data)
{
    Node *current = this->head;
    Node *prev = current;
 
    while (current->pNext != nullptr)
    {
        if (current->data == data)
        {
            if (current == prev)
                this->head = this->head->pNext;
            else
                prev->pNext = current->pNext;
            delete current;
            Size--;
            return true;
        }
        else
        {
            current = current->pNext;
            prev = current;
        }
    }
 
    return false;
}
 
int main()
{
    List list1;
    
    for(int i = 0; i < 5; i++)
    {
        list1.push_back(i);
    }
 
    cout << "==============================" << endl;
 
    cout << list1.find(2) << endl;
    // list1.remove(4);
    // list1.show_nodes();
    // cout << list1.GetSize();
 
    return 0;
}

@L0M · 19.01.2019, 19:44

Сообщение от ajaxlord

сам алгоритм написан правильно

Это смотря что называть алгоритмом

и что считать как "правильно"

Сообщение от ajaxlord

Не работают методы поиска и удаления

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

В общем посмотрите. Я там поправил.

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
#include <iostream>
using namespace std;
 
// Связанный список
 
class List {
public:
    List();
    ~List();
 
    void push_back(int data);
    bool remove(int data);
    void show_nodes() const;
    bool find(int data) const;
    int GetSize()  const { return Size; }
 
private:
    class Node {
    public:
        Node *pNext;
        int data;
 
        Node(int data, Node *pNext = nullptr) // default parameter
        {
            this->data = data;
            this->pNext = pNext;
        }
    };
 
    int Size;
    Node *head;
    Node *tail;
 
    bool _find(int data, List::Node *&prev, List::Node *&target) const;
};
 
List::List() {
    Size = 0;
    head = nullptr;
    tail = nullptr;
}
 
List::~List() {
    Node *cur;
    while (head != nullptr) {
        cur = head;
        head = head->pNext;
        delete cur;
    }
}
 
void List::push_back(int data) {
    Node *newnode = new Node(data);
    if (this->head == nullptr)
        this->tail = this->head = newnode;
    else {
        this->tail->pNext = newnode;
        this->tail = newnode;
    }
 
    cout << this->tail->data << endl;
    Size++;
}
 
void List::show_nodes() const {
    Node *current = this->head;
 
    cout << "--- head ---\n";
    while (current != nullptr) {
        cout << current->data << endl;
        current = current->pNext;
    }
    cout << "--- tail ---\n";
}
 
bool List::_find(int data, List::Node *&prev, List::Node *&target) const {
    prev = nullptr;
    target = this->head;
 
    while (target != nullptr) {
        if (target->data == data) {
            return true;
        }
        prev = target;
        target = target->pNext;
    }
    return false;
}
 
bool List::find(int data) const {
    Node *p, *t;
    return _find(data, p, t);
}
 
bool List::remove(int data) {
    Node *prev, *target;
    if (head == nullptr || !_find(data, prev, target))
        return false;
 
    if (tail == target)
        tail = prev;
    if (prev == nullptr)
        head = head->pNext;
    else
        prev->pNext = target->pNext;
    delete target;
    --Size;
}
 
int main() {
    List list1, list2;
 
    for (int i = 0; i < 5; i++) {
        list1.push_back(i);
        list2.push_back(i + 1);
    }
 
    cout << "==============================" << endl;
 
    cout << "List size: " << list1.GetSize() << endl;
    list1.show_nodes();
    cout << "Find data: " << (list1.find(2) ? "ok" : "fail") << endl;
 
    list1.remove(1);
    list1.show_nodes();
 
    list1.remove(4);
    list1.show_nodes();
 
    list1.remove(0);
    list1.show_nodes();
 
    list1.remove(2);
    list1.show_nodes();
 
    list1.remove(3);
    list1.show_nodes();
    cout << "List size: " << list1.GetSize() << endl;
 
    return 0;
}

В методе List::find(int) возвращать то же значение, которое было передано в параметре - не имеет смысла. Ещё меньше смысла возвращать значение 0 как флаг того, что элемент не найден (т.е. существующий элемент со значением 0 никогда не будет найден). Поэтому тип возвращаемого значения поменял на bool.

Добавил код в деструктор.

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

Новые блоги и статьи Все статьи Все блоги /
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .

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

Решение