A股上市公司传智教育(股票代码 003032)旗下技术交流社区北京昌平校区

 找回密码
 加入黑马

QQ登录

只需一步,快速开始

© 十字路口 中级黑马   /  2013-6-1 00:32  /  1522 人查看  /  4 人回复  /   0 人收藏 转载请遵从CC协议 禁止商业使用本文

本帖最后由 十字路口 于 2013-6-1 10:13 编辑

public class Quicksort{
public static int partion(int values[],int left,int right){
  int i=left;
  int j=right;
  int temp;
  int key=values[left];
  while(i<j){
   while(values<=key){
    i++;
   }
   while(values[j]>=key){
    j--;
   }
   if(i<=j){
    temp=values;
    values=values[j];
    values[j]=temp;
    i++;
    j--;
   }
  }
  return i;
}
public static  void qsort(int values[],int left,int right){
  int temp=0;
  if(left<right){
   temp=partion(values, left, right);
   qsort(values, left, temp-1);
   qsort(values, temp+1, right);
  }
}
public static void main(String[] args) {
  int a[]={34,43,6,12,78,37,92,14};
  qsort(a, 0, a.length-1);
  for(int i=0;i<a.length;i++){
   System.out.print(a+"  ");
  }
}
}
为什么这是个死循环。。。 while(values<=key){
i++;
}
while(values[j]>=key){
j--;
}
把段代码改成
while(values<key){
i++;
}
while(values[j]>key){
j--;
}
结果又不正确为什么呢?

评分

参与人数 1技术分 +1 收起 理由
袁梦希 + 1 很给力!

查看全部评分

4 个回复

倒序浏览
首先这里while(values<=key)可以写等于,但是还缺少一个判断,就是i<j,因为如果不加这个判断,角标肯定会出界,所以运行时会出现错误。
这里可以写成while(i<j&&values<=key)
还有  if(i<=j){
     temp=values;
     values=values[j];
     values[j]=temp;
     i++;
     j--;
    }
   这里不能这么写,比如只有三个数,你去代入下就知道肯定会排出错误结果了

我之前写的一段代码,你可以看一下:
  1. /*
  2. 需求:使用快速排序的方法对数组进行升序排列
  3. 思路:1、快速排序是先利用数组的一个元素为参照,也叫枢轴。从左右两边分别与其比较,这是可以定义角标变量,左角标变量向右移,右的向左右,知道左大于等于右时,完成第一轮排序比较。
  4.                 第一轮比较后将大于这个元素的数放到右边,小于这个元素的数放到左边
  5.           2、再对左右两边根据第一步的思想进行排序。如:把左边的的部分当作一个数组,再从中选一个元素做参照。以此反复。
  6. */
  7. class QuickSort
  8. {       
  9.         public static void quickSort(int[] arr ,int left,int right)
  10.         {
  11.                 int l=left,r=right,pivot=arr[l];//pivot定义枢轴

  12.                 while(l<r)//判断一轮是否进行完了
  13.                 {
  14.                         while(l<r && arr[r]>=pivot)//判断右边的数是否都大于pivot
  15.                                 r--;

  16.                         //如果小于pivot,就将小于的元素放到左边去
  17.                         if(l<r)
  18.                         {
  19.                                 arr[l]=arr[r];
  20.                                 l++;//l++是为了避免重复判断
  21.                         }

  22.                         while(l<r && arr[l]<=pivot)//判断左边的数是否都小于pivot
  23.                                 l++;

  24.                         //如果大于privot,就将大于的元素放到右边去
  25.                         if(l<r)
  26.                         {
  27.                                 arr[r]=arr[l];
  28.                                 r--;
  29.                         }       
  30.                 }

  31.                 arr[l]=pivot;//将作为参照的元素放到分割位置
  32.                        
  33.                 //当一轮比较完,l!=left,说明左边还有大于1个元素,那么继续排序
  34.                 if(l>left)
  35.                 {
  36.                         quickSort(arr,left,l-1);
  37.                 }

  38.                 //当一轮比较完,r!=right,说明右边边还有大于1个元素,那么继续排序
  39.                 if(r<right)
  40.                 {
  41.                         quickSort(arr,l+1,right);
  42.                 }
  43.         }


  44.         //遍历数组
  45.         public static void printArray(int[] arr)
  46.         {
  47.                 System.out.print("[");
  48.                 for(int x=0;x<arr.length-1;x++)
  49.                         System.out.print(arr[x]+",");
  50.                 System.out.println(arr[arr.length-1]+"]");
  51.         }
  52. }


  53. class Test
  54. {
  55.         public static void main(String[] args)
  56.         {
  57.                 int[] arr1={1,2};
  58.                 int[] arr2={34,43,6,12,78,37,92,14};
  59.                 int[] arr3={2,2,2,10,2,3,3,3,3,2,2,2};

  60.                 QuickSort.printArray(arr1);
  61.                 QuickSort.quickSort(arr1,0,arr1.length-1);
  62.                 QuickSort.printArray(arr1);

  63.                 QuickSort.printArray(arr2);
  64.                 QuickSort.quickSort(arr2,0,arr2.length-1);
  65.                 QuickSort.printArray(arr2);

  66.                 QuickSort.printArray(arr3);
  67.                 QuickSort.quickSort(arr3,0,arr3.length-1);
  68.                 QuickSort.printArray(arr3);

  69.         }
  70. }
复制代码

评分

参与人数 1技术分 +1 收起 理由
袁梦希 + 1 很给力!

查看全部评分

回复 使用道具 举报
首先,楼主的代码从头到尾没有对第一个数组元素的操作,仔细分析一下34的位置永远在首位
我跟着程序的思路理了一个程序运行流程给楼主,至于这个问题怎么解决,代码重构了
首先:34,43,6,12,78,37,92,14
第一次排序 i =1, j = 6;交换i,j的位置得到34,92,6,12,78,37,43,14,;i++后i=2,j--后j=5,返回2
第二次排序为前一截(values, 0, 1)就是给34,92排序,这里面不会执行交换,但是因为34<92,i会自加两下,所以最后返回的是i=2;
然后就无限重复第二步、、

这个问题出现的关键在于楼主代码没有对用作标杆的元素换位置,快速排序的核心思路就是在每一轮排序后,标杆元素在中间,左边全比它小,右边全比它大。然后再对左右分别进行排序。
下面是一个比较简洁的快速排序代码,希望能帮到你
  1. /**
  2.      * 交换指定数组a的两个变量的值
  3.      * @param a 数组应用
  4.      * @param i 数组下标
  5.      * @param j 数组下标
  6.      */
  7.     public void swap(int a[], int i, int j) {
  8.       
  9.         if(i == j) return;

  10.         int tmp = a[i];

  11.         a[i] = a[j];

  12.         a[j] = tmp;

  13.     }

  14.     /**
  15.      *
  16.      * @param array 待排序数组
  17.      * @param low 数组下标下界
  18.      * @param high 数组下标上界
  19.      * @return pivot
  20.      */
  21.     public int partition(int array[], int low, int high) {
  22.         //当前位置为第一个元素所在位置
  23.         int p_pos = low;
  24.         //采用第一个元素为轴
  25.         int pivot = array[p_pos];
  26.         
  27.         for (int i = low + 1; i <= high; i++) {

  28.             if (array[i] < pivot) {            
  29.                
  30.                 p_pos++;

  31.                 swap(array, p_pos, i);

  32.             }

  33.         }

  34.         swap(array, low, p_pos);

  35.         return p_pos;

  36.     }
  37.     /**
  38.      * 快速排序实现
  39.      * @param array
  40.      * @param low
  41.      * @param high
  42.      */
  43.     public void quickSort(int array[], int low, int high) {

  44.         if (low < high) {

  45.             int pivot = partition(array, low, high);

  46.             quickSort(array, low, pivot - 1);

  47.             quickSort(array, pivot + 1, high);

  48.         }

  49.     }
复制代码

点评

哥们,好犀利!学习了!  发表于 2013-6-1 14:56

评分

参与人数 1技术分 +1 收起 理由
袁梦希 + 1 神马都是浮云

查看全部评分

回复 使用道具 举报 1 0
灰常感谢各位大虾们,看了你们的提示,学到了很多。问题解决了。。。。。。
回复 使用道具 举报
您需要登录后才可以回帖 登录 | 加入黑马