Question

我写小项目来实施合并排序。这是我的代码：

template <typename T>
void merge(T arr[], int begin, int mid, int end)
{
    int len = end - begin + 1;
    T *temp = new T[len];
    int i = begin;
    int j = mid + 1;
    int k = 0;
    while (i <= mid && j <= end)
    {
        if(arr[i] <= arr[j])
            temp[k++] = arr[i++];
        else
            temp[k++] = arr[j++];
    }
    while (i <= mid)
        temp[k++] = arr[i++];
    while(j <= end)
        temp[k++] = arr[j++];

    memcpy(arr + begin, temp, len*sizeof(T));
}


//merge sort
template <typename T>
void mergeSort(T arr[], int begin, int end)
{
    if (begin >= end)
        return;

    int mid = (end + begin) / 2;
    mergeSort(arr, begin, mid);
    mergeSort(arr, mid + 1, end);
    merge(arr, begin, mid, end);
}

int main()
{
    const int N = 10;
    int arr[N];
    for_each(arr, arr + N, [](int &val){ val = rand() % 100; });

    copy(arr, arr+N, ostream_iterator<int>(cout, " "));
    cout<<endl;

    mergeSort(arr, 0, N - 1);

    copy(arr, arr+N, ostream_iterator<int>(cout, " "));
    cout<<endl;
}

有时候我得到了正确的答案，有时我得错了答案。这个问题困扰了我很多时间，何时可以帮助找到错误？

Answer 1

是的，rand（）实现在两个平台上都不同，它是stackoverflow中的一个旧帖子，请参考link here

Answer 2

您在代码中犯的错误是在计算要排序的数组长度时。实际上int len = end - begin;给出了9的长度，而你的数组包含10个元素。

将索引发送到函数时应该小心。所有std::函数都与迭代器一起使用，end迭代器总是指示一个超过最后一个元素。

我建议您更改代码时考虑到这一点，尝试在主服务器中调用mergeSort(arr, 0, N);，让其他功能正常运行。

我写了一个合并排序，但它有一些bug

2 个答案: