Zgadywanie liczb - problem

Question

Witam

Zabrałam się za napisanie krótkiej gerki w zgadywanie liczb. Program losuje liczbę, my mamy zgadnąć jaką. Przy błędnej odpowiedzi pisze czy podaliśmy za dużą czy za małą liczbę. Przy każdej błędnej odpowiedzi pisze która to jest nasza próba. Jeśli zgadniemy to wypisuje ile prób nam to zajęło. Program jest zabezpieczony przed wpisaniem błędnych znaków np. liter. Mam tylko jeden problem jak wpiszę litery zamiast liczby w którejś z kolei próbie to program zapętla się bez kontroli. Jeśli wpisze litery przy pierwszej próbie jest ok. Proszę o pomoc jak to naprawić. Jeśli ktoś ma jakieś inne uwagi w sprawie kodu chętnie wysłucham.

#include <iostream>
#include <cstdlib>
#include <ctime>

using namespace std;

int main()
{
    int liczba, liczba_z, licznik=1;
    srand(time(NULL));
    liczba = (rand () % 1000) +1;
    cout << "Pomyslalem sobie liczbe od 1 do 1000. Zgadnij jaka:" << endl;
    cout << liczba << endl;
    cin >> liczba_z ;
        if (cin.good() && liczba_z>=1 && liczba_z<=1000)
        {
            for (int i=1; liczba!=liczba_z; i++)
            {
               licznik ++;
                if (liczba>liczba_z)
               {
                   cout << "Za malo. To twoja " << licznik << " proba. Sprobuj jeszcze raz:" << endl;
                   cin.sync(); cin.clear();
                   cin >> liczba_z ;
               }
               else if (liczba<liczba_z)
               {
                   cout << "Za duzo. To twoja " << licznik << " proba. Sprobuj jeszcze raz:" << endl;
                   cin.sync(); cin.clear();
                   cin >> liczba_z ;
               }
               else
               {
                   cout << "Wprowadziles bledne dane!!!" << endl;
                    cin.sync(); cin.clear();
               }
            }
            if (liczba==liczba_z)
            {
                cout << "Zgadles!!! Gratulacje. Ilosc prob:" << licznik << endl;
            }
        }
        else
        {
            cout << "Wprowadziles bledne dane!!!" << endl;
            cin.sync(); cin.clear();
        }

    return 0;
}

 

Comments

cin.good() jest twoim przyjacielem. Przykład:

while(true) {
    cin >> liczba_z ;
    if (cin.good())
        break;
    cout << "Nie rób sobie jaj!. Wprowadź liczbę!" << endl;
}

 

Answer 1

Sprawdz ten kod:

#include <iostream>
#include <cstdlib>
#include <ctime>

using namespace std;

int main()
{
    int liczba, liczba_z=0, licznik=0;
    srand(time(NULL));
    liczba = (rand () % 1000) +1;
    cout << "Pomyslalem sobie liczbe od 1 do 1000. Zgadnij jaka:" << endl;
    cout << liczba << endl;
    for (int i=1; liczba!=liczba_z; i++)
    {
        cin >> liczba_z ;
        licznik ++;
        if (cin.good()==false)
            cout << "Wprowadziles bledne dane!!!" << endl;
            
        else if (liczba>liczba_z)
            cout << "Za malo. To twoja " << licznik << " proba. Sprobuj jeszcze raz:" << endl;
            
        else if (liczba<liczba_z)
            cout << "Za duzo. To twoja " << licznik << " proba. Sprobuj jeszcze raz:" << endl;
            
        cin.clear();
        cin.sync();
    }
    cout << "Zgadles!!! Gratulacje. Ilosc prob:" << licznik << endl;
    return 0;
}

 

Comments

Dzięki

Po przeanalizowaniu znalazłam wiele błędów. Takich jak nie potrzebne pętle.

Najważniejszym jednak błędem było zamiana dwóch funkcji czyszczących bufor

cin.sync();
cin.clear();

A poprawnie jest

cin.clear();
cin.sync();

To sprawiało najwięcej problemów.

Answer 2

Ok, ale jakoś nie chce mi działać. Może wstawiam w złym miejscu tą pętle? próbowałam chyba wszystkiego z cin.good i nie chce zadziałać.

Zobaczyłam inny błąd więc wklejam poprawiony kod:

#include <iostream>
#include <cstdlib>
#include <ctime>

using namespace std;

int main()
{
    int liczba, liczba_z, licznik=1;
    srand(time(NULL));
    liczba = (rand () % 1000) +1;
    cout << "Pomyslalem sobie liczbe od 1 do 1000. Zgadnij jaka:" << endl;
    cout << liczba << endl;
    cin >> liczba_z ;
        if (cin.good())
        {
            if (liczba_z>=1 && liczba_z<=1000)
            {
                for (int i=1; liczba!=liczba_z; i++)
                {
                   licznik ++;
                    if (liczba>liczba_z)
                   {
                       cout << "Za malo. To twoja " << licznik << " proba. Sprobuj jeszcze raz:" << endl;
                       cin.sync(); cin.clear();
                       cin >> liczba_z ;
                   }
                   else if (liczba<liczba_z)
                   {
                       cout << "Za duzo. To twoja " << licznik << " proba. Sprobuj jeszcze raz:" << endl;
                       cin.sync(); cin.clear();
                       cin >> liczba_z ;
                   }
                   else if (cin.good())
                   {
                        cout << "Wprowadziles bledne dane!!!" << endl;
                        cin.sync(); cin.clear();
                   }
                }
                if (liczba==liczba_z)
                {
                    cout << "Zgadles!!! Gratulacje. Ilosc prob:" << licznik << endl;
                }
            }
             else
            {
                cout << "Wprowadz liczbe od 1 do 1000" << endl;
                cin.sync(); cin.clear();
                cin >> liczba_z;
            }
        }
        else
        {
            cout << "Wprowadziles bledne dane!!!" << endl;
            cin.sync(); cin.clear();
        }

    return 0;
}

 

Comments

Witam.

Więc tak, oczywiście to, co napiszę to moje zdanie, a że każdy ma własne...

Ja bym 'łapał' błędy inaczej, nie puszczając w ogóle liter dalej, do pętli sprawdzającej. Użył bym np. wpisania liczby do 'string', a w 'try...catch' była by próba zmiany w 'int' przez 'stoi()'.

Mogę to wpisać, ale mnie znowu zjadą, że gotowce podrzucam, więc tylko samo sprawdzenie:

	try
	{
		liczba = stoi(znaki);
	}
	catch (exception& err)
	{
		cerr<<"Wprowadz poprawna liczbe!\n";
		liczba = 0;
	}

A wszystko w pętli 'while', dopóki liczba == 0

Może to pomoże.

Pozdrawiam.

---

W liniach 26, 32 i 49 pobierasz dane ze strumienia cin, ale nie sprawdzasz czy si cię to powiodło. Musisz sprawdzić cin.good() za każdym razem gdy pobierzesz dane z tego strumienia.

---

Niestety jeszcze nie do końca rozumiem o czym piszesz. Chyba jeszcze nie ten poziom.

---

Czy rozumiesz co robi poniższy kod?
 

int liczba;
cin >> liczba;