为什么 std::sort() 会改变排序向量？

Question

问题是：

在我的第一堂课中，我有一个向量、一个双精度变量，并且我重载了比较运算符。这是相关的代码：

class City
{
    double distance;
    std::vector<int> coordinates;

    bool operator<(const City& city) const
    {   
        return this->distance < city.distance;
    } 

    // same for the greater-than operator but changing "<" to ">"
};

在另一个类中，我有一个城市向量，每次满足条件时我都必须对其进行排序。为此，我有一个定义如下的结构：

编辑：（引用而不是值）

struct CitySortHelper {
    bool operator() (const City &x, const City &y) const { return x < y; } 
} city_sort;

现在问题部分，当我对向量进行排序时，出现新的 City 对象，我无法解释原因：

编辑：

// this prints all current objects in the vector
for (int i = 0; i < totalCities; i++) {
    std::cout << cities->at(i) << std::endl;
}

// after the following line I get new City objects in the 
// vector, that weren't there before the sort. The new objects
// always have distance = 0 and random values in the coordinates
std::sort(cities->begin(), cities->end(), city_sort);

// using the sort with no predicate also gives the same faulty results
std::sort(cities->begin(), cities->end());

编辑：（复制构造函数和赋值运算符）

City(const City &city)
{
    this->distance = city.distance;
    this->coordinates = city.coordinates;
}

City& operator= (const City &city)
{
    this->distance = city.distance;
    this->coordinates = city.coordinates;

    return *this;
}

奇怪的是，只有当我按升序对 City 对象进行排序时才会发生这种情况，即如果我将 CitySortHelper 中的比较运算符从“<”更改为到“>”一切正常。

有什么想法为什么会发生这种情况吗？任何帮助表示赞赏。

Answer 1

CitySortHelper 需要通过 const 引用而不是值来获取参数。另一件需要记住的事情是，排序对 City 使用赋值运算符；检查您的赋值运算符是否正常工作。解决这两个问题应该可以解决问题。

Answer 2

将您的排序助手更改为具有

bool operator() ( const City& x , const City& y) const

并且还要检查 City 复制构造函数和赋值运算符是否执行正确的操作

Answer 3

如果您想保留顺序，则不应使用 std::sort()，而应使用 std::stable_sort()。 stable_sort 保证元素保持其相对顺序，而 sort 则不然。

另外，似乎 sort 不是你的问题。似乎有 City 对象被推入向量中的某个地方，而您没有注意到它们，因为您正在检查变量的大小而不是向量的迭代器。尝试像这样打印并告诉我们结果：

for (std::vector <City> ::iterator it = cities->begin(); it != cities->end(); ++it) {
    std::cout << *it << std::endl;
}