Wykonywanie pętli w nieskończoność

Question

Dlaczego pętla wykonuje się w nieskończoność?

 
 if(wyborwalka==1)
            {
                do{
                cout<<"Atakujesz...";
                Sleep(2000);
                system("cls");
                cout<<""<<imie<<endl;
                cout<<"HP:60"<<endl;
                cout<<"OBRAZENIA:20"<<endl;
                cout<<"****************"<<endl;
                cout<<"GOBLIN"<<endl;
                cout<<"HP:"<<goblinhp-bohaterobrazenia<<endl;
                cout<<"OBRAZENIA:7"<<endl;
                Sleep(2000);
                system("cls");
                cout<<"Goblin atakuje...";
                Sleep(2000);
                cout<<""<<imie<<endl;
                cout<<"HP:"<<bohaterhp-goblinobrazenia<<endl;
                cout<<"OBRAZENIA:20"<<endl;
                cout<<"****************"<<endl;
                cout<<"GOBLIN"<<endl;
                cout<<"HP:"<<goblinhp-bohaterobrazenia<<endl;
                cout<<"OBRAZENIA:7"<<endl;
                }while(goblinhp!=0);
                cout<<"Wygrales!"<<endl;
            }

Answer 1

Pewnie zmienna goblinhp zawsze jest różna od 0, ile ona wynosi na początku? Nie widzę też zmiany jej wartości.

Comments

Na początku wynosi 25

---

No właśnie i dlaczego miała by mieć nagle 0, żeby pętla się przerwała? Nigdzie odjąłeś jej tych 25.

goblinhp = goblinhp - 25

 

Answer 2

W warunku pętli jest "robic petle dopóki hp goblina bedzie rozne od zera". Ale jak będzie -5 to też będzie robić pętle. Zmień warunek na:

while(goblinhp>0);

 

Comments

Nie pomaga

---

Problem jest w tym, że nie uaktualniasz zmiennej goblinhp o nową wartość. Spójrz na moją odpowiedź.

Answer 3

Jeśli bohater zadaje cios to zmiana ilości hp goblina musi nastąpić za pomocą ponownego przypisania ilości hp do zmiennej goblinhp, a u Ciebie tego nie ma. Jest:
 

... << goblinhp-bohaterobrazenia << ...

To co tutaj robisz to tworzysz zmienną anonimową w postaci uaktualnionego życia goblina, ale nie przypisujesz tej wartości do zmiennej goblinhp. Powinno być:
 

goblinhp -= bohaterobrazenia; 
//... 
... << goblinhp << ...

I jak Hipcio, słusznie zauważył, warunek wyjścia z pętli powinien być zmieniony.

Comments

Dzięki, już działa