gsl :: span <t>和gsl :: span <const t =“”>重载是不明确的

时间:2016-03-07 10:36:32

标签: c++ c++14 cpp-core-guidelines

C++ Core Guidelines宣传using span的做法。

问题在于const和可变范围。这就是我试图做的事情:

auto foo(gsl::span<int>);         // 1st
auto foo(gsl::span<const int>);   // 2nd

但如果没有明确的span演员/构造参数,就无法调用它们:

std::vector<int> v;
const std::vector<int> cv;
const std::vector<int>& crv = v;

// ambiguous
// want to call 1st
foo(v);

// ambiguous, although 1st is illegal (static_assert kicks in)
// want to call 2nd
foo(cv); // ambiguous
foo(crv); // ambiguous

处理这个问题的正确方法是什么?

这似乎应该是微不足道的,类似于const T&T&重载,但它不是(或者我只是看不到它)。

只是在同一页面上,foo(gsl::span<int>{v})很麻烦,我想避免它,让调用者保持简单:foo(v)

我概括了这个问题,但是如果这是一个XY问题,这就是我实际尝试做的事情:

auto split(gsl::cstring_span<> str) -> std::vector<gsl::cstring_span<>>;
auto split(gsl::string_span<> str) -> std::vector<gsl::string_span<>>;

并且想要使用[const] char *[const] string等参数进行调用。

2 个答案:

答案 0 :(得分:3)

根据P0122R1span类的相关构造函数是:

template <class Container>
constexpr span(Container& cont);

所以你所有的3个例子都很不幸。除非 Container::value_type&可转换为span::value_type& Container与span兼容,否则可以通过要求将此构造函数从重载决策中移除来使第二个成为合法的。

即使我们这样做,我也看不到允许数字1和3,因为两个重载只需要一个用户定义的隐式转换。

通常的解决方法是添加另一个级别:

template<class T>
auto foo( T && x ) { return foo_impl( as_span(std::forward<T>(x) ) ); }

auto foo_impl(gsl::span<int>);         // 1st
auto foo_impl(gsl::span<const int>);   // 2nd

请注意,as_span不在P0122R1中,但它是在Microsoft GSL中实现的。它起作用,因为它检查类型并返回span<typename Container::value_type>

答案 1 :(得分:0)

sbabbi's answer as_span所示,span是一个不错的解决方案。但是没有as_string_span,这将解决我的真正问题。

这是我的简单实现:

template <class Char_t, gslx::size_t N>
auto as_basic_string_span(gsl::basic_string_span<Char_t, N> str)
    -> gsl::basic_string_span<Char_t, N>
{
    return str;
}

template <class Char_ptr_t>
auto as_basic_string_span(Char_ptr_t ptr)
    -> std::enable_if_t<
          stdx::is_pointer_v<Char_ptr_t>,
          gsl::basic_string_span<std::remove_pointer_t<Char_ptr_t>>>
{
    Expects(ptr != nullptr);
    return {ptr, gslx::size_cast(stdx::char_traits_length(ptr))};
}

template <class CharT, gslx::size_t N>
auto as_basic_string_span(stdx::c_array_t<CharT, N>& arr)
    -> gsl::basic_string_span<CharT, N - 1>
{
    Expects(N > 0 && arr[N - 1] == '\0');
    return arr;
}

template <class Char_t, class Traits, class Allocator>
auto as_basic_string_span(std::basic_string<Char_t, Traits, Allocator>& str)
    -> gsl::basic_string_span<Char_t>
{
    return {const_cast<Char_t*>(str.data()), gslx::size_cast(str.size())};
}

template <class Char_t, class Traits, class Allocator>
auto as_basic_string_span(
    const std::basic_string<Char_t, Traits, Allocator>& str)
    -> gsl::basic_string_span<const Char_t>
{
    return {str.data(), gslx::size_cast(str.size())};
}

template <class Char_t, class Traits, class Allocator>
auto as_basic_string_span(std::basic_string<Char_t, Traits, Allocator>&& str) =
    delete;

奖金as_const_basic_string_span

template <class Char_t, gslx::size_t N>
auto as_const_basic_string_span(gsl::basic_string_span<Char_t, N> str)
    -> gsl::basic_string_span<const Char_t, N>
{
    return str;
}

template <class... Args>
auto as_const_basic_string_span(Args&&... args)
    -> decltype(as_const_basic_string_span(
        as_basic_string_span(std::forward<Args>(args)...)))
{
    return as_const_basic_string_span(
        as_basic_string_span(std::forward<Args>(args)...));
}

用法:

template <class CharT>
auto split(gsl::basic_string_span<CharT> str)
    -> std::vector<gsl::basic_string_span<CharT>>

template <class T>
auto split(T&& str)
{
    return split(as_basic_string_span(std::forward<T>(str)));
}