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

Question

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

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 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

在移动构造函数中，std::move 的使用是否正确？

是的。

尤其是list::insert的第一行代码

这也是正确的。

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

第二，

inline node(Person &&personToInsert)

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

第三，

void list::insert(Person &&personToInsert)

根本不是构造函数——它是一个普通的成员函数，碰巧接受一个右值参数。