Как я могу использовать BOOST FOREACH с контейнером, поддерживающим только итератор const?

у меня есть этот контейнер:

class /*final*/ Row
{
public:
  typedef FieldIterator const_iterator;
  typedef FieldIterator iterator;
  FieldIterator begin() const;
  FieldIterator end() const;
  FieldIterator begin();
  FieldIterator end();
  ...
};

учитывая, что следующий код компилируется просто замечательно:

BOOST_FOREACH(Field field, row)
{
}

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

class /*final*/ Row
{
public:
  typedef FieldIterator const_iterator;
  FieldIterator begin() const;
  FieldIterator end() const;
  ...
};

но теперь тот же цикл foreach не компилируется:

1>o:cboost_1_48_0boostforeach.hpp(364): error C2039: 'type' : is not a member of 'boost::mpl::eval_if<C,F1,F2>'
1>          with
1>          [
1>              C=boost::mpl::false_,
1>              F1=boost::range_const_iterator<sqlserver::Row>,
1>              F2=boost::range_mutable_iterator<sqlserver::Row>
1>          ]
1>          c:devinternalplaymssqlceplaymssqlce.cpp(29) : see reference to class template instantiation 'boost::foreach_detail_::foreach_iterator<T,C>' being compiled
1>          with
1>          [
1>              T=sqlserver::Row,
1>              C=boost::mpl::false_
1>          ]
...

из сообщения об ошибке я понимаю, что BOOST_FOREACH пытается создать экземпляр range_mutable_iterator тип, который, очевидно, терпит неудачу. Как я сделать это, чтобы создать экземпляр постоянного диапазона вместо этого?

спасибо.

редактировать

вот полные объявления класса для Row и FieldIterator:

class /*final*/ Row
{
  const BYTE *m_buffer;
  const DBBINDING *m_pColumnBindings;
  int m_columnBindingCount;
  FieldIterator m_end;
public:
  typedef FieldIterator const_iterator;
  typedef FieldIterator iterator;
  Row(const BYTE *buffer, const DBBINDING *pColumnBindings, int columnBindingCount);
  bool isSameRow(const Row& r) const;
  int fieldCount() const;
  Field field(int i) const;
  Field& field(int i, void *fieldBuffer) const;
  FieldIterator begin() const;
  FieldIterator end() const;
  FieldIterator begin();
  FieldIterator end();
};

class FieldIterator : public iterator_facade<FieldIterator, Field, boost::random_access_traversal_tag>
{
  const Row *m_pRow;
  int m_index;
  mutable BYTE m_fieldBuffer[sizeof(Field)];
public:
  FieldIterator(const Row *pRow = NULL, int index = 0);
private:
  friend class boost::iterator_core_access;
  void increment();
  void decrement();
  void advance(difference_type n);
  difference_type distance_to(FieldIterator it);
  reference dereference() const;
  bool equal(const FieldIterator& rhs) const;
};

4 ответов


A обойти если вы действительно хотите, чтобы избежать iterator member должен использовать пару итераторов.

BOOST_FOREACH(Field field, std::make_pair(row.begin(), row.end()))

что случилось с вашим исходным кодом?

некоторые из стандартных контейнеров библиотеки, как std::set и std::multiset, есть итераторы, которые все const (обновление не разрешено). Стандарт конкретно говорит:

для ассоциативных контейнеров, где тип значения совпадает с типом ключа, итератор и const_iterator являются постоянными итераторами. Это энное ли Iterator и const_iterator такие же тип.

вам, вероятно, сойдет с рук

typedef const_iterator iterator;

в своем классе.


FieldIterator кажется, тот же класс итератора для методов итератора const и non-const. BOOST_FOREACH работает с любым контейнером, включая массивы C-style, что заставляет меня думать, что проблема в FieldIterator класса. Можете ли вы опубликовать код для него?


С boost 1.52 (я не тестировал с другими версиями), BOOST_FOREACH(Field field, const_cast<Row const&>(row)) тоже будет работать.