Tracenie żyć w zgaduj zgaduli .

Question

#include <iostream>
#include <cstdlib>
#include <time.h>
using namespace std;
int liczba , strzal , zycia=11;
int wybor;
int main()
{
    cout << "Menu Glowne" << endl;
    cout << "------------" << endl;
    cout << "Zagraj w zgaduj zgadule!" << endl;
    cout << "1. Liczby od 1 - 100 (EASY) " << endl;
    cout << "2. Liczby od 100 - 1000 (MEDIUM) " << endl;
    cout << "3. Liczby od 10000 - 100000 (HARD) " << endl;
    cout << "4. Wyjdz z gry. " << endl;
    cout << endl;
    cout << "Co wybierasz? : ";
    cin >> wybor;


switch (wybor)
{
    case 1 :
    {
        {
            system ("cls");
        }
    cout << "Pomyslalem sobie liczbe od 1...100" << endl;
    cout << "Masz 10 zyc na odgadniecie " << endl;

    srand(time(NULL));
     liczba = rand()%100+1;
     while(strzal!=liczba)
     {
         zycia--;
         cout << "Zgadnij jaka to liczba: (Jest to twoje "  << zycia << " zycie)" << endl;
         cin >> strzal;

     if (strzal==liczba)
         cout << "Trafiles!" << endl;
     else if (strzal>liczba)
        cout << "To za duzo!" << endl;
     else if (strzal<liczba)
        cout << "To za malo!" << endl;
        else if (zycia==0)
            cout << "Przegrales ! " << endl;

     }
    }
  break;

Witam !!
Jestem początkującym programistą i mam pewien problem.
Chcę aby gracz grający w moją grę miał 10 szans na odgadnięcie prawidłowej liczby.(Zastosowałem if'y)
 Komputer po straceniu 10 żyć nie pokazuje mi że przegrałem i dalsze próby naliczają życia na  minus.
W tym momencie kieruje pytanie do was czy istnieje inny sposób lub inne zastosowanie "if'ów" aby komputer normalnie kończył grę po straceniu 10 żyć?
Poniżej wklejam kod.

Pozdrawiam i z góry dziękuję za odpowiedź!

Comments

Jezu czy tak ciężko wstawić kod w przeznaczoną do tego ramkę ?

Jak to zrobić: http://forum.pasja-informatyki.pl/faq#faq1

---

Już to zrobiłem. Jest to mój pierwszy post na forum następnym razem już wkleję od razu przepraszam za utrudnienia.

---

Nie wjeżdżaj tak na ludzi, jak widać 150pkt - tzn. na forum od chwili.

---

No dobra ale regulamin i FAQ jak wstawiać pytania się powinno przeczytać lub chociaż obejrzeć. :)

Answer 1

Przyjrzyj się temu fragmentowi kodu:

if (strzal==liczba)
         cout << "Trafiles!" << endl;
     else if (strzal>liczba)
        cout << "To za duzo!" << endl;
     else if (strzal<liczba)
        cout << "To za malo!" << endl;
        else if (zycia==0)
            cout << "Przegrales ! " << endl;

Ostatnia instrukcja if nigdy się nie wykona, bo któraś ze wcześniejszych instrukcji zawsze musi się wykonać (bo sprawdzasz w nich wszystkie trzy opcje: strzał jest mniejszy, większy lub równy od zgadywanej liczby - któraś z tych opcji zawsze jest prawdziwa). Pomyśl w jaki sposób wstawić odpowiednio ifa ze sprawdzaniem życia, żeby miał on szansę się wykonać. I przydałoby się wyjść z pętli, jeżeli ten if zostanie spełniony - zastosuj instrukcję break.

Comments

Dziękuję za pomoc ! Pogłówkuje trochę i dam znać czy udało mi się uzyskać zamierzany efekt!

Answer 2

Wstaw kod w znacznikach. :P

Comments

Chodzi o to co zrobiłem?

Answer 3

Zamiast robić ten ostatni else if daj tam po prostu

if (zycia==0) {

cout << "przegrałeś";

strzal = liczba ;  // dzięki temu wyjdzie Ci z while

}

Answer 4

A nigdzie nie odejmujesz od liczby żyć, jeśli użytkownik nie zgadnie liczby.

Comments

Odejmuje. Jest zapis zycia-- i od podanej liczby 11 zmniejsza liczbę o 1 za każdym razem gdy gracz poda swój strzał.

---

A faktycznie, to przerwij pętle jak warunek zostanie spełniony, tzn będzie 0 żyć.

---

Hmm napisałem

else if (zycia==0)
            cout << "Przegrales ! " << endl;

lecz przy życiach wynoszących 0 on zaczyna liczyć  dalej na - i nie wykonuje mojego warunku. Właśnie tutaj mam problem bo on go ignoruje i nie może zerwać pętli.

---

Haha, nigdzie nie napisałeś żadnego warunku końca pętli w przypadku zerowej ilości żyć. Napisałeś tylko że przegrałeś, a program kontynuuje pracę :)

W osobnym komentarzu napisałem rozwiązanie twojego problemu.

Answer 5

Nic dziwnego że tak się dzieje. Twoja pętla działa do momentu aż ktoś nie trafi. W else if na temat końca gry dopisz funkcję zrywającą pętlę break; 

 

PS. Nie zapomnij tez oddzielić tego else if'a od tamtych sprawdzających strzał, bo w innym wypadku nigdy nie dojdzie do sprawdzenia żyć :) 

Comments

Dzięki za pomoc ! Już rozwiązałem ten problem.