[CR] C++ (#4): Liczby losowe (pseudolosowe)

Question

CR = Code Review. O co chodzi? Zajrzyj tutaj
Pełna lista wszystkich Code Review? Zajrzyj tutaj

https://www.youtube.com/watch?v=QapjTCGV7GM

Kod z odcinka - lotto:

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

using namespace std;

int liczba;

int main()
{
    cout<<"Witaj w losowaniu! Za 3 sekundy nastapi zwolnienie blokady";
    Sleep(3000);
    cout<<endl;

    srand(time(NULL));

    for (int i=1; i<=6; i++)
    {
        liczba = rand()%49+1;
        Sleep(1000);
        cout<<liczba<<"\a"<<endl;
    }

    return 0;
}

Kod z odcinka - zgadula:

#include <iostream>
#include <cstdlib>
#include <time.h>
#include <stdio.h>

using namespace std;

int liczba, strzal, ile_prob=0;

int main()
{
    cout<<"Witaj! Pomyslalem sobie liczbe 1..100"<<endl;
    srand(time(NULL));
    liczba = rand()%100+1;

    while(strzal!=liczba)
    {
      ile_prob++;

      cout<<"Zgadnij jaka (to Twoja "<<ile_prob<<" proba): ";
      cin>>strzal;

      if(strzal==liczba)
        cout<<"Udalo sie! Wygrywasz w "<<ile_prob<<" probie"<<endl;

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

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

    getchar();getchar(); //dla pewnosci 2 razy

    return 0;
}

Paczka z odcinka: POBIERZ​

Answer 1

Czyste czepialstwo odnośnie "głupot" w kodzie, ale żeby CR nie stał pusty to 2 uwagi dotyczące kodu (pomijam moje uwagi z poprzednich CR, w tym znowu mamy "redundancję logiczną": Listing 2, linia 29):

1. Nagłówki bibliotek... Jeżeli używamy C++, to używajmy już tych w wersji <c[nazwa]>, a nie <[nazwa].h>.
2. Dlaczego getchar 2 razy? Jeżeli chcemy mieć pewność, że jakimś cudem nie prześlizgnie się żaden nadprogramowy znak to nie jest lepiej najpierw wyczyścić bufor i wykonać getchara, tylko raz?

Comments

@szymon jak wyczyścić bufor ?

---

​std::cin.ignore(INT_MAX);
std::cin.clear();

http://www.cplusplus.com/reference/istream/istream/ignore/

http://www.cplusplus.com/reference/ios/ios/clear/