Question

我试图尝试实现自己的选择排序代码。到目前为止，代码排序工作......它表现得很奇怪。

class HelloWorld {
 public static void main(String[] args) {
    long startTime = System.currentTimeMillis();

    int a[] = {
        3,7,8,4,22,13,9,5,10
    };

    int max_pos = a.length-1;
    int counter = 0;
    int min_sort = 0;
    int j = -1;
    int min_pos = 0;
    int i=0;

    while (j < max_pos) {
        i=j+1;
        min_sort = a[i];
        // traverse trough unsorted portion
        while (i < max_pos) {
            i++;
            if (min_sort > a[i]) {
                min_sort  = a[i]; // minimum unsorted
                min_pos = i;
            }
        }

        // sorted portion
        j++;
        a[min_pos] = a[j];
        a[j] = min_sort ;
        for(int x=0; x < a.length; x++) {
            System.out.print(a[x] + ", ");
        }
        System.out.print("\t\t");
        System.out.println("i: " + i + " j: " + j + " min pos: " + min_pos + " min val: "+ min_sort);
    }
  }
 }

我试过跟踪它，输出如下：

3, 7, 8, 4, 22, 13, 9, 5, 10,           i: 8 j: 0 min pos: 0 min val: 3          
3, 4, 8, 7, 22, 13, 9, 5, 10,           i: 8 j: 1 min pos: 3 min val: 4          
3, 4, 5, 7, 22, 13, 9, 8, 10,           i: 8 j: 2 min pos: 7 min val: 5          
3, 4, 5, 7, 22, 13, 9, 7, 10,           i: 8 j: 3 min pos: 7 min val: 7          
3, 4, 5, 7, 7, 13, 9, 22, 10,           i: 8 j: 4 min pos: 7 min val: 7          
3, 4, 5, 7, 7, 9, 13, 22, 10,           i: 8 j: 5 min pos: 6 min val: 9          
3, 4, 5, 7, 7, 9, 10, 22, 13,           i: 8 j: 6 min pos: 8 min val: 10         
3, 4, 5, 7, 7, 9, 10, 13, 22,           i: 8 j: 7 min pos: 8 min val: 13         
3, 4, 5, 7, 7, 9, 10, 13, 22,           i: 8 j: 8 min pos: 8 min val: 22

Answer 1

由于这几乎肯定是您的教育过程（真正的代码只会调用Arrays.sort()），我最初不会给出直接答案。

相反，您应该注意到，j == 3似乎是8神奇地变身为7时的行。{/ p>

所以你应该开始磨练你的调试技巧。通过调试器运行代码直到前一行的输出结束，然后单步执行每一条指令。在某些时候，您会看到7成为8和您需要关注的代码。

一旦你完成了这项工作，你将有希望能够分辨出问题所在。但是，如果您仍然卡住（确保首先尝试，或者您永远不会改进），请参阅下文。吨---

这是对您的问题的详细分析。每次找到的元素小于在外部循环的每次迭代中设置的第一个元素时，您将存储有关最小值（值min_sort及其位置min_pos）的信息：

if (min_sort > a[i]) {
    min_sort  = a[i]; // minimum unsorted
    min_pos = i;
}

现在这是一个很好的方法，这两个信息后来用于交换过程^（1）：

j++;
a[min_pos] = a[j];
a[j] = min_sort;

但是，在这种情况下，找不到小于第一个元素的元素，例如下面的位置3保持值{{1超出该值的所有值都大于7：

在这种情况下，3, 4, 5, 7, 22, 13, 9, 8, 10 ^语句的主体将永远不会运行，并且迭代开始时的初始条件仍然适用：

if

注意那里缺少什么？比如，让我们看看，这个位置被设定为什么？

宾果！如果在迭代中检查的第一个元素已经处于正确的位置，i=j+1; min_sort = a[i];将保持设置为之前的任何值，并且当您交换时，这将不会很漂亮。您可以通过将外部min_pos循环内的代码更改为：

来解决此问题

while

你会看到输出的大大改进（即正确）：

i = j + 1;
min_pos = i;          // add this line
min_sort = a[i];

^（1）顺便说一下，当最小的剩余值已经在正确的位置时，交换是不必要的，尽管它没有任何危害。如果您想避免不必要的交换，可以使用：

3, 7, 8, 4, 22, 13, 9, 5, 10, i: 8 j: 0 min pos: 0 min val: 3
3, 4, 8, 7, 22, 13, 9, 5, 10, i: 8 j: 1 min pos: 3 min val: 4
3, 4, 5, 7, 22, 13, 9, 8, 10, i: 8 j: 2 min pos: 7 min val: 5
3, 4, 5, 7, 22, 13, 9, 8, 10, i: 8 j: 3 min pos: 3 min val: 7
3, 4, 5, 7, 8, 13, 9, 22, 10, i: 8 j: 4 min pos: 7 min val: 8
3, 4, 5, 7, 8, 9, 13, 22, 10, i: 8 j: 5 min pos: 6 min val: 9
3, 4, 5, 7, 8, 9, 10, 22, 13, i: 8 j: 6 min pos: 8 min val: 10
3, 4, 5, 7, 8, 9, 10, 13, 22, i: 8 j: 7 min pos: 8 min val: 13
3, 4, 5, 7, 8, 9, 10, 13, 22, i: 8 j: 8 min pos: 8 min val: 22

但是，如上所述，进行交换并不是一个问题，所以这取决于你。

奇怪的选择排序行为

1 个答案: