raz wylosowana liczba powtarza sie do konca programu

Question

moj kod:

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

using namespace std;

 

int main()
{

    int strzal;
    srand(time(NULL));
    int losowa_liczba;
    losowa_liczba = rand()%50+1;
    do
    {
     cout << "wymyslilem sobie liczbe z zakresu od 1 do 50" << endl;
     cout<<"zgadnij jaka to liczba :)"<<endl;
     do
     {

      cin>>strzal;
      int i=1;
      if(strzal==losowa_liczba)
      {
          cout<<"proba nr:"<<i<<"  brawo! udalo ci sie zgadnac! :)"<<endl;
      }
      else if(strzal<losowa_liczba)
      {
          cout<<"proba nr:"<<i<<"  to za malo"<<endl;
      }
      else if
         (strzal>losowa_liczba)
      {
          cout<<"proba nr:"<<i<<"  to za duzo"<<endl;
      }
      i++;
     }while(strzal!=losowa_liczba);
     cout<<"jesli chcesz zagrac jeszcze raz to nacisnij 1";
     int c;
     cin>>c;
     system("cls");
    }while(c==1);
    return 0;
}

problem jest taki ze jesli juz user zgadnie wylosowana liczbe i bedzie chcial zagrac jeszcze raz po nacisnieciu 1 to bedzie ta sama liczba wylosowana, nie rozumiem czemu...

Answer 1

Ponieważ, umieścileś to przed pętlą, funkcja rand zadziała tyle razy ile razy ją wywołasz. Wywołałeś ją tylko 1 raz na samym początku, działa to na podstawie takiego konstruktora. Na początku wykonuje się cała instrukcja od przypisywania danych do pętli, warunków itd, starczy że umieścisz losowa_liczba = rand()%50+1; przed pierwszym coutem

Answer 2

Ponieważ losowa_liczba się nie zmienia, i użyłeś rand tylko jeden raz. Prostym pomysłem jest zrobienie kolejnego rand po napisaniu 1

Answer 3

no tak teraz juz widze ten blad, dziekuje:). Jest jeszcze jeden problem a niechce zakladac nowego tematu dla takiego drobiazga, otoz program nie chce sie skompilowac a codeblocks pokazuje ze problem jest w 46 linijce, tam gdzie jest zakonczenie petli, przy tym:    }while(c==1) ;

ps. na ten problem nie wplywa gdzie wstawie funkcje rand

Comments

Zadeklaruj zmienną c pomiędzy using namecpace a int main()

Oto poprawiony kod:

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

using namespace std;

int c;

int main()
{

    int strzal;
    srand(time(NULL));
    int losowa_liczba;

    do
    {
    losowa_liczba = rand()%50+1;
     cout << "wymyslilem sobie liczbe z zakresu od 1 do 50" << endl;
     cout<<"zgadnij jaka to liczba :)"<<endl;
     do
     {

      cin>>strzal;
      int i=1;
      if(strzal==losowa_liczba)
      {
          cout<<"proba nr:"<<i<<"  brawo! udalo ci sie zgadnac! :)"<<endl;
      }
      else if(strzal<losowa_liczba)
      {
          cout<<"proba nr:"<<i<<"  to za malo"<<endl;
      }
      else if
         (strzal>losowa_liczba)
      {
          cout<<"proba nr:"<<i<<"  to za duzo"<<endl;
      }
      i++;
     }while(strzal!=losowa_liczba);
     cout<<"jesli chcesz zagrac jeszcze raz to nacisnij 1";

     cin>>c;
     system("cls");
    }while(c==1);
    return 0;
}

---

a moglbys powiedziec czemu?, i co to da?, bo nie zabardzo ogarniam

---

Ta zmienna musi byc przed pętlą aby ta pętla ją "widziała"