最小和完美转发

Question

min 算法通常是这样表示的：

template <typename T>
const T& min(const T& x, const T& y)
{
    return y < x ? y : x;
}

但是，这不允许min(a, b) = 0 形式的构造。您可以通过额外的重载来实现：

template <typename T>
T& min(T& x, T& y)
{
    return y < x ? y : x;
}

我想做的是通过完美转发统一这两个重载：

template <typename T>
T&& min(T&& x, T&& y)
{
    return y < x ? std::forward<T>(y) : std::forward<T>(x);
}

但是，g++ 4.5.0 向min(2, 4) 发出警告，我返回对临时的引用。我是不是做错了什么？

---

好的，我明白了。问题在于条件运算符。在我的第一个解决方案中，如果我调用 min(2, 4)，条件运算符会看到一个 xvalue，因此会从转发的 x 移动以生成一个临时对象。当然，通过引用返回它是危险的！如果我分别转发整个表达式而不是 x 和 y，编译器就不会再抱怨了：

template <typename T>
T&& min(T&& x, T&& y)
{
    return std::forward<T>(y < x ? y : x);
}

---

好的，我摆脱了算术类型的引用:)

#include <type_traits>

template <typename T>
typename std::enable_if<std::is_arithmetic<T>::value, T>::type
min(T x, T y)
{
    return y < x ? y : x;
}

template <typename T>
typename std::enable_if<!std::is_arithmetic<T>::value, T&&>::type
min(T&& x, T&& y)
{
    return std::forward<T>(y < x ? y : x);
}

Answer 1

在我看来，您试图将问题过于简单化。不幸的是，让它完全正确绝对不是微不足道的。如果您还没有阅读N2199，那么现在是阅读的好时机。右值引用继续发展，因此它的 min 和 max 的引用实现可能不再完全正确，但它至少应该是一个相当不错的起点。警告：参考实现比你想象的要复杂很多！

Answer 2

您不希望完美转发，在这里，您希望返回T& 或const T& 而永远不要返回T&&。 std::forward 旨在将您的一个参数传递给另一个函数，而不是用于返回值。

我想你想要的是：

template <typename T>
min(T&& x, T&& y) -> decltype(x)
{
    return y < x ? y : x;
}

编辑以避免悬空引用问题：

template <typename T>
struct dedangle { typedef T type; }

template <typename T>
struct dedangle<const T&> { typedef T type; }

template <typename T, typename U>
min(T&& x, U&& y) -> dedangle<decltype(0?y:x)>::type
{
    return y < x ? y : x;
}

// dedangle is re-usable by max, etc, so its cost is amortized