Question

已经花了好几个小时才发现这个代码有什么问题，而且我用google搜索的所有mergesort的算法看起来都差不多，这个算法不起作用; 试图将mergesort做到指针数组：

输出： 2 | 2 | 3 | 3 | 4 | 5 | 4 | 4 | 7 | 7 |

#include <stdlib.h>
#include <stdio.h>
void merge( int ** pointerArray, int pivot,int size){
int leftSize=pivot;
int rightSize=size;
 int leftIndex=0, rightIndex=pivot, newIndex=0;

int **tempArray= (int **)malloc(sizeof(int *)*size);

while (leftIndex < leftSize && rightIndex < rightSize) {//merging two sorted array

    if ((*pointerArray)[leftIndex]<=(*pointerArray)[rightIndex] )
        (tempArray)[newIndex++]=(pointerArray)[leftIndex++];
    else
        (tempArray)[newIndex++]=(pointerArray)[rightIndex++];



}

while (leftIndex<leftSize )//rest of left array
    (tempArray)[newIndex++]=(pointerArray)[leftIndex++];

while (rightIndex<rightSize)//rest of right array
    (tempArray)[newIndex++]=(pointerArray)[rightIndex++];


    for (int i=0; i<size; i++)//copying the tempArray to the original array
    pointerArray[i]=tempArray[i];


free(tempArray);

}
void mergeSort( int ** ptrarray, int size){
if (size==1)
    return;
int pivot=size/2;
mergeSort(ptrarray, pivot);
mergeSort(ptrarray+pivot, size-pivot);
merge(ptrarray, pivot, size);



}

int** pointerSort(int* arr, unsigned int size,
              char ascend_flag)
{
int **pointerArray= (int **)malloc(sizeof(int *)*size);

for (int i=0; i<size; i++)
    pointerArray[i]=&arr[i];

mergeSort(pointerArray, size);

return pointerArray;
}

int  main()
{
int size=10;
int array[10]={3,2,3,5,4,7,2,7,4,4};
char ascend_flag =1;
int ** pointer= pointerSort(array, size, ascend_flag);

for (int **p=pointer; p-pointer<size; p++)//print sorted array
    printf(" %d|", *(*p));


}

Answer 1

虽然不清楚为什么你要对指针数组进行排序而不是仅仅复制整数，但它肯定会让你误入歧途。当你说：

if ((*pointerArray)[leftIndex] <= (*pointerArray)[rightIndex])

你没有比较你想要的东西。

*pointerArray与pointerArray[0]或列表中第一个int *相同。您正在将pointerArray[0][leftIndex]与pointerArray[0][rightIndex]进行比较，当您想要的实际上是pointerArray[leftIndex][0]和pointerArray[rightIndex][0]时，就像这样：

if (*(pointerArray[leftIndex]) <= *(pointerArray[rightIndex]))

工作示例：https://ideone.com/Kgl3oO

p.s。这个数组在您的版本中显示出类似排序的事实是一个幸福的事故。尝试使用更混合的初始数组，例如：

int array[10] = {97, 4, 2, 107, 6, 7, 14, 22, 89, 6};

你会看到像

这样的结果

4| 97| 7| 14| 22| 6| 2| 6| 107| 89|

Mergesort不起作用

1 个答案: