为何我写的文件操作代码在switch case中失效但单独运行正常？

我一眼就看出问题所在了——这是输入缓冲区残留的换行符加上gets()函数的特性导致的，单独运行没问题但放到switch里就翻车。

核心原因拆解 #

当你用cin >> menu读取用户选择的数字后，用户按下的回车键（也就是换行符\n）并没有被cin吃掉，而是留在了输入缓冲区里。接下来调用gets(info)时，这个函数会从缓冲区里读取字符，直到遇到换行符为止——结果它一上来就碰到了残留的\n，直接认为已经读完一行，根本不会等你输入内容，自然没法把信息写入文件。

而单独执行这段文件操作代码时，输入缓冲区是空的，gets()会正常等待用户输入，所以能正常工作。

修复方案 #

我给你两个关键修复点，再加上一些安全性优化：

1. 清理输入缓冲区
   在cin >> menu之后，添加代码清空缓冲区里的残留字符，确保后续输入能正常读取：

cin.ignore(numeric_limits<streamsize>::max(), '\n');

记得要包含<limits>头文件才能使用numeric_limits。

2. 替换不安全的gets()
gets()已经被C++标准废弃了，它不检查输入长度，很容易造成缓冲区溢出。换成更安全的fgets()：

fgets(info, sizeof(info), stdin);

另外，fgets()会把用户输入的换行符也读进去，如果不需要的话，可以手动去掉：

info[strcspn(info, "\n")] = '\0';

修改后的完整代码 #

#include<iostream>
#include<stdio.h>
#include<limits>
#include<cstring>
using namespace std;

int main () {
    int menu;
    FILE *files;
    char info[150];
    
    cout<< " \n ** MENU ** \n";
    cout<< "\n 1- Create File";
    cout<< "\n 2- Copy File";
    cout<< "\n 3- Copy without spaces";
    cout<< "\n 4- Merge Files";
    cout<< "\n 5- Quit \n";
    cout<< "\n Please Select : ";
    
    cin>>menu;
    // 清理输入缓冲区的残留换行符
    cin.ignore(numeric_limits<streamsize>::max(), '\n');
    
    switch (menu) {
        case 1:
            files = fopen("example.txt","w");
            // 一定要检查文件是否成功打开
            if(files == NULL){
                cout << "Failed to open file!" << endl;
                break;
            }
            printf("\n Information :");
            fgets(info, sizeof(info), stdin);
            // 移除换行符
            info[strcspn(info, "\n")] = '\0';
            // 用格式化字符串更安全，避免注入风险
            fprintf(files, "%s", info);
            fclose(files);
            cout << "File created and content saved successfully!" << endl;
            break;
        case 5:
            cout << "Quitting program..." << endl;
            break;
        default:
            cout << "\n Error: Invalid selection!" << endl;
    }
    return 0;
}

额外注意事项 #

• 永远要检查文件是否成功打开（if(files == NULL)），否则后续操作会导致未定义行为。
• 使用fprintf时，尽量用%s这类格式化占位符，不要直接把用户输入作为格式字符串，防止潜在的安全问题。

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