Question

我试图制作链表类型的数据结构，目前只有一个char作为数据，但我无法正确分配。当我运行以下代码时：

#include <string.h>
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <sys/types.h>

FILE* outfile;
FILE* infile;

int linkedlist_size;
struct linked_list
{
    char* data;
    struct linked_list* next;
    struct linked_list* previous;
};

int main()
{
    outfile = fdopen(STDOUT_FILENO,"w");
    infile = fdopen(STDIN_FILENO,"r");

    struct linked_list line_buffer;
    struct linked_list* current = &line_buffer;
    int linkedlist_size = 0;

    int input_char;
    char input_cast;
    for (input_char = fgetc(infile); (input_char != EOF && input_char != '\n') ;input_char = fgetc(infile))
    {
        input_cast = input_char;
        current->data = malloc(sizeof(char));
        (current->data)[0] = input_cast;
        linkedlist_size++;
        current->next = malloc(sizeof(struct linked_list));
        current = current->next;
        printf("\nMy address is: %p",current);
        printf("\nMy number is: %d",input_char);
        printf("\nMy char cast is: %c",input_cast);
        printf("\nMy char is: %s",current->data);
    }

    return 0;
}

使用gcc ll_test.c编译，使用./a.out运行，并使用something作为键盘的输入，我得到以下输出：

My address is: 0x10558a0
My number is: 115
My char cast is: s
My char is: (null)
My address is: 0x1055cf0
My number is: 111
My char cast is: o
My char is: (null)
My address is: 0x1055d30
My number is: 109
My char cast is: m
My char is: (null)
My address is: 0x1055d70
My number is: 101
My char cast is: e
My char is: (null)
My address is: 0x1055db0
My number is: 116
My char cast is: t
My char is: (null)
My address is: 0x1055df0
My number is: 104
My char cast is: h
My char is: (null)
My address is: 0x1055e30
My number is: 105
My char cast is: i
My char is: (null)
My address is: 0x1055e70
My number is: 110
My char cast is: n
My char is: (null)
My address is: 0x1055eb0
My number is: 103
My char cast is: g
My char is: (null)

这意味着这些字母正确地进入STDIN，正在被正确解释（在输入\n后循环停止）并且演员正在完成，但是分配不起作用。对于它的价值，我也尝试linked_list.data定期char并直接分配（通过current->data = input_cast）并收到类似的结果（空白输出，而非(null)，暗示a \0被“打印”）。我认为这是关于我不熟悉的结构的一些挑剔的观点，但我不能为我的生活弄清楚它是什么。随意抓取/编译/测试代码。

另外，我知道存在内存泄漏......这是一个来自更大代码的修改片段，因此很多功能都不是学术上的完美。我只是想证明我的行为。

谢谢大家！

编辑：如下所述，错误是我在切换到下一个空节点后尝试打印当前节点的字符。我这个愚蠢的逻辑错误。

Answer 1

printf("\nMy char is: %s",current->data);

应该是

printf("\nMy char is: %c", *(current->data));

或

printf("\nMy char is: %c", current->data[0]);

也就是说，格式说明符应该是单个char而不是字符串，并且需要取消引用数据指针才能获取字符。如果仍然不清楚，C中的字符串是 NUL终止字符序列。您只有一个字符而不是字符串。

Answer 2

您需要分配2个字节，如下所示：current->data = malloc(2);第一个字节将存储字符，第二个字节将存储字符串终结符'\0'，之后您可以将其打印为字符串。您忘了使用以前的字段：

 current->next = malloc(sizeof(struct linked_list));
 current->next->previous=current;
 current = current->next;

您从新分配的节点打印字符串，它不会在您打印时及时初始化。将您的行current = current->next;移到printf语句之上。

结构字符未正确分配

2 个答案: