Создание настраиваемого класса диапазона, который оборачивает адаптеры повышения

Question

У меня есть класс, который применяет некоторые адаптеры буст-преобразования к диапазону (например, в действительности это намного сложнее):

struct Foo {
    auto range() const {
        return boost::irange(0, 10)
            | boost::adaptors::transformed([] (auto x) { return x * 2; });
    }

    auto begin() const { return range().begin(); }
    auto end() const { return range().end(); }
};

Только это позволяет нам перебирать Foo используя диапазон для:

for (auto x : Foo()) {
    std::cout << num << std::endl;
}

Тем не менее, это не очень хорошо сочетается с другими буст-адаптерами или диапазонными операциями (например, boost::join):

auto bad = boost::join(boost::irange(0, 10), Foo());
auto also_bad = Foo() | boost::adaptors::transformed([] (auto x) { return x + 1; });

Оба вышеперечисленных провоцируют некоторые неприятные ошибки шаблона. Бывший (bad):

In file included from test.cpp:4:
/usr/local/include/boost/range/join.hpp:30:70: error: no type named 'type' in
      'boost::range_iterator<const Foo, void>'
            BOOST_DEDUCED_TYPENAME range_iterator<SinglePassRange1>::type,
            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~
/usr/local/include/boost/range/join.hpp:44:28: note: in instantiation of template class
      'boost::range_detail::joined_type<const Foo, const boost::integer_range<int> >' requested
      here
    : public range_detail::joined_type<SinglePassRange1, SinglePassRange2>::type
                           ^
test.cpp:34:16: note: in instantiation of template class 'boost::range::joined_range<const Foo,
      const boost::integer_range<int> >' requested here
    auto bad = boost::join(Foo(), range);
               ^

...

И последняя (also_bad):

In file included from test.cpp:1:
In file included from /usr/local/include/boost/range/any_range.hpp:17:
In file included from /usr/local/include/boost/range/detail/any_iterator.hpp:22:
In file included from /usr/local/include/boost/range/detail/any_iterator_wrapper.hpp:16:
In file included from /usr/local/include/boost/range/concepts.hpp:24:
/usr/local/include/boost/range/value_type.hpp:26:70: error: no type named 'type' in
      'boost::range_iterator<Foo, void>'
    struct range_value : iterator_value< typename range_iterator<T>::type >
                                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~
/usr/local/include/boost/range/adaptor/replaced.hpp:109:40: note: in instantiation of template
      class 'boost::range_value<Foo>' requested here
                BOOST_DEDUCED_TYPENAME range_value<SinglePassRange>::type>& f )
                                       ^
test.cpp:35:27: note: while substituting deduced template arguments into function template
      'operator|' [with SinglePassRange = Foo]
    auto also_bad = Foo() | boost::adaptors::transformed([] (auto x) { return x * 2; });
                          ^
...

Обе ошибки, кажется, жалуются, что Foo не диапазон Я пытался добавить operator OutContainer() и typedefs для iterator/const_iterator как предложено здесь безрезультатно. Что я должен сделать, чтобы Foo позволить ему хорошо играть с этими операциями диапазона?

Answer 1

Ошибка очень полезна в этом случае. Ваш тип должен моделировать SinglePassRange и это не так. Там есть страница о том, как это сделать, и пока вы предоставили begin() а также end(), вы не предоставили псевдонимы типа для iterator а также const_iterator, Следовательно, вы не моделируете SinglePassRange,

Но на самом деле очень хорошо, что ваш код провалился, потому что это тоже плохо. У тебя есть begin() вызов range().begin() а также end() вызов range().end(), Это итераторы в разных диапазонах. Так что это в любом случае неопределенное поведение - вы не соответствуете семантическим понятиям SinglePassRange,

Более простое решение - просто использовать Foo() непосредственно. Это уже работает:

auto good = boost::join(boost::irange(0, 10), Foo().range());
auto also_good = Foo().range() | boost::adaptors::transformed([] (auto x) { return x + 1; });

А значит, вам просто нужно написать одну функцию: range()