c++ - C++函数运行超出预期

当我运行程序时，用户可以登录，但是如果输入错误的用户名，它将再次运行检查用户名循环，表示他们没有输入有效的用户名。除了一件事情，这工作正常。假设他们尝试登录三次，而第三次尝试是正确的，则提示输入密码。他们输入后，要求输入第二个密码，然后再输入第三个密码。似乎正在完成其他尝试的功能。我想不出一种方法来检查这一点。有任何想法吗。

如果您查看它，可以看到我正在UserCheck内呼叫getNameIndex。我几乎肯定这是发生错误的地方。

检查用户的功能：

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

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

    while (fin >> username >> password)
    {
        Usernames.push_back(username);
        Password.push_back(password);
        ++sizeOfDatabase; // This may or may not be needed elsewhere.
    }

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

    getNameIndex();

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

    if(!PasswordMatches())
    {
        cout << "Access denied";
    }
    else
    {
        cout << "Success! You have logged in.";
    }
}

这是用户名检查功能

void User_Psw::getNameIndex()
{
    userThere = false;

    for(int i=0; i < sizeOfDatabase; i++)
    {
        if (Usernames[i] == username)
        {
            index = i;
            userThere = true;
        }
    }
    if (userThere == false)
    {
        cout << "\nThat user name does not exsist. \n";
        cout << "Please try again. \n\n";
        UserCheck();
    }
}

最佳答案

您的程序结构错误。

而不是让getNameIndex再次调用UserCheck（），您应该让getNameIndex返回一个布尔值-成功时为true，失败时为false。在循环中运行它，如下所示：

bool success = false;

while (!success)
{
 cout << "Username: ";
 cin >> username;

 success = getNameIndex();
}

另外，应该使用全局变量而不是全局变量。就像是：

success = getNameIndex(username);

和getNameIndex()不应执行任何I / O-调用getNameIndex()的函数也应负责打印错误消息。想象一下，如果您在不同的上下文中使用了getNameIndex()，例如当该程序正在由另一个程序运行或以自动方式运行时–那么打印到控制台将毫无意义。