Program dziwaczeje po wpisaniu litery zamiast liczby w losowaniu pseudolosowych - Dlaczego i jak zaradzić ?

Question

Jak w temacie, szaleje po wpisaniu litery , poza tym na liczbach normalnie działa.

#include <iostream>
#include <windows.h>
#include <stdlib.h>
#include <stdio.h>
#include <time.h>

int liczba;
int strzal;
int proba=0;
using namespace std;

int main()
{
    cout << "Hello world!" << endl;
    cout<<"Pomyslalem liczbe od 1 do 100 , zgaduj"<<endl;

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




while (liczba!=strzal)
{
    proba++;
    cout<<"Zgdnij jaka"<<"To twoja "<<proba<<" proba"<<endl;
    cin>>strzal;

    if (liczba==strzal)
       cout<< "Udalo Ci sie"<<endl;

       else if (liczba<strzal)
        cout<<"To za duzo"<<endl;

       else if (liczba>strzal)
        cout<<"To za malo"<<endl;
}



getchar(); getchar();
    return 0;
}

 

Comments

1) Zachowanie jest naturalne, skoro do typu liczby calkowitej wstaeiamy typ zanku... mieszasz typy i tyle. Albo dodasz else, albo trzeba by bylo sprawdzac znaki. Lepiej dac else, tak to u mnie dzialalo

2) zmienne wstac do main

3) angieleksie nazwy zmiennych

4)ctime, cstdlib, zamiast .h

5)Jak sie nir myle, to do losowania liczb losowych, jest nowsza funkcja zgodnie z nowym standardem

Answer 1

Proszę, zapomnij o srand() i rand...

Na końcu masz przykład:

http://en.cppreference.com/w/cpp/numeric/random/uniform_int_distribution

 

PS. Za zmienne globalne, okładamy witkami wierzbowymi po piętach :-)

Answer 2

Próbujesz do int-a przypisać zmienną tekstową gdy podana zostanie litera lub ciąg znaków. Jeżeli chcesz być na to odporny to pobierz tekst a następnie zamień go na liczbę.