Problem C++. Pentla nieskończona ?

Question

http://pastebin.com/ZsdytJyh Witam. Otóż jest to fragment programu, trochę po mojemu z kursu pana Mirosława.

Tu moje pytanie, dlaczego funkcja wykouje się w nieskończoność? Jeżeli... return stawiam nie w while, tylko return 0; wstawiam w funkcji main. Generalnie  pentla wypisuje komunikat albo w nieskończoność, że większa lub mniejsza? Z góry dziękuje za wyjaśnienie

 

Umieszczam link do kodu u góry. 

Answer 1

Cop w tym dziwnego? skoro podajesz raz liczbę przed pętlą, potem losujesz drugą i sprawdzasz w pętli warunek, jeżeli  podana przez ciebie liczba nie jest równa wylosowanej to pętla leci dalej i będzie leciałą dopóki podana przez ciebie liczba nie będzie równa wylosowanej, a że podajesz te dane przed pętlą to ta pętla nigdy się nie skończy bo  każdy kolejny obrót pracuje z tymi samymi danymi;]

 

#include <iostream>
#include <stdio.h>
#include <cstdlib> //funkcja rand
#include <time.h>

using namespace std;


int liczba, strzelaj;

int main()
{

        cout<<"Witamy w losowaniu! Wybierz sobie liczbe w przedziale 1..100:"<<endl;
        cout<<"Podaj liczbe z tego przedzialu: "<<endl;
        cin>>strzelaj;
        int licznik=0;


        srand(time(NULL));                               //start randomaizing, zacznij generowac liczy losowe


                        /* funkcja rand() oznacza przedzial dla 100 sa to liczby 0-99, wiec*/                                   // musimy dodac 1,

        while(strzelaj!=liczba)
{
 liczba= rand()% 100+1;
        if(strzelaj>liczba)
{
        cout<<"TWOJA liczba jest za duza "<<endl;
}
        else if(strzelaj == liczba)
{
        cout<<"Wygrales, liczba podana przez Ciebie jest prawidlowa"<<endl;
}
        else if(strzelaj < liczba)
{
        cout<<"Twoja liczba jest za mala. "<<endl;
}
licznik++;
   }
        cout<<"wypiszesz to gnoju"<< "liczba prob"<<licznik<<endl;
        return 0;

}

 

Comments

wstawiłem po prostu break do pentli i działa poprawnie. Mimo pobierania 2 liczb, przed pentlą.

---

Owszem, ale wyniku ci nie znajdzie poprawnego, bo pętla się przerwie po break, w takim razie pętla jest ci nie potrzebna bo i tak raz tylko sprawdzasz wynik.

Answer 2

W każdej pętli musi być zmiana warunku pętli aby mogła ona się skończyć, bo tak za każdym obiegiem pętli warunek sie nie zmieni co powoduję, że pętla będzie nieskonczona

Answer 3

Wystarczy, że wytniesz ten kod na początku:

cout<<"Podaj liczbe z tego przedzialu: "<<endl;
cin>>strzelaj;

I wkleisz go wewnątrz pętli.

Pętla Ci ciągle sprawdza warunek dla jednej liczby. A liczba powinna się zmieniać za każdym obejściem pętli.

Wstaw ten kod przed if'ami w pętli.

Poza tym użyj pętli "do..while", ponieważ pętla ta sprawdzi warunek przynajmniej raz. W pętli "while" jeśli na samym początku liczba podana byłaby równa liczbie szukanej, to program by się zwiesił.

Pozdrawiam.

Comments

#include <iostream>
#include <stdio.h>
#include <cstdlib> //funkcja rand
#include <time.h>
 
using namespace std;
 
 
int liczba, strzelaj;
 
int main()
{              
       
        cout<<"Witamy w losowaniu! Wybierz sobie liczbe w przedziale 1..100:"<<endl;

 
 
        srand(time(NULL));                               //start randomaizing, zacznij generowac liczy losowe
        liczba= rand()% 100+1;  
       
                        /* funkcja rand() oznacza przedzial dla 100 sa to liczby 0-99, wiec*/                                   // musimy dodac 1,
       
        do
{      
 cout<<"Podaj liczbe z przedzialu: "<<endl;
        cin>>strzelaj;
        if(strzelaj>liczba)
{
        cout<<"TWOJA liczba jest za duza "<<endl;
}
        else if(strzelaj == liczba)
{
        cout<<"Wygrales, liczba podana przez Ciebie jest prawidlowa"<<endl;
}
        else if(strzelaj < liczba)
{
        cout<<"Twoja liczba jest za mala. "<<endl;
}
       
   } while(strzelaj!=liczba);

        return 0;
               

Tutaj masz przykład o co mi chodzi.