Question

我的程序在main函数中声明了3个int数组（指针）。用户输入数组A的长度，然后用随机数填充。

然后，调用一个函数，它将所有3个数组作为参数。它从数组A获取所有偶数，并将它们放入数组B，将所有奇数放入C。 B和C的大小需要与其元素的数量相同。然后打印B的元素。

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

int velA, velB, velC; //the sizes of all the arrays

void napravi(int a[], int *b, int *c);

void main() {
    int *a, *b, *c;
    int i;
    srand(time(NULL));

    printf("Enter array lenght:"); 
    scanf("%d", &velA);
    getchar();

    a = (int*)calloc(velA, sizeof(int));
    b = (int*)malloc(4); //I have to initialize the variable in order to pass it as an argument ?
    c = (int*)malloc(4);

    for(i = 0; i < velA; i++) {
        a[i] = rand() %101;
    }

    napravi(a, b, c);

    for(i = 0; i < velB; i++) {
        printf("%d  ", b[i]);
    }

    free(a);
//  free(b); //Windows has triggered a breakpoint in .exe
//  free(c);
    getchar();
}

void napravi(int a[], int *b, int *c) {
    int i;
    int j = 0, k = 0;

    for(i = 0; i < velA; i++) {
        if(a[i] % 2 == 0) {
            j++;
        }
        else {
            k++;
        }
    }
    free(b);
    free(c);
    b = (int*)calloc(j, sizeof(int));
    c = (int*)calloc(k, sizeof(int));


    j = 0; k = 0;

    for(i = 0; i < velA; i++) {
        if(a[i] % 2 == 0) {
            b[j] = a[i];
//          printf("\n%d | %d", a[i], b[j]);
            j++;
        }
        else {
            c[k] = a[i];
            k++;
        }
    }
    velB = j;
    velC = k;
}

此代码存在2个问题。

首先，如果用户输入的值大于~420左右，则会开始打印垃圾值，如-17987777 ......
其次，当我尝试在结束时释放（b）和释放（c）时，程序崩溃并出现错误＆＃34; Windows在my.exe中触发了断点＆＃34;。

Answer 1

在C中

，函数参数按值传递。因此，从napravi()函数，您无法更改b本身。但是，您当然可以更改b 指向的地址的值，即*b。

如果您需要从b更改napravi()，则需要将指针从b传递给main()。

如果用户输入的值大于~420左右，则会开始打印垃圾值，如-17987777 ......

您在代码中面对undefined behaviour。

for(i = 0; i < velB; i++) { printf("%d ", b[i]); }

在上面的代码中，velB值已正确更新（正如代码所预期的那样），但b不是（{em>正如预期{{1 }}功能）。因此，您将超出分配的内存区域，创建UB。

当我尝试在结束时释放（b）和释放（c）时，程序崩溃并出现错误＆＃34; Windows在my.exe中触发了断点＆＃34;。

UB的结果。

注意：请napravi() malloc()和C中的家人返回do not cast。

Answer 2

试试这个

int main(void) {
    int *a, *b, *c;
    int i;
    srand(time(NULL));

    printf("Enter array lenght:"); 
    scanf("%d", &velA);
    getchar();

    a = (int*)calloc(velA, sizeof(int));
    //b = NULL;//There is no need to set the value
    //c = NULL;//NULL is better than malloc(4)

    for(i = 0; i < velA; i++) {
        a[i] = rand() %101;
    }

    napravi(a, &b, &c);

    for(i = 0; i < velB; i++) {
        printf("%d  ", b[i]);
    }

    free(a);
    free(b);
    free(c);
    getchar();
    return 0;
}

void napravi(int a[], int **b, int **c) {
    int i;
    int j = 0, k = 0;

    *b = *c = NULL;
    velB = velC = 0;
    if(a == NULL || velA == 0)
        return;

    for(i = 0; i < velA; i++) {
        if(a[i] % 2 == 0) {
            j++;
        }
    }

    if(velB = j)
        *b = (int*)malloc(velB * sizeof(int));
    if(velC = velA - velB)
        *c = (int*)malloc(velC * sizeof(int));


    j = 0; k = 0;

    for(i = 0; i < velA; i++) {
        if(a[i] % 2 == 0)
            (*b)[j++] = a[i];
        else
            (*c)[k++] = a[i];
    }
}

C动态内存分配数组

2 个答案: