Question

当我在程序顶部全局初始化数组直到堆栈溢出时，我的C合并代码才有效。我正在尝试使用malloc初始化数组，但是当我这样做时，代码将只读取两个整数并停止运行。

该程序从名为alg.txt的文件中提取随机数，然后对它们进行排序。同样，当将程序顶部的z定义为要排序的整数数量时，代码可以工作（直到500k整数），并且全局声明数组等于arr [z]。我如何弄清楚发生了什么？

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

int count = 0;
void merge(int arr[], int l, int m, int r)
{
    int i, j, k;
    int n1 = m - l + 1;
    int n2 =  r - m;

    int L[n1], R[n2];

    for (i = 0; i < n1; i++)
        L[i] = arr[l + i];
        count++;

    for (j = 0; j < n2; j++)
        R[j] = arr[m + 1+ j];
        count++;

    i = 0;
    j = 0;
    k = l;
    while (i < n1 && j < n2)
    {
        if (L[i] <= R[j])
        {
            arr[k] = L[i];
            i++;
            count++;
        }
        else
        {
            arr[k] = R[j];
            j++;
            count++;
        }
        k++;
        count++;
    }

    while (i < n1)
    {
        arr[k] = L[i];
        i++;
        k++;
        count++;
    }

    while (j < n2)
    {
        arr[k] = R[j];
        j++;
        k++;
        count++;
    }
}

void mergeSort(int arr[], int l, int r)
{
    if (l < r)
    {
        int m = l+(r-l)/2;

        mergeSort(arr, l, m);
        mergeSort(arr, m+1, r);

        merge(arr, l, m, r);
    }
}

void printArray(int A[], int size)
{
    int i;
    for (i=0; i < size; i++)
        printf("%d ", A[i]);
    printf("\n");
}

int main()
{
    int i;
    FILE *myFile;
    myFile = fopen("alg.txt", "r");

    int z;
    printf("Enter the size of the array: ");
    scanf("%d", &z);
    int *arr = (int *)malloc(z*sizeof(int));
    int n = sizeof(arr)/sizeof(arr[0]);
    for(i=0; i < z; i++)
    {
        fscanf(myFile, "%d,", &arr[i]);
    }

    mergeSort(arr, 0, n - 1);

    printf("\nSorted array is \n");
    printArray(arr, n);
    printf("count is %d\n", count);
    return 0;
}

Answer 1

您在本地（在merge函数内）声明了 L 和 R 数组。它们在每个函数调用上都使用堆栈。您应该全局初始化它们，以便它们可以存储在堆上。此外，您应该只声明一个数组（如下面代码中的 temp_arr ），因为不需要两个单独的数组，如 L 和 R < /强>

请参阅Local vs Global variable storage

您可以改进合并功能：

int temp_arr[500000]; // Global declaration void merge(int arr[], int l, int m, int r) { int i=l, j=m+1, k=l; while(i<=m && j<=r) { if(arr[i]<arr[j]) temp_arr[++k] = arr[++i]; else temp_arr[++k] = arr[++j]; } while(i<=m) temp_arr[++k] = arr[++i]; while(j<=r) temp_arr[++k] = arr[++j]; i = l; while(i<=r) arr[i] = temp_arr[++i]; // Storing stored array in arr }

为什么这个程序堆栈溢出？

1 个答案: