C - malloc 不适用于列表的第二个元素

Question

该函数应该创建一个结构链表，我在调试器中看到它为第一个元素分配内存，但是程序在为第二个元素分配内存期间崩溃。

struct Tipologia{
int mq;
int prezzo;
int n;
char descrizione[100];
};
struct Nodo{
struct Tipologia t;
struct Nodo *next;
};
typedef struct Nodo* NODO;
struct Tipologia creaTipologia(){
struct Tipologia t;
printf("MQ?n");
scanf("%d", &(t.mq));
printf("PREZZO?n");
scanf("%d", &(t.prezzo));
printf("DISPONIBILI?n");
scanf("%d", &(t.n));
printf("DESCRIZIONE?n");
int c;
while ( (c = getc(stdin)) != EOF && c != 'n');
fgets((t.descrizione), 100, stdin);
return t;
}
NODO costruisciPalazzo(){
int continua;
NODO h= NULL;
printf("VUOI COSTRUIRE UN APPARTAMENTO? 1 SI, 0 NOn");
scanf("%d", &continua);
if(continua){
NODO n= malloc(sizeof(NODO));
h= n;
n->t= creaTipologia();
n->next= NULL;
printf("VUOI COSTRUIRE UN APPARTAMENTO? 1 SI, 0 NOn");
scanf("%d", &continua);
while(continua){
NODO nodo= malloc(sizeof(NODO));
n->next= nodo;
n= nodo;
n->t= creaTipologia();
printf("VUOI COSTRUIRE UN APPARTAMENTO? 1 SI, 0 NOn");
scanf("%d", &continua);
}
n->next= NULL;
}
return h;
}

我一直在遵循教授的指示，但它一直在崩溃，没有任何错误来解释实际发生的事情。它似乎对我的同学有用，我们无法弄清楚问题是什么。请帮忙

Answer 1

问题是您正在使用一个隐藏NODO是指针这一事实的typedef。这很糟糕，因为它会造成令人困惑的情况，即您希望类型具有一定的大小并与特定的语法一起使用，但实际上它是完全不同的东西。

例如，您必须这样做：

h= n;
n->t= creaTipologia();
n->next= NULL;

令人困惑的是，h和n都没有显式声明为指针，但您必须使用箭头表示法。

我的建议是，要么完全删除typedef并在代码中使用struct nodo，要么至少从typedef中删除指针。对其他结构执行相同的操作以保持一致性：

typedef struct {
int mq;
int prezzo;
int n;
char descrizione[100];
} TIPOLOGIA;
typedef struct Nodo {
TIPOLOGIA t;
struct Nodo *next;
} NODO;

您还可以使用对象作为其大小的参考来简化malloc。例如：

NODO *h = malloc(sizeof *h);

这样就避免了在调用malloc时指定h类型的需要。重复使用也更好。

Answer 2

这个问题可以通过简单地使用 malloc(sizeof(struct Nodo((而不是 malloc(sizeof(NODO(( 来解决，因为使用 NODO 它只会分配指针，而对于结构体 Nodo，它分配整个元素