3

我的要求与使用迭代器隐藏内部容器并在 stackoverflow上实现对基本容器[ 1 ] 的通用操作的问题相同。我有一个通用的纯虚拟基容器类,它需要提供一个应该是 STL 投诉的迭代器,以便我可以将它们与 cpp 算法一起使用#include <algorithm>。我的实现只使用一个类而不是 [ 1 ] 解决方案中的两个类。

基础纯虚类

class BaseItr
{
  public:
    class iterator : public std::iterator<std::input_iterator_tag, int>
    {
      public:
        iterator() : _in(NULL) {}
        inline iterator(const iterator& org) : _in(org._in) {}
        inline iterator& operator=(const iterator& other) { _in = other._in; return *this; }
        virtual inline int operator * () { return _in->operator*(); }
        virtual inline iterator& operator++() { (*_in)++; return *this; }
        virtual inline iterator& operator++(int unused) { (*_in)++; return *this; }
        virtual inline bool operator==(const iterator& other) 
        {
          return *(*_in) == *(*(other._in));
        }
        virtual inline bool operator!=(const iterator& other)
        {
          return *(*_in) != *(*(other._in));
        }
        // would use shared pointer insted of this
        //~iterator() { if(_in) { delete _in; } }
        static inline iterator New(iterator *in) { return iterator(in); }
      private:
        iterator(iterator *in) : _in(in) {}
        iterator *_in;
    };

    virtual iterator begin() = 0;
    virtual iterator end() = 0;
};

执行

class Itr : public BaseItr
{
  private:
    class iterator : public BaseItr::iterator
    {
      public:
        iterator(int val) : _val(val), BaseItr::iterator() {}
        int operator * () { return _val; }
        inline iterator& operator++() { ++_val; return *this; }
        inline iterator& operator++(int unused) { _val++; return *this; }
      private:
        int _val;
    };
    BaseItr::iterator _begin;
    BaseItr::iterator _end;
  public:
    inline Itr(int start, int end)
    {
      _begin = BaseItr::iterator::New(new iterator(start));
      _end = BaseItr::iterator::New(new iterator(end));
    }

    BaseItr::iterator begin() { return _begin; }
    BaseItr::iterator end() { return _end; }
};

我的实现工作是需要的,我想知道这个实现是否有任何缺点,请帮助我决定我的设计以使用适当的实现。我已经在 github:gist https://gist.github.com/3847688中添加了我的完整工作示例代码

参考:

4

1 回答 1

5

最明显的问题:您的迭代器没有值语义。

如果他们愿意,STL 算法可以自由地复制迭代器。例如假设:

template <typename It>
It find(It b, It e, typename std::iterator_traits<It>::const_reference t) {
    for (; b != e; ++b) {
        if (*b == t) { return b; }
    }
    return e;
}

问题是,如果您使用 调用此算法BaseItr&,则结果是类型的BaseItr,因此您将暴露于对象切片,这是未定义的行为

为了给迭代器提供值语义,您需要围绕抽象实现创建一个包装器类,并让包装器通过虚拟clone方法正确管理副本。如果您的迭代器以虚拟方法结束,那么您做错了。

于 2012-10-07T11:04:07.240 回答