设为首页收藏本站
切换到窄版

鱼C论坛

 找回密码
 立即注册
鱼C论坛»论坛 › 编程语言专区 › C\C++交流 › 单链表的这个代码错误在哪？
返回列表 发新帖
查看: 1493|回复: 14

[已解决]单链表的这个代码错误在哪？

[复制链接]
召唤师
发表于 2020-3-6 09:34:30 | 显示全部楼层 |阅读模式

马上注册，结交更多好友，享用更多功能^_^

您需要 登录 才可以下载或查看，没有账号？立即注册

x
#include <stdio.h>
#include <stdlib.h>

void addBook(struct Book **library);
void getinput(struct Book *book);
void printfbook(struct Book *library);
void freelibrary(struct Book *book);
struct Book{
        char title[128];
        char author[40];
        struct Book *next;
};

void getinput(struct Book *book){
        printf("请输入书名[title]\n");
        scanf("%s",book->title);
        printf("请输入作者[author]\n");
        scanf("%s",book->author);
}

void addBook(struct Book **library){
        struct Book *book,*temp;
        book=NULL;
        temp=NULL;
        book=(struct Book*)malloc(sizeof(struct Book));
        if(book==NULL){
                printf("内存分配失败了");
                exit(1);
        }
        else{
                getinput(book);
        }
        if (*library != 0)                 
        {                                
                temp = *library;
                *library = book;
                book->next = temp;
        }
        else                           
        {
                *library = book;
                book->next = NULL;
        }
}

void printfbook(struct Book *library){
        struct Book *book;
        book=library;
        int i=1;
        while(book!=NULL){
        printf("您录入的第%d本书的书名是：%s\n",i,book->title);
        printf("您录入的第%d本书的作者是：%s\n",i,book->author);       
        i++;
        book=book->next;
        }
       
}

void freelibrary(struct Book *library){
        struct Book* temp;//temp跳出函数之后会自动释放，所以不用操心他的存在
        while(library!=NULL){
        temp=library->next;
        free(library);
        library=temp;
        }
}

int main(){
        struct Book *library=NULL;
        addBook(&library);
        char ans;


        while(1){
                printf("是否继续录入书本？Y/N\n");
                scanf("%s",&ans);
        if(ans=='Y'){
                printf("请继续录入书本信息\n");
                addBook(&library);
        }
        else{
        break;
        }
        }
       
        printf("是否打印目录?Y/N\n");
        scanf("%s",&ans);
                if(ans=='Y'){
                        printf("开始打印书目\n");
                        printfbook(library);
                }
                else{
                        printf("再见了您累");
                }
        freelibrary(library);
        return 0;
}



运行之后，打印的书名和作者会乱码。并显示单链表.EXE已停止工作。  麻烦帮我看看错误在哪，查了半天没看出来。
最佳答案
月排行榜 / 总排行榜
jzzc
2020-3-6 15:06:41
main函数换成这个，除了排版一共改了4处地方，功能应该正常
  1. int main(void)
  2. {
  3.         struct Book *library = NULL;
  4.         addBook(&library);
  5.         char ans;


  8.         while(1)
  9.         {
  10.                 printf("是否继续录入书本？Y/N\n");
  11.                 getchar();           //加入这个回收enter键
  12.                 scanf("%c", &ans); //char类型，不是字符串，别用%s
  13.                
  14.                 if(ans == 'Y')
  15.                 {
  16.                                 printf("请继续录入书本信息\n");
  17.                         addBook(&library);
  18.                 }
  19.                 else
  20.                 {
  21.                         break;
  22.                 }
  23.         }
  24.         
  25.         printf("是否打印目录?Y/N\n");
  26.         
  27.         getchar();           //加入这个回收enter键
  28.         scanf("%c", &ans); //一样的，char类型
  29.         
  30.         if(ans=='Y')
  31.         {
  32.                 printf("开始打印书目\n");
  33.                 printfbook(library);
  34.         }
  35.         else
  36.         {
  37.                 printf("再见了您累");
  38.         }
  39.         
  40.         freelibrary(library);
  41.         
  42.         return 0;
  43. }
复制代码




小甲鱼最新课程 -> https://ilovefishc.com
回复

使用道具 举报

qiuyouzhi
发表于 2020-3-6 09:56:49 | 显示全部楼层
不要把结构体变量和结构体指针定义在函数里，要么放在main里，要么
放在结构体定义的地方
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

召唤师
 楼主| 发表于 2020-3-6 13:05:55 | 显示全部楼层
qiuyouzhi 发表于 2020-3-6 09:56
不要把结构体变量和结构体指针定义在函数里，要么放在main里，要么
放在结构体定义的地方

不好意思，还是不太明白？能稍微具体点吗？
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

qiuyouzhi
发表于 2020-3-6 13:06:52 | 显示全部楼层
召唤师 发表于 2020-3-6 13:05
不好意思，还是不太明白？能稍微具体点吗？


你不是在函数里写了一堆struct什么的吗？
把他们放到结构体定义的地方
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

召唤师
 楼主| 发表于 2020-3-6 13:18:17 | 显示全部楼层
qiuyouzhi 发表于 2020-3-6 13:06
你不是在函数里写了一堆struct什么的吗？
把他们放到结构体定义的地方

#include <stdio.h>
#include <stdlib.h>
# include <malloc.h>

void addBook(struct Book **library);
void getinput(struct Book *book);
void printfbook(struct Book *library);
void freelibrary(struct Book *book);
struct Book{
        char title[128];
        char author[40];
        struct Book *next;
}*book,*temp;

void getinput(struct Book *book){
        printf("请输入书名[title]\n");
        scanf("%s",book->title);
        printf("请输入作者[author]\n");
        scanf("%s",book->author);
}

void addBook(struct Book **library){
        //struct Book *book,*temp;
        book=NULL;
        temp=NULL;
        book=(struct Book*)malloc(sizeof(struct Book));
        if(book==NULL){
                printf("内存分配失败了");
                exit(1);
        }
        else{
                getinput(book);
        }
        if (*library != 0)                 
        {                                
                temp = *library;
                *library = book;
                book->next = temp;
        }
        else                           
        {
                *library = book;
                book->next = NULL;
        }
}

void printfbook(struct Book *library){
//        struct Book *book;
        book=library;
        int i=1;
        while(book!=NULL){
        printf("您录入的第%d本书的书名是：%s\n",i,book->title);
        printf("您录入的第%d本书的作者是：%s\n",i,book->author);       
        i++;
        book=book->next;
        }
       
}

void freelibrary(struct Book *library){
        //struct Book* temp;//temp跳出函数之后会自动释放，所以不用操心他的存在
        while(library!=NULL){
        temp=library->next;
        free(library);
        library=temp;
        }
}

int main(){
        struct Book *library=NULL;
        addBook(&library);
        char ans;


        while(1){
                printf("是否继续录入书本？Y/N\n");
                scanf("%s",&ans);
        if(ans=='Y'){
                printf("请继续录入书本信息\n");
                addBook(&library);
                    }
        else{
             break;
             }
        }
       
        printf("是否打印目录?Y/N\n");
        scanf("%s",&ans);
                if(ans=='Y'){
                        printf("开始打印书目\n");
                        printfbook(library);
                }
                else{
                        printf("再见了您勒");
                }
        freelibrary(library);
        return 0;
}

这样也不行好像。
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

qiuyouzhi
发表于 2020-3-6 13:28:35 | 显示全部楼层
召唤师 发表于 2020-3-6 13:18
#include
#include
# include

哪有把声明写在结构体定义前面的。。。
  1. #include <stdio.h>
  2. #include <stdlib.h>
  3. # include <malloc.h>

  5. struct Book{
  6.          char title[128];
  7.          char author[40];
  8.          struct Book *next;
  9. }*book,*temp;
  10.   void addBook(struct Book **library);
  11. void getinput(struct Book *book);
  12. void printfbook(struct Book *library);
  13. void freelibrary(struct Book *book);

  15. void getinput(struct Book *book){
  16.          printf("请输入书名[title]\n");
  17.          scanf("%s",book->title);
  18.          printf("请输入作者[author]\n");
  19.          scanf("%s",book->author);
  20. }

  22. void addBook(struct Book **library){
  23.          //struct Book *book,*temp;
  24.          book=NULL;
  25.          temp=NULL;
  26.          book=(struct Book*)malloc(sizeof(struct Book));
  27.          if(book==NULL){
  28.                  printf("内存分配失败了");
  29.                  exit(1);
  30.          }
  31.          else{
  32.                  getinput(book);
  33.          }
  34.          if (*library != 0)                 
  35.          {                                
  36.                  temp = *library;
  37.                  *library = book;
  38.                  book->next = temp;
  39.          }
  40.          else                           
  41.          {
  42.                  *library = book;
  43.                  book->next = NULL;
  44.          }
  45. }

  47. void printfbook(struct Book *library){
  48. //        struct Book *book;
  49.          book=library;
  50.          int i=1;
  51.          while(book!=NULL){
  52.          printf("您录入的第%d本书的书名是：%s\n",i,book->title);
  53.          printf("您录入的第%d本书的作者是：%s\n",i,book->author);        
  54.          i++;
  55.          book=book->next;
  56.          }
  57.          
  58. }

  60. void freelibrary(struct Book *library){
  61.          //struct Book* temp;//temp跳出函数之后会自动释放，所以不用操心他的存在
  62.         while(library!=NULL){
  63.          temp=library->next;
  64.          free(library);
  65.          library=temp;
  66.          }
  67. }

  69. int main(){
  70.          struct Book *library=NULL;
  71.          addBook(&library);
  72.          char ans;


  75.          while(1){
  76.                  printf("是否继续录入书本？Y/N\n");
  77.                  scanf("%s",&ans);
  78.          if(ans=='Y'){
  79.                  printf("请继续录入书本信息\n");
  80.                  addBook(&library);
  81.                      }
  82.          else{
  83.               break;
  84.               }
  85.          }
  86.          
  87.          printf("是否打印目录?Y/N\n");
  88.          scanf("%s",&ans);
  89.                  if(ans=='Y'){
  90.                          printf("开始打印书目\n");
  91.                          printfbook(library);
  92.                  }
  93.                  else{
  94.                          printf("再见了您勒");
  95.                  }
  96.          freelibrary(library);
  97.          return 0;
  98. }
复制代码
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

召唤师
 楼主| 发表于 2020-3-6 13:58:37 | 显示全部楼层
qiuyouzhi 发表于 2020-3-6 13:28
哪有把声明写在结构体定义前面的。。。

你说的这个我知道了，但是还是不行……
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

qiuyouzhi
发表于 2020-3-6 13:59:27 | 显示全部楼层
召唤师 发表于 2020-3-6 13:58
你说的这个我知道了，但是还是不行……

可我这里编译过了啊
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

召唤师
 楼主| 发表于 2020-3-6 14:18:50 | 显示全部楼层
qiuyouzhi 发表于 2020-3-6 13:59
可我这里编译过了啊

编译完 输入两本书的内容，打印一下可以吗？我编译可以啊，运行不行啊
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

jzzc
发表于 2020-3-6 15:06:41 | 显示全部楼层    本楼为最佳答案   
main函数换成这个，除了排版一共改了4处地方，功能应该正常
  1. int main(void)
  2. {
  3.         struct Book *library = NULL;
  4.         addBook(&library);
  5.         char ans;


  8.         while(1)
  9.         {
  10.                 printf("是否继续录入书本？Y/N\n");
  11.                 getchar();           //加入这个回收enter键
  12.                 scanf("%c", &ans); //char类型，不是字符串，别用%s
  13.                
  14.                 if(ans == 'Y')
  15.                 {
  16.                                 printf("请继续录入书本信息\n");
  17.                         addBook(&library);
  18.                 }
  19.                 else
  20.                 {
  21.                         break;
  22.                 }
  23.         }
  24.         
  25.         printf("是否打印目录?Y/N\n");
  26.         
  27.         getchar();           //加入这个回收enter键
  28.         scanf("%c", &ans); //一样的，char类型
  29.         
  30.         if(ans=='Y')
  31.         {
  32.                 printf("开始打印书目\n");
  33.                 printfbook(library);
  34.         }
  35.         else
  36.         {
  37.                 printf("再见了您累");
  38.         }
  39.         
  40.         freelibrary(library);
  41.         
  42.         return 0;
  43. }
复制代码




小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

召唤师
 楼主| 发表于 2020-3-6 15:26:10 | 显示全部楼层
jzzc 发表于 2020-3-6 15:06
main函数换成这个，除了排版一共改了4处地方，功能应该正常

大哥，终于可以了，没想到啊。我排查了快一天了。谢啦。为什么不用getchar会出现这样的差异？
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

jzzc
发表于 2020-3-6 15:32:54 | 显示全部楼层
召唤师 发表于 2020-3-6 15:26
大哥，终于可以了，没想到啊。我排查了快一天了。谢啦。为什么不用getchar会出现这样的差异？

导致出现exe停止应该是因为%c写成了%s，和getchar()应该没有关系，加getchar()是因为在输入完作者的信息后摁了回车，这导致scanf读取到了回车直接break了，所以加一个getchar()把这个回车给消灭掉，不让他影响我们的代码逻辑
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

召唤师
 楼主| 发表于 2020-3-6 15:48:27 | 显示全部楼层
jzzc 发表于 2020-3-6 15:32
导致出现exe停止应该是因为%c写成了%s，和getchar()应该没有关系，加getchar()是因为在输入完作者的信息 ...

有影响，不过我明白了，谢谢！
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

jzzc
发表于 2020-3-6 15:52:09 | 显示全部楼层
召唤师 发表于 2020-3-6 15:48
有影响，不过我明白了，谢谢！

嗯嗯
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

返回列表 发新帖
高级模式
B Color Image Link Quote Code Smilies
您需要登录后才可以回帖 登录 | 立即注册

本版积分规则

小黑屋|手机版|Archiver|鱼C工作室 ( 粤ICP备18085999号-1 | 粤公网安备 44051102000585号)

GMT+8, 2025-7-2 05:25

Powered by Discuz! X3.4

© 2001-2023 Discuz! Team.

快速回复 返回顶部 返回列表