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

@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.

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

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

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .
Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .

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

Решение