【问题标题】:merge sort stack overflow in c++C ++中的合并排序堆栈溢出
【发布时间】:2018-03-23 00:08:27
【问题描述】:

此代码在数组大小为 75.000 时有效,但在为 100.000 时出现堆栈溢出错误,我该如何解决?程序需要计算两种算法的经过时间和平均时间并进行比较

这是错误开始的地方

 //_______________________________
    // MERGE SORT
    void merge(int arr[], int l, int m, int r)
    {
        int i, j, k;
        int n1 = m - l + 1;
        int n2 = r - m;

        /* create temp arrays */
    int L[100000], R[100000];

    /* Copy data to temp arrays L[] and R[] */
    for (i = 0; i < n1; i++)
        L[i] = arr[l + i];
    for (j = 0; j < n2; j++)
        R[j] = arr[m + 1 + j];

    /* Merge the temp arrays back into arr[l..r]*/
    i = 0; // Initial index of first subarray
    j = 0; // Initial index of second subarray
    k = l; // Initial index of merged subarray
    while (i < n1 && j < n2)
    {
        if (L[i] <= R[j])
        {
            arr[k] = L[i];
            i++;
        }
        else
        {
            arr[k] = R[j];
            j++;
        }
        k++;
    }

    /* Copy the remaining elements of L[], if there
    are any */
    while (i < n1)
    {
        arr[k] = L[i];
        i++;
        k++;
    }

    /* Copy the remaining elements of R[], if there
    are any */
    while (j < n2)
    {
        arr[k] = R[j];
        j++;
        k++;
    }
}

发布的一些细节!..

/* l is for left index and r is right index of the
sub-array of arr to be sorted */
void mergeSort(int arr[], int l, int r)
{
    if (l < r)
    {
        // Same as (l+r)/2, but avoids overflow for
        // large l and h
        int m = l + (r - l) / 2;

        // Sort first and second halves
        mergeSort(arr, l, m);
        mergeSort(arr, m + 1, r);

        merge(arr, l, m, r);
    }
}
// END MERGE SORT
//_______________________________

插入部分不会在 100.000 上出现任何错误,但合并部分会出现

【问题讨论】:

  • 我的上帝……这会是 Stooge Sort 的工作吗?
  • int L[100000], R[100000]; 每次迭代至少占用 400,000 字节,并且可能超过 1.5 兆字节。幸运的是,这不是递归的函数,但它并没有给你留下太多的回旋余地。不管怎样,做一个你想要排序的非常小的列表,或者开始使用调试器,或者添加大量的打印语句来观察发生了什么。

标签: c++ stack-overflow mergesort insertion-sort


【解决方案1】:

堆栈大小非常小,因此请尝试将这两个大数组分配在堆中

int *L = new int[100000];
int *R = new int[100000];

... merge stuff ...

delete [] L;
delete [] R;

由于L的大小是n1,R的大小是n2,你可以这样做

int *L = new int[n1];
int *R = new int[n2];

【讨论】:

  • 通过分配L和R一次(在最外层的代码处)并重用它们,与此相比可以提高效率。
  • 现在它给出:抛出异常:写访问冲突。 L 为 0x1118235。
  • @user205160 您能否更具体地说明它是如何发生的(如何修改代码,程序在哪一行崩溃)。我运行你的代码就好了here
  • @hgminh 看起来它正在您编辑的我的版本中运行,但我认为它实际上并没有做任何事情。合并功能仅需约 0.01 秒。
  • 对包含 10^5 个元素的数组进行排序应该不会花费太多时间。可以尝试增加到 10^7 个元素,大约需要 2s (ideone.com/r52zvC)
猜你喜欢
  • 1970-01-01
  • 2023-03-11
  • 1970-01-01
  • 2021-01-19
  • 2013-06-01
  • 2020-03-26
  • 2011-01-13
  • 1970-01-01
  • 1970-01-01
相关资源
最近更新 更多