两个排序算法在同一个数组(quickSort和heapSort)上给我两个不同的输出!

时间:2014-03-25 15:50:04

标签: c++ sorting

我不明白他们编译时为什么给我不同的输出。例如......当我只编译一个算法时,答案是好的,另一个算法也是如此,但是当我同时编译它们时,它们会给我一些奇怪的输出。

我的代码:

#include <iostream>
using namespace std;


int parent(int i){
    return i/2;
}
int leftChild(int i){
    return 2*i+1;
}
int rightChild(int i){
    return 2*i+2;
}
void maxHeapify(int a[], int i, int n){
    int largest;
    int temp;
    int l = leftChild(i);
    int r = rightChild(i);
    //   p.countOperation("CMPbottomUp",n);
    if (l <= n && (a[l] > a[i]))
        largest = l;
    else
        largest = i;
    //      p.countOperation("CMPbottomUp",n);
    if (r <= n && (a[r] > a[largest]))
        largest = r;
    if (largest != i){
        //    p.countOperation("ATTbottomUp",n);
        temp = a[i];
        //  p.countOperation("ATTbottomUp",n);
        a[i] = a[largest];
        //p.countOperation("ATTbottomUp",n);
        a[largest] = temp;
        maxHeapify(a, largest, n);
    }
}

void buildMaxHeap(int a[], int n){
    for (int i=n/2; i>=0; i--){
        maxHeapify(a, i, n);
    }
}
void heapSort(int a[],int n){
    buildMaxHeap(a,n);
    int n1=n;
    int temp;
    for(int i=n1;i>0;i--){
        temp = a[0];
        a[0] = a[i];
        a[i] = temp;
        n1--;
        maxHeapify(a,0,n1);
    }

}

int partitionArray(int arr[], int left, int right){
    int i = left, j = right;
    int tmp;
    int pivot = arr[(left + right) / 2];
    while (i <= j) {
        while (arr[i] < pivot)
            i++;
        while (arr[j] > pivot)
            j--;
        if (i <= j) {
            tmp = arr[i];
            arr[i] = arr[j];
            arr[j] = tmp;
            i++;
            j--;
        }
    }
    return i;
}

void quickSort(int arr[], int left, int right) {
    int index;
    index = partitionArray(arr, left, right);
    if (left < index - 1)
        quickSort(arr, left, index - 1);
    if (index < right)
        quickSort(arr, index, right);
}

int main(){
    int x[8]= {5,87,21,4,12,7,44,3};
    int a[8];
    for(int i=0;i<8;i++){
        a[i] = x[i];
    }
    heapSort(x,8);
    quickSort(a,0,8);

    for(int i=0;i<8;i++){
        cout<<a[i]<<' ';
    }
    cout<<endl;

    for(int j=0;j<8;j++){
        cout<<x[j]<<' ';
    }

    return 0;
}

示例输出:

1)当我只编译一个算法时,输出为:3,4,5,7,12,21,44,87(这很好)

2)当我在代码中编译它们时,输出为:87,4,5,7,12,21,44,87(quickSort)和3,3,4,5,7,12,21 ,44(heapSort)

2 个答案:

答案 0 :(得分:0)

阵列ax在堆栈中彼此相邻。看看你在输出中如何重复值87,你的排序函数似乎访问你给它们的数组之外的内存。这是缓冲区溢出,一种未定义的行为。有了这个,你的代码可以做任何事情,因为你已经损坏了变量值(或者更糟糕的是,地址/指针已损坏)。

仔细检查您如何访问数组。请记住,长度为8的数组的C数组索引是0..7!

答案 1 :(得分:0)

我认为应该有效:

heapSort(x,7);
quickSort(a,0,7);