Question

在我们项目的代码库中，我找到了类似的东西：

struct MeshData {
    MeshData() {}
    MeshData(MeshData&& obj) { *this = std::move(obj); }
    MeshData& operator=(MeshData&& obj) {
        if (this != &obj) {
            indexes = std::move(obj.indexes);
        }
        return *this;
    }
    std::vector<int> indexes;
};

在移动分配方面实施移动构造对我来说似乎是一个聪明的想法，可以减少代码重复，但在查找信息之后，我没有找到任何关于此的具体建议。

我的问题是：这是反模式还是有任何情况不应该这样做？

Answer 1

如果您的类型的数据成员没有默认构造函数，则无法执行此操作，如果它们具有昂贵的默认构造函数，则不应该执行此操作：

struct NoDefaultCtor {
    NoDefaultCtor() = delete;
    NoDefaultCtor(int){}
};

struct Foo {
    Foo(Foo&& obj) { *this = std::move(obj); }
    Foo& operator=(Foo&& obj) {
        if (this != &obj) {
            thing = std::move(obj.thing);
        }
        return *this;
    }
    NoDefaultCtor thing;
};

给出了这个错误：

<source>: In constructor 'Foo::Foo(Foo&&)':
<source>:10:20: error: use of deleted function 'NoDefaultCtor::NoDefaultCtor()'
     Foo(Foo&& obj) { *this = std::move(obj); }
                    ^
<source>:5:5: note: declared here
     NoDefaultCtor() = delete;

这是因为必须在输入构造函数体之前构造所有数据成员。

但是，最好的建议是遵循Rule of Zero并避免写下这些特殊成员。

Answer 2

当移动构造函数和移动成员变量的运算符可能产生不同的结果时，这种方法的一个缺陷就会表现出来。例如，如果indexes是使用vector评估为std::allocator_traits<allocator_type>::propagate_on_container_move_assignment::value的自定义分配器的true，则会发生这种情况。在这种情况下，专用移动构造函数将移动分配器和数据而不抛出异常，而调用移动分配将导致分配新存储并且所有项目移动构造导致相当大的开销并阻止移动构造函数不被抛出。

根据移动赋值运算符

2 个答案: