2

我有这样的基本结构

typedef struct struck {
    char* id;
    char* mat;
    int value;
    char* place;
} *Truck;

像这样的函数会创建该结构的新“实例”：

Truck CTruck(char* id, char* mat, int value, char* place) {
    Truck nT = (Truck) malloc(sizeof (Truck));
    nT->value = value;
    strcpy(nT->id, id);
    strcpy(nT->mat, mat);
    strcpy(nT->place, place);
    return nT;
}

我在第一个中遇到错误strcpy。它编译没有问题。

4

3 回答 3

11

您的 typedef 定义Truck为 a struct struck *，即指针。所以它的大小将4取决于8架构而不是结构的大小

用于sizeof(*Truck)获取结构的实际大小。

您还需要为字符分配内存。最简单的方法是使用strdup().

Truck CTruck(const char* id, const char* mat, int value, const char* place) {
    Truck nT = malloc(sizeof (*Truck));
    nT->value = value;
    nT->id = strdup(id);
    nT->mat = strdup(mat);
    nT->place = strdup(place);
    return nT;
}

但是，我建议更改您的 typedef，使其成为结构的别名，而不是指向它的指针：

typedef struct {
    char* id;
    char* mat;
    int value;
    char* place;
} Truck;

然后在你的函数中使用这个：

Truck *nT = malloc(sizeof(Truck));
于 2012-04-13T15:16:59.920 回答
7

nT->id只是一个指针。需要malloc将字符串复制到内存中。其他人也一样。

于 2012-04-13T15:16:53.963 回答
4

你的用法sizeof不正确。通常，参数 tomalloc()需要是“返回的指针指向的大小”。换句话说，您需要sizeof *nT. 看看这也消除了重复类型名称 ( Truck) 吗？

此外，在 C 中，您不需要转换 ; 的返回值malloc()。它没有任何作用，可以隐藏实际错误，并使代码更难阅读。

正如其他人指出的那样，您也没有为任何字符串数据分配空间，您所拥有的只是结构中的指针。

于 2012-04-13T15:19:00.007 回答