Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.85/13: Рейтинг темы: голосов - 13, средняя оценка - 4.85
 Аватар для anapshy
531 / 272 / 220
Регистрация: 14.11.2016
Сообщений: 1,052

Последовательный список: функция-член pop_front() удаляет не все элементы

19.09.2017, 16:28. Показов 2577. Ответов 4
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
1) Проблема в том, что pop_front() удаляет не все элементы.
Возможно накосячил в методе pop_front() или push_front()
2) Не работает swap(List&)
+ ещё возможно проблема с pop_back()
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
 // List.h
#include <stdexcept>
class List
{
public:
    typedef double  value_type;
    typedef size_t  size;
private:
    struct Node
    {
        Node(const value_type& a) : item(a) {};
        Node(void) {};
        value_type item;
        Node *next;
        Node *prev;
    };
public:
    class iterator
    {
        friend class List;
        iterator(Node *el) : elem(el) {};
    public:
        // Конструкторы
        iterator(void) : elem(0) {};
        iterator(const iterator &it) : elem(it.elem) {};
        // Сравнение итераторов
        bool operator==(const iterator& it) const { return (elem == it.elem); }
        bool operator!=(const iterator& it) const { return !(*this == it); }
        // Перемещение итератор
        iterator& operator++(void)
        {
            if (elem != 0)
                elem = elem->next;
            return *this;
        }
        iterator& operator--(void)
        {
            if (elem != 0)
                elem = elem->prev;
            return *this;
        }
        value_type& operator*(void) const
        {
            if (elem != 0)
                return elem->item;
            else
                throw std::out_of_range("Nulliterator");
        }
    private:
        Node *elem;
    };
// Конструкторы / копирование / присваивание
    List(void) : Head(new Node()), Tail(Head), count(0)
    {
        Tail->next = Tail->prev = 0;
        head = iterator(Head);
        tail = iterator(Tail);
    }
    List(const value_type& a, size n = 1)
    {
        List tmp;
        for (size i = 0; i < n; ++i)
            tmp.push_front(a);
        *this = tmp;
    }
    List(iterator first, iterator last)
    {
        List tmp;
        for (iterator ip = first; ip != last; ++ip)
            tmp.push_front(*ip);
        *this = tmp;
    }
    List(const List& r)
    {
        List tmp(r.begin(), r.end());
        *this = tmp;
    }
    ~List(void)
    {
        Node *delete_Node = Head;
        for (Node *p = Head; p != Tail;)
        {
            p = p->next;
            delete delete_Node;
            --count;
            delete_Node = p;
        }
        delete delete_Node;
    }
    List& operator=(const List& t)
    {
        List tmp(t);
        swap(tmp);
        return *this;
    }
// Итераторы
    iterator begin(void) { return head; }
    iterator end(void) { return tail; }
    iterator begin(void) const { return head; }
    iterator end(void) const { return tail; }
// Размеры
    bool empty(void) const { return (Head == Tail); }
    size length(void) const { return count; }
// Доступ к элементам
    value_type& front(void) { return *begin(); }
    value_type& back(void) { iterator it = end(); --it; return *it; }
// Модификаторы контейнера
    void push_front(const value_type &value)
    {
        Node *new_Node = new Node(value);
        new_Node->prev = 0;
        new_Node->next = Head;
        Head->prev = new_Node;
        Head = new_Node;
        head = iterator(Head);
        ++count;
    }
    void pop_front(void)
    {
        Node *el = begin().elem;
        Node *nextel = el->next;
        nextel->prev = 0;
        delete el;
        Head = nextel;
        head = iterator(Head);
        --count;
    }
    void pop_back(void)
    {
        Node *el = (--end()).elem;
        Node *prevel = el->prev;
        Node *nextel = el->next;
        prevel->next = nextel;
        nextel->prev = prevel;
        delete el;
        --count;
    }
    void erase(iterator it)
    {
        if (it == begin())
            pop_front();
        else if (it == --end())
            pop_back();
        else
        {
            Node *el = it.elem;
            Node *prevel = it.elem->prev;
            Node *nextel = it.elem->next;
            prevel->next = nextel;
            nextel->prev = prevel;
            delete el;
        }
        --count;
    }
    void erase(iterator first, iterator last)
    {
        while (first != last)
            erase(first);
    }
    void swap(List &other)
    {
        if (this != &other)
        {
            std::swap(this->count, other.count);
            std::swap(this->Head, other.Head);
            std::swap(this->Tail, other.Tail);
            
            this->head = iterator(this->Head);
            this->tail = iterator(this->Tail);
            other.head = iterator(other.Head);
            other.tail = iterator(other.Tail);
        }
    }
private:
    long count;
    Node *Head;
    Node *Tail;
    iterator head, tail;
};
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
// main.cpp
#include "List.h"
#include <iostream>
 
int main()
{
    List a;
    a.push_front(1);
    a.push_front(2);
    a.push_front(3);
    a.push_front(4);
 
    for (List::iterator it = a.begin(); it != a.end(); ++it)
        std::cout << *it << ' ';
    std::cout << std::endl << "length: " << a.length() << std::endl << std::endl;
    
    for (size_t i(0); i < a.length(); ++i) // должно удалять 4 элемента, но удаляет только первые 2
        a.pop_front();
 
    for (List::iterator it = a.begin(); it != a.end(); ++it)
        std::cout << *it << ' ';
    std::cout << std::endl << "length: " << a.length() << std::endl << std::endl;
}
Добавлено через 1 час 51 минуту
Исправил pop_front()
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
void pop_front(void)
    {
        if (begin() != end())
        {
            Node *del = begin().elem;
            Node *nextel = del->next;
            nextel->prev = 0;
            delete del;
            Head = nextel;
            head = iterator(Head);
            --count;
        }
    }
0
Лучшие ответы (1)
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
19.09.2017, 16:28
Ответы с готовыми решениями:

Функция удаляет из списка все отрицательные элементы
функция удаляет из списка все отрицательные элементы

Двусвязный список. Программа подвисает при вызове pop_front()
Пытаюсь реализовать двусвязный список. Столкнулся с проблемой. В случае одного элемента в списке и вызрве ф-йии pop_front() программа...

Некорректно удаляет элементы(односвязный список)
#include &quot;stdafx.h&quot; #include &lt;iostream&gt; #include &lt;cstring&gt; #include &lt;conio.h&gt; using namespace std; struct WORD//СОЗДАЕМ...

4
Модератор
Эксперт CЭксперт С++
 Аватар для sourcerer
5288 / 2376 / 342
Регистрация: 20.02.2013
Сообщений: 5,773
Записей в блоге: 20
19.09.2017, 16:28
 Комментарий модератора 
anapshy, пожалуйста, прочитайте правила форума.
Особое внимание обратите на пункт 4.3 (порядок именования тем).

.
0
 Аватар для anapshy
531 / 272 / 220
Регистрация: 14.11.2016
Сообщений: 1,052
19.09.2017, 16:41  [ТС]
Поторопился, pop_front - опять глючит. Вопрос не решен.
0
284 / 232 / 114
Регистрация: 07.09.2016
Сообщений: 584
19.09.2017, 18:55
Лучший ответ Сообщение было отмечено anapshy как решение

Решение

C++
1
2
    for (size_t i(0); i < a.length(); ++i) // должно удалять 4 элемента, но удаляет только первые 2
        a.pop_front();
у вас тут уменьшается размер списка, да еще i увеличивается.
итого на третей итерации i == 2, length тоже равно 2. т.е. условие i < a.length() == false и поэтому тело цикла не выполняется. поэтому всего два вызова pop_front
1
 Аватар для anapshy
531 / 272 / 220
Регистрация: 14.11.2016
Сообщений: 1,052
19.09.2017, 22:03  [ТС]
DU3, хах... LOL и правда.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
19.09.2017, 22:03
Помогаю со студенческими работами здесь

функция которая удаляет элементы с одинаковым значением JavaScript
Нужна функция, которая принимает в качестве аргумента последовательность и возвращает список элементов без каких-либо элементов с...

Не удаляет все положительные элементы из вектора
Добрый день, форумчане! Читала на форуме, как правильно удалять элементы из вектора по индексу... Но почему-то у меня не получается....

Рекурсивная функция, не возвращающая значения, которая удаляет из заданной строки все точки
плиииз...(((

Рекурсивная функция, не возвращающая значения, которая удаляет из заданной строки все точки
вот код...есть ещё вариант с массивом, но он тоже не работает =( тем более препод сказал, что через массив делать не стоит...=( #include...

Вызов удаляет все элементы из объекта х типа ArrayList
A) x.Remove() B) x.Delete() C) x.Empty() D) х.С1еаг()


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
5
Ответ Создать тему
Новые блоги и статьи
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами
8Observer8 20.02.2026
Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .
Конвертировать закладки radiotray-ng в m3u-плейлист
damix 19.02.2026
Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .
Семь 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. На борту пять. . .
Камера 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. Пошагово создадим проект для загрузки изображения. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru