C语言图书馆管理系统书籍删除功能异常问题求助

看起来你的图书馆管理系统里的书籍删除功能遇到了实打实的问题——明明执行了删除操作，文件里的记录却纹丝不动。我仔细看了你的代码，发现了几个关键的逻辑和操作错误，下面帮你一一梳理并给出修复方案。

代码中的核心问题分析 #

• findbook变量未初始化：你定义了char findbook;但没给初始值，当没找到匹配ISBN时，这个变量的值是随机的，可能会误判为"找到记录"，导致后续逻辑全乱套。
• getch()输入处理太粗糙：直接用getch() == 'y'判断，既不支持大写Y，也没处理输入缓冲区里的残留字符，很容易导致后续输入读取异常。
• 冗余的fseek调用：向临时文件写入记录时，fseek(fd, 0, SEEK_CUR);完全多余，wb+模式打开的文件，fwrite会自动移动指针到正确位置，这个调用反而可能干扰写入。
• fflush(stdin)是未定义行为：C标准里fflush(stdin)没有统一规范，不同编译器处理方式不一样，不能用来清空输入缓冲区。
• 文件资源泄漏：用户选择不删除时直接跳回主菜单，没关闭之前打开的fp文件指针，会导致文件资源无法正常释放。
• findbook未重置：循环处理下一个删除请求时，findbook还保留着上一次的结果，会导致判断错误。

修复后的完整删除函数代码 #

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

// 假设你的书籍信息结构体是这样的（根据代码推断）
typedef struct {
    int isbn;
    char name[50];
    char Author[50]; // 这里假设Author是字符串，若实际是单个字符可改回char类型
} BookInfo;

BookInfo info;
FILE *fp, *fd;

void deletingbooks() {
    system("cls");
    int d;
    char findbook;
    char another = 'y';
    
    // 支持大小写输入，循环处理多次删除请求
    while (toupper(another) == 'Y') {
        system("cls");
        findbook = 'f'; // 每次循环初始化"未找到"状态
        printf("Enter the Book ID to delete: ");
        
        // 处理无效输入（比如用户输入非数字）
        while (scanf("%d", &d) != 1) {
            // 清空输入缓冲区的无效内容
            while (getchar() != '\n');
            printf("Invalid input! Please enter a valid Book ID: ");
        }
        // 清空输入缓冲区里的换行符
        while (getchar() != '\n');

        // 打开文件并检查是否成功
        fp = fopen("library.dat", "rb+");
        if (fp == NULL) {
            printf("Error opening library.dat! Can't proceed.\n");
            printf("Press any key to return to main menu...");
            getch();
            mainmenu();
            return;
        }

        rewind(fp);
        // 找到匹配记录后直接退出循环，提升效率
        while (fread(&info, sizeof(info), 1, fp) == 1) {
            if (info.isbn == d) {
                printf("The book record is available:\n\n");
                printf("Book name: %s\n", info.name);
                printf("Author name: %s\n", info.Author); // 若Author是单个字符，改回%c
                findbook = 't';
                break;
            }
        }

        if (findbook != 't') {
            printf("No record found!\n");
            fclose(fp); // 必须关闭文件再返回
            printf("Press any key to return to main menu...");
            getch();
            mainmenu();
            return;
        }

        printf("\nDo you want to delete this record? (Y/N): ");
        char choice = toupper(getch()); // 统一转成大写，支持大小写输入
        printf("\n");

        if (choice == 'Y') {
            fd = fopen("test.dat", "wb"); // 用wb模式即可，不需要wb+
            if (fd == NULL) {
                printf("Error creating temporary file!\n");
                fclose(fp);
                printf("Press any key to return to main menu...");
                getch();
                mainmenu();
                return;
            }

            rewind(fp);
            // 把除了要删除的记录之外的内容写入临时文件
            while (fread(&info, sizeof(info), 1, fp) == 1) {
                if (info.isbn != d) {
                    fwrite(&info, sizeof(info), 1, fd);
                }
            }

            // 关闭所有打开的文件
            fclose(fd);
            fclose(fp);

            // 替换原文件
            remove("library.dat");
            rename("test.dat", "library.dat");

            printf("Record deleted successfully!\n");
        } else {
            fclose(fp); // 取消删除时也要关闭文件
            printf("Deletion cancelled.\n");
        }

        printf("\nDelete another record? (Y/N): ");
        another = toupper(getch());
        printf("\n");
    }

    mainmenu();
}

// 假设你的主菜单函数存在
void mainmenu() {
    // 主菜单逻辑
}

关键修复说明 #

1. 初始化findbook：每次循环开始时将其设为'f'，确保"找到/未找到"的判断绝对可靠。
2. 完善输入处理：
   • 检查scanf的返回值，处理用户输入非数字的情况。
   • 用getchar()循环清空输入缓冲区，替代不安全的fflush(stdin)。
   • 用toupper()统一处理输入大小写，提升用户体验。
3. 优化文件操作：
   • 打开文件后必做空指针检查，避免程序崩溃。
   • 找到匹配记录后立即跳出循环，不用遍历整个文件。
   • 删除冗余的fseek调用，简化临时文件写入逻辑。
   • 确保所有打开的文件都被关闭，杜绝资源泄漏。
4. 修复Author打印：如果你的info.Author是字符串类型，把%c改成%s；如果确实是单个字符，改回%c即可。

内容的提问来源于stack exchange，提问作者Asma Maryam