学习开发C++登录系统时读取txt凭证文件失败，如何调整存储与读取逻辑

凭证存储格式建议 #

• 无需分文件存储，单文件存储凭证足够满足练习场景需求，反而能降低IO操作的复杂度。
• 推荐单用户占一行，邮箱和密码用特殊分隔符隔开，选择不会出现在合法邮箱、密码中的字符作为分隔符即可，比如英文竖线|、英文逗号,，示例行格式：test@example.com|12345678。
• 不建议账号密码分行存储，多用户场景下无法匹配账号密码对应关系，维护成本极高。

现有代码的核心问题 #

1. 读取逻辑错误：当前while循环遍历完整个文件后，temp_email只会保留文件最后一行的内容，除了最后一个注册的用户，其他所有用户都无法正常登录。同时while(!read.eof())是经典错误写法，eof标记是读取到文件末尾后才会触发，会导致最后一行多读一次空内容。
2. 存储逻辑存在歧义：直接拼接邮箱和密码写入文件，没有分隔符，会出现无法区分账号密码边界的问题，比如邮箱为a123密码为456、邮箱为a密码为123456的写入结果完全一致，校验时会出错。
3. 输入逻辑冗余：注册环节调用getline(cin >> ws, email)后又执行cin.ignore(1000, '\n')，会导致输入的邮箱后缀被意外截断，属于多余操作。
4. 函数返回值缺失：login函数声明为bool返回类型，但是多个分支没有写return语句，编译会触发警告，运行时可能出现未定义行为。
5. 全局变量滥用：计数器、账号、密码等变量全部定义为全局变量，逻辑耦合度极高，很容易出现变量值意外篡改的问题。
6. 流程逻辑缺失：注册完成后程序直接退出，没有自动进入登录流程，不符合常规登录系统的交互逻辑。

核心逻辑修改参考 #

写入凭证逻辑调整 #

写入时添加分隔符，避免边界歧义：

ofstream file_2("credentials.txt", ios::out | ios::app);
file_2 << email << "|" << new_passcode << "\n";
file_2.close();

读取凭证逻辑调整 #

读取时拆分每行内容，存储所有用户的账号密码对：

#include <vector>
// 读取前先定义容器存储所有凭证
vector<pair<string, string>> cred_list;
string line;
// 替换原来的eof循环
while (getline(read, line)) {
    size_t split_idx = line.find('|');
    if (split_idx != string::npos) {
        string em = line.substr(0, split_idx);
        string pw = line.substr(split_idx + 1);
        cred_list.emplace_back(em, pw);
    }
}

登录校验逻辑调整 #

遍历所有凭证匹配用户输入：

bool is_match = false;
for (auto &item : cred_list) {
    if (item.first == email && item.second == passcode) {
        is_match = true;
        break;
    }
}
if (is_match && counter < 3) {
    cout << "login successful";
    return true;
}

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