C++ Program Liczb losowych

Question

Ma ktos jakis dobry Pomysl jak to rozwiazac? Po uruchomieniu Programu w C++ pojawia sie blad i chociaz ten Program jest dobrze napisany on i tak sie pokazuje.

Napisany przezemnie Program z Odcinka 4 Programowania C++ od Miroslawa Zelenta do polepszenia Umiejetnosci nie dziala poprawnie. Logike rozumiem, no ale nie chcialbym aby Klient nie mogl wlaczyci skoniczonego Programu na Zlecenie.

Aby sie upewnic na 100% skopiowalem Kod z Code review https://forum.pasja-informatyki.pl/126490/cr-c-%234-liczby-losowe-pseudolosowe, i wstawilem go do Katalogu main.cpp.

Dobrzy Ludzie zaproponowali mi użyć biblioteki <random> i <chrono>, bo C++11 to już raczej podstawa.

Program z i bez tych Bibliothek nie dziala poprawnie,

Pytanie jest: istnieja jakies nowsze Bibliotheki do Wersji C++ 17.12?

 

main.cpp i c++0x_warning,h

#include <iostream>
#include <cstdlib>
#include <time.h>
#include <chrono>
#include <random>

using namespace std;

int liczba, strzal, ile_prob=0;

int main()
{
    cout << "Witaj! Pomyslalem sobie liczby 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;


}

#if __cplusplus < 201103L
#error This file requires compiler and library support for the \
ISO C++ 2011 standard. This support is currently experimental, and must be \
enabled with the -std=c++11 or -std=gnu++11 compiler options.
#endif

#endif

 

Comments

Nie używaj rand()! C++ ma specjalne klasy do losowania liczb losowych.

#include <random>
#include <chrono>
#include <iostream>

int main(){
    std::default_random_engine random_generator(std::chrono::steady_clock::now().time_since_epoch().count());
    std::uniform_int_distribution<int> distribution(1, 10); 
    for(int i=0;i<10;++i){
        std::cout<<i<<". "<<distribution(random_generator)<<"\n";
    }
}

http://cpp.sh/4cvto

---

int main()
{
    cout << "Witaj! Pomyslalem sobie liczby 1..100"<< endl;
    srand(time(NULL));
    Liczba = std::default_random_engine random_generator(std::chrono::steady_clock::now().time_since_epoch().count());
    std::uniform_int_distribution<int> distribution(1, 10);
    for(int i=0;i<10;++i){
        std::cout<<i<<". "<<distribution(random_generator)<<"\n";

Nie rozumiem, w ten Sposob?, sam podany twoj Kod nie dziala a specjalne klasy do losowania? przeciez na Kursie c++ byl poddany caly kod?

---

Kurs ma już 5 lat..

Poprawiłem trochę Twój kod: http://cpp.sh/9uund

A zaraz wrzucę cały kod, w najlepszej możliwej na moje siły wersji.

---

#include <iostream>
#include <cstdlib>
#include <chrono>
#include <random>

int main()
{
    int RandomNumber, UserInput, CountOfAttempts=0;
    std::cout << "Witaj! Pomyslalem sobie liczby 1..100\n";

    std::default_random_engine RandomGenerator(std::chrono::steady_clock::now().time_since_epoch().count());
    std::uniform_int_distribution<int> distribution(1, 100);
    RandomNumber = distribution(RandomGenerator);

    while(UserInput!=RandomNumber)
    {
        ++CountOfAttempts;

        do{
            std::cin.clear();
            std::cin.sync();
            std::cout<<"Zgadnij jaka (to Twoja "<<CountOfAttempts<<" proba):\n";
        }while(!(std::cin>>UserInput));


        if(RandomNumber==UserInput) std::cout<<"Udalo sie! Wygrywasz w "<<CountOfAttempts<<" probie\n";
        else if(UserInput<RandomNumber) std::cout<<"To za malo\n";
        else std::cout<<"To za duzo\n";
    }

    getchar();

    return 0;
}

 

---

Jeśli chodzi o ten warning, to https://stackoverflow.com/questions/18174988/how-can-i-add-c11-support-to-codeblocks-compiler (jeśli piszesz w codeblocks)

---

@Jason_Nr_1,

Nie wiem jakiego kompilatora używasz, ale musisz aktywować odpowiednie flagi, np:

-std=c++14 -Wall -Wextra -pedantic-errors

Poniżej kod, który używa wymienionych przez Ciebie bibliotek

#include <chrono>
#include <iostream>
#include <random>

int main()
{
    int quess = 0;

    // wylosowanie liczby
    unsigned seed = std::chrono::steady_clock::now().time_since_epoch().count();
    std::default_random_engine rd(seed);
    std::uniform_int_distribution<int> dist(1, 100);
    int draw = dist(rd);
    std::cout << "Wylosowano liczbe z przedzialu 1-100." << std::endl;

    // pierwsza próba odgadnięcia liczby
    int number;
    std::cout << "Sprobuj ja odgadnac: ";
    while (!(std::cin >> number)) {
        std::cin.clear();
        while (std::cin.get() != '\n') {}
        std::cout << "Bledne dane, podaj liczbe: ";
    }

    // kolejne próby, aż do skutku - przy użyciu pętli while
    while(number != draw) {
        quess++;
        if(number < draw) {
            std::cout << "Liczba jest zbyt mala.";
        } else {
            std::cout << "Za duza liczba.";
        }
        std::cout << " Sprobuj jeszcze raz: ";
        while (!(std::cin >> number)) {
            std::cin.clear();
            while (std::cin.get() != '\n') {}
            std::cout << "Bledne dane, podaj liczbe: ";
        }
    }
    std::cout << "Celny strzal :) Brawo!" << std::endl;
    std::cout << "Liczba prob: " << quess << std::endl;

    return 0;
}

 

---

Nie lepiej od razu z c++1z (c++17)? Dodatkowo ja zawsze używam też O3 (maksymalna optymalizacja)

---

@Hiskiel, Dzieki Stary, :D

Szkoda ze Kurs nie jest aktulizowany regularnie, pomoglo by to wiekszosci Osob zaczynajcych.