Question

我从大学开始编写一个程序，它接收一个整数数组及其大小，并将一个排序的指针数组返回到原始数组的值。必须使用mergeSort算法完成排序，并且我不允许触摸原始数组，只能更改指针数组。

任何人都可以帮助我吗？这是我的代码：

void sortedPointerArray (int **arr1[], int size1, int **arr2[], int size2, int **arr3[])
{
    int read1, read2, writeind;

    read1 = read2 = writeind = 0;
    while( (read1<size1) && (read2 < size2))
    {
            if (*(arr1[read1]) < *(arr2[read2]))
            {
                    arr3[writeind] = arr1[read1];
                    writeind++;
                    read1++;
            }
            else
            {
                    arr3[writeind] = arr2[read2];
                    writeind++;
                    read2++;
            }

    }
    while (read1 < size1)
    {
            arr3[writeind] = arr1[read1];
            writeind++;
            read1++;
    }
    while (read2 < size2)
    {
            arr3[writeind] = arr2[read2];
            writeind++;
            read2++;
    }
}

void mergeSort (int **arr, int size)
{
    int temp, i;

    if(size <=1)
            return;
    else
    {
            mergeSort(arr, size/2);
            mergeSort(arr+(size/2), size-(size/2));
            int** temp = (int **)malloc(size*sizeof(int *));
            sortedPointerArray(arr,size/2,arr+(size/2),size-(size/2),temp);
            for (i=0; i<size; i++)
                    (*arr)[i] = (*temp)[i];
            free(temp);
    }
}

由于某些未知原因我无法编译它并且它不起作用...任何帮助将不胜感激！

Answer 1

sortedPointerArray只需要接收int []，而不是int**[]。目前，在合并排序的数组时，它会比较指针而不是整数。 mergeSort应该更改调用代码指向的数组，因此需要int**（或c ++中的int*&）。 mergeSort也应该避免释放完全排序的数组。

在一个不相关的说明中，我认为您的代码可以使用更好的变量名称。例如，arr1，arr2和arr3可以分别替换为in1，in2和out。您还可以使用const注释数组，以降低意外更改输入数组的风险。

Answer 2

#include <stdlib.h>
#include <stdio.h>

void mergeSort (int **arr, size_t size);
void sortedPointerArray (int **arr1, size_t size1, int **arr2, size_t size2, int **arr3);

void sortedPointerArray (int **arr1, size_t size1, int **arr2, size_t size2, int **arr3)
{
    size_t read1, read2, writeind;

    for(read1=read2=writeind=0; read1 < size1 && read2 < size2; )
            arr3[writeind++] = (*arr1[read1] < *arr2[read2])
                   ? arr1[read1++]
                   : arr2[read2++]
                   ;

    while (read1 < size1)
            arr3[writeind++] = arr1[read1++];

    while (read2 < size2)
            arr3[writeind++] = arr2[read2++];
}

void mergeSort (int **arr, size_t size)
{
    size_t idx;
    int **temp ;

    if(size < 2) return;

    mergeSort(arr, size/2);
    mergeSort(arr+(size/2), size-(size/2));

    temp = malloc(size*sizeof *temp);
    sortedPointerArray(arr,size/2,arr+(size/2),size-(size/2),temp);
    for (idx=0; idx<size; idx++)
            arr[idx] = temp[idx];
    free(temp);
}


#define COUNTOF(a) (sizeof a/ sizeof a[0])
int array[] = { 3,2,4,5,9,8,7,1,6,0};

int *ptrs[COUNTOF( array)];

int main(void)
{
unsigned idx;

for(idx=0; idx < COUNTOF( array); idx++ ) ptrs[idx] = array+idx;

mergeSort(ptrs, COUNTOF(ptrs) );

for(idx=0; idx < COUNTOF(ptrs); idx++ ) {
        printf("%u: %p %d\n", idx, (void*) ptrs[idx], *ptrs[idx] );
        }
return 0;
}

输出：

0: 0x601064 0
1: 0x60105c 1
2: 0x601044 2
3: 0x601040 3
4: 0x601048 4
5: 0x60104c 5
6: 0x601060 6
7: 0x601058 7
8: 0x601054 8
9: 0x601050 9

评论/更改：

删除了令人困惑的（和错误的）额外级别的间接
删除了额外的括号
删除了malloc
压缩合并循环以使用dst[aaa++] = src[bbb++];
将所有索引更改为unsigned（size_t）类型。
在if (size <2) return;
添加了一个驱动程序。
删除了大括号并引入了三元运算符来挫败老师

指针数组上的MergeSort实现

2 个答案: