实现链表的选择排序

Question

我正在尝试实现一个选择排序算法，该算法将与链表一起工作，并将使用迭代器来滚动它们。选择排序算法如下:对于列表中的每个元素，除了最后一个元素(我们称之为K)，它将从我们当前所在的位置寻找最小的元素(所以它将从K开始直到最后一个元素)。之后是swap K and the smallest element.

我认为我的错误在第一个for循环;我很不确定--a.end()是前最后一个元素。我得到一些输出，虽然它是错误的。

#include <iostream>
#include <list>
using namespace std;
void sort_list(list<int>& a)
{
    //from the first until the pre-last element
    for(list<int> :: iterator itr = a.begin(); itr != (--a.end()); ++itr)
    {
            int smallest = *itr;
        //get smallest element after current index
         list<int> :: iterator itr2 =itr;
          ++itr2;
    for(; itr2 != a.end(); ++itr2)
        {
                if (smallest > *itr2)
                   {
                       smallest = *itr2;
                   } 
        }
        //swap smallest and current index
        int tmp = *itr;
        *itr = smallest;
        smallest = tmp;
    }
}
int main()
{
    //create a list and some elements
    list<int> listi;
    listi.push_back(5);
    listi.push_back(4);
    listi.push_back(3);
    listi.push_back(2);
    listi.push_back(1);
    // sort the list
    sort_list(listi);
    //print all of the elements
    for(list<int> :: iterator itr = listi.begin(); itr != listi.end(); ++itr)
    {
            cout << *itr << endl;
    }
    return 0;
}

Answer 1

当您执行itr2 = ++itr时，您也更改了itr的值，因此您应该执行如下操作

list<int> :: iterator itr2 = itr;
for(++itr2; itr2 != a.end(); ++itr2) {
    ...
}

此外，如果以后要交换，必须保留指向最小元素的指针，如下所示:

 int* smallest = &(*itr);

这还需要一些其他的更改，您可以在这里找到您的代码的工作示例

Answer 2

问题是您在初始化itr2时破坏了itr。