先说结论能编译通过、能生成 exe编译器只报了 2个警告真正的问题集中在逻辑层其中有两个会直接崩溃/死循环。按严重程度排序如下。---一、崩溃级错误必改会损坏内存/程序崩溃1. contact.c:211 — 文件读取写进了指针变量本身且结构体大小读错fread(newnode, sizeof(struct PersonNode), 1, fp) // ← 严重错误- newnode 是 PersonNode**把整个结构体的字节写进栈上的指针变量8 字节里后面还照读 104 字节 → 栈被破坏newnod变成垃圾值运行必崩。- 且读的是 sizeof(PersonNode)而 Savecontactcontact.c:195写入的是 sizeof(structPerson)——存/读格式不一致即使去掉 也会把记录读串位。- ✅ 应改为fread(newnode, sizeof(struct Person), 1, fp)2. contact.c:58-78 —删除时循环永不推进删除失败即死循环while (cur ! NULL) {if (strcmp(cur-data.name, name) 0) { ... }} // ←循环体内没有 cur cur-next;- cur 永远停在 head。只有删除的是表头才能成功删任何一个中间/尾部联系人或名字不存在时 →无限死循环。- ✅ 循环末尾补 cur cur-next;。---二、逻辑错误功能做错需改3. contact.c:89 —修改联系人用错判断if (strcmp(cur-data.name, name) 1) // 应为 0- strcmp 只保证 0相等、0/0不等1 仅在首个不同字符恰好差 1 时才成立 →几乎永远找不到修改功能基本失效。- ✅ 改为 0。4. test.c:34-60 —菜单与枚举错位 缺少排序选项- 枚举顺序是 MODIFY3, SEARCH4但菜单写的是 “3.查找联系人 4.修改联系人”→用户选 3 实际执行的是修改选 4实际执行查找两个功能对调了。- 菜单有 “6.按名字进行排序”但switch 里没有 case SORT: →选 6 直接掉进 default 报错排序功能完全用不了。- ✅ 把菜单 3/4 文字对调并补上 case SORT: Sortcontact(con); break;。5. contact.c:215-216 —加载时 prev 未初始化newnode-next NULL;newnode-next NULL; // ←重复第二行本应 newnode-prev NULL;- 链表头节点的 prev 是 malloc 的随机垃圾值。Deletecontact 里用 cur-prev ! NULL 判断可能把垃圾当指针用 →随机崩溃。- ✅ 第二行改成 newnode-prev NULL;。---三、编译警告能过编译但应修6. test.c:2 —warning C4067#include test.h; // ←include 后面不能加分号- ✅ 去掉行尾 ;。7. test.c:3 —warning C4431menu() 缺返回类型隐式 intmenu() { ... }- C 已不支持默认 int建议改成 void menu(void)且声明进 test.h。---四、健壮性隐患新手阶段可后续加强- scanf(%s, ...) 无宽度限制contact.c:18,20,24,26,56,86,93,95,99,101,112输入超过 NAME/PHONE数组容量就缓冲区溢出。建议写成 scanf(%19s, ...) 等。- scanf(%s, newnode-data.name) 的 是错的contact.c:18、20、24、26、93、95、99、101数组名 是char(*)[20]%s 要的是 char*。碰巧能跑但属于类型不匹配应去掉 。age 的 是对的。- test.c:38,62 —int main() 里 return; 不带值MSVC 不报错我实测验证过但 GCC/Clang 会当错误且 main的返回值不确定。建议 return 0;。- test.c:33 —scanf(%d, input) 输入非数字时会失败input 保持旧值 →菜单死循环。建议检查 scanf 返回值。- contact.c:229 —提示信息写的是 contacts.dat实际文件名是 contacts.data错字。---五、补充说明- 刚才 Rebuild 报的 LNK1104: 无法打开 test0826.exe 不是代码问题——exe正被运行中的程序占用你说的已经在桌面打开应该就是这个程序在跑关掉程序再编译即可。- 源码是 GBK 编码在 VS 里正常用其它工具按 UTF-8 打开会显示乱码属正常现象不是错误。---注意两点1. exe 还在运行中之前 Rebuild 报 LNK1104 就是因为程序被占用。先在桌面上关掉正在跑的那个通讯录程序再去 VS里重新编译否则还会报无法打开 exe。2. 旧数据文件兼容旧代码的保存Save本来写的就是 struct Person新的加载改用相同大小所以现有 contacts.data能正常读出来。文件保持了原来的 GBK 编码、CRLF 换行和 Tab 缩进VS 里中文显示正常。插入《数据结构》通讯录搜索优化增加哈希表。降低查找时间复杂度
先说结论能编译通过、能生成 exe编译器只报了 2个警告真正的问题集中在逻辑层其中有两个会直接崩溃/死循环。按严重程度排序如下。---一、崩溃级错误必改会损坏内存/程序崩溃1. contact.c:211 — 文件读取写进了指针变量本身且结构体大小读错fread(newnode, sizeof(struct PersonNode), 1, fp) // ← 严重错误- newnode 是 PersonNode**把整个结构体的字节写进栈上的指针变量8 字节里后面还照读 104 字节 → 栈被破坏newnod变成垃圾值运行必崩。- 且读的是 sizeof(PersonNode)而 Savecontactcontact.c:195写入的是 sizeof(structPerson)——存/读格式不一致即使去掉 也会把记录读串位。- ✅ 应改为fread(newnode, sizeof(struct Person), 1, fp)2. contact.c:58-78 —删除时循环永不推进删除失败即死循环while (cur ! NULL) {if (strcmp(cur-data.name, name) 0) { ... }} // ←循环体内没有 cur cur-next;- cur 永远停在 head。只有删除的是表头才能成功删任何一个中间/尾部联系人或名字不存在时 →无限死循环。- ✅ 循环末尾补 cur cur-next;。---二、逻辑错误功能做错需改3. contact.c:89 —修改联系人用错判断if (strcmp(cur-data.name, name) 1) // 应为 0- strcmp 只保证 0相等、0/0不等1 仅在首个不同字符恰好差 1 时才成立 →几乎永远找不到修改功能基本失效。- ✅ 改为 0。4. test.c:34-60 —菜单与枚举错位 缺少排序选项- 枚举顺序是 MODIFY3, SEARCH4但菜单写的是 “3.查找联系人 4.修改联系人”→用户选 3 实际执行的是修改选 4实际执行查找两个功能对调了。- 菜单有 “6.按名字进行排序”但switch 里没有 case SORT: →选 6 直接掉进 default 报错排序功能完全用不了。- ✅ 把菜单 3/4 文字对调并补上 case SORT: Sortcontact(con); break;。5. contact.c:215-216 —加载时 prev 未初始化newnode-next NULL;newnode-next NULL; // ←重复第二行本应 newnode-prev NULL;- 链表头节点的 prev 是 malloc 的随机垃圾值。Deletecontact 里用 cur-prev ! NULL 判断可能把垃圾当指针用 →随机崩溃。- ✅ 第二行改成 newnode-prev NULL;。---三、编译警告能过编译但应修6. test.c:2 —warning C4067#include test.h; // ←include 后面不能加分号- ✅ 去掉行尾 ;。7. test.c:3 —warning C4431menu() 缺返回类型隐式 intmenu() { ... }- C 已不支持默认 int建议改成 void menu(void)且声明进 test.h。---四、健壮性隐患新手阶段可后续加强- scanf(%s, ...) 无宽度限制contact.c:18,20,24,26,56,86,93,95,99,101,112输入超过 NAME/PHONE数组容量就缓冲区溢出。建议写成 scanf(%19s, ...) 等。- scanf(%s, newnode-data.name) 的 是错的contact.c:18、20、24、26、93、95、99、101数组名 是char(*)[20]%s 要的是 char*。碰巧能跑但属于类型不匹配应去掉 。age 的 是对的。- test.c:38,62 —int main() 里 return; 不带值MSVC 不报错我实测验证过但 GCC/Clang 会当错误且 main的返回值不确定。建议 return 0;。- test.c:33 —scanf(%d, input) 输入非数字时会失败input 保持旧值 →菜单死循环。建议检查 scanf 返回值。- contact.c:229 —提示信息写的是 contacts.dat实际文件名是 contacts.data错字。---五、补充说明- 刚才 Rebuild 报的 LNK1104: 无法打开 test0826.exe 不是代码问题——exe正被运行中的程序占用你说的已经在桌面打开应该就是这个程序在跑关掉程序再编译即可。- 源码是 GBK 编码在 VS 里正常用其它工具按 UTF-8 打开会显示乱码属正常现象不是错误。---注意两点1. exe 还在运行中之前 Rebuild 报 LNK1104 就是因为程序被占用。先在桌面上关掉正在跑的那个通讯录程序再去 VS里重新编译否则还会报无法打开 exe。2. 旧数据文件兼容旧代码的保存Save本来写的就是 struct Person新的加载改用相同大小所以现有 contacts.data能正常读出来。文件保持了原来的 GBK 编码、CRLF 换行和 Tab 缩进VS 里中文显示正常。插入《数据结构》通讯录搜索优化增加哈希表。降低查找时间复杂度
