为链表实现选择排序

Question

我正在尝试实现一种选择排序算法，该算法将与链表一起使用，并将使用迭代器来遍历它们。选择排序算法如下：对于列表中除最后一个之外的每个元素（我们称之为K），它将从我们当前所在的位置开始寻找最小的on（因此它将从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);

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

Answer 2

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