Question

我正在尝试遍历我的链表并将每个节点字段组合成一个字符串，然后将该字符串添加到字符串数组中。

void listToArray(){
   //create array of strings
    char *list[numNodes];
    int i = 0, n;

  while(head != NULL){
    // add string to array
    printf("[%d] %s %s:%d\n ", i, head->fileName, head->ip, head->port);
    n = sprintf(list[i], "[%d] %s %s:%d\n", i, head->fileName, head->ip, head->port);
    head = head->next;
    printf("%s\n", list[i]);
    i++;

 }

printf语句工作正常，这表明它不是导致问题的节点字段，但是当我使用sprintf语句将字符串复制到数组的索引中时。我遇到了分段错误。

Answer 1

您只需声明

char *list[numNodes];

但不为它们分配内存。在使用list[i]之前使用malloc分配内存。要知道sprintf使用snprintf生成的字符串的大小。感谢user3121023找到了我们这个功能。

void listToArray(){
   //create array of strings
    char *list[numNodes];
    int i = 0, n;

  while(head != NULL){
    printf("[%d] %s %s:%d\n ", i, head->fileName, head->ip, head->port);

    n = snprintf(NULL, 0, "[%d] %s %s:%d\n", i, head->fileName, head->ip, head->port);
    list[i] = malloc((n+1)*sizeof(char));

    n = sprintf(list[i], "[%d] %s %s:%d\n", i, head->fileName, head->ip, head->port);
    head = head->next;
    printf("%s\n", list[i]);
    i++;

 }

Answer 2

您没有初始化char指针数组 - sprintf正在将数据写入某个随机位置。

在调用sprintf之前，应将每个char指针初始化为具有malloc的缓冲区。

Answer 3

列表已定义，但每个元素中都没有/随机指针。上面的答案已经完成，但可以简化一下：

void listToArray(){
    //create array of strings
    char *list[numNodes];
    char buf[5000];
    int i = 0, n;

    while (head != NULL) {
        snprintf(buf,sizeof(buf),"[%d] %s %s:%d\n ",
            i,head->fileName,head->ip,head->port);

        // add string to array
        list[i] = strdup(buf);

        // output
        fputs(list[i],stdout);

        head = head->next;
        i++;
    }
}

你只需要做一个printf / sprintf。那是缓慢的部分。执行单个snprintf和strdup比执行2-3 * printf调用更快更简单。我猜你每个项目只需要一个输出行，剩下的就是调试。

无法将字符串复制到C中的字符串数组

3 个答案: