java - heapsort 的一部分发生了变化并且不起作用
问题描述
我已经完成了堆排序的代码,效果很好。但是当我改变函数heapify中的部分时()
int largest = left + 1 < size && arr[left + 1] > arr[left] ? left + 1 : left;
largest = arr[largest] > arr[index] ? largest : index;
至
int largest = (left + 1) < size && arr[left +1 ] < arr[left] ? left : (left + 1);
largest = arr[largest] > arr[index]? largest : index;
然后代码不起作用。我认为这两个部分是相同的。我不知道是什么问题。我已经检查过很多次了。
import java.util.Arrays;
public class Code_03_HeapSort {
public static void heapSort(int[] arr) {
if (arr == null || arr.length < 2) {
return;
}
for (int i = 0; i < arr.length; i++) {
heapInsert(arr, i);
}
int size = arr.length;
swap(arr, 0, --size);
while (size > 0) {
heapify(arr, 0, size);
swap(arr, 0, --size);
}
}
public static void heapInsert(int[] arr, int index) {
while (arr[index] > arr[(index - 1) / 2]) {
swap(arr, index, (index - 1) / 2);
index = (index - 1) / 2;
}
}
// 1
public static void heapify(int[] arr, int index, int size) {
int left = index * 2 + 1;
while (left < size) {
int largest = left + 1 < size && arr[left + 1] > arr[left] ? left + 1 : left;
largest = arr[largest] > arr[index] ? largest : index;
if (largest == index) {
break;
}
swap(arr, largest, index);
index = largest;
left = index * 2 + 1;
}
}
public static void swap(int[] arr, int i, int j) {
int tmp = arr[i];
arr[i] = arr[j];
arr[j] = tmp;
}
// for test
public static void comparator(int[] arr) {
Arrays.sort(arr);
}
// for test
public static int[] generateRandomArray(int maxSize, int maxValue) {
int[] arr = new int[(int) ((maxSize + 1) * Math.random())];
for (int i = 0; i < arr.length; i++) {
arr[i] = (int) ((maxValue + 1) * Math.random()) - (int) (maxValue * Math.random());
}
return arr;
}
// for test
public static int[] copyArray(int[] arr) {
if (arr == null) {
return null;
}
int[] res = new int[arr.length];
for (int i = 0; i < arr.length; i++) {
res[i] = arr[i];
}
return res;
}
// for test
public static boolean isEqual(int[] arr1, int[] arr2) {
if ((arr1 == null && arr2 != null) || (arr1 != null && arr2 == null)) {
return false;
}
if (arr1 == null && arr2 == null) {
return true;
}
if (arr1.length != arr2.length) {
return false;
}
for (int i = 0; i < arr1.length; i++) {
if (arr1[i] != arr2[i]) {
return false;
}
}
return true;
}
// for test
public static void printArray(int[] arr) {
if (arr == null) {
return;
}
for (int i = 0; i < arr.length; i++) {
System.out.print(arr[i] + " ");
}
System.out.println();
}
// for test
public static void main(String[] args) {
int testTime = 500000;
int maxSize = 100;
int maxValue = 100;
boolean succeed = true;
for (int i = 0; i < testTime; i++) {
int[] arr1 = generateRandomArray(maxSize, maxValue);
int[] arr2 = copyArray(arr1);
heapSort(arr1);
comparator(arr2);
if (!isEqual(arr1, arr2)) {
succeed = false;
break;
}
}
System.out.println(succeed ? "Nice!" : "Fucking fucked!");
int[] arr = generateRandomArray(maxSize, maxValue);
printArray(arr);
heapSort(arr);
printArray(arr);
}
}
解决方案
int largest = left + 1 < size && arr[left + 1] > arr[left] ? left + 1 : left;
largest = arr[largest] > arr[index] ? largest : index;
和
int largest = (left + 1) < size && arr[left +1 ] < arr[left] ? left : (left + 1);
largest = arr[largest] > arr[index]? largest : index;
不相等。想象一下arr[left + 1]
是1
和arr[left]
也是1
。在第一种情况下largest
会left
在第二种情况下largest
会是(left + 1)
。这会导致您的索引关闭。
免责声明:我没有用代码对此进行测试。只是指出您更改了“等于”案例的行为
推荐阅读
- node.js - Mailgun 批量发送不适用于个性化数据
- sql - 如何将计算选择的结果保存在列中?
- c++ - 将动态二维数组作为参数传递给 c++ 中的函数
- flutter - Flutter - MissingPluginException(在通道 plugins.flutter.io/image_picker 上找不到方法 pickImage 的实现)
- javascript - 如何更换 PyV8,因为 PyV8 不再更新
- javascript - 从字典对象中的键获取所有值
- flutter - 高架按钮主题数据边框 - Flutter
- angular - 在 Effect 中获取商店项目
- java - Java类库中的VMSystem是什么?
- node.js - 在heroku中部署错误,拒绝错误