c程序不按预期输出

Question

所以我有这个程序，我把头从头放在一起c。它来自第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);（格式中没有换行符，因为该名称仍包含换行符）或在012xx（或两者）调用之前用printf("Printing list:\n");标记显示）来帮助自己。如果您从名称中删除换行符，则需要调整“回显”操作。

学习如何创建有用的诊断消息是一种有价值的技术;其中一个关键点是确保输出行以换行符结束，因此您可以很好地看到打印的发生，而不是稍后的某个不确定时间。另一个关键点是打印输入，以便您知道代码的工作内容，而不是认为您知道代码的工作内容。在每次迭代中打印完整列表还有助于确保正确构建列表。您可以在SO上找到列表未正确构建的示例（例如First address of struct）。如果每次迭代都打印完整列表，问题就会更加明显。

因此，麻烦似乎是您的期望与代码设计的目标不符。

Answer 2

“我的问题是，输出显示所有以前列出的条目作为 以及最后输入的名称为标准输入。”

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

顺便说一下，你告诉我们你的问题，但你没有告诉我们，你的程序应该做