Re: Problem with wring my own iterator

Thanks a lot Victor

Not sure what I was doing wrong but decided to copy the original code
from the example I had and I got it to work with a simple A class. I
believe what you mentioned about the constructor taking a
'map::iterator' as its argument was the missing elements. Anyway.
Thought I would post the working code for other people to use later
one. Thanks again for your kind help...

ps: to your question about 'what does the constructor do' when I use
_i(), nope I am not sure what it does exactly ;-( but I will try to
find out.

-c

class A
{
public:
    A() : _map() {}
    typedef std::map<std::string, int> AttributeMap;

    class Iterator;

    Iterator begin ();
    Iterator end ();
    Iterator find (const char name[]);

private:

    AttributeMap _map;
};

//----------
// Iterators
//----------

class A::Iterator
{
  public:

    Iterator ();
    Iterator (const A::AttributeMap::iterator &i);

    Iterator & operator ++ ();
    Iterator operator ++ (int);

    const std::string & name () const;
    int & attribute () const;

  private:

    //friend class A::ConstIterator;

    A::AttributeMap::iterator _i;
};

//-----------------
// Inline Functions
//-----------------

A::Iterator
A::begin ()
{
    return _map.begin();
}

A::Iterator
A::end ()
{
    return _map.end();
}

A::Iterator
A::find (const char name[])
{
    return _map.find (name);
}

inline
A::Iterator::Iterator (): _i()
{
    // empty
}

inline
A::Iterator::Iterator (const A::AttributeMap::iterator &i): _i (i)
{
    // empty
}

inline A::Iterator &
A::Iterator::operator ++ ()
{
    ++_i;
    return *this;
}

inline A::Iterator
A::Iterator::operator ++ (int)
{
    Iterator tmp = *this;
    ++_i;
    return tmp;
}

inline const std::string &
A::Iterator::name () const
{
    return _i->first;
}

inline int &
A::Iterator::attribute () const
{
    return _i->second;
}