Java：LinkedList上的QuickSort给了我例外

Question

我正在学习考试（算法和数据结构），并且正在尝试使LinkedList quicksort工作，但它给了我ListIndexOutOfBoundsException 。

对于前一段时间的作业，我使用了straightinsertion插入对ArrayList和Vector进行排序，现在我想了解LinkedList QuickSor t（我在理论上是这样做的）。 我对linkedlist不太熟悉，但是它与ArrayList应该没有太大不同吗？

public class Sort {

public static void quickSort(LinkedList<Oseba> a) {
    sort(a, 0, a.size() - 1); // this is line 16
}

public static void sort(LinkedList<Oseba> a, int l, int r) {
    int i = l;
    int j = r;

    Oseba x = a.get((l + r) / 2), w;

    do {
        while (a.get(i).mlajsi(x)) {
            ++i;
        }
        while (x.mlajsi(a.get(j))) { // this is line 31
            --j;
        }
        if (i <= j) {
            w = a.get(i);
            a.set(i, a.get(j));
            a.set(j, w);
            ++i;
            --j;
        }
    } while (i <= j);

    if (l < j) {
        sort(a, l, j);
    }

    if (i < r) {
        sort(a, i, r);
    }
}
}

Oseba意思是“一个人”，这是我为测试各种方法（例如排序，比较）而制作的一门课

public class Oseba implements Comparable<Oseba> {

protected String priimekIme; //surnameName
protected int letoRojstva; //year of birth
protected Spol spol; //gender (enum)

public Oseba(String priimekIme, int letoRojstva, Spol spol) {
    this.priimekIme = priimekIme;
    this.letoRojstva = letoRojstva;
    this.spol = spol;
}

@Override
public int compareTo(Oseba o) {
    if (this.letoRojstva < o.letoRojstva) {
        return -1;
    } else if (this.letoRojstva > o.letoRojstva) {
        return 1;
    } else {
        return this.priimekIme.compareTo(o.priimekIme);
    }
}

public boolean mlajsi(Oseba o) { //younger
    return (o.letoRojstva - this.letoRojstva <= 0);
}

@Override
public String toString() {
    String s = priimekIme + ", " + spol.getKratko() + ", " + letoRojstva;
    return s;
}
}

这是我得到的错误：

Exception in thread "main" java.lang.IndexOutOfBoundsException: Index: -1, Size: 6
    at java.util.LinkedList.checkElementIndex(LinkedList.java:553)
    at java.util.LinkedList.get(LinkedList.java:474)
    at javaapplication1.Sort.sort(Sort.java:31)
    at javaapplication1.Sort.quickSort(Sort.java:16)
    at javaapplication1.JavaApplication1.main(JavaApplication1.java:55)
Java Result: 1

这种quicksort方法应该与Vector或ArrayList ，我不知道为什么它不与LinkedList ？

谢谢！

Answer 1

好吧，您不必在循环中检查边界。

   while (a.get(i).mlajsi(x)) {
        ++i;
    }
    while (x.mlajsi(a.get(j))) { // this is line 31
        --j;
    }

应该

   while (i <= r && a.get(i).mlajsi(x)) {
        ++i;
    }
    while (j >= l && x.mlajsi(a.get(j))) { // this is line 31
        --j;
    }

和

} while (i <= j);

严格来说，还应该考虑到i和j在边界之内（但我认为这不是必需的）。 它将解决异常问题，但是我没有验证算法的正确性。

Answer 2

Java（通常是OO）的一大规则是“代码是接口，而不是实现”。 现在，您正在编码到List接口的LinkedList实现。 确保此代码可与任何List（ Vector ， ArrayList等）一起使用的唯一方法是更改​​声明。 例如：

public static void quickSort(LinkedList<Oseba> a) {

应该成为

public static void quickSort(List<Oseba> a) {

并且与排序类似：

public static void sort(List<Oseba> a, int l, int r) {

现在，无论何时声明一个人，它都应如下所示：

List<Oseba> a = LinkedList<Oseba>();

但是，可以用LinkedList代替任何其他类型的列表。

这并不能回答您的代码为什么失败的问题-我认为UmNyobe的建议很好，尽管我没有对其进行测试-但是它回答了您的一个较小的问题，即为什么该代码的行为不像其他列表类型。 这是因为您要编码实现，因此应该在其中使用接口。