Справочные и установочные функции rvalue

Question

Я хочу использовать удобные функции при создании экземпляров класса в моей функции main(), чтобы сделать вещи более понятными.

Вот минимальный пример:

class MyClass 
{ 
    public: 
      MyClass() : value{ -1 }, str{ "hello" } {} 
      MyClass(const MyClass &&other) 
      { 
        value = move(other.value); 
        str = move(other.str); 
        file = move(other.file); //Use of deleted function... 
      } 

      void open() 
      { 
        file.open("myfile"); 
      } 

    private: 
      MyClass(const MyClass &) = delete; 
      MyClass operator=(const MyClass &) = delete; 
      MyClass &operator=(const MyClass &&) = delete; 

      ofstream file; 
      int value; 
      string str; 
}; 

inline MyClass setup_myclass() 
{ 
    MyClass ret; 
    ret.open(); 

    return ret; 
} 

int main(int argc, char **argv) 
{ 
    MyClass &&mc = setup_myclass(); 

    return 0; 
} 

Проблема заключается в том, когда мой класс содержит такие вещи, как fstream или нить, которая имеет свой ход конструкторов удалены.

Я компилирую с g ++ 5.1.1 и arm-linux-g ++ 5.2.0 (raspberrypi, buildroot).

Что должен выглядеть мой конструктор перемещения, когда у меня есть члены с удаленными конструкторами перемещения?

Как я могу изменить свой код на ту же самую чистую основную функцию?

Answer 1

Здесь есть две проблемы. Во-первых, ваш ход конструктор имеет неправильную подпись:

MyClass(const MyClass &&other) 

Вы не можете двигаться, из const RValue. Вы имели в виду:

MyClass(MyClass &&other) 

и более того, вы на самом деле имел в виду:

MyClass(MyClass &&other) = default; 

См Rule of Zero. Все ваши участники имеют конструкторы перемещения (std::thread и std::ofstream тоже!), Поэтому просто используйте их. Кроме того, это:

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

должны быть public и выглядеть следующим образом:

MyClass &operator=(MyClass &&) = default; 

Почему включить конструктор перемещения, но delete задание двигаться?

Во-вторых, это плохо:

MyClass &&mc = setup_myclass(); 

вы просто взяли ссылку на временный, который разрушается в конце линии. Теперь у вас свисающая ссылка. То, что вы хотите сделать, это просто:

MyClass mc = setup_myclass(); 

Благодаря РВО, не будет на самом деле движение вызывается здесь. Временной в пределах setup_myclass() фактически будет построен на месте в mc.