重载operator=导致std::sort失效的原因排查

你的std::sort失效完全是因为重载的operator=犯了一个致命错误——它根本没有修改当前对象，而是返回了一个全新的对象，这直接破坏了std::sort依赖的元素赋值逻辑。

具体错误分析 #

你写的赋值运算符是这样的：

Person operator= (const Person& p) {
    Person newP;
    newP.name = p.name;
    newP.age = p.age;
    return newP;
}

这里的核心问题有两个：

1. 返回值是Person（值类型）而非Person&（引用类型）
2. 函数内部创建了一个新的Person对象并赋值返回，完全没有对调用该运算符的当前对象（*this）做任何修改

当std::sort在排序过程中需要交换或移动元素时，它会依赖赋值运算符来修改容器中的元素。但你的版本只是返回了一个无关的新对象，容器里原来的元素根本没有被改变，等于排序操作完全没有生效。

修正后的赋值运算符 #

正确的赋值运算符应该修改当前对象，并返回自身的引用，这样才能满足std::sort的需求，同时也符合C++的通用惯例：

Person& operator=(const Person& p) {
    // 先判断自赋值，避免不必要的操作（可选但推荐）
    if (this != &p) {
        this->name = p.name;
        this->age = p.age;
    }
    return *this;
}

返回*this的引用不仅能让赋值支持链式操作（比如a = b = c），更关键的是它真正修改了调用该运算符的对象，让std::sort能正确调整容器内元素的状态。

额外的规范建议 #

你的operator<也应该加上const限定符，因为std::sort在比较时可能会处理const对象，加上const能让运算符更通用、更符合const正确性：

bool operator<(const Person& p) const { 
    return this->age < p.age; 
}

补充说明 #

其实如果你的Person类没有涉及动态内存分配、资源管理这类特殊需求，完全不需要手动重载operator=——编译器自动生成的默认赋值运算符已经能正确处理std::string和uint32_t这些成员，而且不会出错。

内容的提问来源于stack exchange，提问作者yano