Losowanie bez powtórzeń

pytanie zadane 17 sierpnia 2015 w C i C++ przez Iras Obywatel (1,800 p.)

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

using namespace std;

int wylosuj()
{
    return (rand()%10)+1;
}

bool czyWylosowanaLiczbaPadla(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;
}
void dodawanieLiczbDoTablicy(int tab[])
{
    int liczba=wylosuj();
    int wylosowana=0;
    do{
        if(czyWylosowanaLiczbaPadla(liczba,tab,wylosowana)==false)
        tab[wylosowana]=liczba;
        wylosowana++;
    }while(wylosowana<5);
}


void OdczytywanieLiczbZTablicy(int tab[])
{
    int wylosowana=0;
    do{
        cout<<tab[wylosowana]<<endl;
        wylosowana++;
    }while(wylosowana<5);
}
int main()
{
    srand(time(NULL));

    int wylosowane[5];
    OdczytywanieLiczbZTablicy(wylosowane);
    dodawanieLiczbDoTablicy(wylosowane);
    OdczytywanieLiczbZTablicy(wylosowane);
    return 0;
}

Cześć, Mam problem mianowicie mniej więcej wiem gdzie jest problem ale nie wiem gdzie bo według mnie program wylosowuje pierwsza liczbe i później nic nie robi bo kolejne miejsca w tablicy to miejsca w pamięci. Proszę o pomoc

4 odpowiedzi

odpowiedź 17 sierpnia 2015 przez Dorion300 Szeryf (90,250 p.)

odpowiedź 17 sierpnia 2015 przez Szykem2 Nałogowiec (29,510 p.)

Strasznie dziwny ten kod i nie do końca wiem co chcesz powiedzieć w tym komentarzu do kodu, ale napiszę jak ten kod działa najpierw sobie wypisuje tablice(niezaincjalizowaną) z wartościami śmieciowymi. Dalej dodaje liczby do tablicy(przynajmniej taki jest zamysł) ale liczba nie jest losowana za każdym razem wejścia do pętli więc za kazym wywołaniem(poza pierwszym) funkcja czyWylosowanaLiczbaPadla zwraca false i pętla do .. while się kończy a dodana jest tylko wartość dla indeksu 0.

void dodawanieLiczbDoTablicy(int tab[])
{
    int liczba;
    int wylosowana=0;
    do{
        liczba = wylosuj();
        if(czyWylosowanaLiczbaPadla(liczba,tab,wylosowana)==false)
        {
             tab[wylosowana]=liczba;
             wylosowana++;
        }
        else continue;
    }while(wylosowana<5);
}

wydaje mi się, że o to chodzi

komentarz 17 sierpnia 2015 przez Iras Obywatel (1,800 p.)

tak to już sam poprawiłem dlaczego był błąd tylko że teraz kompilator mi się zawiesza za każdym razem i nie wiem co jest nie tak w tym drugim kodzie

komentarz 17 sierpnia 2015 przez Szykem2 Nałogowiec (29,510 p.)

Mi ten kod, który wkleiłem działa bez problemów

odpowiedź 17 sierpnia 2015 przez Iras Obywatel (1,800 p.)

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

using namespace std;

int wylosuj()
{
    return (rand()%10)+1;
}

bool czyWylosowanaLiczbaPadla(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;
}
void dodawanieLiczbDoTablicy(int tab[])
{

    int wylosowana=0;

    do{
        int liczba=wylosuj();
        if(czyWylosowanaLiczbaPadla(liczba,tab,wylosowana)==false){
        tab[wylosowana]=liczba;
        wylosowana++;}
    }while(wylosowana<5);
}

void OdczytywanieLiczbZTablicy(int tab[])
{
    int wylosowana=0;
    do{
        cout<<tab[wylosowana]<<endl;
        wylosowana++;
    }while(wylosowana<5);
}
int main()
{
    srand(time(NULL));

    int wylosowane[5];
    OdczytywanieLiczbZTablicy(wylosowane);
    dodawanieLiczbDoTablicy(wylosowane);
    OdczytywanieLiczbZTablicy(wylosowane);
    return 0;
}

Trochę poprawiłem i wydaje mi się że powinien działać poprawnie, ale po kompilacji program sie zatrzymuje i nic nie robi