Перемещающий конструктор и оператор перемещающего присваивания

@Index1 · Регистрация: 17.12.2016

Author24 — интернет-сервис помощи студентам

Правильно у меня написаны перемещающий конструктор и оператор присваивания? И исправьте ошибку в операторе
"ссылка rvalue не может быть привязана к lvalue"

C++

class MyClass
{
private:
    int a;
    double b;
 
public:
    MyClass()
    {
        a = 0;
        b = 0.0;
    }
 
    MyClass(const MyClass &rhs)
    {
        a = rhs.a;
        b = rhs.b;
    }
    
    MyClass(MyClass &&rhs)
    {
        a = rhs.a;      // Правильно?
        b = rhs.b;      // ...
    }
 
    MyClass& operator=(const MyClass &rhs)
    {
        a = rhs.a;
        b = rhs.b;
 
        return *this;
    }
 
    MyClass&& operator = (MyClass &&rhs)
    {
        a = rhs.a;      // Правильно?
        b = rhs.b;      // ...
 
        return *this;   // Ошибка ссылка rvalue не может быть привязана к lvalue
    }
};

@DrOffset · 20.12.2016, 14:56

Сообщение от Index1

C++

    MyClass&& operator = (MyClass &&rhs)
    {
        a = rhs.a;      // Правильно?
        b = rhs.b;      // ...
 
        return *this;   // Ошибка ссылка rvalue не может быть привязана к lvalue
    }

C++

    MyClass& operator = (MyClass &&rhs)
    {
        a = rhs.a;
        b = rhs.b;
        return *this;
    }

Сообщение от Index1

C++

    MyClass(MyClass &&rhs)
    {
        a = rhs.a;      // Правильно?
        b = rhs.b;      // ...
    }

C++

1
2
3

    MyClass(MyClass &&rhs)
        : a(rhs.a), b(rhs.b) 
    { }

@Peoples · 20.12.2016, 14:57

Как минимум

C++

 MyClass& operator = (MyClass &&rhs)
    {
        a = rhs.a;      // Правильно?
        b = rhs.b;      // ...
 
        return *this;   // Ошибка ссылка rvalue не может быть привязана к lvalue
    }

C++

1	MyClass(MyClass &&rhs):a(rhs.a),b(rhs.b){}

Опередили немного

@Index1 · 20.12.2016, 15:05 **[ТС]**

Благодарю

notAll · 20.12.2016, 16:34

Сообщение от Index1

Правильно у меня написаны перемещающий конструктор и оператор присваивания?

Если уж пишем функции перемещения, то не забываем про noexcept иначе контейнеры ничего перемещать не будут при резайзинге:

C++

class MyClass
{
    int a = 0;
    double b = 0.0;
 
public:
    MyClass() = default;
 
    MyClass(const MyClass &rhs) : a(rhs.a), b(rhs.b)
    {
        //puts(__PRETTY_FUNCTION__);
    }
 
    MyClass(MyClass &&rhs) noexcept : a(rhs.a), b(rhs.b)
    {
        puts(__PRETTY_FUNCTION__);
        rhs.a = 0;
        rhs.b = 0.0;
    }
 
    MyClass& operator=(const MyClass &rhs)
    {
        a = rhs.a;
        b = rhs.b;
 
        return *this;
    }
 
    MyClass& operator = (MyClass &&rhs) noexcept
    {
        puts(__PRETTY_FUNCTION__);
        a = rhs.a;
        b = rhs.b;
        rhs.a = 0;
        rhs.b = 0.0;
 
        return *this;
    }
};
 
int main()
{
    std::vector<MyClass> v;
    v.reserve(4);
    for (int i = 0; i < 17; ++i)
    {
        std::cout << "size: " << v.size() << ", capacity: " << v.capacity() << '\n';
        MyClass mc;
        v.push_back(mc);
    }
}

Без noexcept все будет копироваться.

@Index1 · 20.12.2016, 18:27 **[ТС]**

notAll, да мне бы как-нибудь написать, я сегодня только начал изучать перемещающие конструкторы и присваивания. Главное работает и хорошо, для начала этого вполне хватит. А так все равно спасибо за совет, на будущее учту про noexcept

Перемещающий конструктор и оператор перемещающего присваивания

Решение

Решение

Решение

@Index1 0 / 0 / 0 Регистрация: 17.12.2016 Сообщений: 14
	20.12.2016, 15:05 [ТС]	4
	Благодарю 0

@Index1 0 / 0 / 0 Регистрация: 17.12.2016 Сообщений: 14
	20.12.2016, 18:27 [ТС]	6
	notAll, да мне бы как-нибудь написать, я сегодня только начал изучать перемещающие конструкторы и присваивания. Главное работает и хорошо, для начала этого вполне хватит. А так все равно спасибо за совет, на будущее учту про noexcept 0