problem z warunkami w funkcji "if"

Question

Problem polega na tym że kiedy wpisuje poprawnie login a źle hasło funkcja uznaje że została wykonana poprawnie mimo że nie został spełniony warunek 3 "&&(haslo=="lampart")" chociaż przy większej ilości zmiennych program powinien analizować od lewej do prawej i w tym przypadku powinien zanegować funkcje. Co robię źle?

#include <iostream>

using namespace std;

string login,haslo;

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

    if ((login=="ogien")||(login=="Ogien")&&(haslo=="lampart"))
    {
        cout<<"Zgadza sie :)";
    }
    else
    {
        cout<<"Nie zgadza sie probuj dalej :)";
    }

    return 0;
}

Answer 1

http://en.cppreference.com/w/cpp/language/operator_precedence

&& jest ponad || w kolejności działania. Kompilator powinien wywalać ostrzeżenie o tym. Warunki z || daj we wspólnym nawiasie.

Answer 2

Musiałbyś pokombinować troszkę z nawiasami:

if ((login=="ogien"||login=="Ogien")&&haslo=="lampart")
{
    cout<<"Zgadza sie :)";
}

Najpierw jest nawias if()-a. Potem daję do nawiasu warunek ||, czyli albo "ogien", albo "Ogien". Jeśli ten warunek będzie spełniony, to wtedy pójdzie do następnego warunku, &&, czyli haslo=="lampart".

A najlepiej to zrobić to tak:

if ((tolower(login)=="ogien"&&haslo=="lampart")
{
    cout<<"Zgadza sie :)";
}

Funkcja tolower() (znajdująca się w bibliotece cctype, więc musisz ją zainkludować) zwraca łańcuch znaków, tylko że wszystkie litery zostają zamienione na małe. I po problemie :)

Answer 3

if(((login==ogien)||(login=="Ogien))&&(haslo=="lampart"))

Answer 4

Napisz w 2 if'ach najpierw login a w środku hasło a z kodu wynika, że kiedy login jest równy Ogień sprawdza czy hasło jest poprawne