Question

在下面的代码中，我初始化了L，R和数组like this：

int *L = (int *) malloc(sizeof(int) * n1);
int *R = (int *) malloc(sizeof(int) * n2);

for (i = 1; i <= n1; i++){
    L[i] = A[p + i - 1];
}

for (j = 1; j <= n2; j++){
    R[j] = A[q + j];
}


int *array = (int *) malloc(sizeof(int) * n);

...
for (c = 0; c < n; c++){
    scanf("%d", &array[c]);
}

但是遇到了段错误。你能告诉我这个初始化有什么问题吗？

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

void merge(int *A, int p, int q, int r)
{
    int n = r - p + 1; /* number of elements in array A */
    int n1 = q - p + 1;
    int n2 = r - q;
    printf("n2 = %d\n",n2);
    int  i, j, k;

    int *L = (int *) malloc(sizeof(int) * n1);
    int *R = (int *) malloc(sizeof(int) * n2);

    for (i = 1; i <= n1; i++){
        L[i] = A[p + i - 1];
    }

    for (j = 1; j <= n2; j++){
        R[j] = A[q + j];
    }

    L[n1+1] = INT_MAX;
    R[n2+1] = INT_MAX;

    i = 1;
    j = 1;

    printf("p = %d\n",p);
    printf("r = %d\n",r);

    for (k = p; k <= r; k++){
        printf("k=%d\n",k);
        if (L[i] <= R[j]){
            A[k] = L[i];
            i += 1;
        }

        else{
            A[k] = R[j];
            j += 1;
        }
    }

    free(L);
    free(R);
}

void merge_sort(int *A, int p, int r)
{
    int i;
/*  for (i=0; i<=r-p;i++){
        printf("%d\n",A[i]);
    }*/
    int q;
    if (p < r){
    /*  printf("merge_sort p = %d\n",p);
        printf("merge_sort r = %d\n",r);*/
        q = (p + r)/2;
/*      printf("q = %f\n",q);*/
        merge_sort(A, p, q);
        merge_sort(A, q+1, r);
/*      printf("done\n");*/
        merge(A, p, q, r);
    }
}


int main()
{
    int x, c;
    int n;
    int *array = (int *) malloc(sizeof(int) * n);
    printf("Enter number of elements\n");
    scanf("%d",&n);

    printf("Enter %d elements\n",n);

    for (c = 0; c < n; c++){
        scanf("%d", &array[c]);
    }

    merge_sort(array, 0, n-1);

    for (c = 0; c < n; c++){
        printf("%d\n", array[c]);
    }

    free(array);
    return 0;
}

Answer 1

分别为malloc和n1中n2和int L的{{1}}内存。这使得通过R和L[0]到L[n1 - 1]编写索引R[0]是有效的，但是您的循环写入R[n2 - 1]和L[n1]到期条件为R[n2]和i<=n1。将每个循环的开始更改为j<=n2，并从每个结束条件中删除0。

（检查你的其他代码是否有同样的错误也是一个好主意，也就是说，记住0是第一个索引，它计入数组的长度。）

Answer 2

每个数组的第一个索引始终为0.

array: int tab[3];
elements: tab[0], tab[1], tab[2]

没有元素tab[3]（seg fault）。

有趣的是，main(...)函数中的循环被写入正确，但merge(...)中的另外两个有不同想法迭代数组。

Answer 3

您可能需要查看memset()

在c中初始化指针数组

3 个答案: