Warunek if/else if/else

Question

Chcę używając warunków if/else if/else stworzyć program do logowania, gdzie poprawny login to "login" a hasło to "hasło" lub "hasło2", oraz żeby w przypadku braku poprawności jednej z tych danych program o tym informował, np. wpisując niepoprawny login i poprawne hasło, pojawiał się komunikat: "Niepoprawny login". W moim kodzie natomiast wpisując zły login z poprawnym hasłem wyskakuje błędny komunikat - "Niepoprawne hasło". To samo tyczy się innych warunków. Proszę powiedzieć mi co robię źle.

#include <iostream>

using namespace std;

string login,haslo;

int main ()
{
cout << "Wprowadz login: " << endl;
cin >> login;
cout << "Wprowadz haslo: " << endl;
cin >> haslo;

if (login=="login" && haslo=="haslo"||haslo=="haslo2")
{
cout << "Poprawne dane";
}
else if (login!="login" && haslo=="haslo"||haslo=="haslo2")
{
cout << "Niepoprawny login";
}
else if (login=="login" && haslo!="haslo"||haslo!="haslo2")
{
cout << "Niepoprawne haslo";
}
else
{
cout << "Niepoprawny login i haslo";
}

return 0;
}

 

Answer 1

Operator AND (&&) ma wyższy priorytet niż OR (||). Priorytety operatorów w programowaniu mają się do siebie tak jak działania w mnożeniu. Jedno działanie wykonuje się przed innym. W Twoim wypadku w każdym z wyrażeń warunkowych, ciąg operacji jest następujący:
1. Sprawdzenie czy "login" jest równy (nie równy) zmiennej login.
2. Sprawdzenie czy "haslo" jest równe (nie równe) zmiennej haslo.
3. Wykonanie iloczynu logicznego (AND) na wartościach wyznaczonych w punktach 1 oraz 2.
4. Sprawdzenie czy "haslo2" jest równe (nie równe) zmiennej haslo.
5. Wykonanie sumy logicznej (OR) na wyznaczonych wartościach z punktów 3 oraz 4.

Rozwiązaniem Twojego problemu pewnie będzie wzięcie w  nawias wyrażeń po znaku iloczynu logicznego (&&). Wykonanie tego sprawi,że sprawdzanie warunku będzie wyglądało następująco:
1.Sprawdzenie czy "haslo" jest równe (nie równe) zmiennej haslo.
2. Sprawdzenie czy "haslo2" jest równe (nie równe) zmiennej haslo.
3. Suma logiczna (OR) na wyznaczonych wartościach z punktów 1 oraz 2.
4. Sprawdzenie czy "login" jest równy (nie równy) zmiennej login.
5. Iloczyn logiczny na wyznaczonych wartościach z punktów 4 oraz 3.

Wśród wyrażeń logicznych najwyższych priorytet ma negacja (NOT), dalej jest iloczyn logiczny (AND) oraz suma logiczna (OR). Działania w nawiasach wykonywane są jakie pierwsze, tak jak na matematyce w podstawówce.;) Tak było w podstawówce, takie same priorytety działań sobie w algebrze Boola i takie są w programowaniu.

Answer 2

Pilnuj łączenia operatorów logicznych. Prawdopodobnie wszędzie gdzie napisałeś

if (A && B || C)

Powinieneś napisać:

if(A && (B || C))

Answer 3

Warunki od haseł wez w dodatkowy nawias. Tzn. np.: 

if(login == "login" && (haslo == "haslo" || haslo == "haslo2") )

I w ostatnim else if tak samo te hasla w nawias i zamiast || wrzuć &&. W sposob jaki masz teraz nawet jesli podane haslo to "haslo2" to i tak zostanie odrzucone bo nie jest "haslo".

Answer 4

Warunek logiczny || powinieneś wziąć w nawias