c - printf 函数打印不应该打印的内容，为什么？

Question

正如您从下面所附的图像中看到的那样，添加列表顶部的第二个元素会为 printf 函数创建一个奇怪的行为。

添加列表节点的函数是:

void add_QUEUEnode_top(Item a)
{
    if (head==NULL)
    {
        QUEUEinit(a);
    }
    else
    {
        QUEUEput_top(a);
    }
    return;
}

void QUEUEinit(Item a)
{
    head=(link)malloc(sizeof(link*));
    head->next=NULL;
    head->item=a;
    tail=head;
    printf("Coda iniziallizata...\n\n");

}

void QUEUEput_top(Item a)
{
    link tmp;
    tmp=(link)malloc(sizeof(link*));
    tmp->item=a;
    tmp->next=head;
    head=tmp;
    return;
}

这里是处理项目的函数:

Item fill_item()
{
    Item a;
    int i;
    for(i=0; i<DIM-1; i++)
    {
        a.stringa[i]=rand();
    }
    a.stringa[DIM-1]='\0';
    a.numero=rand();
    printf("\nOggetto generato: \n");
    print_item(a);
    return a;
}

void print_item(Item a)
{
    printf("\nStringa elemento: ");
    printf("%s", a.stringa);
    printf("\nNumero elemento:  %d\n", a.numero);
}

Here是我正在编辑的代码块项目的链接。调用“错误”printf 的函数 print_item(Item a) 在 item.c 模块中，而生成列表项的函数在 中list.c 模块。

知道是什么导致了这个问题吗？

PS:对于意大利语的捕获，我很抱歉

编辑:项目的定义:

typedef struct
{
    char stringa[DIM];
    int numero;
} Item;

链接指针的定义:

typedef struct QUEUEnode *link;

链接结构的定义:

struct QUEUEnode
    {
        Item item;
        link next;
    };

Answer 1

您有几个错误 - 我会尝试为您找出更多错误，但您询问的具体问题与此代码相关:

for(i=0; i<DIM-1; i++)
{
    a.stringa[i]=rand();
}
a.stringa[DIM-1]='\0';

您只是在每个字符中放置了一个随机数 - 其中许多可能不是您的字符集中有趣的字符，甚至是不可打印的字符。这就是为什么你会得到你所看到的疯狂输出。如果您想将随机可打印字符放入字符串中，请以更能识别字符集的方式进行。

还有一些问题:

1. 这些分配行是错误的:

   head=(link)malloc(sizeof(link*));
   tmp=(link)malloc(sizeof(link*));
   

   它们应该是:

   head = malloc(sizeof(struct QUEUEnode));
   tmp = malloc(sizeof(struct QUEUEnode));
   

   也就是说，您应该为整个 struct QUEUEnode 分配足够的大小，而不仅仅是指向一个的指针。 (或者指向一个指针的指针，这就是你所拥有的)。像使用 typedef struct QUEUEnode *link; 那样将指针类型隐藏在 typedef 中是那些有争议的样式选择之一 - 我个人不喜欢这样做，因为它混淆了诸如此类的问题这个非常快。

2. 您在此程序中按值传递了很多结构。这是很好的和有效的 C，它只是有点不惯用。通常人们会传递指针。如果您的结构增长到任何可观的大小，由于所有隐式内存复制正在进行，性能可能会开始受到严重影响。