包含仅移动类型的类的构造函数是否应通过引用或右值引用接收仅移动类型?

时间:2019-06-23 20:24:23

标签: c++ move-semantics

我最近开始学习关于移动语义的问题,几天来我一直在思考以下问题:

假设我们有一个不可复制的类,如下所示:

class Texture
{
public:

   Texture(unsigned int texID);
   ~Texture();

   Texture(const Texture&) = delete;
   Texture& operator=(const Texture&) = delete;

   Texture(Texture&& rhs);
   Texture& operator=(Texture&& rhs);

   // ...

private:

   unsigned int mTexID;
};

对于那些想知道的人,在使用OpenGL时通常有这样的包装器类。该ID用于访问存储在GPU中的数据,并且用于告诉GPU销毁该数据,这是在该包装类的析构函数中完成的。这就是为什么它是不可复制的类。

现在让我们说我们还有另一个不可复制的类,如下所示:

class Mesh
{
public:

   // Notice how the constructor receives the vector of Texture objects (a move-only type) by reference
   Mesh(const std::vector<unsigned int>& indices, std::vector<Texture>& textures)
      : mIndices(indices)
      , mTextures(std::move(textures))
   {
      // ...
   }
   ~Mesh();

   Mesh(const Mesh&) = delete;
   Mesh& operator=(const Mesh&) = delete;

   Mesh(Mesh&& rhs);
   Mesh& operator=(Mesh&& rhs);

   // ...

private:

   std::vector<unsigned int> mIndices;
   std::vector<Texture>      mTextures;
};

使用现在的构造方法,客户端可以通过执行以下操作来创建Mesh

std::vector<unsigned int> indices;
std::vector<Texture> textures;

// ...

Mesh mesh(indices, textures); // Client is unaware that the textures vector has been moved from

我的问题是,像这样声明Mesh类的构造函数是否会更好:

// Notice how the constructor receives the vector of Texture objects (a move-only type) by rvalue reference
Mesh::Mesh(const std::vector<unsigned int>& indices, std::vector<Texture>&& textures)
   : mIndices(indices)
   , mTextures(std::move(textures))
{
   // ...
}

使用这个新的构造函数,在创建Mesh对象时,客户端将被迫执行以下操作:

std::vector<unsigned int> indices;
std::vector<Texture> textures;

// ...

Mesh mesh(indices, std::move(textures)); // Client is fully aware that the textures vector has been moved from

当然可以输入更多信息,但是现在用户已经完全意识到纹理矢量已从中移出,我看不到任何性能影响。

所以我想我的问题是:是否有什么准则来指导接收仅移动类型的最佳方式?通过引用const进行接收可以清楚地表明该类型将不会被移走,那么如何相反呢?如何告诉客户类型将从何而来?

1 个答案:

答案 0 :(得分:1)

如果传递的值可能是prvalue,则使用rvalue引用显然更好:

struct A {};
struct B {B(A&);};
struct C {C(A&&);};
A get();
A a;

B b{a};              // OK: a is an lvalue
B b2{get()};         // error: prvalue
C c{a};              // error: lvalue
C c2{std::move(a)};  // OK: xvalue
C c3{get()};         // OK: prvalue