Person 对象列表的移动构造函数

Question

我正在编写一个简单的代码，其中有一个 class 人的对象列表。这个人 class

class Person
{
    private:
        std::string name;
        std::string surname;
        int year;
        
    public:
        Person(const std::string& personName,
               const std::string& personSurname, 
               const int& personYear ):
               name(personName), surname(personSurname), year(personYear) {}
               
        Person(const Person& p): name(p.name), surname(p.surname), year(p.year) { }
        
        Person(Person &&p) noexcept: name(std::move(p.name)), surname(std::move(p.surname)), year(std::move(p.year)) { }
        
        int getYear()
        {
            return year;
        }
        
        void print()
        {
            std::cout << name << " " << surname << " " << year << std::endl;
        }
};

其中，移动构造函数为

        Person(Person &&p) noexcept: name(std::move(p.name)), surname(std::move(p.surname)), year(std::move(p.year)) { }

节点结构我也有

struct node
{
    Person data;
    node *next;
    inline node(const std::string& personName,
                const std::string& personSurname, 
                const int& personYear): data(personName, personSurname, personYear), next(nullptr) { }
    inline node(const Person& personToInsert): data(personToInsert), next(nullptr) {} 
    inline node(Person &&personToInsert): data(std::move(personToInsert)), next(nullptr) {}
};

谁的移动构造函数是

    inline node(Person &&personToInsert): data(std::move(personToInsert)), next(nullptr) {}

最后我得到了列表 class

class list
{
    private:
    node *head;
    
    public:
    list();
    ~list();
    void insert(const std::string&, const std::string&, const int& );
    void insert(const Person& );
    void insert(Person &&);
    void print();
};

谁的移动构造函数是

void list::insert(Person &&personToInsert) {
    node *new_node = new node(std::move(personToInsert));
    if(head == nullptr || (head->data).getYear() >= (new_node->data).getYear())
    {
        new_node->next = head;
        head = new_node;
    }
    else
    {
        node *current = head;
        while(current->next != nullptr && (current->next->data).getYear() < (new_node->data).getYear())
        {
            current = current->next;
        }
        new_node->next = current->next;
        current->next = new_node;
    } 
}

我的问题是：在移动构造函数中，使用std::move是否正确？特别是在list::insert

的第一行代码

Answer 1

inside the move constructors, is the use of std::move correct?

是的。

Particularly in the first line of code of list::insert

这也是正确的。

但是请注意，我想指出两件小事。首先，不需要为 Person 手动定义移动构造函数。如果您的 class 不进行手动资源处理（内存、IO 句柄等），只需依赖编译器为您生成的特殊成员函数。在这种情况下，我将只删除 Person 的移动和复制构造函数。

第二，

inline node(Person &&personToInsert)

是不是移动构造函数。它是一个普通的构造函数，恰好采用右值引用参数。移动构造函数用于从其自身类型的右值引用构造对象。

第三，

void list::insert(Person &&personToInsert)

根本不是构造函数 - 它是一个普通的成员函数，恰好接受右值参数。

Person 对象列表的移动构造函数

Move constructor for a list of Person objects

c++

constructor

c++17

stdmove