losowanie bez powtarzania

Question

Program w ogóle nie chce działać. Pomoże ktoś?

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

using namespace std;

int losowacz()
{
    return rand()%10+1;

}

bool spraw(int liczba, int tab[], int ile)
{
    if (ile<=0) return false;

    int i=0;
    do
    {
        if ( liczba == tab [ ile ] ) return true;
       ile ++;
    }while(i<ile);
    return false;
}

int main()
{
    srand(time(NULL));
    int los;
    int i=0;
    int numer=1;
    int liczby [5];
    do
    {
       los=losowacz();

       if (spraw(los, liczby, i) == false)
       {
           los=liczby[i];
           i++;
       }
    }while (i<5);

     i=0;
     do
     {
        cout << "wylosowana liczba numer " << numer <<": "<< liczby[i] << endl;
        i++;
        numer++;
     }while (i<5);

    return 0;
}

 

Answer 1

Proponuję dodanie komentarzy czy czegokolwiek wyjaśniającego jak to ma dokładnie działać bo ok można się domyśleć z kodu ale po pierwsze ten kod jak widzę nie do końca działa poprawnie a poza tym z komentarzami by było łatwiej. A więc moje sugestie:

masz tablicę liczby[5] do której nie wprowadzasz danych, więc znajdują tam się śmieci, jakieś losowe liczby które możesz odczytać, ale dalej są losowe i brzydkie, 

a więc w programie losujesz jakąś liczbę, potem sprawdzasz czy jest w tablicy i tutaj pojawia się pierwszy błąd bo w tablicy nie ma nic więc są śmieci więc raczej nie znajdziesz tam ładnych liczb z przedziału <1;10>, a dalej ponieważ nie ma tej liczby w tablicy to wykonuje się linijka:

los=liczby[i];

czyli przypisujesz wartość z tablicy do zmiennej los (czy to nie powinno być na odwrót ?????????)

liczby[i] = los;

a na końcu czytasz liczby z tablicy w której dalej są śmieci które są wypisywane.

Generalnie wszystko działa ale chyba nie powinno działać w ten sposób, a więc chyba powinieneś zrobić dwie rzeczy: odwrócić ten zapis który pokazałem powyżej i najlepiej uzupełnić tablicę liczby takimi liczbami które nie pojawią się w losowaniu np. zerami lub -1, to drugie jest opcjonalne ale nie zaszkodzi (gdybyś tak zrobił na początku to błąd łatwiej było by wykryć)

Comments

Działa. Dziękuję za wnikliwe wytłumaczenie. Pomysł z komentarzami bardzo dobry. Sama będę szybciej się orientować. Dzięki.

Answer 2

while(i<ile); <-- ten warunek będzie zawsze prawdziwy. Pętla wykonuje Ci się w nieskończoność.

Answer 3

Poprawiłam pętle ale wychodzą za duże liczby. Analizowałam wszystko tysiąc razy, nie mam pomysłu co jest źle.

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

using namespace std;

int losowacz()
{
    return rand()%10+1;

}

bool spraw(int liczba, int tab[], int ile)
{
    if (ile<=0) return false;

    int i=0;
    do
    {
        if ( liczba == tab [ i ] ) return true;
       i ++;
    }while(i<ile);
    return false;
}

int main()
{
    srand(time(NULL));
    int los;
    int i=0;
    int numer=1;
    int liczby [5];
    do
    {
       los=losowacz();

       if (spraw(los, liczby, i) == false)
       {
           los=liczby[i];
           i++;
       }
    }while (i<5);

     i=0;
     do
     {
        cout << "wylosowana liczba numer " << numer <<": "<< liczby[i] << endl;
        i++;
        numer++;
     }while (i<5);

    return 0;
}

 

Comments

los=liczby[i];

Raczej na odwrot ;)