提问人:Bernard 提问时间:1/17/2020 最后编辑:Bernard 更新时间:5/4/2021 访问量:1258
shift_right() 打算如何在 C++20 中实现?
How is shift_right() intended to be implemented in C++20?
问:
在 C++20 中,标头获得了两个新算法:shift_left()
和 shift_right()。
它们都接受任何 LegacyForwardIterator。对于 ,指定“移动按从 ”开始的递增顺序执行“;对于 ,指定“如果满足 LegacyBidirectionalIterator 要求,则移动将按从 开始的降序执行。<algorithm>
shift_left()
i
0
shift_right()
ForwardIt
i
last - first - n - 1
我能想到一个相当简单的方法来实现:shift_left()
template <typename ForwardIt>
constexpr inline ForwardIt shift_left(ForwardIt first, ForwardIt last, typename std::iterator_traits<ForwardIt>::difference_type n) {
if (n <= 0) return last;
ForwardIt it = first;
for (; n > 0; --n, ++it) {
if (it == last) return first;
}
return std::move(it, last, first);
}
如果满足 LegacyBidirectionalIterator 的要求,我可以看到它可以以与 .然而,目前还不清楚如何实现非双向前向迭代器。ForwardIt
shift_right()
shift_left()
shift_right()
我想出了一种算法,它使用空间作为暂存空间来交换元素,但它似乎比上面的算法更浪费:[first, first+n)
shift_left()
template <typename ForwardIt>
constexpr inline ForwardIt shift_right(ForwardIt first, ForwardIt last, typename std::iterator_traits<ForwardIt>::difference_type n) {
if (n <= 0) return first;
ForwardIt it = first;
for (; n > 0; --n, ++it) {
if (it == last) return last;
}
ForwardIt ret = it;
ForwardIt ret_it = first;
for (; it != last; ++it) {
std::iter_swap(ret_it, it);
ret_it++;
if (ret_it == ret) ret_it = first;
}
return ret;
}
会有更好或“预期”的实施方式吗?shift_right()
答:
7赞
VLL
1/17/2020
#1
以下是班次的示例实现:https://github.com/danra/shift_proposal/blob/master/shift_proposal.h
该链接来自提案文件:http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2017/p0769r0.pdf
#include <algorithm>
#include <iterator>
#include <type_traits>
#include <utility>
template<class I>
using difference_type_t = typename std::iterator_traits<I>::difference_type;
template<class I>
using iterator_category_t = typename std::iterator_traits<I>::iterator_category;
template<class I, class Tag, class = void>
constexpr bool is_category = false;
template<class I, class Tag>
constexpr bool is_category<I, Tag, std::enable_if_t<
std::is_convertible_v<iterator_category_t<I>, Tag>>> = true;
/// Increment (decrement for negative n) i |n| times or until i == bound,
/// whichever comes first. Returns n - the difference between i's final position
/// and its initial position. (Note: "advance" has overloads with this behavior
/// in the Ranges TS.)
template<class I>
constexpr difference_type_t<I> bounded_advance(
I& i, difference_type_t<I> n, I const bound)
{
if constexpr (is_category<I, std::bidirectional_iterator_tag>) {
for (; n < 0 && i != bound; ++n, void(--i)) {
;
}
}
for(; n > 0 && i != bound; --n, void(++i)) {
;
}
return n;
}
template<class ForwardIt>
ForwardIt shift_left(ForwardIt first, ForwardIt last, difference_type_t<ForwardIt> n)
{
if (n <= 0) {
return last;
}
auto mid = first;
if (::bounded_advance(mid, n, last)) {
return first;
}
return std::move(std::move(mid), std::move(last), std::move(first));
}
template<class ForwardIt>
ForwardIt shift_right(ForwardIt first, ForwardIt last, difference_type_t<ForwardIt> n)
{
if (n <= 0) {
return first;
}
if constexpr (is_category<ForwardIt, std::bidirectional_iterator_tag>) {
auto mid = last;
if (::bounded_advance(mid, -n, first)) {
return last;
}
return std::move_backward(std::move(first), std::move(mid), std::move(last));
} else {
auto result = first;
if (::bounded_advance(result, n, last)) {
return last;
}
// Invariant: next(first, n) == result
// Invariant: next(trail, n) == lead
auto lead = result;
auto trail = first;
for (; trail != result; ++lead, void(++trail)) {
if (lead == last) {
// The range looks like:
//
// |-- (n - k) elements --|-- k elements --|-- (n - k) elements --|
// ^-first trail-^ ^-result last-^
//
// Note that distance(first, trail) == distance(result, last)
std::move(std::move(first), std::move(trail), std::move(result));
return result;
}
}
for (;;) {
for (auto mid = first; mid != result; ++lead, void(++trail), ++mid) {
if (lead == last) {
// The range looks like:
//
// |-- (n - k) elements --|-- k elements --|-- ... --|-- n elements --|
// ^-first mid-^ result-^ ^-trail last-^
//
trail = std::move(mid, result, std::move(trail));
std::move(std::move(first), std::move(mid), std::move(trail));
return result;
}
std::iter_swap(mid, trail);
}
}
}
}
评论
0赞
Caleth
1/17/2020
@YSC防止过度热心的“丢弃结果”警告
0赞
YSC
1/17/2020
@vll我就是这么想的。
5赞
walnut
1/17/2020
@YSC 可能是为了防止不应调用的重载逗号运算符。
0赞
Bernard
1/17/2020
这似乎是我正在做的事情。
评论
std::move
std::copy