Итератор для своего контейнерного класса-списка

@Eliont · Регистрация: 10.12.2010

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

Здравствуйте.
Есть класс List, и мне надо помимо стандартных методов приделать к нему итератор.

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
#include <cstdlib>
#include <iostream>
#include <string>
using namespace std;
 
template <typename universum> class List{
    
    public:
        List(){
            root = new node;
            root->next = 0;
            root->prev = 0;
            root->empty = true;
            cur = root;
            };
            
        void append(universum value){
            node *current,*prev;
            current = root;
            prev = root;
            
            if (root->empty){
                root->value = value;
                root->empty = false;
                return;}
                
            while (current){
                prev = current;
                current = current->next;}
                      
            current = new node;
            current->next = 0;
            current->prev = prev;
            prev->next = current;
            current->value = value;
            current->empty = false;
            //cout << "C = " << current<< " "<< current->prev <<  endl;
        };
        
        bool empty(){
            if ((root->next==0) && (root->empty)) return true;
            else return false;
        }
        
        int size(){
            int count;
            node *current;
            current = root;
            while(current){
                count++;
                current = current->next;}
            return count;
        }
        
        universum *begin(){
            int s = size();
            
            //static universum m[s];
            static int *ptr;
            ptr = new int[s];
            node *current;
            current = root;
            // m = &current->value;
            for (int i=0;i<s;i++){
                ptr[i] = current->value;
                current = current->next;}
            return ptr;}
            
        universum *end(){
            node *current;
            current = root;
            while(current->next){
                current = current->next;}
            return &current->value;
        }
        
        void tst(){
            cout << root << " " << root->next << endl;
            };
 
    private:        
        typedef struct node{
            node *next;
            node *prev;
            bool empty;
            universum value;
        };
        node *cur;
        node *root;
 
};
 
 
 
int main()
{
    List <int> list;
    List <string> str;
    int r;
    
    cout << "List empty = "<< list.empty() <<endl;
    
    for (int i=0;i<10;i++) {
        r = rand()%50;
        cout << r << " ";
        list.append(r);}
    
    int *s,*e;
    s = list.begin();
    e = list.end();
    
    cout<< endl << "List empty = "<< list.empty()<< endl;
    cout<< endl << "List size = "<< list.size()<< endl;
  
    for (int i = 0;i<list.size()+1;i++){
        cout << (s+i) << " " << *(s+i) << endl;
    }
    cout<< endl<< endl;
    cout<< endl << s << " " << *s << endl;
    cout<< endl << e << " " << *e << endl;
    
    system("PAUSE");
    return EXIT_SUCCESS;
}

Но в таком виде как сейчас - работает, а если попробовать вызывать begin / end в форе (в смысле в объявлении) то вылетает. Также, кстати, совершенно непонятно поччему вылетает если вставить cout << endl; между первым фором и объяалением указателей.
Так же у меня есть код встраиваемого итератора,

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
class myiterator : public iterator<bidirectional_iterator_tag, universum>
{
  universum* p;
public:
  myiterator(universum* x) :p(x) {}
  myiterator(const myiterator& mit) : p(mit.p) {}
  myiterator& operator++() {++p;return *this;}
  myiterator& operator++(int) {static myiterator tmp(*this); operator++(); return tmp;}
  myiterator& operator--() {--p;return *this;}
  myiterator& operator--(int) {static myiterator tmp(*this); operator--(); return tmp;}
  bool operator==(const myiterator& rhs) {return p==rhs.p;}
  bool operator!=(const myiterator& rhs) {return p!=rhs.p;}
  const universum& operator*() {return *p;}
  const universum& operator$() {return *p;}
};

Но я не понимаю как его прицепить к списку который на указателях а не в непрерывном куске памяти.

Не по теме:

Помогите, пожалуйста. Мой язык - Питон, а Си я, откровенно говоря, ненавижу. Но сейчас припёрло.

@Aye Aye · 10.12.2010, 08:25

ну типа если есть двунаправленный список:

C++
1
2
3
4
5
6
7
8
9
10
11
12
template <typename T> class List{
    struct node{
       T value;
       node *next;
       node *prev;
    };
    struct iterator {
        node *nodeptr;
    };
    node *head;
    node *tail;
};

то двунаправленный итератор реализовать можно так:

C++
1
2
3
4
5
6
7
8
9
10
11
12
class iterator{
    iterator &operator++(int i)
    {
        nodeptr = nodeptr->next;
        return *this;
    }
    iterator &operator--(int i)
    {
        nodeptr = nodeptr->prev;
        return *this;
    }
};

ну и все остальное.

@Eliont · 10.12.2010, 08:46 **[ТС]**

Спасибо.
Этот класс-итератор надо встроить в класс-список и объявлять так же как обычный итератор?

C++
1
2
3
        for (List<int>:iterator it=s.begin(); it!=s.end(); it++) {
            cout << *it << " ";
        }

Методы, создающие итераторы:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
        myiterator begin(){
            static iterator b(head); 
            return b;
        }
        
        myiterator end(){
            node *current;
            current = root;
            while(current->next){
                                      current = current->next;}
            staticyiterator e(current); 
            return e;
        }

@Aye Aye · 10.12.2010, 09:46

абсолютно верно, только в методе iterator end() надо возвращать поле tail, потому что это ссылка на последний элемент списка. Конструирование списка должно быть устроено так, чтобы при доступе к последнему элементу не приходилось проходить весь список. И не забыть унарный оператор *

C++
1
2
3
4
T &iterator::operator*()
{
    return nodeptr->value;
}

Добавлено через 32 минуты
что то я не заметил, а почему временная переменная e в посте №3 объявлена static? Если так то метод будет всегда одно и тоже возвращать (то что было при первом вызове метода).

@Eliont · 10.12.2010, 10:07 **[ТС]**

Ура! Работает.

Если ещё кому понадобится (или я что неправильно сделал, хотя работает):
До public:

C++
1
2
3
4
5
6
7
8
9
    private:
        typedef struct node{
            node *next;
            node *prev;
            bool empty;
            universum value;
        };
        node *root;
        node *last;

Сразу после конструктора:

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
        class scanner{
            node *nodeptr;
            public:
                scanner(node* ptr):nodeptr(ptr) {}
                scanner &operator++(int i)
                {   
                    nodeptr = nodeptr->next;
                    return *this;
                }
                scanner &operator--(int i)
                {
                    nodeptr = nodeptr->prev;
                    return *this;
                }
                universum &operator*()
                {
                    return nodeptr->value;
                }
                bool operator==(const scanner& ptr) 
                {   
                    return nodeptr==ptr.nodeptr;
                }
                bool operator!=(const scanner& ptr) 
                {   
                    return nodeptr!=ptr.nodeptr;
                }
        };
        
        scanner begin(){
            scanner b(root); 
            return b;
        }
        
        scanner end(){
            scanner e(last); 
            return e;
        }

last = current->next; в методе, добавляющем узлы.

C++
1
2
3
    for (List<int>::scanner it=list.begin(); it!=list.end(); it++) {
                cout << *it << " ";
        }

Но один вопрос остался:

C++
1
scanner(node* ptr):nodeptr(ptr) {}

А что это за странный конструктор? В смысле зачем одинарное двоеточие и почему он пустой?

Статики убрал. Работает.
Но когда я встраивал готовый итератор, тот что в <iterator> , компилятор выдавал что возвращается адрес локальной переменной.

@ForEveR · 10.12.2010, 13:04

Eliont, Список инициализации конструктора. Работает быстрее + лучше делать именно так.

Советую в самом списке написать

C++
1
typedef /*typename*/ scanner iterator

@Eliont · 10.12.2010, 15:09 **[ТС]**

Тут ещё вопрос возник...
А чтобы сделать простой и константный итераторы надо повторить класс или можно как-то в одном?

@ForEveR · 10.12.2010, 17:34

Eliont, По сути можно сделать так.

C++
1
typedef const scanner const_iterator

вроде бы.
А вот по концепции лучше сделать еще один.

@STArSka · 26.09.2017, 21:19

Вы написали такой цикл:

C++
1
2
3
4
 
for (List<int>::scanner it=list.begin(); it!=list.end(); it++) {
                cout << *it << " ";
        }

Прошу заметить, что при таком подходе последний элемент списка не выведется, поскольку ваша реализация метода end(), такова:

C++
1
2
3
4
scanner end(){
            scanner e(last); 
            return e;
        }

Он вернет последний элемент списка, и когда итератор в цикле будет равен ему(посл. элементу) цикл завершиться, так и не выведя последний элемент.

Для решения ситуации можно поступить так(если nullptr у Вас "конечный" указатель в списке):

C++
1
scanner end(){return nullptr;}

Или завести какую то константу "конец" и ссылаться на нее вместо nullptr.

Поправьте меня, если я не прав.
В том же std::vector метод end() возвращает указатель на некий специальный "закрывающий" элемент(формально) но никак не на последний.

Не по теме:

Тема старая, но ошибка есть ошибка

@no swear · 26.09.2017, 22:05

У тебя есть ошибка но я покажу её тебе только через 7 лет

ЖДИ!

Новые блоги и статьи Все статьи Все блоги /
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. Пошагово создадим проект для загрузки изображения. . .

@Eliont 2 / 2 / 0 Регистрация: 10.12.2010 Сообщений: 29
	10.12.2010, 15:09 [ТС]
	Тут ещё вопрос возник... А чтобы сделать простой и константный итераторы надо повторить класс или можно как-то в одном? 0

@no swear 192 / 166 / 82 Регистрация: 01.07.2016 Сообщений: 943
	26.09.2017, 22:05
	У тебя есть ошибка но я покажу её тебе только через 7 лет ЖДИ! 0