Błąd w kodzie

Question

Witam, dopiero zacząłem uczyć się języka c++ i napotkałem się na błąd:

#include <iostream>

using namespace std;

int main()
{
    int login, haslo;
    cout<<"Witamy na facebooku"<<endl;
    cout<<"By sie zalogowac prosze podac swoj login i haslo"<<endl;

    cout<<"Podaj login: "<<endl;
    cin>>login;
    cout<<"Podaj haslo: "<<endl;
    cin>>haslo;

 if ((login=="admin")&&(haslo=="haslo12345"))
    {
        cout<<"Haslo poprawne"<<endl;
    }

    else
    {
        cout<<"Prosze zalogowac sie jeszcze raz"<<endl;
    }

    return 0;
}

w linijce 16 tam gdzie zaczyna się instrukcja warunkowa pokazało mi że jest błąd nie wiem o co chodzi może ktoś mi pomóc ?

Answer 1

Wydaje mi się,że login powinien być typu string, a nie int jak ty to zadeklarowałeś.Chodzi o to,że int może zawierać tylko liczby,więc porównanie nie jest prowidłowe.

Comments

A no tak przecież login to słowo a słowa trzeba zadeklarować w typie string. Dzięki :)

---

Niekoniecznie, string jest wszechstronny, obsługuje i liczby i znaki. Jednak to co w nim jest, jest traktowane jako znak nie jako liczba więc np

string a="15",b="20"; to std::cout<<a+b"<<std::endl; // wyświetli wynik 1520

cóż podsumowując, string obsługuje wszystko, lecz wszystko traktuje jako tekst, jeżeli chciałbyś dodać zgaduje musisz castować stringa do inta.

Answer 2

Jako iż jesteś nowy to napiszę od razu że zawsze warto podać treść błędu bo wtedy łatwiej pomóc. Tutaj dość mały kod więc widać od razu że masz zmienne "login" i "haslo" zdefiniowane jako int (liczba) a chcesz porównać czy nie są one tekstem i pewnie też ten tekst próbujesz do nich wpisywać. No tak się nie da, jeśli chcemy żeby w zmiennej przechowywać tekst musi być typu string.

Czyli powinno być przy deklaracji 

string login, haslo;

A poza tym ten warunek może wyglądać tak, te dodatkowe nawiasy są w tym przypadku zbędne

if (login=="admin" && haslo=="haslo12345")

Answer 3

Zmień z int na string przy login i hasło (7 linijka)