I have the following code to determine whether or not 2 types are comparable.
template<typename T, typename U, typename = std::void_t<>>
struct is_comparable
: std::false_type
{};
template<typename T, typename U>
struct is_comparable<T, U, std::void_t<decltype((std::declval<T>() == std::declval<U>()))>>
: std::true_type
{};
Is this an acceptable way of archieving what I am trying to do? Can you see any problems with this design?
EDIT
Keeping cdhowie's comment and Henri Menki's answer in mind, this is how the code looks now.
namespace meta
{
template<typename T, typename U, typename = std::void_t<>>
struct has_equal_to_operator
: std::false_type
{};
template<typename R, typename T, typename U, typename = std::void_t<>>
struct has_equal_to_operator_r
: std::false_type
{};
template<typename T, typename U, typename = std::void_t<>>
struct has_nothrow_equal_to_operator
: std::false_type
{};
template<typename R, typename T, typename U, typename = std::void_t<>>
struct has_nothrow_equal_to_operator_r
: std::false_type
{};
template<typename T, typename U>
struct has_equal_to_operator<T, U, std::void_t<decltype(std::declval<T>() == std::declval<U>())>>
: std::true_type
{};
template<typename R, typename T, typename U>
struct has_equal_to_operator_r<R, T, U, std::void_t<decltype(std::declval<T>() == std::declval<U>())>>
: std::is_convertible<decltype(std::declval<T>() == std::declval<U>()), R>
{};
template<typename T, typename U>
struct has_nothrow_equal_to_operator<T, U, std::void_t<decltype(std::declval<T>() == std::declval<U>())>>
: std::bool_constant<noexcept(std::declval<T>() == std::declval<U>())>
{};
template<typename R, typename T, typename U>
struct has_nothrow_equal_to_operator_r<R, T, U, std::void_t<decltype(std::declval<T>() == std::declval<U>())>>
: std::bool_constant<(noexcept(std::declval<T>() == std::declval<U>()) && std::is_convertible_v<decltype(std::declval<T>() == std::declval<U>()), R>)>
{};
template<typename T, typename U>
inline constexpr auto has_equal_to_operator_v = has_equal_to_operator<T, U>::value;
template<typename R, typename T, typename U>
inline constexpr auto has_equal_to_operator_r_v = has_equal_to_operator_r<R, T, U>::value;
template<typename T, typename U>
inline constexpr auto has_nothrow_equal_to_operator_v = has_nothrow_equal_to_operator<T, U>::value;
template<typename R, typename T, typename U>
inline constexpr auto has_nothrow_equal_to_operator_r_v = has_nothrow_equal_to_operator_r<R, T, U>::value;
}
Here is a solution with void_t like in the question. Additionally I would check if the comparison produces the correct type (bool in this case).
#include <type_traits>
#include <iostream>
template < typename T, typename U >
using equality_comparison_t = decltype(std::declval<T&>() == std::declval<U&>());
template < typename T, typename U, typename = std::void_t<> >
struct is_equality_comparable
: std::false_type
{};
template < typename T, typename U >
struct is_equality_comparable < T, U, std::void_t< equality_comparison_t<T,U> > >
: std::is_same< equality_comparison_t<T,U>, bool >
{};
struct X {};
struct Y { int operator==(Y const&) { return 1; } };
int main()
{
static_assert(false == is_equality_comparable<X, X>(), "!");
static_assert( true == is_equality_comparable<std::string, std::string>(), "!!");
static_assert(false == is_equality_comparable<int, std::string>(), "!!!");
static_assert( true == is_equality_comparable<int, int>(), "!!!!");
static_assert(false == is_equality_comparable<Y, Y>(), "!!!!!");
}
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With