C++实现文件存储用户注册登录校验系统的代码问题修复

已定位的逻辑缺陷 #

代码读不全用户数据、运行异常是多个逻辑错误叠加导致的，逐个列清：

• 注册写文件默认用覆盖模式：ofstream File("Test.txt")每次打开都会清空原有文件内容，永远只能存储最后一个注册的用户
• 登录校验时文件提前关闭：读文件的while循环第一遍遍历就执行了input.close()，后续内容根本读不到，自然只能匹配到文件开头的内容
• 校验规则完全错位：逐行匹配时只要任意一行命中用户名/密码/ID就打合法标记，不校验三个信息是否属于同一个用户，甚至把密码匹配的分支写错成了给IsNamevalid赋值
• 文件打开模式错误：读文件的ifstream不需要传ios::app参数，这个参数是给写文件追加内容用的
• 控制流混乱：main函数顺序调用所有功能函数，各个功能函数内部又递归调用Menu，调用栈会无限堆积，逻辑跳转完全不可控
• 校验标记初始值错误：密码校验标记IsPassvalid初始值设为true，默认密码永远判定为正确
• 循环判断逻辑失效：do-while条件用了逗号表达式，实际只校验ID是否合法，用户名、密码的校验结果不参与循环判断
• 无效代码：登录成功分支只写了字符串字面量，没有通过cout输出，用户看不到成功提示
• 文件读取写法有坑：用while(!eof())做循环条件会多读一次末尾空行，容易引发匹配错误

修复后完整可运行代码 #

#include <iostream>
#include <string>
#include <fstream>
using namespace std;

class UserFile {
private:
    string Name, password;
    string ID;
public:
    void Menu();
    void Register();
    bool Login();
};

int main()
{
    UserFile Obj;
    while(1) {
        Obj.Menu();
    }
    return 0;
}

void UserFile::Menu()
{
    int Option = 0;
    system("cls");
    cout << "1: 注册" << endl;
    cout << "2: 登录" << endl;
    cout << "3: 退出" << endl;
    cout << "请选择选项: " << endl;
    cin >> Option;

    switch(Option)
    {
        case 1: 
            Register();
            break;
        case 2:
            if(Login()) {
                cout << "登录成功！" << endl;
            } else {
                cout << "用户名/密码/ID错误，请重试" << endl;
            }
            system("pause");
            break;
        case 3:
            exit(0);
            break;
        default:
            cout << "输入错误，请重新选择" << endl;
            system("pause");
            break;
    }
}

void UserFile::Register()
{
    system("cls");
    // 追加模式打开文件，不覆盖原有用户数据
    ofstream File("Test.txt", ios::app); 
    cout << "注册页面: \n\n\n";

    cout << "输入用户名: ";
    cin >> Name;
    File << Name << endl;

    cout << "输入密码: ";
    cin >> password;
    File << password << endl;
        
    cout << "输入ID: ";
    cin >> ID;
    File << ID << endl;

    File.close();
    cout << "注册完成！" << endl;
    system("pause");
}

bool UserFile::Login()
{
    string storedName, storedPass, storedID;
    string inputName, inputPass, inputID;
    system("cls");

    cout << "登录页面: \n";
    cout << "输入用户名: " << endl;
    cin >> inputName;
    cout << "输入密码: " << endl;
    cin >> inputPass;
    cout << "输入ID: " << endl;
    cin >> inputID;

    ifstream input("Test.txt");
    if(!input.is_open()) {
        cout << "用户数据文件不存在，请先注册" << endl;
        return false;
    }

    // 按组读取：每次读三行对应用户名、密码、ID，整组匹配
    while (getline(input, storedName) && getline(input, storedPass) && getline(input, storedID))
    {
        if (inputName == storedName && inputPass == storedPass && inputID == storedID)
        {
            input.close();
            return true;
        }
    }
    input.close();
    return false;
}

关键修复点说明 #

• 注册写入改用追加模式，支持多用户数据存储，不会覆盖历史数据
• 登录读文件时按「用户名-密码-ID」三行一组读取，整组信息和用户输入做比对，不会出现零散匹配的逻辑错误
• 文件关闭操作移到读取循环结束后，保证能遍历完全部存储的用户数据
• 重构控制流，去掉函数之间的递归调用，用main函数的外层循环承接菜单跳转，避免调用栈溢出
• 修正原逻辑里错误的布尔值赋值、无效的逗号表达式判断、无输出的无效代码
• 增加文件不存在的异常判断，避免首次运行无数据文件时程序崩溃

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