Czy mógłby mi ktoś powiedzieć, co tu mam źle ? :D

Question

Witam, jestem zielony w tych sprawach i nie za bardzo wiem, co zorbiłem źle :

#include <iostream>
#include <time.h>
#include <windows.h>

using namespace std;

int liczba, strzal;

int main()
{
    cout << "Pomyslalem liczbe od 1 do 100." << endl;
    Sleep (2000);

    srand(time(NULL));
    liczba=rand()%100+1;

    while(liczba != strzal);
    {
     cout << "Jaka to liczba ?" << endl;
     cin >> strzal;
      if (strzal==liczba)
        cout << "Wygrales! :)";
      else if (strzal > liczba)
      cout << "Za duzo :C" ;
      else if (strzal < liczba)
        cout << "Za malo :C" ;
    }
    return 0;
}

Chodzi o to, że nie pojawia mi się napis "Jaka to liczba ?" i nie mogę przez to zrobic nic dalej.

Z góry dziękuję za odpowiedź :)

Answer 1

Wywal średnik po warunku pętli while

Answer 2

usuń średnik za while;

 

while(liczba != strzal);  <-ten tutaj

Answer 3

Przeanalizuj co zmieniłem. 

#include <iostream>
#include <time.h>
#include <windows.h>

using namespace std;

int liczba, strzal;

int main()
{
    cout << "Pomyslalem liczbe od 1 do 100." << endl;

    srand(time(NULL));
    liczba=rand()%100+1;

    do
    {
     cout << "Jaka to liczba ?" << endl;
     cin >> strzal;
      if (strzal==liczba)
        cout << "Wygrales! :)";
      else if (strzal > liczba)
      cout << "Za duzo :C" ;
      else if (strzal < liczba)
        cout << "Za malo :C" ;
    }while(liczba != strzal);
    return 0;
}

Sleep bylo nie potrzebne. Używaj w takim przypadku do while. Ta pętla najpierw się wykonuje, potem sprawdza prawdziwość warunku.

Answer 4

Średnik po while jest niepotrzebny.

#include <iostream>
#include <time.h>
#include <windows.h>

using namespace std;

int liczba, strzal;

int main()
{
    cout << "Pomyslalem liczbe od 1 do 100." << endl;
    Sleep (2000);

    srand(time(NULL));
    liczba=rand()%100+1;

    while(liczba != strzal)
    {
     cout << "Jaka to liczba ?" << endl;
     cin >> strzal;
      if (strzal==liczba)
        cout << "Wygrales! :)";
      else if (strzal > liczba)
      cout << "Za duzo :C" ;
      else if (strzal < liczba)
        cout << "Za malo :C" ;
    }
    return 0;
}

Pozdrawiam.

Answer 5

Dziękuję wszystkim bardzo :)