C Copying to a const char *

Question

我有一个项目，我得到了一个头文件，其中包含:

typedef struct User_struct {
    const char *name;
    Friends amigos;
} User;

Friends是另一个不在我的问题范围内的结构体

在我需要创建的源文件中，我创建了一个结构体和随后的全局变量:

typedef struct UserList{
        User *user;
        struct UserList *next;
}UserList;
static UserList *users;

有一个初始化函数create_amigonet，其中我将内存分配给users，另一个分配给users->user。我的问题是这个函数:

void addUser( const char *name )
{
        UserList *new_users;
        UserList *iter;
        if(users->user->name == NULL)
        {
                users->user->name = malloc(strlen(name));
                strncpy((users->user->name,name,strlen(name));
                return;
        }
        for(iter = users; iter != NULL; iter = iter->next)
        {
                if(iter->user->name == name)
                        return;
        }
        new_users = malloc(sizeof(UserList));
        if(new_users != NULL)
        {
                new_users->user = malloc(sizeof(User));
                new_users->user->name = malloc(sizeof(name)+1);
                strncpy(new_users->user->name,name,sizeof(name)+1);
                new_users->user->amigos = NULL;
                new_users->next = users;
                users = new_users;
        }
        return;
}

我需要将const char *复制到const char *中，并且不允许更改User结构体或函数的定义。我不确定我应该如何复制/初始化我的结构中的任何节点。我可以改变UserList，这是我创建的一个结构体作为"用户"的链表。

Answer 1

这会导致一个问题:

users->user->name = malloc(strlen(name));
strncpy((users->user->name,name,strlen(name));

似乎你不知道null终止。假设您稍后想从users->user->name中读取数据。你怎么知道名字什么时候结束?你没有把strlen(name)信息保存在任何地方。

由于这个原因，C中的字符串在它们的末尾有一个标记。该标记是一个空字节(值为0的字节)。然后，字符串处理函数知道字符串的末尾在哪里(通常是分配的内存的末尾)。 因此，您需要分配一个额外的字节，并使用适当的复制函数:

users->user->name = malloc(strlen(name) + 1);
strcpy(users->user->name, name);

作为经验法则，永远不要使用strncpy 。strcpy或snprintf函数更合适。strncpy是远古时代的遗迹。

你有一个类似的(但更糟糕的是，因为你误入使用sizeof没有真正理解sizeof做什么)在你的程序后面。希望你现在也能解决这个问题，在阅读我的答案后。

我需要将const char *复制到const char *中，

你需要停止认为指针"包含"任何东西。指针指向内存的其他部分，这些部分本身必须存在。

你需要将一些字节从一个地方复制到另一个地方，在那里你有指向两个位置的指针。

现在，您不能通过const char *写入位置。这就是const的作用。然而，在只读内存中的位置是而不是:您只是malloc了它。所以这个位置绝对是安全的。您只需要重新安排代码，以便将非const指针传递给编写函数。一种方法是:

char *new_name = malloc(strlen(name) + 1);
strcpy(new_name, name);
users->user->name = new_name;

另一种方法，如果您在POSIX系统上，将是:

users->user->name = strdup(name);