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

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

Answer 2

执行itr2 = ++itr时，您也会更改itr的值，因此您应该执行类似

的操作

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

此外，如果你想稍后交换它，你必须保留一个指向最小元素的指针，如下所示：

 int* smallest = &(*itr);

这还需要进行一些其他更改，您可以找到代码here的实际示例。

实现链接列表的选择排序

2 个答案: