funkcja if else

Question

Witam, nie mogę znaleźć błędu w programie.
Chodziło mi o to, aby program po podaniu loginu sprawdził od razu czy jest on poprawny i dopiero wtedy prosił o hasło. Jeśli login jest zły to ma dać komunikat. Niestety program po podaniu złego loginu wyświetla również informacje o złym haśle, którego jeszcze nie podano. Czy da się to jakoś zrobić?

 

#include <iostream>

using namespace std;
string login,haslo;
int main()
{
    cout << "Podaj login: ";
    cin >> login;
    if(login!="admin")
    cout << "zly login";
    else
{
    cout << "Podaj haslo: ";
    cin >> haslo;
}
    if(haslo!="123")

    cout <<"zle haslo";
    else
    cout << "zalogowano";
    return 0;
}

 

Answer 1

Jak inicjalizujesz zmienna haslo to domyślnie ma wartość null, więc jest różna od 123 i dlatego wypisuje komunikat.

Zamknij sekcje wprowadzania hasla w else loginu, aby wywoływało się tylko kiedy login jest poprawny.

Comments

Domyślnie zmienne typu int, jeśli są globalne mają wartość 0. Ale reszta i tak się zgadza.

 

Edit: nvm, byłem przekonany że hasło było podane jako int, sorry :)

Answer 2

Dzięki za info. Te zmienne to były string a nie int o ile to coś zmienia w ich domyślnej wartości. Zmieniłem kod programu, bo wersja pierwotna była trochę do d...y. Teraz jest to co chciałem. Może nie do końca trafnie sformułowałem pytanie na wstępnie i dodam, że dopiero raczkuje w temacie programowania. Oto efekt:

#include <iostream>

using namespace std;
string login,haslo;
int main()
{
    while (login!="admin")
    {
    cout << "Podaj login: ";
    cin >> login;
    if(login!="admin")
    cout << "zly login" << endl;
    }
    while (haslo!="123")
    {
    cout << "Podaj haslo: ";
    cin >> haslo;
    if(haslo!="123")
    cout <<"zle haslo"<< endl;
    }
    cout << "zalogowano";
    return 0;
}

Teraz po źle wpisanym loginie lub haśle prosi o ponowienie próby :)

Dzięki za pomoc 

Comments

 Te zmienne to były string a nie int o ile to coś zmienia w ich domyślnej wartości.

zmienne std::string są obiektami, więc ich inicjalizacją zajmuje się konstruktor - zawsze mają tą samą wartość domyślną czyli pusty string (niezależnie czy to obiekt globalny, lokalny czy jaki sobie nie wymyślisz). Tak dla wyjaśnienia