C如何迭代未知大小的函数int数组作为参数

Question

我有以下代码：

intlist* create_list(int *arr) {

    intlist* l = malloc(sizeof(intlist));

    while(*arr++) {
        l->sum += *arr; 
        //*arr++;
    }

    return l;
}

void printl(intlist* l) {
    printf("%d\n", l->sum);
}

当我在 while 循环内打印 *arr 时，它会显示正确的值。问题是在我退出函数 create_list 并尝试访问列表的总和后，这是完全错误的。例如，它不是 1 + 2 + 3 = 6，而是类似于 -123871239012。我做错了什么？

我不是在寻找将数组大小传递给的函数，因为根据我的观点和经验，在大多数情况下使用这种方法非常危险。我不想传递数组的大小，因为如果我不小心写了“1239123”而不是“1239122”，我可能会做坏事。或者更糟糕的是，如果我在最后一个地方没有按“3”时，不小心写了“1239123”而不是“123912”，那将会产生毁灭性的影响。

Answer 1

函数create_list有两个问题

intlist* create_list(int *arr) {

    intlist* l = malloc(sizeof(intlist));

    while(*arr++) {
        l->sum += *arr; 
        //*arr++;
    }

    return l;
}

第一个是数据成员sum没有初始化。

第二个是因为while循环的条件后自增而忽略了数组的第一个元素

    while(*arr++) {

你至少应该写

    while(*arr) {
        l->sum += *arr; 
        arr++;
    }

请注意，在将控件传递给while循环之前，您应该检查是否为intlist类型的对象成功分配了内存。

该函数期望数组包含一个标记值等于 0 的元素。

如果函数必须处理不具有标记值元素的数组，那么您必须声明第二个参数，该参数将指定传递的数组中有多少元素。

注意第一个参数应该有限定符const，因为传递的数组在函数内没有被改变。

就是函数应该这样声明

intlist * create_list( const int *arr, size_t n );

Answer 2

一些错误：

while(*arr++) {
    l->sum += *arr; 
    //*arr++;
}

您将在 arr 增量之后添加*arr 。

为了说明这个问题：https://godbolt.org/z/5qe9cE

必须是：

while(*arr) {
    l->sum += *arr++; 
    //*arr++;
}

分配的内存未初始化，结构成员具有未确定的值（可以是任何值）。在循环之前你需要

    l->sum = 0; 

例如，数组必须以0 终止

int array[] = {1,2,3,4,5,6,0};

https://godbolt.org/z/c8cPGs