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

Question

我正在写一个简单的代码，其中有一个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;
}
};

其中，move构造函数是

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

我的问题是：在move构造函数中，std::move的使用正确吗？特别是在list::insert的第一行代码中

Answer 1

在move构造函数中，std::move的使用正确吗？

是。

特别是在list::insert的第一行代码中

这也是正确的。

不过，请注意，我想指出两件小事。首先，不需要手动定义Person的移动构造函数。如果您的类不进行手动资源处理(内存、IO处理等(，只需依赖编译器为您生成的特殊成员函数即可。在这种情况下，我只需要删除Person的move-and-copy构造函数。

第二，

inline node(Person &&personToInsert)

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

第三，

void list::insert(Person &&personToInsert)

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