Question

我为了好玩而编写了一个简单的插入排序算法。它似乎在一个问题上运行得很好，它只适用于数组中的所有项目（它试图排序的东西）是单个数字整数。如果一个元素是一个多位整数，它会将所有内容排序到该整数，然后停止并抛出IndexOutOfBoundsException.

注意：只是让你理解源代码，我的程序的工作方式是这样的：我将InsertionSort类传递给构造中的原始int数组。然后我将其转换为ArrayList list。我知道ArrayList list已经复制了正确的值，因为我将其打印出来并匹配起来。

示例运行：

工作：

{ 9, 2, 8, 5, 1, 6, 6, 7, 1}; //works perfectly - ＆gt;

Unsorted List: [9, 2, 8, 5, 1, 6, 6, 7, 1]
Step 1: [9]
Step 2: [2, 9]
Step 3: [2, 8, 9]
Step 4: [2, 5, 8, 9]
Step 5: [1, 2, 5, 8, 9]
Step 6: [1, 2, 5, 6, 8, 9]
Step 7: [1, 2, 5, 6, 6, 8, 9]
Step 8: [1, 2, 5, 6, 6, 7, 8, 9]
Step 9: [1, 1, 2, 5, 6, 6, 7, 8, 9]
Sorted List: [1, 1, 2, 5, 6, 6, 7, 8, 9]

不工作：

{ 9, 2, 8, 5, 1, 6, 6, 7, 1, 22, 823, 30, 244, 45, 5}; //doesn't work - ＆gt;

Unsorted List: [9, 2, 8, 5, 1, 6, 6, 7, 1, 22, 823, 30, 244, 45, 5]
Step 1: [9]
Step 2: [2, 9]
Step 3: [2, 8, 9]
Step 4: [2, 5, 8, 9]
Step 5: [1, 2, 5, 8, 9]
Step 6: [1, 2, 5, 6, 8, 9]
Step 7: [1, 2, 5, 6, 6, 8, 9]
Step 8: [1, 2, 5, 6, 6, 7, 8, 9]
Step 9: [1, 1, 2, 5, 6, 6, 7, 8, 9]
java.lang.IndexOutOfBoundsException: Index: 9, Size: 9
Sorted List: [1, 1, 2, 5, 6, 6, 7, 8, 9]
    at java.util.ArrayList.rangeCheck(Unknown Source)
    at java.util.ArrayList.get(Unknown Source)
    at InsertionSort.sort(InsertionSort.java:41)
    at Executer.main(Executer.java:7)

以下是相关的排序代码：

static ArrayList<Integer> list = new<Integer> ArrayList();
static ArrayList<Integer> list2 = new<Integer> ArrayList();

public static int[] sort() {
    int s = 0;
    System.out.println("Unsorted List: " + list.toString());

    try {
        for (int i = 0; i < list.size(); i++) {
        s++;
            if (i == 0) {
                list2.add(list.get(i));
                System.out.println("Step " + s + ": " + list2.toString());
                continue;

            } else {
                int z = 0;
                while (list2.get(z) < list.get(i)) {
                    z++;
                }

                list2.add(z, list.get(i));

            }
            System.out.println("Step " + s + ": " + list2.toString());
        }

    } catch (Exception e) {
        e.printStackTrace();

    }
    System.out.println("Sorted List: " + list2.toString());

    return toPrimative(list.toArray(new Integer[list.size()]));
}

Answer 1

介意调试。问题出在这里：

while (list2.get(z) < list.get(i)) {
                    z++;
                }

如果新值最大，则

z超过list2计数少数例子中的第一个值9隐藏了这个问题。

我希望解决方案足够清楚

Answer 2

它与单个或多个数字无关，在第一个例子中，您很幸运将最大数字作为列表中的第一个元素。当你为下一个元素选择一个位置时，有一个明显的错误：

int z = 0;
while (list2.get(z) < list.get(i)) {
    z++;
}

你增加索引但不进行边界检查，所以当遇到第22个时你得到一个例外。正确的是

while (list2.get(z) < list.get(i) && z <= list2.size())

为什么我的插入排序算法只适用于单个数字整数？

2 个答案: