Expected initializer before int - o co chodzi?

Question

Witam. Jestem początkującym koderem i miałem ochotę napisać swój pierwszy autorski program. Jest to program liczący nasze BMI. Tutaj mam kod:

#include <iostream>

using namespace std;

int waga, wzrost, x //x to nasze BMI

int main()
{
    cout << "Podaj swoja waga: " << ;
    cin >> waga

    cout << "Podaj swoj wzrost" << ;
    cin >> wzrost

    x=waga/(wzrost*wzrost)

    cout << "Twoje BMI to " <<x << endl;

    if (x<18.5);
    cout << "Niedowaga";
    if (x>18.6 && x<=25);
    cout << "Norma";
    if (x>26 && x<=30);
    cout << "Nadwaga";
    if (x>31);
    cout << "Otylosc";

    return 0;
}

Kiedy próbowałem ten program skompilować wyskakuje mi błąd na linii 7 - expected initializer before int.

Jestem początkującym koderem. Proszę o pomoc oraz dobre rady, jak uniknąć takich problemów w przyszłości. Jeszcze dodam, że uczę się z kursu pana Mirka Zelenta - to jest bardzo dobry nauczyciel.

Kompilator to Code::Blocks

Answer 1

1. Słusznie, słusznie... w linijce 5 brakuje średnika 

int waga, wzrost, x; // Teraz dobrze

2. To nie wszystko. Po co takie coś:

cout << "Podaj swoja waga: " << ;

Powinno być tak:

cout << "Podaj swoja waga: ";

3. 

if (x<18.5); // źle!
    cout << "Niedowaga";

Twój program ZAWSZE wypisze "Niedowaga" ponieważ postawiłeś średnik bezpośrednio po nawiasie if. Z resztą if'ów jest podobnie. Pokasuj te średniki:

if (x<18.5)        // Teraz if zadziała :)
    cout << "Niedowaga";

4. Kolejny błąd, to to, że waga, wzrost i x są typu całkowitego int. A skoro te zmienne są całkowite, to wyrażenie:
x=waga/(wzrost*wzrost)
przypisze Ci do zmiennej x liczbę całkowitą. A skoro x jest liczbą całkowitą, to bez sensu sprawdzać taki warunek:
if( x < 18.5 )
Mój kompilator wypisuje mi warninga, że porównuję int z double. Ponieważ w C++ 18.5 jest traktowane jako double. Są to tak zwane stałe dosłowne... kiedyś się dowiesz ;-)
Zmień "int waga, wzrost, x;" ------------> "double waga, wzrost, x;"

No no... moja rada... uważnie pisz kod dopóki nie będziesz tego robił odruchowo ;-)

Answer 2

Zapomniałeś ; na końcu 5 linii

Answer 3

Kompilator to Code::Blocks

C::B to edytor, kompilator to G++ (GCC).

Niestety, G++ nie dał zbyt pomocnej podpowiedzi, może błąd Clanga Ci pomoże:

main.cpp:5:20: error: expected ';' after top level declarator
int waga, wzrost, x //x to nasze BMI
                   ^
                   ;

Answer 4

int float waga, wzrost, x;

---

cout << "Podaj swoja waga: " << ;
cin >> waga;
 
cout << "Podaj swoj wzrost" << ;
cin >> wzrost;

---

x=waga/(wzrost*wzrost);

---

if (x<18.5);
cout << "Niedowaga";
if (x>18.6 && x<=25);
cout << "Norma";
if (x>26 && x<=30);
cout << "Nadwaga";
if (x>31);
cout << "Otylosc";

---

Nie, kompilator to nie Code::Blocks.

Answer 5

Poprawiłem, oto kod:

#include <iostream>

using namespace std;

double waga, wzrost, x; //x to nasze BMI

int main()
{
    cout << "Podaj swoja waga: " <<;
    cin >> waga

    cout << "Podaj swoj wzrost" <<;
    cin >> wzrost

    x=waga/(wzrost*wzrost)

    cout << "Twoje BMI to " <<x << endl;

    if (x<18.5)
    cout << "Niedowaga";
    if (x>18.6 && x<=25)
    cout << "Norma";
    if (x>26 && x<=30)
    cout << "Nadwaga";
    if (x>31)
    cout << "Otylosc";

    return 0;
}

Ale mi teraz wywala nowe błędy:

linijka 9 i 12 - expected primary-expression before ';' token

linijka 17 - expected ';' before 'cout'

Comments

W linijce 10, 13 i 15 zapomniałeś ;

---

#include <iostream>

using namespace std;

double waga, wzrost, x; //x to nasze BMI

int main()
{
    cout << "Podaj swoja waga: ";
    ;cin >> waga

    ;cout << "Podaj swoj wzrost: ";
    ;cin >> wzrost

    ;x=waga/(wzrost*wzrost)

    ;cout << "Twoje BMI to " <<x<<endl
    ;

    if (x<18.5)
    cout << "Niedowaga";
    if (x>18.6 && x<=25)
    cout << "Norma";
    if (x>26 && x<=30)
    cout << "Nadwaga";
    if (x>31)
    cout << "Otylosc";

    return 0;
}

Następnym razem muszę bacznie przyglądać się kodowi. Ludzie się uczą na błędach. Jak powiedział kiedyś pan Zelent: Nie od razu Kraków zbudowano:) Zamykam

---

@UP Matko Boska co to jest XDDDD

---

Nie mam siły tłumaczyć co zrobiłeś źle. Czasami to, że działa nie wystarczy... Twój kod woła o pomstę do nieba :-D
Poprawię za Ciebie, bo nie wytrzymam XD

#include <iostream>

using namespace std;

double waga, wzrost, x; //x to nasze BMI

int main()
{
    cout << "Podaj swoja waga: ";
    cin >> waga;

    cout << "Podaj swoj wzrost: ";
    cin >> wzrost;

    x=waga/(wzrost*wzrost);

    cout << "Twoje BMI to " << x << endl;

    if (x < 18.5)
        cout << "Niedowaga";
    if (x > 18.6 && x <= 25)
        cout << "Norma";
    if (x > 26 && x <= 30)
        cout << "Nadwaga";
    if (x >31 )
        cout << "Otylosc";

    return 0;
}

Teraz jest dopiero pełna profeska