在C递归函数中作为引用传递的数组的动态内存分配[关闭]

Question

我想计算我的递归嵌套了多少，并返回函数内部创建的数组中的每个级别编号，但该代码无法正常工作（运行时显示“分段错误”）：

void test (int* result[], int* counter) {


  if (*counter == 0)
        *result = (int**) malloc(0);

    if (*counter == 5)
        return;

    else {
        *counter = *counter + 1;
        *result = (int**) realloc(*result, sizeof(int) * (*counter));
        *result[*counter - 1] = *counter; // This line is wrong, and I don't know why.
        test(result, counter);
    }
}

void main () {
    int** result;
    int counter = 0;

    test(result, &counter);

    int i;
    for (i = 0; i < 5; i++)
        printf("%d\n", *result[i]);
}

我知道我可以使用单指针作为参数和返回值来做到这一点。关键是，我只是好奇。

Answer 1

test() 的代码在我看来表面上没问题（我没有测试过，所以我不能肯定），但在 main() 中这是错误的：

for (i = 0; i < 5; i++)
    printf("%d\n", *result[i]);
}

你想要(*result)[i]。数组索引比指针解引用具有更高的优先级。

你也不应该强制转换malloc() 的返回值（参见Do I cast the result of malloc?），尤其是因为你做错了：*result 的类型是 int *，所以如果你想保留这个转换，在至少将其更改为int *，但我建议完全取消它。

更新：

这一行也是如此：

*result[*counter - 1] = *counter;

应该是：

(*result)[*counter - 1] = *counter;

Answer 2

所有有问题或错误的行的简要说明：

*result = (int**) malloc(0);

该行要么将*result 设置为0，要么设置为某个必须为freed 的唯一值。此外，演员表是多余的并且是错误的，因为*result 的类型为int*。

*counter = *counter + 1;

上面对标准增量运算符大喊大叫：++*counter;

*result = (int**) realloc(*result, sizeof(int) * (*counter));

除了上面malloc标识的类型和转换问题，最好这样写以避免使用错误的大小：*result = realloc(*result,*counter * sizeof *result);

*result[*counter - 1] = *counter; // This line is wrong, and I don't know why.

运算符优先级表示需要加括号：(*result)[*counter - 1] = *counter;

void main () {

main 的唯一祝福原型是 int main(void) 和 int main(int argc, char* argv[argc]) 或兼容的。实现也可能允许其他人，尽管它们是不可移植的。

int** result;

x 你想要一个int* 在这里，因为你想用它作为test 的引用参数。

test(result, &counter);

注释x表示需要取上面result的地址。

for (i = 0; i < 5; i++)

您可能会考虑使用 *counter 作为通用性的限制，但在您的情况下，test 实际上总是将其设置为 5。

printf("%d\n", *result[i]);

如评论 x 所说，删除取消引用以获得良好结果。