我有这样的基本结构
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
。 它编译没有问题。
你的typedef定义Truck
的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));
nT->id
只是一个指针。 需要malloc
内存字符串复制到。 同上,用于其他人。
你的使用sizeof
是不正确。 一般情况下,参数malloc()
需要“的那个,其返回的指针所指向的大小”。 换句话说,你需要sizeof *nT
。 怎么看也消除重复类型名称( Truck
)?
此外,在C你不需要投的返回值malloc()
; 也是没有用处的,可以隐藏实际的错误,并且使代码难以阅读。
正如其他人所指出的那样,你也没有任何字符串数据分配空间,你必须在你的结构的指针。