Question

我目前正在尝试学习C而且我遇到了一个我无法解决的问题。

考虑：

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

#define ELEMENTS 5

void make(char **array, int *array_size) {
    int i;
    char *t = "Hello, World!";

    array = malloc(ELEMENTS * sizeof(char *));

    for (i = 0; i < ELEMENTS; ++i) {
        array[i] = malloc(strlen(t) + 1 * sizeof(char));
        array[i] = strdup(t);
    }
}

int main(int argc, char **argv) {
    char **array;
    int size;
    int i;

    make(array, &size);

    for (i = 0; i < size; ++i) {
        printf("%s\n", array[i]);
    }

    return 0;
}

我不知道为什么上面的内容在创建之后无法回读数组的内容。我花了一个小时的时间试图理解它为什么会失败但却空手而归。毫无疑问，这是微不足道的。

干杯，

Answer 1

您需要将“array”的地址传递给函数。也就是说，你需要char ***。这是因为您需要通过为其分配内存来更改数组的值。

编辑：只是为了使它更完整，在函数声明中你需要有类似

的东西

void make(char ***array, int *array_size)

然后你需要使用

来调用它

make(&array, &size);

在函数 make 中，使用

分配内存

*array = malloc(ELEMENTS * sizeof(char *));

相应地改变其他地方。

另外，正如kauppi所指出的，strdup会为你分配内存，所以你不需要在每个字符串上做malloc。

Answer 2

以下是工作代码：

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

#define ELEMENTS 5

void make(char ***array) {
    char *t = "Hello, World!";

    *array = malloc(ELEMENTS * sizeof(char *));

    int i;
    for (i = 0; i < ELEMENTS; ++i) {
        (*array)[i] = strdup(t);
    }
}

int main(int argc, char **argv) {
    char **array;
    make(&array);

    int i;
    for (i = 0; i < ELEMENTS; ++i) {
        printf("%s\n", array[i]);
        free(array[i]);
    }
    free(array);
    return 0;
}

正如另一个发布的那样 - 有未使用的大小，strdup自己分配内存，之后释放内存很好......

Answer 3

请参阅PolyThinker的评论，这是绝对的观点。

除了传递数组的方式之外，还应该检查一些其他问题：

也许你应该在make（...）？
strdup（char *）分配内存，不需要malloc for array [i]。
在您不再需要它之后，您应该释放所有分配的内存。

Answer 4

您正在传递数组的当前值以作为副本（在堆栈上）。当您在make（）中更改数组时，您只是更改副本，而不是实际变量。尝试通过＆amp;引用传递，或者使它成为char ***并使用* array = ...

Answer 5

声明了大小但没有赋值（这应该发生在函数make中，我想）。

在C中传递多维数组

5 个答案: