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 是我正在编辑的代码块项目的链接。调用“bugged” 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，只是有点不习惯。通常人们会传递指针。如果您的结构增长到任何可察觉的大小，性能可能会因为所有隐式内存复制的进行而开始受到严重影响。

Answer 2

您将随机文本分配给stringa，然后将numero 设置为一个随机数。当您打印这些东西时，您会得到随机输出。因为您没有将随机化限制为可打印字符，所以会发生奇怪的事情。您正在打印控制字符。您的第二个屏幕截图表明您打印了一个carriage return 字符。我确定你不想那样做！

程序正在按照您的要求执行。因为你没有说你想从哪个字符集分配给stringa，所以我真的不能说你的程序应该是什么。您可以通过此更改使其选择 A 到 Z 范围内的值：

a.stringa[i] = 'A' + rand() % 26;

我敢打赌，您的代码还有其他问题，但由于问题的主要焦点与打印有关，我不会再深入研究。