Gra "Walka bokserów" - algorytm

[Skayfer 0]

Witam

 

Piszę właśnie program w c++,tak jak w tytule gra "walka bokserów". Mam problem z algorytmem walki. W Założeniach ma byc 12 rund, przy czym walka może zostaćprzerwana jeżeli parametr wytrzymałość jednego z bokserów spadnie do 0.

 

Oto kod

http://pastebin.com/M8dyLwNv

 

Za wszelką pomoc będę bardzo wdzięczny.

[mineralwaser 0]

Nie wiesz jak coś zrobić, czy coś nie działa? W każdym razie ta pętelka:

do
{
 ...
} while( W1 || W2 < 0 )

zatrzyma sie wtedy i tylko wtedy, gdy W1 == 0 i jednocześnie W2 >= 0.

Jeśli chciałbyś przerwać w momencie, gdy któraś ze zmiennych będzie <= 0, to bardziej pasowałoby coś w stylu ( W1 > 0 && W2 > 0 ).

[Skayfer 0]

Ogólnie to chodzi mi oto żeby program działał prawidłowo . Wiem, że jest tam jeszcze sporo błędów i dopiero powoli dochodzę gdzie się one znajdują i na czym polegają.

 

 

Dzięki za spostrzeżenie z tą pętlą. Jak zauważę jakiś konkretniejszy problem to się spytam.

[Skayfer 0]

Kolejna wersja mojego programiku http://pastebin.com/yg80VB6E

 

Dodałem pętlę dla rund od 2 do 12 i tu jest problem, bo program omija tą część programu (wykonuje się kod tylko dla pierwszej rundy).

[tomek32td 0]

No to jest coś takiego jak debugger, który pozwala prześledzić działanie programu.

 

A jeśli chodzi o problem to może warto by sprawdzić warunki w tym forze czy oby na pewno pozwalają one na wykonanie instrukcji w pętli.

[Skayfer 0]

Właśnie tak myślałem, że może być coś na rzeczy z tymi warunkami. Albo w for, albo w if, ale kompletnie nie wiem gdzie dokładnie i na czym polega błąd.

[tomek32td 0]

Nie no, jak grochem o ścianę. Takie błędy szybko wyłapiesz za pomocą debuggera.

 

Ale pójdę nawet krok dalej. Skoro już wiesz że błąd jest w pętli, to może warto było by prześledzić jakie wartości mają zmienne z warunku pętli, otóż:

for (i = 2; i > 12 || W1 == 0 || W2 == 0; i++)

Mamy naszą pętle i na pierwszy rzut oka widać że pierwszy warunek tzn i>12 nie jest spełniony bo przecież nadajemy jej wartość początkową 2.

Teraz wystarczy się zastanowić czy jest szansa na to że po pierwszej rundzie któraś ze zmiennych W1 i W2 będzie miała wartość zero.

Tą zagadkę pozostawiam dla Ciebie. I na przyszłość trochę więcej własnej inicjatywy.

Dziękuje za uwagę.

[mineralwaser 0]

Wydaje mi się, iż autor po prostu myślał, że środkowy składnik fora jest warunkiem przerywającym pętle.

( i>12 || W1==0 || W2==0 ) jest zaprzeczniem ( i<=12 && W1!=0 && W2!=0 ), czyli tego co sprawiłoby, że kod działałby tak jak (chyba) chciał tego autor. (chociaż biorąc pod uwagę to, że W1 i W2 chyba mogą spaść poniżej zera, lepiej byłoby po prostu sprawdzać czy są > 0)