选择排序程序显示异常结果

Question

我用 C++ 编写了一个选择排序程序。我已经检查并重新检查了代码，但逻辑非常好。但是代码没有正确排序数组：

    // Selection Sort

#include <iostream>

using namespace std;

inline void swap( int& x, int& y) {
    int temp = x;
    x = y;
    y = temp;
}

int main() {
    const int n = 10;
    int list[n];

    string line(14, '-');
    cout << '\n' << line << "\nSelection Sort\n" << line << '\n';
    cout << "Enter " << n << " elements:\n";

    for( int i = 0; i < n; ++i ) {
        cin >> list[i];
    }

    for( int i = 0; i < n-1; ++i ) {
        int small = i;
        for( int j = 1; j < n; ++j ) {
            if( list[j] < list[small] ) {
                small = j;
            }
        }
        if( list[small] < list[i] ) {
            swap( list[i], list[small]);
        }
    }

    cout << "Sorted Array:\n";
    for( int i = 0; i < n; ++i ) {
        cout << list[i] << ' ';
    }

    return 0;
}

我哪里错了？选择排序的算法在这里给出：Selection Sort- wikipedia

10 个数字的示例输入：

7 8 9 10 11 12 13 14 15 16

输出：

7 9 10 11 12 13 14 15 8 16

Answer 1

这里的内部循环肯定是错误的：

for( int j = 1; j < n; ++j ) {

应该是

for( int j = i+1; j < n; ++j ) {

还有循环结束的条件

if( list[small] < list[i] ) {
    swap( list[i], list[small]);
}

过分了。内循环退出后按定义满足。

请参阅：Sorting algorithms - selection sort 获取伪代码和关于不同类型数据的精美动画演示。

Answer 2

在排序循环的内部循环中，你写了：for( int j = 1; ...)
现在，每次，内部循环不是从 1 到 n-1，而是从 i+1 到 n-1 >
所以把循环改成for( int j = i+1; ...)

此外，您可以将交换条件更改为 small != i，但这是可选的。

Answer 3

我检查了您的代码，但您有一个逻辑错误，选择排序成功。在内循环中，内循环的初始化总是j = 1。这应该更改为：

for( int j = i+1; j < n; ++j ) {
        if( list[j] < list[small] ) {
            small = j;
        }
}

并将外部循环的条件值更改为i < n，以确保循环从头到尾。

Answer 4

内循环应该是

  for(j=i+1;j<n;j++)
  {
    if( list[j] < list[small] ) {
        small = j;
    }
  }

如果它们小不等于 I 变量则交换

  if(small != i)
  {
   swap();
  }