段错误strcpy的中()段错误strcpy的中()(Segmentation Fault in s

2019-05-13 12:17发布

我有这样的基本结构

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 。 它编译没有问题。

Answer 1:

你的typedef定义Truckstruct struck * ,即一个指针。 所以,它的大小将是48取决于架构,而不是结构的大小

使用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));


Answer 2:

nT->id只是一个指针。 需要malloc内存字符串复制到。 同上,用于其他人。



Answer 3:

你的使用sizeof是不正确。 一般情况下,参数malloc()需要“的那个,其返回的指针所指向的大小”。 换句话说,你需要sizeof *nT 。 怎么看也消除重复类型名称( Truck )?

此外,在C你不需要投的返回值malloc() ; 也是没有用处的,可以隐藏实际的错误,并且使代码难以阅读。

正如其他人所指出的那样,你也没有任何字符串数据分配空间,你必须在你的结构的指针。



文章来源: Segmentation Fault in strcpy()