c ++合并排序不起作用并崩溃

Question

我试图通过以下方式在 C++ 中实现归并排序：

#include <iostream>
using std::cout;
using std::endl;
using std::string;
using std::memcpy;

void mergeSort(int *array, int *temp, int leftStart, int rightEnd);
void mergeHalves(int *array, int *temp, int leftStart, int rightEnd);

void printArray(int* array, int size,string message){
    cout<<message<<endl;
    for (int i = 0; i < size; i++)
        cout << *(array+i) << "\t";
    cout << endl;
}
int main(){
    const int size = 10;
    int myInts[size] = {9, 8, 7, 6, 5, 4, 3, 2, 1, 0};
    int temp[size] = {0};
    printArray(myInts,size,"Input");
    mergeSort(myInts, temp, 0, size-1);
    printArray(myInts, size, "Output:");

    return 0;
}

void mergeSort(int *array, int *temp, int leftStart, int rightEnd){
    if (leftStart >= rightEnd)
        return;

    int middle = (leftStart + rightEnd) / 2;

    mergeSort(array, temp, leftStart, middle);
    mergeSort(array, temp, middle + 1, rightEnd);
    mergeHalves(array,temp,leftStart,rightEnd);
    return;
}

void mergeHalves(int *array, int *temp, int leftStart, int rightEnd){
    int leftEnd = (rightEnd + leftStart) / 2;
    int rightStart = leftEnd + 1;
    int size = rightEnd - leftStart + 1;

    int left = leftStart;
    int right = rightStart;
    int index = leftStart;

    while (left <= leftEnd && right <= rightEnd){
        if (*(array+left) <= *(array+right)){
            *(temp+index) = *(array+left);
            left++;
        }else{
            *(temp+index) = *(array+right);
            right++;
        }
        index++;
    }

    memcpy(temp + index, array + left, (leftEnd - left + 1) * sizeof(int));
    memcpy(temp + index, array + right, (rightEnd - right + 1) * sizeof(int));
    memcpy(array, temp, size * sizeof(int));
}

我检查代码的次数越来越多，但我无法理解我缺少什么。
该算法无法正常工作，初始值为

myInts[size] = {3, 6, 4, 2, 7, 8, 9, 1, 0, 5}

因为它只对数组的前半部分进行了排序。
然后我尝试用代码中显示的输入来改变输入，我得到了以下输出：

Input
9   8   7   6   5   4   3   2   1   0   
Output:
3   2   1   0   -161742639  6   5   4   7   8   
Abort trap: 6

过了一会儿，我重新编译并运行了相同的代码并得到了这个输出：

Input
9   8   7   6   5   4   3   2   1   0   
Output:
3   2   1   0   6   5   4   7   8   9

在我看来，某些赋值操作或复制操作也随机出错，但对我来说一切似乎都是正确的..

我已经尝试逐步调试，并且程序的流程似乎没问题。

我在 MacOS X Sierra 10.12.6 上运行，我正在使用 g++ 进行编译。

我错过了什么？我快疯了..提前谢谢！

编辑：我似乎已经解决了Abort Trap 问题，我在main 函数中编辑了传递size-1 而不是size 的代码。但仍然无法订购

Answer 1

在为您进行大量调试后，我发现了您的问题。您通过不同的函数将多个变量命名为同一事物。其中一些变量的含义完全相同（leftStart、rightEnd），而其他变量（例如 size）则表示完全不同的两个含义。

在mergHalves 的末尾，您使用memcpy(array, temp, size * sizeof(int)); 可能是因为您认为size 是整个数组的大小，因此您将所有内容复制回来。但是，它不是。

那行应该是memcpy(array+leftStart, temp+leftStart, size * sizeof(int));，因为在这个函数中你已经声明了大小为int size = rightEnd - leftStart + 1;。这是一个很好的例子，说明为什么让变量名更长、更具描述性可以为您节省几个小时的调试时间（并且必须在 StackOverflow 上发布）。

在ideone 上查看正在执行的修复。

最后，顺便说一句，我知道您想尝试在没有矢量的情况下“艰难地”执行此操作，但请永远不要做类似*(temp+index) 的事情。 subscript operator 正是为了这个目的而制作的，它意味着完全相同的东西，只是更具可读性。 （即，改为使用temp[index]）。