我目前正在尝试学习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;
}

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

干杯,

有帮助吗?

解决方案

以下是工作代码:

#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自己分配内存,之后释放内存很好......

其他提示

您需要传递“数组”的地址进入功能。也就是说,你需要char ***。这是因为您需要通过为其分配内存来更改数组的值。

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

的东西
void make(char ***array, int *array_size)

然后你需要使用

来调用它
make(&array, &size);

在函数 make 中,使用

分配内存
*array = malloc(ELEMENTS * sizeof(char *));

相应地改变其他地方。

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

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

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

  1. 也许你应该在make(...)?
  2. 中为array_size分配一些东西
  3. strdup(char *)分配内存,不需要malloc for array [i]。
  4. 在您不再需要它之后,您应该释放所有分配的内存。

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

声明了大小但没有赋值(这应该发生在函数make中,我想)。

许可以下: CC-BY-SA归因
不隶属于 StackOverflow
scroll top