Question

我正在尝试创建一个链排序算法，当我尝试递归调用它时，它给了我堆栈溢出错误

递归调用是我要将数组a排序为两个排序数组（preme，NoNpreme）的地方。之后，我将它们合并回a。我试图弄乱条件，但我发现只有1个元素时才对数组进行排序，这是我递归的出口。但这不起作用，我也不知道为什么。

我的算法：

int* urediSPrameni(int* a, int n)
{
    if (n <= 1)
        return a;

    int pe=a[0];
    int* preme = new int[n];
    preme[0] = a[0];
    int j = 1;
    int k = 0;
    int* NoNpreme = new int[n];
    for(int i=0;i<n;i++)
    {
        if (a[i] > pe)
        {
            pe = a[i];
            preme[j] = a[i];
            j++;
        }
        else
        {
            NoNpreme[k] = a[i];
            k++;
        }
    }
    NoNpreme = urediSPrameni(NoNpreme, k);


    // My merge algorithm:

    //Zlivanje
    int h = 0;//NoNpreme
    int s = 0;//Preme
    int i = 0;
    while (h < k && s < j)
    {
            if (preme[h] <= NoNpreme[s])
            {
                a[i] = preme[h];
                s++;
            }
            else
            {
                a[i] = NoNpreme[s];
                h++;
            }
            i++;
    }
    if (h > k)
    {
        for (int b = s;b < j;b++)
        {
            a[i] = NoNpreme[b];
            i++;
        }
    }
    else
    {
        for (int b = h;b < k;b++)
        {
            a[i] = preme[b];
            i++;
        }
    }
    return a;
}

我这样调用函数：

int n = 7;
int drek[] = { 5,1,2,20,5,28,7 };
int* urejenoZap = urediSPrameni(drek, n);
for (int i = 0;i < n;i++)
    cout << urejenoZap[i] << " ";
system("PAUSE");

drek未排序且urejenoZap指向已排序数组drek的指针。

Answer 1

您将a的第一个元素分配给pe，并将该元素的副本存储到preme中。根据{{1}}的哪一侧将a的元素排序到单独的数组中时，请再次从元素0开始（已将其放置在pe中），其中，因为preme将为 false ，因此会将此元素放入if (a[i] > pe)中。一旦数组到达一个以上元素且第一个元素最小的点，所有所有元素将被复制到NoNpreme中。此后的每个递归调用都将执行相同的操作，因此递归永远不会结束，最终您将耗尽堆栈空间。

解决方案是从第二个元素开始for循环：

NoNpreme

递归调用函数时发生堆栈溢出

1 个答案: