Gdzie jest błąd? Krótki i prosty program.

Question

#include <iostream>

using namespace std;

string uzytkownik, haslo, powtorzonehaslo, login, password;

int main ()

{{{{
    cout << "Zajerestruj sie" << endl;
    cout << "utworz login:";
    cin >> uzytkownik;

    cout <<"utworz haslo:";
    cin >> haslo;

    cout <<"powtorz haslo:";
    cin >> powtorzonehaslo;

    if  ((powtorzonehaslo==haslo));
    {{{
    cout <<"dziekujemy za rejestracje" << endl;
    cout <<"zaloguj sie"<<endl;

    cout <<"podaj login:";
    cin >> login;

    cout <<"podaj haslo:";
    cin >> password;

    if ((login==uzytkownik)&&(haslo==password));
    {
        cout <<"zalogowano do systemu";
    }

    else

    {
        cout <<"nie zalogowano do systemu";
    }

    }}} // tu jest instrukcja poprawna

    else
    {{{
      while ((powtorzonehaslo==haslo));

      cout << "Zajerestruj sie" << endl;
      cout << "utworz login:";
      cin >> uzytkownik;

      cout <<"utworz haslo:";
      cin >> haslo;

      cout <<"powtorz haslo:";
      cin >> powtorzonehaslo;

    if  ((powtorzonehaslo==haslo));

   {{
    cout <<"dziekujemy za rejestracje" << endl;
    cout <<"zaloguj sie"<<endl;

    cout <<"podaj login:";
    cin >> login;

    cout <<"podaj haslo:";
    cin >> password;

    if ((login==uzytkownik)&&(haslo==password));
    {
        cout <<"zalogowano do systemu";
    }

    else
    {
        cout <<"nie zalogowano do systemu";
    }

    }}

    return 0;
}}}}

 

Answer 1

Słuchaj, po co ci tyle tych klamerek? I czemu nie podałeś błędu? Chociaż i tak w tym wypadku wszyscy go widzą...

Comments

Klamerki są jakby dla mnie, żeby móc sobie wszystko rozróżnić, bo jak widać dość zamotany jest ten program, bo w końcu to mój pierwszy program, praktycznie. Czy mógłbyś mi przybliżyć ten błąd?

---

Nie nie nie, tych klamerek jest po prostu za dużo ;) Nie potrzeba ich aż tyle. Bez nich jest O WIELE czytelniej, a przynajmniej dla... Dla wszystkich ;) Jak chcesz, to sobie rób tyle tych klamerek, ale tylko jeśli jesteś tak uparty ;) Musisz w sobie zaszczepić dobre nawyki, a robienie tylu klamer jest... Dziwne ;D A do tego dajesz średniki na końcu if()-ów i while()-ów. Tak nie można! Wtedy nic się nie wykona ;) Oto gotowy kod, dokładnie go przestudiuj i jakby co to pytaj ;)

#include <iostream>

using namespace std;

string uzytkownik,haslo,powtorzonehaslo,login,password;

int main ()
{
    cout << "Zajerestruj sie" << endl;
    cout << "utworz login:";
    cin >> uzytkownik;

    cout <<"utworz haslo:";
    cin >> haslo;

    cout <<"powtorz haslo:";
    cin >> powtorzonehaslo;

    if(powtorzonehaslo==haslo)
    {
        cout <<"dziekujemy za rejestracje" << endl;
        cout <<"zaloguj sie"<<endl;

        cout <<"podaj login:";
        cin >> login;

        cout <<"podaj haslo:";
        cin >> password;

        if((login==uzytkownik)&&(haslo==password))
        {
            cout <<"zalogowano do systemu";
        }
        else
        {
            cout <<"nie zalogowano do systemu";
        }
    } // tu jest instrukcja poprawna
    else
    {
        while((powtorzonehaslo==haslo))
        {
            cout << "Zajerestruj sie" << endl;
            cout << "utworz login:";
            cin >> uzytkownik;

            cout <<"utworz haslo:";
            cin >> haslo;

            cout <<"powtorz haslo:";
            cin >> powtorzonehaslo;

            if((powtorzonehaslo==haslo))
            {
                cout <<"dziekujemy za rejestracje" << endl;
                cout <<"zaloguj sie"<<endl;

                cout <<"podaj login:";
                cin >> login;

                cout <<"podaj haslo:";
                cin >> password;

                if((login==uzytkownik)&&(haslo==password))
                    cout <<"zalogowano do systemu";
                else
                    cout <<"nie zalogowano do systemu";
            }
        }
    }
    return 0;
}

 

---

Dzięki wielkie :) Jeszcze jestem niedokładny jeśli chodzi o te ważne detale, ale mam czas, zeby się nauczyć i wprawić ;) Niestety, ten poprawiony kod dalej nie działa tak jak chciałem, bo po wprowadzeniu błędnego, powtórzonego hasła program kończy się, nie zapętla rejestracji od nowa... :/

---

użyj pętli do while poczytaj o niej, pomyśl jak jej użyć.

---

@filowsky - ale chciałbyś, żebym ci ten kod poprawił i powiedział co poprawiłem, czy powiedzieć co sam masz zrobić?

---

Nie, nie, sam pomyślę w takim razie, dzięki za pomoc :)

---

W takim razie dam ci tylko parę rad:

• Nie musisz używać w if()-ach i while()-ach dwóch par nawiasów.

if((zmienna==5)) //<--- nie trzeba tak
if(zmienna==5) //<--- wystarczy tak ;)

• Jak tworzysz zmienne, to nazywaj je tak, żeby były bardziej czytelne:

int zmiennadruga; //<--- dwa wyrazy połączone w całość
int zmiennaDruga; //<--- widać gdzie się zaczyna kolejny wyraz
int zmienna_druga; //<--- tego nie lubię, bo jest zbyt długie, ale widać wyrazy ;)

Takie dwie małe porady. Działanie kodu będzie oczywiście to samo, ale kod będzie wyglądał przejrzyściej ;)

A jeśli chodzi o twój kod, to mała podpowiedź do niego: możesz zastąpić ten pierwszy blok if() - else jednym while()-em, tyle wystarczy ;)

---

Poszło miękko wszystko, dzięki za dobre rady! :D

---

Nie ma za co, ucz się dalej ;)

---

wystarczy ze w petli while wstawisz while(powtorzonehaslo!=haslo) zamiast while (powtorzonehaslo==haslo).
pzdr

Answer 2

Mam wrażenie, że nikt tego nie czyta przed zadaniem pytania, a wiele to by przyspieszyło i ułatwiło:

https://forum.pasja-informatyki.pl/faq

Comments

Wiem, bez sensu jest to pytanie, bo coś mi się pomyliło i niechcący kliknąłem dodaj w ogóle nie uwzględniając problemu. Otóż, zacząłem sobie pisać dosłownie wczoraj w C++ i postanowiłem napisać program, w którym można się zarejestrować, a następnie po wprowadzeniu odpowiednich danych zalogować. Program ma działać tak:
1. Rejestrujesz się i jeśli powtórzysz hasło bez pomyłki, program przenosi do logowania, jeżeli podasz odpowiednie dane przy logowaniu wyskakuje komunikat "zalogowano do systemu", jeżeli dane są błędne to wyświetla "nie zalogowano do systemu".
2. Inna opcja przy rejestracji: podajesz login, haslo i jesli powtarzasz błędnie hasło to program do skutku przenosi się do "panelu" rejestracji od początku. Jeżeli uda się wpisać w końcu poprawne dane przy rejestracji znowu mamy logowanie, które może zakończyć się powodzeniem lub nie itd.

No i problem pojawia się w tym, że gdy jestem w rejestracji i podaje złe powtórzone hasło to program nie zapętla mi ciągle tej czynności z powrotem do początku rejestracji tylko robi to raz i na tym program się kończy. Co tam trzeba poprawić?

---

wystarczy ze w petli while wstawisz while(powtorzonehaslo!=haslo) zamiast while (powtorzonehaslo==haslo).
pzdr

Answer 3

W linijce pętli / klasy nigdy nie ma średnika