2

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

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

干杯,

4

5 回答 5

6

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

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

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

然后你需要使用它来调用它

make(&array, &size);

在函数make中,分配内存

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

并相应地改变其他地方。

此外,正如 kauppi 所指出的,strdup 将为您分配内存,因此您无需对每个字符串执行 malloc。

于 2009-01-08T08:00:42.310 回答
5

这是工作代码:

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

于 2009-01-08T08:09:21.417 回答
4

请参阅 PolyThinker 的评论,这绝对是正确的。

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

  1. 也许您应该在 make(...) 中为 array_size 分配一些东西?
  2. strdup(char*) 分配内存,array[i] 的 malloc 不是必需的。
  3. 您应该在不再需要后释放所有分配的内存。
于 2009-01-08T07:59:50.813 回答
3

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

于 2009-01-08T08:05:25.140 回答
0

声明了 size 但没有分配任何值(我想这应该发生在函数 make 中)。

于 2009-01-08T08:04:50.897 回答