Wybór randomowej linii z pliku tekstowego

Question

A więc chciałem zrobić wisielca, w którym zawsze musi być randomowe słowo. Próbowałem coś zrobić, ale ostatecznie się pogubiłem i zrobił się bajzel. Z tego co zrobiłem ostatecznie wynika że wylosowana liczba w ogóle nie bierze udziału w wyborze linii i nie wiem jak ją tam wkleić. Kod:

int nr_lini;
string linia;

int main()
{
    srand(time(NULL));
    nr_lini=rand()%20+1;
    nr_lini=rand()%20+1;
    nr_lini=rand()%20+1;
    nr_lini=rand()%20+1;

    fstream slowa;
    slowa.open("slowa.txt", ios::in);
    if(slowa.good()==false)
    {
       cout << "Taki plik nie istnieje!";
    }

    (getline(slowa, linia));
    {
        slowo=linia;
    }
    slowa.close();

    cout << slowo;

 

Comments

Hej

Jak chcesz losować z przedziału to nie używaj modulo (%) aby zawięzić zbiór. Gdy to zrobisz wprowadzasz "modulo bias" i Twój rozkład losowy nie jest już taki równy.
Zachcęcam do lektury dowolnego wyjaśnienie np. tutaj:

https://stackoverflow.com/questions/10984974/why-do-people-say-there-is-modulo-bias-when-using-a-random-number-generator

Dodatkowo, funkcja rand() jest dosyć "przewidywalna", dlatego do nauki jest spoko (bo jest prosta), ale nauczysz się złego designu. Do losowania liczb radzę użyć nowszych technik.

Tutaj przykład jak wylosować liczbę od 0 do 10000:

std::uniform_int_distribution<int> distribution(0, 10000);
std::random_device rd;
std::mt19937 engine(rd());
int randomNumber = distribution(engine);

(Powyższy fragment pochodzi z zasad SEI CERT dla C++, gdzie jest wytłumaczone, czemu rand jest niebezpieczny. Link tutaj: MSC50-CPP. Do not use std::rand() for generating pseudorandom numbers)

Answer 1

    nr_lini=rand()%20+1;
    nr_lini=rand()%20+1;
    nr_lini=rand()%20+1;
    nr_lini=rand()%20+1;

czemu tyle razy nadpisujesz jedną zmienną, ostateczny wynik będzie i tak wykonaniem 4 lini 

W dalszej częsci kodu nigdzie nie używasz " nr_lini", tylko zmienną linia, która nie jest zainicjalizowana. 

Comments

Ale jak ostatecznie zrobić randomizera linii z pliku? :(

---

int count = 1;
While(count <= 3)
{
    random = rand() % 50;
    while(getline(File, line))
    {
        ++numOfLines;

        if(numOfLines == random)
        {
            cout << line;
        }
    }
    count++;
}

to jest też w miejscu, gdzie twoje rozwiązanie "w miare jest wyjasnione" tylko, że próbując je pisać samemu pomyliłeś nazwenictwo 

random = rand() % 50;
random1 = rand() % 50;
random2 = rand() % 50;
while(getline(File, line))
{
    ++numOfLines;

    if(numOfLines == random || numOFLines == random1 || numOfLines == random2)
    {
        cout << line;
    }
}

w myśl deklaracji zmiennej, potem inicjalizacji i jej braku użycia 

---

Dzięki wielkie

---

owocnej pracy.