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是我正在编辑的codeblock项目的链接。调用“bugged”printf的函数print_item(Item a)位于item.c模块中，而生成列表项的函数位于list.c模块内。

有什么可以导致这个问题的想法吗？

entering the first element of the stack

and that is what happen at the second element

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';

你只是在每个角色中放一个随机数 - 其中许多可能不是很有趣，甚至你的角色集中也没有可打印的字符。这就是为什么你得到你所看到的疯狂输出。如果要将随机可打印字符放入字符串中，请以更符合字符集的方式执行此操作。

还有一些问题：

这些分配行错误：
```
head=(link)malloc(sizeof(link*));
tmp=(link)malloc(sizeof(link*));
```
他们应该是：
```
head = malloc(sizeof(struct QUEUEnode));
tmp = malloc(sizeof(struct QUEUEnode));
```
也就是说，您应该为整个struct QUEUEnode分配足够的大小，而不仅仅是指向一个{I}的指针。（或指向一个指针的指针，这就是你所拥有的）。像typedef一样将指针类型隐藏在typedef struct QUEUEnode *link;内是有争议的风格选择之一 - 我个人不喜欢，因为它很快就会混淆像这样的问题。
你在这个程序中按价值传递了很多结构。这是好的和有效的C，它只是有点非惯用。通常人们会绕过指针。如果你的结构增长到任何可观的大小，由于所有隐式内存复制都会导致性能下降。

Answer 2

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

该计划完全按照您的要求行事。既然你没有说明你要分配给stringa的字符集，我真的不能说你的程序应该是什么。您可以使用此更改选择A到Z范围内的值：

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

我敢打赌，你的代码还有其他问题，但由于问题的主要焦点是打印，我看起来不会更深。

printf函数打印不应该的东西，为什么？

2 个答案: