2013-03-22 29 views
0

我花了很多时间试图找出我没有的权利;调试和所有,但我似乎无法把我的手指,为什么我的快速排序错过了一些排序项目。快速排序除了几个项目之外

我已经将我的代码复制到下面的排序和分区。我有一种感觉,这是我看过的非常明显的东西,但我花了无数时间来调试,研究和重写我的代码,结果总是如此。

// quicksort the subarray from a[lo] to a[hi] 
private void sort(Comparable[] a, int lo, int hi) { 
    if((hi-lo)>1){ 
     int pivot = partition(a,lo,hi); 
     sort(a,lo,pivot); 
     sort(a,pivot+1,hi); 
    } 
} 

// partition the subarray a[lo .. hi] by returning an index j 
// so that a[lo .. j-1] <= a[j] <= a[j+1 .. hi] 
private int partition(Comparable[] a, int lo, int hi) { 
    //find middle 
    int pivotIndex = (hi+lo)/2; 
    //pick pivot 
    Comparable pivot = a[pivotIndex]; 
    //create left and right pointers 
    int left = lo, right = hi; 
    //start comparing 
    //compare until left passes right 
    while(left<right){ 
     //while left is less than pivot move on 
     while(a[left].compareTo(pivot) < 0) left++; 
     //while right is greater than pivot move on 
     while(a[right].compareTo(pivot) > 0) right--; 
     //if the pointers have passed each other we're done 
     //if a[left] is greater than a[right] swap them 
     if(a[left].compareTo(a[right]) > 0){ 
      Comparable holder = a[left]; 
      a[left] = a[right]; 
      a[right] = holder; 
      //increment/decrement 
      left++; right--; 
     } 
    } 
    return right; 
} 
+0

是javascript吗? :p – 2013-03-22 23:24:00

+0

@GrzegorzKaczan很确定它不是。重新签署到Java。 – 2013-03-22 23:29:36

回答

0

只要删除left++; right--;,一切都应该没问题。

+0

我还没有发现自己的错误,但我运行了一个测试用例,对'Character'[] s = {'z','y','x','a','b','c',' w','v','u'};'并删除'left ++;正确 - ;'没有纠正这个问题。 – Simon 2013-03-23 00:50:56

+0

很奇怪..我再次想到了一点,我猜想有关递归调用的问题出错了。但我无法直接看到故障。 只需看看http://www.vogella.com/articles/JavaAlgorithmsQuicksort/article.html(与您的实现非常相似)的实现,但是在分区部分内或多或少地执行递归。 – fish 2013-03-23 12:23:34