在下面的代码中,我一直不明白为什么在使用addUser()方法添加2个(或多个)User_struct(具有不同名称)后运行dump_data()方法时会得到相同的输出(即相同的名称)。
struct Friends_struct {
struct User_struct *this;
struct Friends_struct *next;
};
typedef struct Friends_struct *Friends;
typedef struct User_struct {
const char *name;
Friends amigos;
} User;
static User **userList; // global table of users
static int numUsers; // global number of users
static int tableSize; // global size of table
void create_amigonet() {
tableSize = INIT_TABLE_SIZE;
numUsers = 0;
userList = malloc(tableSize * sizeof(User *));
}
void addUser(const char *name) {
userList[numUsers] = malloc(sizeof(User));
userList[numUsers]->name = name;
userList[numUsers]->amigos = 0;
numUsers++;
if (numUsers == tableSize) {
tableSize += INIT_TABLE_SIZE;
User **moreUsers;
moreUsers = realloc(userList, tableSize * sizeof(User *));
userList = moreUsers;
}
}
void dump_data() {
for (int i=0; i<numUsers; i++) {
printf("%s; friends:", userList[i]->name);
}
printf("n");
}
不同的.c文件:
void do_add(const char *name) {
addUser(name);
}
int main(void) {
create_amigonet();
do_add("Dan");
do_add("Jim");
dump_data();
}
如果运行此程序,我期望的输出将类似
Dan; friends:
Jim; friends:
但是我得到了
Jim; friends:
Jim; friends:
如何/在哪里覆盖我的数据?
问题是在User_struct
中只存储了一个char
指针,而在addUser
调用中传递的指针只是临时的。您应该将name
的类型更改为足够长度的char
数组,并使用strcpy
在addUser
中复制名称。
我无法完全解释为什么这个例子实际上显示了错误,因为我认为两个名称字符串在整个main()
中都应该有效,但生成的代码似乎重用了存储"Dan"的位置,因此在第二次addUser
调用后,您会得到两次"Jim"。