3

更新：澄清，更清晰的重点和缩短的例子：

  • 我可以避免M op+(M&&,M&&)过载吗？假设，我想要很好地处理RValues？我想其他三个重载是必需的。

(&&,&&)我首先过载的原因：

  • 通常我不会提供M op+(&&,&&)，但我似乎需要它：当提供重载时(&&,&)，(&,&&)编译器会变得模棱两可。有没有更好的方法来解决它然后添加另一个实现变体？

您还可以查看完整的代码。

struct Matrix {
...
  // 2ary ops
  friend Matrix operator+(const Matrix &a, Matrix &&b     ) { b+=a; return move(b); }
  friend Matrix operator+(Matrix &&a,      const Matrix &b) { a+=b; return move(a); }
  friend Matrix operator+(const Matrix &a, Matrix v)        { v+=a; return v; }
  friend Matrix operator+(Matrix &&a,      Matrix &&b)      { a+=b; return move(a); }
  // ... same for operator*
  // ... assume impl of operator+=,*= and move semantics
};

int main() {
  Matrix a{2},b{3},c{4},d{5};
  Matrix x = a*b + c*d;  // reuires &&,&& overload
  std::cout << x << std::endl;
}
4

2 回答 2

2

如果它是右值，则以下帮助函数返回第一个值，否则返回第二个值（可能是右值，但可能不是）。

template <class T1, class T2>
typename std::enable_if<! std::is_reference<T1>::value, T1&&>::type 
  get_rvalue(T1&& t1, T2&& t2) { return std::forward<T1>(t1); }

template <class T1, class T2>
typename std::enable_if<std::is_reference<T1>::value, T2&&>::type 
  get_rvalue(T1&& t1, T2&& t2) { return std::forward<T2>(t2); }

以下帮助函数返回上面未返回的其他值。

template <class T1, class T2>
typename std::enable_if<! std::is_reference<T1>::value, T1&&>::type 
  get_non_rvalue(T1&& t1, T2&& t2) { return std::forward<T2>(t2); }

template <class T1, class T2>
typename std::enable_if<std::is_reference<T1>::value, T2&&>::type 
  get_non_rvalue(T1&& t1, T2&& t2) { return std::forward<T1>(t1); }

这只是比较两种类型是否相同，忽略引用和 const。

template <class T1, class T2>
struct is_same_decay : public std::is_same<
  typename std::decay<T1>::type, 
  typename std::decay<T2>::type
> {};

然后我们可以为每个函数（使用模板）只做一个重载，如下所示：

// 2ary ops
template <class M1, class M2>
friend typename std::enable_if< 
  is_same_decay<M1, Matrix>::value &&
  is_same_decay<M2, Matrix>::value,
Matrix>::type
operator+(M1&& a, M2&& b) 
{ 
  Matrix x = get_rvalue(std::forward<M1>(a), std::forward<M2>(b)); 
  x += get_non_rvalue(std::forward<M1>(a), std::forward<M2>(b)); 
  return x; 
}

template <class M1, class M2>
friend typename std::enable_if< 
  is_same_decay<M1, Matrix>::value &&
  is_same_decay<M2, Matrix>::value,
Matrix>::type
operator*(M1&& a, M2&& b) 
{ 
  Matrix x = get_rvalue(std::forward<M1>(a), std::forward<M1>(b)); 
  x *= get_non_rvalue(std::forward<M1>(a), std::forward<M1>(b)); 
  return x; 
}

请注意，如果其中一个M1或M2是右值，get_rvalue(a, b)则将返回一个右值，因此在这种情况下Matrix x将由移动而不是副本填充。命名返回值优化可能会确保不需要复制（甚至移动）到返回值中，因为x将在返回值的位置构造。

完整代码在这里。

于 2011-08-29T12:05:05.857 回答
1 
Matrix& operator=(Matrix&& o) { swap(*this,o); };

首先，您的 Matrix 类没有任何东西（到目前为止）需要移动，因此您不必费心编写一个。像复制构造函数一样，仅在需要时定义一个。除非您有合法的需要（例如存储裸指针），否则让编译器处理它。

第二，你的那个功能不动；它交换。一个“惯用的”基于交换的移动涉及一个临时的，如下所示：

Matrix temp;
swap(o, temp);
swap(temp, *this);

friend Matrix operator+(const Matrix &a, Matrix &&b     ) { b+=a; return move(b); }
friend Matrix operator+(Matrix &&a,      const Matrix &b) { a+=b; return move(a); }
friend Matrix operator+(const Matrix &a, Matrix v)        { v+=a; return v; }
friend Matrix operator+(Matrix &&a,      Matrix &&b)      { a+=b; return move(a); }

你想在这里完成什么？同样，您的对象没有任何被移动的东西；这样做没有意义。仅仅因为你可以移动某些东西，并不意味着你应该这样做。如果您真的想减少代码重复，您可以正常执行以下操作：

friend Matrix operator+(const Matrix &a, const Matrix &b) { Matrix temp = a + b; return temp; }
于 2011-08-24T10:33:43.970 回答