Problem z kodem

Question

To mój kod- gra polegająca na zgadywaniu liczb. Działa spoko i gdy zgadniesz ową liczbę program pyta czy chcesz grać dalej (wykorzystując komendę "do while"). W zamyśle gdy odpowiesz tak program wraca do początku i generuje kolejną losową liczbę. Tak się jednak nie dzieje. Program zadaje pytanie i gdy odpowiadam tak program daje ci możliwość dalszego odpowiadania jednak nie zmienia on liczby do zgadnięcia. Mam nadzieje że ktoś wytłumaczy mi na czym polega błąd i ewentułalnie pokaże jak zoptymalizować kod. Nie chcę jednak by wklejano mi po prostu skrypt podobnie działających programów bo to sam mogę zrobić.  Proszę o wyrozumiałość, dopiero się uczę.   

#include <iostream>
#include <time.h>
#include <cstdlib>
using namespace std;
int x, y, z;
string odpowiedz;
int main ()
{
    do
    {
    cout << "Zagraj w gre. Program wygeneruje losowom liczbe, z przedzialu 1-100 a ty zgadnij jakom." << endl;

    srand(time(NULL));
    y = rand()%100+1;

              cin >> z;
              if(z==y)
              {
              cout << "Brawo!" << endl;
              cout << "Grasz dalej (wpisz tak lub nie)?" << endl;
              cin >> odpowiedz;
              }

              if(z>y)
              {
              cout << "Liczba jest za duza!" << endl;
              }
              for(x=1; x>=1; x++)
              {
              cin >> z;
              if(z>y)
              {
                    cout << "Liczba jest za duza!" << endl;
              }
              if(z==y)
              {
                    cout << "Brawo!" << endl;
                    cout << "Grasz dalej (wpisz tak lub nie)?" << endl;
                    cin >> odpowiedz;
              }
              if(z<y)
              {
                    cout << "Liczba jest za mala!" << endl;
              }
              }

              if(z<y)
              {
              cout << "Liczba jest za mala!" << endl;
              }
              for(x=1; x>=1; x++)
              {
              cin >> z;
              if(z>y)
              {
                    cout << "Liczba jest za duza!" << endl;
              }
              if(z==y)
              {
                    cout << "Brawo!" << endl;
                    cout << "Grasz dalej (wpisz tak lub nie)?" << endl;
                    cin >> odpowiedz;
              }
              if(z<y)
              {

                    cout << "Liczba jest za mala!" << endl;
              }
              }

}while(odpowiedz == "tak");
return 0;
}

 

Answer 1

1. https://forum.pasja-informatyki.pl/faq#jak-wstawic-kod-zrodlowy

2. Jako taka optymalizacja kodu:

int main ()
{
int randomNumber = 0;
int userAnswer = 0;
int points = 0;
int numberOfTries = 0;

    std::cout << "Zagraj w gre. Program wygeneruje losowa liczbe wieksza od 0, z przedzialu 1-100 a ty zgadnij jaka.\n";
    std::cout << "Zeby zakonczyc gre wystarczy wpisac -1\n";

    do {
    srand(time(NULL));
    randomNumber = rand()%100+1;

        std::cin >> userAnswer;

        numberOfTries ++;

        if (userAnswer == randomNumber) {
            std::cout << "Brawo!\n";
            std::cout << "Ogolna liczba prob: " << numberOfTries << "\n";
            std::cout << "Masz " << points << "punktow";
        } else if (userAnswer < randomNumber) {
            std::cout << "Liczba jest za mala!\n";
        } else if (userAnswer > randomNumber) {
            std::cout << "Liczba jest za duza!\n";
        }

} while(userAnswer != -1);

return 0;
}

Czemu nie powatarza sie losowanie liczby u Ciebie? Bo nie wracasz do poczatku petli, to znaczy robisz to gdy, liczba nie zostala odgadnieta, ale gdy user ja odgadnie prosisz o wpisanie tak lub nie dla dalszej gry, po czym uruchamiasz kolejna petle.

for(x=1; x>=1; x++)
              {
              cin >> z;
              if(z>y)
              {
                    cout << "Liczba jest za duza!" << endl;
              }
              if(z==y)
              {
                    cout << "Brawo!" << endl;
                    cout << "Grasz dalej (wpisz tak lub nie)?" << endl;
                    cin >> odpowiedz;
              }
              if(z<y)
              {
                    cout << "Liczba jest za mala!" << endl;
              }
              }

Po co jest tutaj ta petla? Zrobiles ifa, sprawdziles czy liczba jest wieksza lub mniejszai pozniej urochamiasz petle, to jest bez sensu. Co to za warunek, x wieksze lub rowne 1, a inicjalizujesz x jako jeden odrazu

3. Zacznij nazywac zmienne

4. Slyszales o czyms takim jak TAB lub 4 Spacje? Bo ten kod nie czyta sie fajnie i przyjemnie

5. Do while to petla, wiec przy pomocy petli

Comments

Kod z punktu 2: Czemu na zgadnięcie wylosowanej liczby jest tylko jedna szansa (następnie losowana jest kolejna)?

---

@Hipcio,

bo to jest level hard :D, tylko najlepsi potrzebuja jednej proby :P. A serio, to nie wiem, zaimplementowalem to tak, zawsze mozna to zmienic, zeby nastepna liczba byla losowana dopiero gdy user odgadl poprzednia

---

Własnie chodziło mi o to by po odgadnięciu losowana była nastepna liczba. Wariant z losowaniem z kazdym zgadnięciem nowej liczby i wariant gdzie liczba losuje się tylko raz potrafię zrobić. Cały problem tkwi w tym by kolejna liczba losowała się co zgadnięcie. Także średnio mi pomogłeś bo ten mój kod lepiej działał od twojego XD

---

1. Pokaz mi gdzie napisales, ze ma losowac tylko przy zgadnieciu

2. Nie musisz uzywac pogrubonej czcionki

3. Znajac warunek if(), rozwiazesz ten problem w 5 sekund, wiec nie wiem czy to az tak duzy problem

---

1. Jak znasz konkretną dziedzinę to problem z tej konkretnej dziedziny zawsze możesz rozwiązać w 5 sekund ale jak wspomniałem jestem POCZĄTKUJĄCYM i nie wiem czy musze ci to bardziej tłumaczyć.

2.Mogę używać pogrubioną czcionkę choć co prawda użyłem ją przypadkiem ale specialnie dla ciebie ją zostawie.

3. NIE WIEM CZY NAPRAWDĘ MUSZĘ TO TŁUMACZYĆ ALE TAKA GRA DO JAKIEJ TY KOD NAPISAŁEŚ RACZEJ SENSU BY NIE MIAŁA

---

int main ()
{
int randomNumber = 0;
int userAnswer = 0;
int points = 0;
int numberOfTries = 0;

srand(time(NULL));

    std::cout << "Zagraj w gre. Program wygeneruje losowa liczbe wieksza od 0, z przedzialu 1-100 a ty zgadnij jaka.\n";
    std::cout << "Zeby zakonczyc gre wystarczy wpisac -1\n";

    randomNumber = rand()%100+1;
    do {
        std::cin >> userAnswer;

        numberOfTries ++;

        if (userAnswer == randomNumber) {
            std::cout << "Brawo!\n";
            std::cout << "Ogolna liczba prob: " << numberOfTries << "\n";
            std::cout << "Masz " << points << "punktow";
            randomNumber = rand()%100+1;
        } else if (userAnswer < randomNumber) {
            std::cout << "Liczba jest za mala!\n";
        } else if (userAnswer > randomNumber) {
            std::cout << "Liczba jest za duza!\n";
        }

} while(userAnswer != -1);

return 0;
}

Losuje nastepna, tylko gdy poprzednia byla prawidlowa