Co jest nie tak z tym kodem?

Question

#include <iostream>

using namespace std;

string  PIN;

int main()
{
    cout << "witaj w naszym banku!" << endl;
    cout <<"podaj numer PIN: ";
    cin >> PIN;

    if(PIN=="2987");
    {
        cout <<"Poprawny PIN";
    }
    if(PIN!="2987");
    {
        cout << "Podano zły pin";
    }
       return 0;

}

Gdy włączam F9 wyskakuje mi komunikat taki jak powinien czy podaj numer pin a gdy podaje zły wysakują 2 komunikaty zaraz obok siebie : Poprawny pinPodano zły ping i nie wiem jak to rozłączyć to jest jedna rzecz a druga to gdy chcę zastąpić if(PIN!="2987"); funkcją else to wysakuje mi błąd ;/ powiedzccie proszę co jest nie tak ;/ Dziękuję i pozdrawiam

Answer 1

To pewnie dlatego że za ifami nie stawia się średników czyli powinno być np. tak:

if(PIN=="2987")

I w tym drugim ifie tak samo bez średnika.

Poza tym, nie lepiej aby numer pin był liczbą?

Comments

dziękuję za błyskawiczną odpowiedź, dopiero zaczynam naukę programowania:( dziękuję
jak na razie jest to przyuczanie się i biorę to po prostu z kursu

---

Nie ma sprawy, rozumiem ;) Mały błąd a jak widać dużo psuje.

---

PIN jako liczba to średni pomysł. Co jak PIN zaczyna się od 0? :P

---

A no fakt, o tym nie pomyślałem ;)

Answer 2

Drugi if lepiej żeby był else ifem jeśli i tak sprawdzają tą samą rzecz (tzn. jeśli jeden się sprawdzi, to drugi już na pewno nie. Tak na przyszłość - bezpieczniej i szybciej dla komputera.

Answer 3

po ifach nie powinno być średników.

Comments

dziękuję za odpowiedź ;)

Answer 4

już rozwiązane widzę ;d

Comments

TO mi podrzuciłeś czarną magię jak dla mnie przy moim poziomie nauki :D ale dziękuję

---

dlatego już edytowałem, a sam z rozpędu nie zauważyłem średników przy ifie ;)
tak czy owak, powodzenia w nauce! :)

Answer 5

napisz if bez  ;