problem z własnym iteratorem

Witam,

Mam spory problem z napisaniem własnego iteratora w ćwiczeniu.
O ile rozumiem ogólną ideę, metody, tak własna implementacja sprawia mi spory problem - a nie mogę znaleźć nigdzie dobrze wytłumaczonego tego tematu.

#ifndef VECTOR_HPP
#define VECTOR_HPP

/**
 * Generic vector (array) with fixed size
 */
template<class T, int n>
class Vector {
    T v[n];                
    int siz;                
public:
    Vector() { siz = 0; }

    void push_back(T e) {
        if (siz >= n) throw "Vector is full";
        v[siz++] = e;
    }

    T& operator[](int i) {
        if (i < 0 || i >= siz) throw "Index error";
        return v[i];
    }
    class Iterator;
    Iterator begin(){return Iterator(*this);}
    Iterator end(){return Iterator(*this,siz);}
    class Iterator
    {
        private:
            T* act;
            T* last;
        public:
            Iterator() :act(0), last(0) {} 
            Iterator(Vector &st) :act(st.v), last(st.v+st.siz) {} /
            Iterator(Vector &st, int) :act(st.v+st.siz), last(st.v+st.siz) {} 
            Iterator& operator++()
            { // pre increment
                if (act != last) ++act;
                return act;
            }
            Iterator operator++(int)
            { // post increment
                Iterator tmp = *this;
                ++*this;
                return this;
            }

            bool operator!=(const Iterator &i)
            { 
                if(act==i)return true; //nie wiem jak to zrobić
                else return false;
            }

            T& operator*()
            { 
                if (act != 0 && act != last)
                return act;
            }

    };
};
#endif

 

Kod do sprawdzenia działania:

 #include <iostream>
#include "vector.hpp"
using namespace std;

int main() {

        Vector<int, 20> v1;             // int vector with a maximum size of 20.

    for (int i = 0; i < 10; i++)    // fill with elements
        v1.push_back(i);

    Vector<int, 20>::Iterator i1;	// create an iterator

    for (i1 = v1.begin(); i1 != v1.end(); i1++)    // print the elements
        cout << *i1 << endl;

    return 0;

Będę bardzo wdzięczny za jakiekolwiek uwagi - zwyczajnie nie wiem jak to ugryźć.

#include <iostream>
using namespace std;
 
template<class T, unsigned n> class Vector
  {
   T v[n];
   unsigned siz;                
   public:
   Vector():siz(0) {}
 
   void push_back(const T &e)
     {
      if(siz>=n) throw "Vector is full";
      v[siz++]=e;
     }
 
   const T &pop_back(const T &e)
     {
      if(siz<1) throw "Vector is full";
      return v[--siz];
     }
 
   T& operator[](unsigned i)
     {
      if(i>=siz) throw "Index error";
      return v[i];
     }
     
   class iterator
     {
      private:
      T *P;
      public:
      typedef input_iterator_tag iterator_category;
      typedef T value_type;
      typedef int difference_type;
      typedef T *pointer;
      typedef T &reference;
      iterator(T *P=0):P(P) {}
      bool operator==(const iterator &I)const { return P==I.P; }
      bool operator!=(const iterator &I)const { return P!=I.P; }
      bool operator<(const iterator &I)const { return P<I.P; }
      bool operator>(const iterator &I)const { return P>I.P; }
      bool operator<=(const iterator &I)const { return P<=I.P; }
      bool operator>=(const iterator &I)const { return P>=I.P; }
      int operator-(const iterator &I)const { return P-I.P; }
      iterator &operator++() { ++P; return *this; }
      iterator operator++(int) { iterator r(P); ++P; return r; }
      iterator &operator--() { --P; return *this; }
      iterator operator--(int) { iterator r(P); --P; return r; }
      T &operator*() { return *P; }
      T *operator->() { return P; }
     };
   iterator begin() { return iterator(v); }
   iterator end() { return iterator(v+siz); }
  };

int main()
  {
   Vector<int,20> v1;             // int vector with a maximum size of 20.
 
   for(int i=0;i<10;++i) v1.push_back(i);  // fill with elements
   Vector<int,20>::iterator i1;        // create an iterator
   for(i1=v1.begin();i1!=v1.end();++i1) cout<<*i1<<endl;   // print the elements
   cout<<endl;
   for(Vector<int,20>::iterator i=v1.begin();i<v1.end();++i) cout<<*i<<endl;   // print the elements

   cin.sync(); cin.get();
   return 0;
  }

Wielkie dzięki. Wszystko jasne. Nie musiałeś dopisywać dodatkowych metod - to już sam bym ogarnął :)

_13th_Dragon napisał(a)

class iterator
   ...
};

a nie wystarczy po prostu

typedef T* iterator;

? ;-)