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

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

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

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

C++

#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++

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++

template <typename T> class List{
    struct node{
       T value;
       node *next;
       node *prev;
    };
    struct iterator {
        node *nodeptr;
    };
    node *head;
    node *tail;
};

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

C++

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++

        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++

T &iterator::operator*()
{
    return nodeptr->value;
}

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

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

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

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

C++

    private:
        typedef struct node{
            node *next;
            node *prev;
            bool empty;
            universum value;
        };
        node *root;
        node *last;

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

C++

        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++

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

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

C++

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 лет

ЖДИ!

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

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