C++ - losowanie bez powtórzeń

Question

Witam, napisałem przed chwilą program do losowania liczb pseudolosowych bez powtórzeń, ku mojemu zaskoczeniu zadziałał za pierwszym razem. Miał wylosować liczby z przedziału od 1...5. Wylosował po kolei liczby 3 4 2 1 5. Drugi raz uruchamiam program i niestety konsola CMD świeci pustkami. Nie wyskoczył żaden błąd ani w CodeBlocks'ie ani w konsoli CMD. Wyłączyłem CodeBlocka, uruchomiłem jeszcze raz i znowu zadziałał, niestety po ponownym uruchomieniu kicha nic do wyświetlenia. Teraz nawet po wyłączeniu CodeBlocka jest pustka. Wydaję mi się że kod jest czysty ale oczywiście mogę się mylić. Oto on:
 

#include <iostream>
#include <cstdlib>
#include <time.h>
#include <stdio.h>
using namespace std;

int main()
{
    int liczba[5],wylosowana;
    bool prawda=true;
    srand(time(NULL));

    for(int i=0;i<5;i++)
    {
        wylosowana=rand()%5+1;

        for(int j=0;j<5;j++)
        {
            if(liczba[j]==wylosowana)prawda=false;
        }
        if(prawda==true)
        {
         liczba[i]=wylosowana;
        }
        else i--;
    }
    for(int i=0;i<5;i++)
    {
        cout<<liczba[i]<<endl;
    }

    getchar();getchar();
    return 0;
}

Comments

http://forum.pasja-informatyki.pl/faq#jak-wstawic-kod-zrodlowy

Answer 1

Wydaje mi sie ze masz nastepujacy blad: jesli znajde powtorzenie to juz nie ma szans aby zmienna prawda stala sie true, przez co caly czas dekrementowane jest i co powoduje nie mozliwosc wyjscia z petli.

 

PS: Uzywaj odpowiedniego znacznika do wstawiania kodu (przycisk {...}code w edytorze.

Comments

Jak tak dłużej zacząłem się temu przyglądać też to zauważyłem dzięki :)

---

Po zmianie działa jeszcze raz dzięki za trafną uwagę.
 

#include <iostream>
#include <cstdlib>
#include <time.h>
#include <stdio.h>
using namespace std;

int main()
{
    int liczba[5],wylosowana;
    bool prawda;
    srand(time(NULL));

    for(int i=0;i<5;i++)
    {
        wylosowana=rand()%5+1;
        prawda=true;
        for(int j=0;j<5;j++)
        {
            if(liczba[j]==wylosowana)prawda=false;
        }
        if(prawda==true)
        {
         liczba[i]=wylosowana;
        }
        else i--;
    }
    for(int i=0;i<5;i++)
    {
        cout<<liczba[i]<<endl;
    }

    getchar();getchar();
    return 0;
}

 

Answer 2

#include <iostream>
#include <cstdlib>
#include <time.h>
#include <stdio.h>
using namespace std;

int main()
{
    int liczba[5],wylosowana;
    bool prawda=true;
    srand(time(NULL));

    for(int i=0;i<5;i++)
    {
        prawda = true; // gdy w poprzednim rozliczeniu petli byla rowna false
        wylosowana=rand()%5+1;

        for(int j=0;j<5;j++)
        {
            if(liczba[j]==wylosowana)
                prawda=false;
        }
        if(prawda==true)
        {
            liczba[i]=wylosowana;
        }
        else i--;
    }
    for(int i=0;i<5;i++)
    {
        cout<<liczba[i]<<endl;
    }

    getchar();getchar();
    return 0;
}

Nie mniej jednak zrobiłbym jeszcze taki myk:

#include <iostream>
#include <cstdlib>
#include <time.h>
#include <stdio.h>
using namespace std;

int main()
{
    int liczba[5],wylosowana;
    bool prawda=true;
    srand(time(NULL));

    for(int i=0;i<5;i++)
    {
        prawda = true;
        wylosowana=rand()%5+1;

        for(int j=0;j<i;j++) // po co sprawdzac caly zakres jak sa tam przypadkowe wartosci?
        {
            if(liczba[j]==wylosowana)
                prawda=false;
        }
        if(prawda==true)
        {
            liczba[i]=wylosowana;
        }
        else i--;
    }
    for(int i=0;i<5;i++)
    {
        cout<<liczba[i]<<endl;
    }

    getchar();getchar();
    return 0;
}