6

基になる要素のシーケンスでソートされたビューとして機能するクラスを作成しようとしています。constこれまでのところ、非バージョンを思いつきました。現在、機能を提供するように適応させるのに問題がありconst_iteratorます。

これまでのコードは次のようになります。

// forward declare iterator
template <class InputIt>
class sorted_range_iter;

template <class InputIt>
class sorted_range {
    friend class sorted_range_iter<InputIt>;

  private:
    using T = typename InputIt::value_type;
    InputIt _first;
    InputIt _last;
    std::vector<size_t> _indices;

  public:
    using iterator = sorted_range_iter<InputIt>;

    sorted_range() = default;
    sorted_range(InputIt first, InputIt last)
        : _first(first), _last(last), _indices(std::distance(_first, _last)) {
        std::iota(_indices.begin(), _indices.end(), 0);
    };

    template <class Compare = std::less<T>>
    void sort(Compare comp = Compare()) {
        std::sort(_indices.begin(), _indices.end(),
                  [this, &comp](size_t i1, size_t i2) {
                      return comp(*(_first + i1), *(_first + i2));
                  });
    }

    size_t size() const { return _indices.size(); }
    T& operator[](size_t pos) { return *(_first + _indices[pos]); }
    const T& operator[](size_t pos) const { return (*this)[pos]; }

    iterator begin() { return iterator(0, this); }
    iterator end() { return iterator(size(), this); }
};

対応するイテレータは次のようになります。

template <class InputIt>
class sorted_range_iter
    : public std::iterator<std::forward_iterator_tag, InputIt> {

    friend class sorted_range<InputIt>;

  private:
    using T = typename InputIt::value_type;

    size_t _index;

    sorted_range<InputIt>* _range;
    sorted_range_iter(size_t index, sorted_range<InputIt>* range)
        : _index(index), _range(range) {}

  public:
    T& operator*() { return *(_range->_first + _range->_indices[_index]); }

    // pre-increment
    const sorted_range_iter<InputIt>& operator++() {
        _index++;
        return *this;
    }

    // post-increment
    sorted_range_iter<InputIt> operator++(int) {
        sorted_range_iter<InputIt> result = *this;
        ++(*this);
        return result;
    }

    bool operator!=(const sorted_range_iter<InputIt>& other) const {
        return _index != other._index;
    }
};

使用例は次のようになります。

std::vector<int> t{5, 2, 3, 4};
auto rit = ref.begin();
sorted_range<std::vector<int>::iterator> r(begin(t), end(t));
r.sort();

for(auto& x : r)
{
    std::cout << x << std::endl;
}

出力：

2
3
4
5

ケースにイテレータを適応させるにはどうすればよいconstですか? 代わりに、基になる型 (intたとえば) でイテレータをテンプレート化すると、より簡単になりますInputIt。このクラスを定義するより良い方法はありますか?

たとえば、range-v3ライブラリを使用してこれを解決できると思いますが、これ以上依存関係を追加せず、C++11/14 関数に依存しようとしています。

4

1 に答える 1

5

に間違ったタイプを使用しているだけですT。あなたが持っている：

using T = typename InputIt::value_type;

しかし、とvalue_typeは同じです。それらが持っているのは、異なる参照タイプです。あなたは好むべきです：iteratorconst_iterator

using R = typename std::iterator_traits<InputIt>::reference;

R operator*() { ... }

これには、ポインターを操作するという追加の利点があります。

iterator_traitsまたは、イテレータ自体を逆参照することで回避できます。

using R = decltype(*std::declval<InputIt>());

サイドノート、sorted_range建設で自分自身を分類すべきではありませんか? そうしないと、誤用されやすくなります。

于 2015-10-08T15:55:26.660 に答える