java 递归插入排序

Question

我有一个数组，必须使用插入排序对它们进行排序。我尝试使用 CompareTo 方法来遍历数组并查看更大的值。我遇到了一个问题，因为我试图用一个显然不起作用的字符串引用数组索引（即在compareTo(a[key])）。

任何有关如何执行此操作的建议或提示将不胜感激。

这就是我到目前为止所拥有的。这是一个好的开始吗？还是朝着正确的方向开始？

 public void insertionSort() 
    { 
        insertionSort(a.length-1);
    } 



    private void insertionSort(int n)
    {
        String temp; 
        if(n <= 1)
        {
        //Do nothing, easiest case
        }

        else
        {
        for(int i = 1; i < a.length; i++)
        {
        int j;
        String key = a[i];

            while((j >= 0) && (a[i].compareTo(a[key]) > 0))
            {
            a[i+1] = a[i];
            j--;
            }
            a[i+1] = key;
        }   

        insertionSort(n-1);

        }
    } 

Answer 1

我的第一个建议是，如果传递所需的参数，通常会更容易理解方法。完全不清楚 a 是什么；我希望公共 insertionSort 方法将要排序的对象作为参数。 （我想如果你在自己的类似列表的类上定义它，它并没有那么糟糕，但听起来不像是这样的用法）。

同样，我也不完全确定 n 应该是什么（大概是您知道已排序的索引），但您根本不在私有方法的主体中使用它，所以您只是做同样的事情n次。

您似乎还交换了 a 的元素，而在插入排序中不需要这样做。这看起来更像是冒泡排序。

首先尝试将方法编写为伪代码（例如注释）以布局您的方法，然后用一小段代码充实每个注释。这样做可以避免过于陷入细节，通常概念性错误会显得更加明显，也更容易避免。这可能看起来像：

public static int[] insertionSort(int[] input) {
    // Create the array to hold the results

    // Iterate through the contents of input and insert each one into
    // the result array in order (method call here)

    // return the result array
}

private void insertOneElement(int toInsert, int[] resultArray) {
    // Compare toInsert with first element of resultArray

    // etc.
}

Answer 2

替换内循环如下：

j = i - 1;  //initialize j!!!
String key = a[i];   //take value
while((j >= 0) && (a[j].compareTo(key) > 0)){//Compare with key!!!
     a[j+1] = a[j];
     j--;
}
a[j + 1] = key; //index is j here!!!!

Answer 3

只需将其更改为 a[j].compareTo(key) （请注意，您要比较 a[j]，而不是 a[i]）。您还需要初始化 j，正如 smas 所评论的那样。