Программа сбой при попытке доступа к файлу, fstream --C++

Question

Я создаю базу данных пользователей. Когда я пытаюсь открыть "dataBase.txt", который содержит всех пользователей и пароли, консоль появляется (что должно произойти, поскольку это консольное приложение), но в нем говорится, что программа уже завершена. Когда я закрываю его, мой компьютер скажет мне, что программа разбилась. Функция сохраняется в классе.

После некоторой отладки кода, кажется, врезаться в ifstream fin("dataBase.txt");

Ошибка не возвращается из компилятора.

Код для вызываемой функции:

void User_Psw::UserCheck() 
{ 
    // read from the database 
    ifstream fin("dataBase.txt"); 

    while (!fin.eof()) 
    { 
     fin >> Usernames[sizeOfDatabase] >> Password[sizeOfDatabase]; 
     sizeOfDatabase++; //The Number of lines in .txt 
    } 

    // rest of the program 
    cout << "Username: "; 
    cin >> username; 

    getNameIndex(); 

    cout << "Password: "; 
    cin >> password; 

    if(!PasswordMatches()) 
    { 
     cout << "Access denied"; 
    } 
} 

Я могу добавить больше фрагментов кода, если просил.

Answer 1

Don't use fin.eof() to control a loop. Эта функция полезна только после провала чтения.

Вероятная причина сбоя заключается в том, что вы назначаете Usernames[sizeOfDatabase], что может быть за пределами Usernames.capacity(). Канонический способ добавления и элемента к std::vector - это позвонить push_back().

Поскольку ваши контейнеры std::vector<std::string>, это лучший подход ...

std::string username, password; 
while (fin >> username >> password) 
{ 
    Usernames.push_back(username); 
    Passwords.push_back(password); 
    ++sizeOfDatabase; 
} 

Конечно, если вы хотите знать количество имен пользователей и паролей, как только файл был прочитан, вы можете позвонить Usernames.size() (который должен быть таким же, как Passwords.size()); это может устранить необходимость сохранения sizeOfDatabase.

Лично я использовал бы один контейнер для имен пользователей и (salted, hashed) passwords, а не два отдельных контейнера; либо std::map<std::string, std::string>, либо, возможно, std::unordered_map<std::string, std::string>, что делает поиск пароля приятным и быстрым для каждого имени пользователя.

Answer 2

Я думаю, вы должны начать, добавив в конструктор ifstream как вы не указали вы хотите, файл открыт для ввода:

ifstream fin("dataBase.txt", ifstream::in); 

if(!fin.good()) 
{ 
    cout << "Failed to open database file." << endl; 
    return; 
} 

См http://www.cplusplus.com/reference/fstream/ifstream/ifstream/ для дополнительной литературы.