C++条件语句多参数使用问题：用户名密码验证代码调试

Looks like you ran into a classic pitfall with the comma operator in C++! Let's break down what's wrong and fix your code step by step.

The Problem with Your Original Condition #

Your line:

if(username != "admin", password != "therealpassword")

uses the comma operator, which evaluates the first expression (username != "admin") and discards its result, then returns only the value of the second expression (password != "therealpassword"). That means your code only checks if the password is wrong—completely ignoring the username check! Not exactly what you want for proper validation.

The Correct Logic #

We need to check if either the username OR the password is incorrect to trigger the error message and loop. For this, use the logical OR operator ||. Alternatively, you can check if the combination of correct username AND correct password is NOT true (using !(...) with &&). Both approaches work; let's use the first one since it's more straightforward:

Corrected Full Code #

#include <iostream>
#include <string>

using namespace std;

int main() {
    string username;
    string password;

    cout << "Hello there. To access this program, please put in your user name and password." << endl;
    
    do {
        cout << "Username: ";
        cin >> username;
        cout << "Password: ";
        cin >> password;

        if (username != "admin" || password != "therealpassword") {
            cout << "Username or password is incorrect. Please try again." << endl;
        }
    } while (username != "admin" || password != "therealpassword");

    cout << "Access granted! Welcome to the program." << endl;
    return 0;
}

Key Notes #

• Logical OR (||): This returns true if at least one of the conditions is true. So if either username is wrong OR password is wrong, we show the error and repeat the loop.
• Loop Structure: I adjusted your code to use a do-while loop directly (since you want to prompt at least once), which is cleaner than nesting a do inside an if.
• Avoiding Comma Operator: Remember, the comma operator is almost never what you want for condition checks—it's meant for sequencing expressions, not combining logical checks.

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