[已解决]帮我看看哪儿里错了，关于结构体的？

hanghanga

为什么main函数里，用注释的写法就可以正常运行，用现在的写法，输入的时候，第25行会报错：Segmentation fault

1. #include <stdio.h>
   
2. 
   
3. struct Date
   
4. {
   
5.     int year;
   
6.     int month;
   
7.     int day;
   
8. };
   
9. 
   
10. struct Book
    
11. {
    
12.     char title[128];
    
13.     char author[40];
    
14.     float price;
    
15.     struct Date date;
    
16.     char publisher[40];
    
17. };
    
18. 
    
19. struct Book *getInput(struct Book *book);
    
20. void printBook(struct Book *book);
    
21. 
    
22. struct Book *getInput(struct Book *book)
    
23. {
    
24.     printf("请输入书名：");
    
25.     scanf("%s", book->title);
    
26.     printf("请输入作者：");
    
27.     scanf("%s", book->author);
    
28.     printf("请输入售价：");
    
29.     scanf("%f", &book->price);
    
30.     printf("请输入出版日期：");
    
31.     scanf("%d-%d-%d", &book->date.year, &book->date.month, &book->date.day);
    
32.     printf("请输入出版商：");
    
33.     scanf("%s", book->publisher);
    
34. 
    
35.     return book;
    
36. }
    
37. 
    
38. void printBook(struct Book *book)
    
39. {
    
40.     printf("\n\n======开始打印书籍信息======\n\n");
    
41.     printf("书名：%s\n", book->title);
    
42.     printf("作者：%s\n", book->author);
    
43.     printf("售价：%.2f\n", book->price);
    
44.     printf("日期：%4d-%2d-%0.2d\n", book->date.year, book->date.month, book->date.day);
    
45.     printf("出版社：%s\n", book->publisher);
    
46. }
    
47. 
    
48. int main(void)
    
49. {
    
50.     struct Book *book = NULL;
    
51.     // struct Book book;
    
52.     // struct Book *my_book = getInput(&book);
    
53.     struct Book *my_book = getInput(book);
    
54.     printBook(my_book);
    
55. 
    
56.     return 0;
    
57. }

复制代码

最佳答案

月排行榜 / 总排行榜

jackz007

2023-12-25 23:04:10

1. int main(void)
   
2. {
   
3.     struct Book * book = NULL              ; // 【错误】：这样定义的 book 只是一个指针，而指针是不能储存结构体数据的，你需要定义的是一个结构体
   
4.     struct Book * my_book = getInput(book) ; // 【错误】：book 被初始化为 NULL 也就是 0 值，是一个无法访问的内存地址，所以，就出错了。
   
5.     printBook(my_book);
   
6. 
   
7.     return 0;
   
8. }

复制代码

        这样写才对

1. int main(void)
   
2. {
   
3.     struct Book book  ;
   
4.     getInput(& book)  ;
   
5.     printBook(& book) ;
   
6.     return 0;
   
7. }

复制代码

跳转到最佳答案楼层

FishC

问题出在第18行，你需要给字符数组分配足够的空间来存储输入的字符串。当前的定义是`char title`，这只能存储一个字符，应该修改为`char title[50]`，这样就可以存储最多50个字符的字符串了。同样，也需要修改`char author`和`char publisher`的定义。

另外，第25行的错误是因为传递给`getInput`函数的参数`book`为空指针。你可以通过将`struct Book book;`改为`struct Book *book = malloc(sizeof(struct Book));`来动态分配内存，并确保在使用完之后使用`free(book);`来释放内存。

修改后的代码如下：

1. #include <stdio.h>
2. #include <stdlib.h>
4. struct Date
5. {
6.     int year;
7.     int month;
8.     int day;
9. };
11. struct Book
12. {
13.     char title[50];
14.     char author[50];
15.     float price;
16.     struct Date date;
17.     char publisher[50];
18. };
20. struct Book *getInput(struct Book *book);
21. void printBook(struct Book *book);
23. struct Book *getInput(struct Book *book)
24. {
25.     printf("请输入书名：");
26.     scanf("%s", book->title);
27.     printf("请输入作者：");
28.     scanf("%s", book->author);
29.     printf("请输入售价：");
30.     scanf("%f", &book->price);
31.     printf("请输入出版日期：");
32.     scanf("%d-%d-%d", &book->date.year, &book->date.month, &book->date.day);
33.     printf("请输入出版商：");
34.     scanf("%s", book->publisher);
36.     return book;
37. }
39. void printBook(struct Book *book)
40. {
41.     printf("\n\n======开始打印书籍信息======\n\n");
42.     printf("书名：%s\n", book->title);
43.     printf("作者：%s\n", book->author);
44.     printf("售价：%.2f\n", book->price);
45.     printf("日期：%4d-%2d-%0.2d\n", book->date.year, book->date.month, book->date.day);
46.     printf("出版社：%s\n", book->publisher);
47. }
49. int main(void)
50. {
51.     struct Book *book = malloc(sizeof(struct Book));
52.     struct Book *my_book = getInput(book);
53.     printBook(my_book);
54.     free(book);
56.     return 0;
57. }

复制代码

这样修改后，你应该可以正常运行程序并且不会报错了。记得在使用完动态分配的内存后，要及时使用`free`函数释放内存。

以上回复来自 FishC AI 小助理，如未能正确解答您的问题，请继续追问。

hanghanga

FishC 发表于 2023-12-25 22:21
问题出在第18行，你需要给字符数组分配足够的空间来存储输入的字符串。当前的定义是`char title`，这只能存 ...

你的回答虽然不对，但提醒了我，谢谢。

FishC

hanghanga 发表于 2023-12-25 22:25
你爹回答虽然不对，但提醒了我，谢谢。

非常抱歉，如果我的回答有任何错误。如果你有其他问题或需要进一步的帮助，请随时告诉我。我会尽力提供正确的解答。祝你编程愉快！

以上回复来自 FishC AI 小助理，如未能正确解答您的问题，请继续追问。

歌者文明清理员

struct Book *getInput(struct Book *book)
{
    printf("请输入书名：");
    scanf("%s", &book->title);
    printf("请输入作者：");
    scanf("%s", &book->author);
    printf("请输入售价：");
    scanf("%f", &book->price);

没取地址，直接用了野变量（本人不是很擅长c/c++, 有错请指正）

jackz007

1. int main(void)
   
2. {
   
3.     struct Book * book = NULL              ; // 【错误】：这样定义的 book 只是一个指针，而指针是不能储存结构体数据的，你需要定义的是一个结构体
   
4.     struct Book * my_book = getInput(book) ; // 【错误】：book 被初始化为 NULL 也就是 0 值，是一个无法访问的内存地址，所以，就出错了。
   
5.     printBook(my_book);
   
6. 
   
7.     return 0;
   
8. }

复制代码

        这样写才对

1. int main(void)
   
2. {
   
3.     struct Book book  ;
   
4.     getInput(& book)  ;
   
5.     printBook(& book) ;
   
6.     return 0;
   
7. }

复制代码