c 程序未按预期输出

Question

所以我有这个程序，是我从头开始编写的。它来自第 6 章 - 数据结构章节...我的问题是输出显示所有先前列出的条目以及最后输入到标准输入的名称。因此，程序几乎将所有内容打印两次，而不是显示一次打印的所有内容。我很难描述。如果您只需将其复制并粘贴到计算机上的文本编辑器中并运行代码，您就会明白我的意思。

这本书展示了程序使用 < 重定向工具获取岛屿名称文件。当我尝试此操作时，它会打印第一个名字、第二个名字和第一个名字。然后是下一个名字以及第二个和第一个名字...然后是下一个名字以及第三个、第二个和第一个名字...等等，具体取决于有多少个名字。当在终端中以标准输入方式输入文本时，也会发生此行为。

如果我将代码更改为显示(下一个)，它的工作效果更接近我的预期，但它仍然打印出额外的空行，并且可能存在内存泄漏

这段代码几乎超出了我的理解范围，有人能弄清楚为什么它会这样打印吗？

我会先在 C 讨论区的头问，但我想先问 stackoverflow 并立即得到答案。

我的代码如下。如果将其复制并粘贴到文本编辑器中，它看起来不应像一面文本墙。

祝你编码愉快。

#include <stdio.h> // basic input output
#include <stdlib.h>  // for obtaining and releasing heap memory malloc and free...
#include <string.h>  // for the stringdup method

typedef struct island {
    char *name;
    char *opens;
    char *closes;
    struct island *next;
} island;

void display(island *madonna);

island* create(char *name);

void release(island *start);

int main()
{
    /* create islands */
    island *start = NULL;
    island *i = NULL;
    island *next = NULL;
    char name[80];
    puts("enter island name...");
    for(; fgets(name, 80, stdin) != NULL; i = next) {
        next = create(name);
        if(start == NULL)
            start = next;
        if (i != NULL)
            i -> next = next;

        display(start);

    }
    release(start);

}

// display method
void display(island *start)
{
    island *i = start;

    if (i == NULL)
        puts("i equals NULL ");

    for(;i != NULL; i = i ->next) {
        printf("Name: %s open: %s-%s\n", i->name, i->opens, i->closes);
    }
}

// create method
island* create(char *name)
{
    island *i = malloc(sizeof(island));
    i->name = strdup(name);
    i->opens = "09:00";
    i->closes = "17:00";
    i->next = NULL;
    return i;
}

// release method
void release(island *start)
{
    island *i = start;
    island *next = NULL;
    for(; i != NULL; i = next) {
        next = i-> next;
        free(i->name); // must free this first because of strdup uses heap memory
        free(i);
    }
}

Answer 1

代码按设计运行 (WAD)。它的设计目的是在读取每个条目后打印完整的列表 - 这就是 display(start) 在循环中所做的事情。您可以通过回显输入(printf("Read: %s", name);，其中格式中没有换行符，因为名称仍然包含换行符)或通过标记显示来帮助自己printf("Printing list:\n"); 在调用 display() (或两者)之前。如果您删除名称中的换行符，则需要调整“echo”操作。

学习如何创建有用的诊断消息是一项很有值(value)的技术；关键点之一是确保输出行以换行符结尾，因此您很有可能在打印发生时看到打印，而不是在稍后的某个不确定的时间。另一个关键点是打印输入，以便您知道代码正在做什么，而不是认为您知道代码正在做什么。在每次迭代中打印完整列表也有助于确保正确构建列表。您可以在 SO 上找到列表未正确构建的示例(例如 First address of struct )。如果每次迭代都打印完整的列表，问题就会更加明显。

因此，问题似乎是您的期望与代码的设计目的不符。